Changeset 5380 in josm

Timestamp:

Jul 29, 2012 10:18:55 AM (10 months ago)

Author:

bastiK

Message:

fixed #5679 - Validator: value of name tag not in presets if a preset specifies a name value (patch by mrwojo)

File:

• trunk/src/org/openstreetmap/josm/data/validation/tests/TagChecker.java (modified) (2 diffs)

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

@@ -458 +458 @@
             }
             if (checkValues && value != null && value.length() > 0 && presetsValueData != null) {
+                final Set<String> values = presetsValueData.get(key);
+                final boolean keyInPresets = values != null;  
+                final boolean tagInPresets = values != null && (values.isEmpty() || values.contains(prop.getValue()));
+                
                 boolean ignore = false;
                 for (String a : ignoreDataStartsWith) {
@@ -474 +478 @@
                     }
                 }
+                
+                if (!tagInPresets) {
+                    for (IgnoreKeyPair a : ignoreDataKeyPair) {
+                        if (key.equals(a.key) && value.equals(a.value)) {
+                            ignore = true;
+                        }
+                    }
+
+                    for (IgnoreTwoKeyPair a : ignoreDataTwoKeyPair) {
+                        if (key.equals(a.key2) && value.equals(a.value2)) {
+                            ignore = true;
+                        }
+                    }
+                }
+                
                 if (!ignore) {
-                    String i = marktr("Key ''{0}'' not in presets.");
-                    errors.add( new TestError(this, Severity.OTHER, tr("Presets do not contain property key"),
-                            tr(i, key), MessageFormat.format(i, key), INVALID_VALUE, p) );
-                    withErrors.put(p, "UPK");
-                } else {
-                    Set<String> values = presetsValueData.get(key);
-                    if (values != null && values.size() > 0 && !values.contains(prop.getValue())) {
-                        ignore = false;
-                        for (IgnoreKeyPair a : ignoreDataKeyPair) {
-                            if (key.equals(a.key) && value.equals(a.value)) {
-                                ignore = true;
-                            }
-                        }
-
-                        for (IgnoreTwoKeyPair a : ignoreDataTwoKeyPair) {
-                            if (key.equals(a.key2) && value.equals(a.value2)) {
-                                ignore = true;
-                            }
-                        }
-
-                        if (!ignore) {
-                            String i = marktr("Value ''{0}'' for key ''{1}'' not in presets.");
-                            errors.add( new TestError(this, Severity.OTHER, tr("Presets do not contain property value"),
-                                    tr(i, prop.getValue(), key), MessageFormat.format(i, prop.getValue(), key), INVALID_VALUE, p) );
-                            withErrors.put(p, "UPV");
-                        }
+                    if (!keyInPresets) {
+                        String i = marktr("Key ''{0}'' not in presets.");
+                        errors.add( new TestError(this, Severity.OTHER, tr("Presets do not contain property key"),
+                                tr(i, key), MessageFormat.format(i, key), INVALID_VALUE, p) );
+                        withErrors.put(p, "UPK");
+                    } else if (!tagInPresets) {
+                        String i = marktr("Value ''{0}'' for key ''{1}'' not in presets.");
+                        errors.add( new TestError(this, Severity.OTHER, tr("Presets do not contain property value"),
+                                tr(i, prop.getValue(), key), MessageFormat.format(i, prop.getValue(), key), INVALID_VALUE, p) );
+                        withErrors.put(p, "UPV");
                     }
                 }