Changeset 14517 in josm for trunk/src

Timestamp:

2018-12-06T10:55:42+01:00 (5 years ago)

Author:

GerdP

Message:

see #17055 improve TagChecker

• don't suggest fix value when given value is short and Levenshtein distance is 2

surface=u should not be fixed with surface=mud but highway=tra can be fixed with highway=track

File:

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

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

@@ -533 +533 @@
                 } else if (!isTagInPresets(key, value)) {
                     // try to fix common typos and check again if value is still unknown
-                    String fixedValue = harmonizeValue(prop.getValue());
+                    final String harmonizedValue = harmonizeValue(prop.getValue());
+                    String fixedValue = null;
                     Set<String> possibleValues = getPresetValues(key);
                     List<String> fixVals = new ArrayList<>();
                     int maxPresetValueLen = 0;
-                    if (!possibleValues.contains(fixedValue)) {
+                    if (possibleValues.contains(harmonizedValue)) {
+                        fixedValue = harmonizedValue;
+                    } else {
                         // use Levenshtein distance to find typical typos
                         int minDist = MAX_LEVENSHTEIN_DISTANCE + 1;
@@ -544 +547 @@
                             if (possibleVal.isEmpty())
                                 continue;
+                            if (harmonizedValue.length() < 3 && possibleVal.length() >= harmonizedValue.length() + MAX_LEVENSHTEIN_DISTANCE) {
+                                // don't suggest fix value when given value is short and lengths are too different
+                                // for example surface=u would result in surface=mud
+                                continue;
+                            }
                             maxPresetValueLen = Math.max(maxPresetValueLen, possibleVal.length());
-                            int dist = Utils.getLevenshteinDistance(possibleVal, fixedValue);
+                            int dist = Utils.getLevenshteinDistance(possibleVal, harmonizedValue);
                             if (dist < minDist) {
                                 closest = possibleVal;
@@ -555 +563 @@
                             }
                         }
-                        fixedValue = null;
                         if (minDist <= MAX_LEVENSHTEIN_DISTANCE && maxPresetValueLen > MAX_LEVENSHTEIN_DISTANCE) {
                             if (fixVals.size() < 2) {