Julian Simioni
fef92bd34b
Remove the geoname _type from all layers except venue
...
We can't distinguish between geonames of different layers due to an
ambiguity in our Elasticsearch schema that we unfortunately won't be
able to fix for a few weeks.
So, while it's technically true that there are countries, cities, etc
contained in the geonames dataset, it's still better for now to remove
geonames from these layers. We have good coverage of most coarse layers
from quattroshapes alone, so the impact isn't too bad.
9 years ago
Stephen Hess
00d033101b
removed redundant is-ids-an-array check (already covered by _single_scalar_parameter sanitizer)
9 years ago
Julian Simioni
415029e02b
Use groups sanitiser in sanitize_rect and sanitize_point
9 years ago
Julian Simioni
aa08fb9772
Add optional and required group parameter sanitizers
9 years ago
Julian Simioni
2811569bcd
Switch to source:layer:id ids format in /place
9 years ago
Julian Simioni
d11d185429
Fix missing space in error message
9 years ago
Julian Simioni
2505e92a62
Expect an array of types from _ids sanitiser
...
This doesn't have any effect by itself but allows for the 3-part gid
sanitiser to possibly return multiple types (i.e. in the case of
osm:venue:1000)
9 years ago
Julian Simioni
fa44effdac
Add geoname type to most layers
...
The Geonames dataset includes lots of different kinds of places, so add
them to the mapping.
9 years ago
Julian Simioni
8b1037d7c8
Separate real layers and alias layers into separate objects
9 years ago
Julian Simioni
68c9661c70
Remove all mappings other than helper/type_mapping
9 years ago
Stephen Hess
56d8600218
modified warning message for brevity
9 years ago
Stephen Hess
0f6d08c0ec
set boundary.circle.radius to default if caller did not supply it
9 years ago
Stephen Hess
947797f41e
added warning if any of boundary.circle.lat/lon/radius are supplied + tests
9 years ago
Julian Simioni
ebb3f33b2d
Remove unused test file
9 years ago
Stephen Hess
e90f70ce3d
added sanitizer that verifies that all parameters are single scalar values
9 years ago
Julian Simioni
16450087ee
Use simpler, clearer error on invalid ids length
9 years ago
Julian Simioni
edaf175aa7
Provide better, more consistent, errors on invalid ids
...
All error messages for invalid id formats now use a common format that
explains the nature of the error a bit better.
9 years ago
Julian Simioni
73b289e9e5
Consider tests refactored :)
9 years ago
Julian Simioni
9d21832874
Reformat test for clarity
9 years ago
Julian Simioni
8897c71664
Remove complicated error value test
...
It's better to just specify a few cases if needed
9 years ago
Julian Simioni
18de9a6766
Replace giant switch-statement test with multiple tests
9 years ago
Julian Simioni
03387722fb
Move _ids sanitiser tests to separate file
9 years ago
Julian Simioni
5721841fb3
Clarify name and error message in ids test
9 years ago
Julian Simioni
1b6c61c92e
Leave clean.ids undefined on error
9 years ago
Julian Simioni
65b7db94d6
Fix typo
9 years ago
Julian Simioni
cd63209085
Error on multiple ids parameters instead of warning
9 years ago
Julian Simioni
c687102374
Rename /place id parameter to ids
9 years ago
Julian Simioni
8592c37bf6
Expect multiple ids to be specified as a comma-delimited string
...
Disallow the other way that Node.js allows, which is to list the id
parameter multiple times in the querystring. See #272 .
9 years ago
Julian Simioni
47701277a3
Rename test with duplicate, non descriptive name
9 years ago
Julian Simioni
2cc9107114
Add test for multiple ids
9 years ago
Julian Simioni
bd4fbfe3bd
Whitespace
9 years ago
Peter Johnson
82ab899ecc
progress commit
9 years ago
Julian Simioni
d6abe09447
Use flat clean structure in /reverse
9 years ago
Julian Simioni
97044f655b
Flatten structure of clean for /search focus.point.lat|lon
9 years ago
Julian Simioni
4260442f59
Change boundary.circle params to flat string structure
9 years ago
Julian Simioni
e5ed04858b
Test individual clean attribute intead of using deepEquals
9 years ago
Julian Simioni
1bfa2fde40
Remove empty test of invalid lats
9 years ago
Julian Simioni
a5ba251afe
Remove brittle test fixture from search sanitiser
...
Some of the tests were checking the entire clean object, when they only
cared about one tiny element. This made making changes really hard.
9 years ago
Julian Simioni
66e1cab007
Whitespace
9 years ago
Diana Shkolnikov
f1fdc2bc26
Remove categories from response json
9 years ago
Diana Shkolnikov
0348f36671
Remove warning when no size param specified
...
Added tests for _size sanitizer while in there.
9 years ago
Peter Johnson
7c46aed4a7
expose error messages in geojson
9 years ago
Stephen Hess
6c85e5892e
converted warnings to errors
9 years ago
Julian Simioni
4e74c4db86
Add autocomplete sanitiser
...
This is similar to the search sanitiser but with a more limited scope.
It doesn't support layers, sorces, or bounding boxes.
9 years ago
Stephen Hess
ba22912507
implemented Diana's suggestions to use check-types and delete key rather than set to undefined
9 years ago
Stephen Hess
13635a2c16
added boundary.country to search and reverse queries (with test fixtures)
9 years ago
Stephen Hess
78c335d072
modified sanitizer to read from flattened parameter structure
9 years ago
Stephen Hess
d07a501682
requirements updates
...
set boundary country to iso3 instead of iso2
set to undefined if boundary.country input is not a string
9 years ago
Stephen Hess
66bf2bb46b
added sanitizer for boundary country parameter
...
sets to undefined if input country not found as ISO2/3
9 years ago
Peter Johnson
a1efb0c6d0
lodash-ify some array functions; remove 'yes' and 'y' from thruthy
9 years ago