Changeset 14571 in josm for trunk/src

Timestamp:

2018-12-17T15:08:05+01:00 (6 years ago)

Author:

GerdP

Message:

see #17055 When harmonized value is only 3 characters reduce allowed Levenshtein distance from 2 to 1.

When shop=gas is not in presets, we don't want a message like "Value 'gas' for key 'shop' looks like one of [bag, car, yes]."
just because one character of those values matches.

File:

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

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

@@ -567 +567 @@
                             }
                         }
-                        if (minDist <= MAX_LEVENSHTEIN_DISTANCE && maxPresetValueLen > MAX_LEVENSHTEIN_DISTANCE) {
+                        if (minDist <= MAX_LEVENSHTEIN_DISTANCE && maxPresetValueLen > MAX_LEVENSHTEIN_DISTANCE
+                                && (harmonizedValue.length() > 3 || minDist < MAX_LEVENSHTEIN_DISTANCE)) {
                             if (fixVals.size() < 2) {
                                 fixedValue = closest;
@@ -575 +576 @@
                                 errors.add(TestError.builder(this, Severity.WARNING, MISSPELLED_VALUE_NO_FIX)
                                         .message(tr("Misspelled property value"),
-                                                marktr("Value ''{0}'' for key ''{1}'' looks like one of {2}."), prop.getValue(), key, fixVals)
-                                        .primitives(p)
-                                        .build());
+                                                marktr("Value ''{0}'' for key ''{1}'' looks like one of {2}."),
+                                                prop.getValue(), key, fixVals)
+                                        .primitives(p).build());
                                 withErrors.put(p, "WPV");
                                 continue;