Changeset 16154 in josm


Ignore:
Timestamp:
2020-03-16T22:10:18+01:00 (23 months ago)
Author:
Don-vip
Message:

see #17533 - see #18455 - partial revert of r15640, causing untagged nodes to be skipped entirely

Location:
trunk/src/org/openstreetmap/josm/data/validation
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/validation/Test.java

    r16069 r16154  
    116116        public TagTest(String name) {
    117117            super(name);
    118         }
    119 
    120         @Override
    121         public boolean isPrimitiveUsable(OsmPrimitive p) {
    122             return super.isPrimitiveUsable(p) && p.isTagged();
    123118        }
    124119
  • trunk/src/org/openstreetmap/josm/data/validation/tests/ConditionalKeys.java

    r16086 r16154  
    247247    @Override
    248248    public void check(OsmPrimitive p) {
    249         errors.addAll(validatePrimitive(p));
     249        if (p.isTagged()) {
     250            errors.addAll(validatePrimitive(p));
     251        }
    250252    }
    251253}
  • trunk/src/org/openstreetmap/josm/data/validation/tests/OpeningHourTest.java

    r16086 r16154  
    109109    @Override
    110110    public void check(final OsmPrimitive p) {
    111         for (String key : KEYS_TO_CHECK) {
    112             errors.addAll(checkOpeningHourSyntax(key, p.get(key), p, Locale.getDefault()));
     111        if (p.isTagged()) {
     112            for (String key : KEYS_TO_CHECK) {
     113                errors.addAll(checkOpeningHourSyntax(key, p.get(key), p, Locale.getDefault()));
     114            }
    113115        }
    114116    }
  • trunk/src/org/openstreetmap/josm/data/validation/tests/TagChecker.java

    r16072 r16154  
    616616    @Override
    617617    public void check(OsmPrimitive p) {
     618        if (!p.isTagged())
     619            return;
     620
    618621        // Just a collection to know if a primitive has been already marked with error
    619622        MultiMap<OsmPrimitive, String> withErrors = new MultiMap<>();
Note: See TracChangeset for help on using the changeset viewer.