Changeset 8180 in josm

Timestamp:

2015-04-09T21:25:17+02:00 (9 years ago)

Author:

stoecker

Message:

fix #10977 - reverse way handling of left/right together with oneway

Location:

trunk

Files:

• src/org/openstreetmap/josm/corrector/ReverseWayTagCorrector.java (modified) (2 diffs)
• test/unit/org/openstreetmap/josm/corrector/ReverseWayTagCorrectorTest.java (modified) (1 diff)

trunk/src/org/openstreetmap/josm/corrector/ReverseWayTagCorrector.java

@@ -123 +123 @@
                     newValue = OsmUtils.reverseval;
                 }
+                for (StringSwitcher prefixSuffixSwitcher : stringSwitchers) {
+                    newKey = prefixSuffixSwitcher.apply(key);
+                    if (!key.equals(newKey)) {
+                        break;
+                    }
+                }
             } else if (key.startsWith("incline") || key.endsWith("incline")
                     || key.startsWith("direction") || key.endsWith("direction")) {
@@ -132 +138 @@
                 // Change key but not left/right value (fix #8518)
                 newKey = FORWARD_BACKWARD.apply(key);
-
             } else if (!ignoreKeyForCorrection(key)) {
                 for (StringSwitcher prefixSuffixSwitcher : stringSwitchers) {

trunk/test/unit/org/openstreetmap/josm/corrector/ReverseWayTagCorrectorTest.java

@@ -38 +38 @@
             assertSwitch(new Tag(k, "something"), new Tag(k, "something"));
         }
+        // :left/:right with oneway (see #10977)
+        assertSwitch(new Tag("cycleway:left:oneway", "-1"), new Tag("cycleway:right:oneway", "yes"));
         // :forward/:backward (see #8518)
         assertSwitch(new Tag("turn:forward", "right"), new Tag("turn:backward", "right"));