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

Timestamp:

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

Author:

simon04

Message:

fix #19754 - Tag2Link: support multi-values for URLs in website/source/url/image

File:

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

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

@@ -14 +14 @@
 import java.util.List;
 import java.util.Map;
+import java.util.Objects;
 import java.util.Optional;
 import java.util.function.Supplier;
@@ -160 +161 @@
 
         // Common
-        final String validURL = value.startsWith("http:") || value.startsWith("https:")
-                ? value
-                : value.startsWith("www.")
-                ? "http://" + value
-                : null;
-        if (key.matches("^(.+[:_])?website([:_].+)?$") && validURL != null) {
-            linkConsumer.acceptLink(getLinkName(validURL, key), validURL, imageResource.get());
-        }
-        if (key.matches("^(.+[:_])?source([:_].+)?$") && validURL != null) {
-            linkConsumer.acceptLink(getLinkName(validURL, key), validURL, imageResource.get());
-        }
-        if (key.matches("^(.+[:_])?url([:_].+)?$") && validURL != null) {
-            linkConsumer.acceptLink(getLinkName(validURL, key), validURL, imageResource.get());
-        }
-        if (key.matches("image") && validURL != null) {
-            linkConsumer.acceptLink(tr("View image"), validURL, imageResource.get());
+        final List<String> validURLs = value.startsWith("http:") || value.startsWith("https:") || value.startsWith("www.")
+                ? OsmUtils.splitMultipleValues(value)
+                .map(v -> v.startsWith("http:") || v.startsWith("https:")
+                        ? v
+                        : v.startsWith("www.")
+                        ? "http://" + v
+                        : null)
+                .filter(Objects::nonNull)
+                .collect(Collectors.toList())
+                : Collections.emptyList();
+        if (key.matches("^(.+[:_])?website([:_].+)?$") && !validURLs.isEmpty()) {
+            validURLs.forEach(validURL -> linkConsumer.acceptLink(getLinkName(validURL, key), validURL, imageResource.get()));
+        }
+        if (key.matches("^(.+[:_])?source([:_].+)?$") && !validURLs.isEmpty()) {
+            validURLs.forEach(validURL -> linkConsumer.acceptLink(getLinkName(validURL, key), validURL, imageResource.get()));
+        }
+        if (key.matches("^(.+[:_])?url([:_].+)?$") && !validURLs.isEmpty()) {
+            validURLs.forEach(validURL -> linkConsumer.acceptLink(getLinkName(validURL, key), validURL, imageResource.get()));
+        }
+        if (key.matches("image") && !validURLs.isEmpty()) {
+            validURLs.forEach(validURL -> linkConsumer.acceptLink(tr("View image"), validURL, imageResource.get()));
         }