pelias/api - api - CogtoSrc

Commit Graph

Author	SHA1	Message	Date
Peter Johnson	bf42290c7c	refactor sanitizers: first pass	9 years ago
Diana Shkolnikov	c069933d5b	Refactored layers and sources into a common targets module	9 years ago
Diana Shkolnikov	67e034f5f2	Rename `source` to `sources` ... with a dash of refactoring thrown in for good measure	9 years ago
Diana Shkolnikov	feb1b0a6cd	Update /reverse endpoint params and query as per API spec	9 years ago
Stephen Hess	a2b1579b3e	incorporated _private sanitizer into search endpoint	9 years ago
Stephen Hess	c19d56697e	add _private sanitizer	9 years ago
Stephen Hess	55e31af238	extract truthy sanitizer to own file	9 years ago
Stephen Hess	2f5a5b1972	refactored _details tests to only _details sanitizer	9 years ago
Stephen Hess	e52c9e4ab9	moved details tests out to separate file	9 years ago
Julian Simioni	96c58e8658	Rename /reverse lat/lon to point.lat/lon	9 years ago
Julian Simioni	3176a61596	Rename /search lat/lon to focus.point.lat/lon	9 years ago
Julian Simioni	6c1cfbb4ad	Compare latitude values only, instead of full sanitiser output	9 years ago
Julian Simioni	5d69c8016b	Remove lat/lon params from tests that don't use it This just removes clutter and possible confusion from our test cases.	9 years ago
Julian Simioni	fd3b12cfa9	Remove all references to non-existent zoom parameter	9 years ago
Julian Simioni	040871cbe4	Whitespace	9 years ago
Peter Johnson	22322eaa32	change param:input to param:text	9 years ago
Harish Krishna	86424b8bbe	Rename `/search?input`, replacing it with '/search?text' parameter instead changing parameter name input to text https://github.com/pelias/api/issues/172 Rebased by @orangejulius	9 years ago
Julian Simioni	d9ede52e1b	Remove extra input paremter from reverse unit tests Not strictly required for this change, but I noticed there was an input parameter to lots of unit tests for the reverse endpoint. Reverse doesn't take an input(or text) parameter at all, so these are just extra, and probably came from a copy/paste in the tests.	9 years ago
Julian Simioni	dc6ed3d6a9	Remove unused suggest endpoint unit tests	9 years ago
Julian Simioni	3383fa0321	Add source parameter sanitiser	9 years ago
Julian Simioni	2ebc935295	Rename query/indidces to query/types It didn't really contain a list of indices.	9 years ago
Julian Simioni	6f92189519	Remove default_layers_set flag! After refactoring, this flag is no longer needed, as all areas of the code that care about layers do so by setting a key within clean.types, and then the types helper intelligently combines those together later.	9 years ago
Julian Simioni	7a6ac8541b	Separate concerns of address parser The address parser currently does two things: 1.) make some intelligent guesses as to possible admin regions to explicitly search against to improve the quality of results returned 2.) make some intelligent guesses as to when no part of the query needs to search against anything other than admin regions. This somewhat improves the quality of results returned but mostly improves the speed of the Elasticsearch query since it's searching significantly fewer recoords. These two concerns are now split into two separate methods within the query_parser helper module. They are mostly independent today, but don't have to be in the future.	9 years ago
Julian Simioni	bc8eabe4e6	Remove loops and many global checks from sanitiser tests Modifying these sanitiser tests became extremely hard because almost all of them were looping over lots of individual test cases, which places assumptions about the common behavior of potentialy very different test cases, as well as making assertions about huge swaths of output when only a small amount of that output was really under test. Hopefully these changes will make our tests easier to modify, and not really lose any ability to catch bugs.	9 years ago
Julian Simioni	82d3a6603c	Store sanitised types from layers parameter This moves the list of types created by sanitising the layer API parameter from clean.layers to clean.types.from_layers. In subsequent commits, types created from address parsing, and the yet-to-be-implemented source parameter will also live in the clean.types object. This will allow moving logic to set cmd.type out of controllers, and into separate logic that can be a littler smarter. Also, it will no longer require the clean.default_layers_set flag to be passed all around like a nasty global variable.	9 years ago
Julian Simioni	3479325fa8	Add source parameter sanitiser	9 years ago
Julian Simioni	24349a3839	Rename query/indidces to query/types It didn't really contain a list of indices.	9 years ago
Julian Simioni	3ff1752434	Remove default_layers_set flag! After refactoring, this flag is no longer needed, as all areas of the code that care about layers do so by setting a key within clean.types, and then the types helper intelligently combines those together later.	9 years ago
Julian Simioni	b9152dbe29	Separate concerns of address parser The address parser currently does two things: 1.) make some intelligent guesses as to possible admin regions to explicitly search against to improve the quality of results returned 2.) make some intelligent guesses as to when no part of the query needs to search against anything other than admin regions. This somewhat improves the quality of results returned but mostly improves the speed of the Elasticsearch query since it's searching significantly fewer recoords. These two concerns are now split into two separate methods within the query_parser helper module. They are mostly independent today, but don't have to be in the future.	9 years ago
Julian Simioni	d35544cdf4	Remove loops and many global checks from sanitiser tests Modifying these sanitiser tests became extremely hard because almost all of them were looping over lots of individual test cases, which places assumptions about the common behavior of potentialy very different test cases, as well as making assertions about huge swaths of output when only a small amount of that output was really under test. Hopefully these changes will make our tests easier to modify, and not really lose any ability to catch bugs.	9 years ago
Julian Simioni	4037c49c4b	Store sanitised types from layers parameter This moves the list of types created by sanitising the layer API parameter from clean.layers to clean.types.from_layers. In subsequent commits, types created from address parsing, and the yet-to-be-implemented source parameter will also live in the clean.types object. This will allow moving logic to set cmd.type out of controllers, and into separate logic that can be a littler smarter. Also, it will no longer require the clean.default_layers_set flag to be passed all around like a nasty global variable.	9 years ago
Diana Shkolnikov	db0e063d8c	Refactor controllers to be simple midddleware Refactor search and doc controllers to allow for post-processing middleware to handle geojson and response sending. Allows for a much more flexible routing scheme.	9 years ago
Julian Simioni	b547111d52	Remove latitude restrictions	9 years ago
Julian Simioni	396f27dcba	Mark previously invalid lat/lon test values as valid These examples should all now pass with the lat/lon wrapping.	9 years ago
Severyn Kozak	9b77b4434f	Add more test-cases for invalid bbox values. test/unit/sanitiser/search.js -Add some more sanity-check test-cases to verify that bbox validation is working as expected.	9 years ago
Severyn Kozak	762ef85f5c	Fix longitude-sanitization tests. test/unit/sanitiser/(search, suggest, reverse).js -Update/fix all of the tests that started failing as a result of the removal of longitude sanitization in `90a7683`.	9 years ago
Julian Simioni	167547a6c1	Rename doc endpoint to place	9 years ago
Julian Simioni	2fce64f75a	Move files	9 years ago
Julian Simioni	4e74e9b681	Whitespace	9 years ago
Harish Krishna	35105015b2	tests suggest sanitiser	9 years ago
Harish Krishna	e00c5aa2ad	Testing search sanitisers all sanitiser tests should test sanitizing input, lat/lon etc.. query parsing is tested seperately so, using query_parser to set expected.parsed_input	9 years ago
Harish Krishna	95c0e7461d	TESTS!	9 years ago
Harish Krishna	96a21570be	fixing layers related bug - adding a default_layers_set flag to clean obj.	9 years ago
Harish Krishna	050c11ec0b	just using a forked version of addressIt that focuses on US addresses (for now), modifying search queries and ignoring tests for now	9 years ago
Diana Shkolnikov	63ed198320	Details param not working as expected Fixes #135	9 years ago
Diana Shkolnikov	0b44ba68c3	forgot to add test to commit	10 years ago
Diana Shkolnikov	6ae1a4f7df	fix comments and tests	10 years ago
Diana Shkolnikov	8ac17d0cb2	Fix bbox parsing	10 years ago
Diana Shkolnikov	8c22f33dc7	Add details sanitizer to suggest/coarse	10 years ago
Diana Shkolnikov	45ba29cc54	Add categories param to /reverse	10 years ago

1 2 3 4 5

204 Commits (78c83205244e7bc81a8b4018f5bd711f4c7cfccc)