This allows nominatim to make sure that we are requesting the right place and
enables more heuristics on their side.
Signed-off-by: Corentin Noël <corentin.noel@collabora.com>
Extend list of evaluated address-categories to 'building', 'place', 'amenity':
In the long run, a migration to geocodejson-format should be evaluated, as the currently returned tags for categories
are given according to OSM tagging which is not very consistent.
Quoting from https://nominatim.org/release-docs/latest/api/Search/:
"If you are interested in a stable classification of address categories (suburb, city, state, etc),
have a look at the geocodejson format.
All other formats return classifications according to OSM tagging.
There is a much larger set of categories and they are not always consistent, which makes them very hard to work with."
Signed-off-by: umgfoin <umgfoin@users.noreply.github.com>
Rather than having to remember to reset the query builder in each method
that uses it, these are now created on-demand from the db connection.
Signed-off-by: Maximilian Bosch <maximilian@mbosch.me>
Closes#1333
I decided to not cross-check the availability of all all deprecated
things with older majors, so both min and max version are v30 for now.
A list of all things that I've fixed:
* Deprecated SCSS variables updated as documented in the upgrade notes[1].
* Fixed a bunch of query builder deprecations:
* $qb->resetQueryParts() is deprecated, one should just create a new
query-builder object. Done that.
* Calling `$qb->expr()->andX()` (and `orX()`) with no arguments is
deprecated. Instead, one should list all arguments in an array and
pass that to the method. Reworked queries where necessary to do so.
* The PHPUnit version I got by running `make test` downloaded a version
that prohibits non-static data providers. Made all affected data
providers static.
With that, everything from `make test` passes in a local dev
installation of Nextcloud 30 except for two tests:
1) OCA\Maps\Controller\PhotosControllerTest::testAddGetPhotos
Failed asserting that actual size 0 matches expected size 1.
/home/ma27/Projects/nextcloud/apps/maps/tests/Unit/Controller/PhotosControllerTest.php:206
2) OCA\Maps\Controller\TracksControllerTest::testAddGetTracks
Failed asserting that false matches expected true.
/home/ma27/Projects/nextcloud/apps/maps/tests/Unit/Controller/TracksControllerTest.php:205
Both tests failed on the previous commit already, so I decided to leave
those for now.
[1] https://docs.nextcloud.com/server/latest/developer_manual/app_publishing_maintenance/app_upgrade_guide/upgrade_to_30.html#front-end-changes
Signed-off-by: Maximilian Bosch <maximilian@mbosch.me>
Removed:
private function getLastLookupFile():ISimpleFile
private function getLastLookup():int
private function setLastLookup()
Signed-off-by: Arne Hamann <kontakt+github@arne.email>
