Changeset 18046 in josm for trunk/src/org/openstreetmap

Timestamp:

2021-07-17T14:42:03+02:00 (3 years ago)

Author:

Don-vip

Message:

fix #21126 - make sure URL generated for searching Wikimedia Commons images can be converted to URI, needed for HTTP/2

File:

• trunk/src/org/openstreetmap/josm/tools/Mediawiki.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/tools/Mediawiki.java

@@ -95 +95 @@
     public void searchGeoImages(Bounds bounds, BiConsumer<String, LatLon> imageConsumer)
             throws IOException, ParserConfigurationException, SAXException, XPathExpressionException {
-        final URL url = new URL(baseUrl +
-                "?format=xml" +
-                "&action=query" +
-                "&list=geosearch" +
-                "&gsnamespace=6" +
-                "&gslimit=500" +
-                "&gsprop=type|name" +
-                "&gsbbox=" + bounds.getMaxLat() + "|" + bounds.getMinLon() + "|" + bounds.getMinLat() + "|" + bounds.getMaxLon());
+        final URL url = new URL(getGeoImagesUrl(baseUrl, bounds));
         final Document document = getDocument(url);
         final XPath xPath = XPathFactory.newInstance().newXPath();
@@ -113 +106 @@
             imageConsumer.accept(title, new LatLon(lat, lon));
         }
+    }
+
+    /**
+     * Returns the URL for searching geolocated images in given bounds.
+     * @param baseUrl The wiki base URL
+     * @param bounds the bounds of the search area
+     * @return the URL for searching geolocated images in given bounds
+     * @since 18046
+     */
+    public static String getGeoImagesUrl(String baseUrl, Bounds bounds) {
+        String sep = Utils.encodeUrl("|");
+        return baseUrl +
+                "?format=xml" +
+                "&action=query" +
+                "&list=geosearch" +
+                "&gsnamespace=6" +
+                "&gslimit=500" +
+                "&gsprop=type" + sep + "name" +
+                "&gsbbox=" + bounds.getMaxLat() + sep + bounds.getMinLon() + sep + bounds.getMinLat() + sep + bounds.getMaxLon();
     }