Changeset 16839 in josm for trunk/src/org/openstreetmap/josm/tools

Timestamp:

2020-08-03T22:06:42+02:00 (4 years ago)

Author:

simon04

Message:

fix #19622 - Tag2Link: show icons based on presets/styles

File:

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

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

@@ -11 +11 @@
 import java.util.Arrays;
 import java.util.Collections;
+import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
+import java.util.Optional;
+import java.util.function.Supplier;
 import java.util.function.UnaryOperator;
 import java.util.regex.Matcher;
@@ -23 +26 @@
 import javax.json.JsonValue;
 
+import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
 import org.openstreetmap.josm.data.osm.OsmUtils;
 import org.openstreetmap.josm.data.preferences.CachingProperty;
@@ -77 +81 @@
          * @param name the name/label of the link
          * @param url the URL of the link
+         * @param icon the icon to use
          */
-        void acceptLink(String name, String url);
+        void acceptLink(String name, String url, ImageResource icon);
     }
 
@@ -142 +147 @@
         }
 
+        final HashMap<OsmPrimitiveType, Optional<ImageResource>> memoize = new HashMap<>();
+        final Supplier<ImageResource> imageResource = () -> memoize
+                .computeIfAbsent(OsmPrimitiveType.NODE, type -> OsmPrimitiveImageProvider.getResource(key, value, type))
+                .orElse(null);
+
         // Search
         if (key.matches("^(.+[:_])?name([:_]" + languagePattern + ")?$")) {
+            final ImageResource search = new ImageProvider("dialogs/search").getResource();
             PREF_SEARCH_ENGINES.get().forEach(url ->
-                    linkConsumer.acceptLink(tr("Search on {0}", getHost(url, url)), url.replace("$1", Utils.encodeUrl(value))));
+                    linkConsumer.acceptLink(tr("Search on {0}", getHost(url, url)), url.replace("$1", Utils.encodeUrl(value)), search));
         }
 
@@ -155 +166 @@
                 : null;
         if (key.matches("^(.+[:_])?website([:_].+)?$") && validURL != null) {
-            linkConsumer.acceptLink(getLinkName(validURL, key), validURL);
+            linkConsumer.acceptLink(getLinkName(validURL, key), validURL, imageResource.get());
         }
         if (key.matches("^(.+[:_])?source([:_].+)?$") && validURL != null) {
-            linkConsumer.acceptLink(getLinkName(validURL, key), validURL);
+            linkConsumer.acceptLink(getLinkName(validURL, key), validURL, imageResource.get());
         }
         if (key.matches("^(.+[:_])?url([:_].+)?$") && validURL != null) {
-            linkConsumer.acceptLink(getLinkName(validURL, key), validURL);
+            linkConsumer.acceptLink(getLinkName(validURL, key), validURL, imageResource.get());
         }
         if (key.matches("image") && validURL != null) {
-            linkConsumer.acceptLink(tr("View image"), validURL);
+            linkConsumer.acceptLink(tr("View image"), validURL, imageResource.get());
         }
 
@@ -173 +184 @@
             final String lang = Utils.firstNotEmptyString("en", keyMatcher.group("lang"), valueMatcher.group("lang"));
             final String url = "https://" + lang + ".wikipedia.org/wiki/" + valueMatcher.group("article").replace(' ', '_');
-            linkConsumer.acceptLink(tr("View Wikipedia article"), url);
+            linkConsumer.acceptLink(tr("View Wikipedia article"), url, imageResource.get());
         }
         if (key.matches("(.*:)?wikidata")) {
             OsmUtils.splitMultipleValues(value)
-                    .forEach(q -> linkConsumer.acceptLink(tr("View Wikidata item"), "https://www.wikidata.org/wiki/" + q));
+                    .forEach(q -> linkConsumer.acceptLink(tr("View Wikidata item"), "https://www.wikidata.org/wiki/" + q, imageResource.get()));
         }
         if (key.matches("(.*:)?species")) {
             final String url = "https://species.wikimedia.org/wiki/" + value;
-            linkConsumer.acceptLink(getLinkName(url, key), url);
+            linkConsumer.acceptLink(getLinkName(url, key), url, imageResource.get());
         }
         if (key.matches("wikimedia_commons|image") && value.matches("(?i:File):.*")) {
-            linkConsumer.acceptLink(tr("View image on Wikimedia Commons"), "https://commons.wikimedia.org/wiki/" + value);
+            String url = "https://commons.wikimedia.org/wiki/" + value;
+            linkConsumer.acceptLink(tr("View image on Wikimedia Commons"), url, imageResource.get());
         }
         if (key.matches("wikimedia_commons|image") && value.matches("(?i:Category):.*")) {
-            linkConsumer.acceptLink(tr("View category on Wikimedia Commons"), "https://commons.wikimedia.org/wiki/" + value);
+            String url = "https://commons.wikimedia.org/wiki/" + value;
+            linkConsumer.acceptLink(tr("View category on Wikimedia Commons"), url, imageResource.get());
         }
 
@@ -193 +206 @@
             final String formattedValue = valueFormatter.getOrDefault(key, x -> x).apply(value);
             final String url = urlFormatter.replace("$1", formattedValue);
-            linkConsumer.acceptLink(getLinkName(url, key), url);
+            linkConsumer.acceptLink(getLinkName(url, key), url, imageResource.get());
         });
     }