update overpass wizard code

[tyr_asd]

The overpass wizard (javascript) code used in JOSM's Download from Overpass API utility should be updated to use the recently published standalone module: ​https://github.com/tyrasd/overpass-wizard.

The new module has a couple of ​options which modify the output of the wizard. So, it can be set up to produce a query with recursive meta output, xml format and the global bbox parameter set (see ​https://github.com/drolbr/Overpass-API/issues/251). You could even supply your own set of presets which would be searched for free-form input, but I'd still call that feature as experimental for now.

The module doesn't come in a standalone js file (but as a node module), but you can generate one by passing it through browserify (resulting in a 28kB script after minification):

browserify -s overpassWizard index.js | uglifyjs > overpass-wizard.js

[simon04]

In 9704/josm:

fix #12469 - Update overpass wizard code

[Klumbumbus]

One test fails: https://josm.openstreetmap.de/jenkins/job/JOSM/lastCompletedBuild/jdk=JDK7/testReport/org.openstreetmap.josm.tools/OverpassTurboQueryWizardTest/testKeyValue/

[Klumbumbus]

test is skipped since r9815

[simon04]

In 9856/josm:

fix #12547 see #12469 - Fix Overpass API "Build query" for Java 7

[Don-vip]

It seems queries are not expanded, see #12939.

I have checked that running overpassWizard in the command line results in expanded queries (area(...)), while calling the standalone version from JOSM results in raw queries (geocodeArea:...), rejected by Overpass API.

I assumed the fix would be trivial (include expand.js as follows):

browserify -s overpassWizard index.js expand.js | uglifyjs > overpass-wizard.js

but it does not work better, although the file is much bigger.
I don't know how to solve this.

Simon, tyr_asd, can you please help me?

[tyr_asd]

Oh, yeah. Let's see. Does your javascript engine support XMLHttpRequest and getting results out from an async js callback? If yes, than you can ​use the `exand.js` module[¹] that comes with overpass-wizard (even though it's still undocumented – I'll have to work on that, sorry!). Otherwise, you're probably better off re-implementing the necessary ​shortcut expansion code yourself in Java (you can use ​expand.js as a reference). I guess that you already have code to send requests to Nominatim, right? In which cast this should only require some glue code here and there. (For other features of the overpass-wizard you should also implement the relative date and constants expansion, though.)

PS: I see that you're currently hardcoding a bbox-query (even for example if the users typed in a global query) by sending the bbox parameter with every Overpass request. Don't forget to rework that as well. ;-)

[¹] probably use browserify -s expand expand.js to generate a separate compiled module.

[Don-vip]

thanks for your feedback :) I think I will then rewrite the shortcut expansion in Java, as we have everything we need for Nominatim and keeping control on network connections is a better approach for JOSM (we have a lot of centralized stuff for everything network-related).