Modify ↓
Opened 8 years ago
Last modified 8 years ago
#14804 new defect
Unify case handling for boolean tag values
Reported by: | michael2402 | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Core | Version: | |
Keywords: | Cc: |
Description
We currently have:
OsmUtils#getOsmBoolean()
accepting all cases fortrue
,yes
,1
,on
OsmUtils#isTrue()
,OsmUtils#isFalse()
accepting lower casetrue
,yes
,1
,on
OsmPrimitive#isKeyTrue(key)
accepting lower casetrue
,yes
,1
,on
OsmPrimitive#hasAreaTags()
accepts onlyyes
Way#isOneway()
accepting all cases fortrue
,yes
,1
,on
and-1
, but notreverse
- Relation map painting respects
oneway=-1
, but notreverse
ReverseWayTagCorrector
only accepting lower case, but accepting-1
andreverse
In tests:
Highway#testWrongRoundabout
: accepting all cases fortrue
,yes
,1
,on
for oneway, but none of the reversed.UntaggedWay
test accepts only noname=yes
That's what I found so far. We should unify this and decide on a common policy here.
Attachments (0)
Note:
See TracTickets
for help on using tickets.
Conclusion: