Changeset 35811 in osm for applications/editors/josm/plugins/turnlanes/src

Timestamp:

2021-08-25T20:11:20+02:00 (3 years ago)

Author:

Don-vip

Message:

fix #20385 - check for empty ways

File:

• applications/editors/josm/plugins/turnlanes/src/org/openstreetmap/josm/plugins/turnlanes/model/Validator.java (modified) (2 diffs)

applications/editors/josm/plugins/turnlanes/src/org/openstreetmap/josm/plugins/turnlanes/model/Validator.java

@@ -296 +296 @@
             final Way to = TurnlanesUtils.getMemberWay(r, Constants.TURN_ROLE_TO);
 
-            if (from.firstNode().equals(from.lastNode())) {
-                issues.add(Issue.newError(r, from, "The from-way both starts as well as ends at the via-node."));
-            }
-            if (to.firstNode().equals(to.lastNode())) {
-                issues.add(Issue.newError(r, to, "The to-way both starts as well as ends at the via-node."));
+            if (from.isEmpty()) {
+                issues.add(Issue.newError(r, from, tr("The from-way is empty.")));
+            } else if (from.firstNode().equals(from.lastNode())) {
+                issues.add(Issue.newError(r, from, tr("The from-way both starts as well as ends at the via-node.")));
+            }
+            if (to.isEmpty()) {
+                issues.add(Issue.newError(r, from, tr("The to-way is empty.")));
+            } else if (to.firstNode().equals(to.lastNode())) {
+                issues.add(Issue.newError(r, to, tr("The to-way both starts as well as ends at the via-node.")));
             }
             if (!issues.isEmpty()) {
@@ -327 +331 @@
 
                 if (!from.isFirstLastNode(via)) {
-                    issues.add(Issue.newError(r, from, "The from-way does not start or end at the via-node."));
+                    issues.add(Issue.newError(r, from, tr("The from-way does not start or end at the via-node.")));
                 }
                 if (!to.isFirstLastNode(via)) {
-                    issues.add(Issue.newError(r, to, "The to-way does not start or end at the via-node."));
+                    issues.add(Issue.newError(r, to, tr("The to-way does not start or end at the via-node.")));
                 }