Opened 8 years ago

Last modified 8 years ago

#12133 closed enhancement

warn about name=fixme and its variations — at Version 1

Reported by: Klumbumbus Owned by: team
Priority: normal Milestone: 15.12
Component: Core validator Version:
Keywords: Cc:

Description (last modified by Klumbumbus)

name=fixme, name=Fixme and name=FIXME are used 1214 times, see taginfo comparison

the count of all variations of the fixme word is even 1315, see http://overpass-turbo.eu/s/cSb

We should warn about this. Replacing it with fixme=name is not good, because the error can be detected by error tools even wthout this additional tag, see Key:fixme

Here is the test:

*[name][name=~/^(?i)fixme$/] {
  throwWarning: tr("Wrong usage of {0} tag. Remove {1}, because it is clear that the name is missing even without an additional tag.", "{0.key}", "{0.tag}");
  fixRemove: "name";
  assertMatch: "node name=fixme";
  assertMatch: "node name=Fixme";
  assertMatch: "node name=FIXME";
  assertNoMatch: "node name=valid name";
}

Change History (1)

comment:1 by Klumbumbus, 8 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.