Ticket #19871: 19871.patch

File 19871.patch, 1.8 KB (added by GerdP, 5 years ago)
  • src/org/openstreetmap/josm/data/validation/tests/TurnrestrictionTest.java

     
    256256                        .build());
    257257                return;
    258258            }
    259             final Way viaPseudoWay = new Way();
    260             viaPseudoWay.addNode(viaNode);
    261             checkIfConnected(r, fromWay, viaPseudoWay,
    262                     tr("The \"from\" way does not start or end at a \"via\" node."), FROM_VIA_NODE);
    263             checkIfConnected(r, viaPseudoWay, toWay,
    264                     tr("The \"to\" way does not start or end at a \"via\" node."), TO_VIA_NODE);
     259            if (!fromWay.isFirstLastNode(viaNode)) {
     260                errors.add(TestError.builder(this, Severity.WARNING, FROM_VIA_NODE)
     261                        .message(tr("The \"from\" way does not start or end at a \"via\" node."))
     262                        .primitives(r, fromWay, viaNode)
     263                        .highlight(fromWay, viaNode)
     264                        .build());
     265            }
     266            if (!toWay.isFirstLastNode(viaNode)) {
     267                errors.add(TestError.builder(this, Severity.WARNING, TO_VIA_NODE)
     268                        .message(tr("The \"to\" way does not start or end at a \"via\" node."))
     269                        .primitives(r, toWay, viaNode)
     270                        .highlight(toWay, viaNode)
     271                        .build());
     272            }
    265273        } else {
    266274            if (isFullOneway(toWay) && ((Way) via.get(via.size() - 1)).isFirstLastNode(toWay.lastNode(true))) {
    267275                errors.add(TestError.builder(this, Severity.WARNING, SUPERFLUOUS)