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

Timestamp:

2014-12-28T18:59:56+01:00 (9 years ago)

Author:

Don-vip

Message:

see #10696 - allow URLs without protocol

Location:

trunk/src/org/openstreetmap/josm/data/validation

Files:

• routines/UrlValidator.java (modified) (2 diffs)
• tests/InternetTags.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/data/validation/routines/UrlValidator.java

@@ -157 +157 @@
     private static final Pattern PORT_PATTERN = Pattern.compile(PORT_REGEX);
 
-    private static final String SIMPLE_WEBSITE = "^www[.].*";
-    private static final Pattern SIMPLE_WEBSITE_PATTERN = Pattern.compile(SIMPLE_WEBSITE);
-
     /**
      * Holds the set of current validation options.
@@ -301 +298 @@
         if (!isValidScheme(scheme)) {
             setErrorMessage(tr("URL contains an invalid protocol: {0}", scheme));
-            if (SIMPLE_WEBSITE_PATTERN.matcher(value).matches()) {
-                setFix("http://" + value);
-            }
             return false;
         }

trunk/src/org/openstreetmap/josm/data/validation/tests/InternetTags.java

@@ -128 +128 @@
     private TestError doValidateTag(OsmPrimitive p, String k, String v, AbstractValidator validator, int code) {
         TestError error = null;
-        if (!validator.isValid(v != null ? v : p.get(k))) {
-            String msg = tr("''{0}'': {1}", k, validator.getErrorMessage());
+        String value = v != null ? v : p.get(k);
+        if (!validator.isValid(value)) {
+            String errMsg = validator.getErrorMessage();
+            // Special treatment to allow URLs without protocol. See UrlValidator#isValid
+            if (tr("URL contains an invalid protocol: {0}", (String)null).equals(errMsg)) {
+                String proto = validator instanceof EmailValidator ? "mailto://" : "http://";
+                return doValidateTag(p, k, proto+value, validator, code);
+            }
+            String msg = tr("''{0}'': {1}", k, errMsg);
             String fix = validator.getFix();
             if (fix != null) {