Changeset 17736 in josm for trunk/src

Timestamp:

2021-04-09T23:18:01+02:00 (4 years ago)

Author:

simon04

Message:

fix #19771 - Tag2Link: use percent encoding for Wikimedia Commons URLs

File:

• trunk/src/org/openstreetmap/josm/tools/Tag2Link.java (modified) (3 diffs)

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

@@ -27 +27 @@
 import javax.json.JsonValue;
 
+import com.google.gdata.util.common.base.PercentEscaper;
 import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
 import org.openstreetmap.josm.data.osm.OsmUtils;
@@ -202 +203 @@
         if (key.matches("wikimedia_commons|image") && value.matches("(?i:File):.*")) {
             OsmUtils.splitMultipleValues(value).forEach(i -> linkConsumer.acceptLink(
-                    tr("View image on Wikimedia Commons"), "https://commons.wikimedia.org/wiki/" + i, imageResource.get()));
+                    tr("View image on Wikimedia Commons"), getWikimediaCommonsUrl(i), imageResource.get()));
         }
         if (key.matches("wikimedia_commons|image") && value.matches("(?i:Category):.*")) {
             OsmUtils.splitMultipleValues(value).forEach(i -> linkConsumer.acceptLink(
-                    tr("View category on Wikimedia Commons"), "https://commons.wikimedia.org/wiki/" + i, imageResource.get()));
+                    tr("View category on Wikimedia Commons"), getWikimediaCommonsUrl(i), imageResource.get()));
         }
 
@@ -216 +217 @@
     }
 
+    private static String getWikimediaCommonsUrl(String i) {
+        return "https://commons.wikimedia.org/wiki/" + new PercentEscaper(PercentEscaper.SAFEPATHCHARS_URLENCODER, false).escape(i);
+    }
+
     private static String getLinkName(String url, String fallback) {
         return tr("Open {0}", getHost(url, fallback));