#11499 closed defect (fixed)
Wrong validator result: Value 'yes' for key 'payment:credit_cards' not in preset.
Reported by: | mdk | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | 15.08 |
Component: | Core validator | Version: | latest |
Keywords: | Cc: |
Description
The node 1589860488 contains three payment types:
payment:cash=yes payment:credit_cards=yes payment:telephone_cards=yes
But only for payment:credit_cards the value 'yes' is unknown (the key itself is known)!
Attachments (0)
Change History (13)
comment:1 by , 10 years ago
follow-up: 3 comment:2 by , 10 years ago
Is there a possibility to give the mapper a more helpful hint than Value 'yes' for key 'payment:credit_cards' not in preset.
? Somethink like: Use more detailed 'payment:<credit card>' keys instead!
and perhaps - let me dream - a link to the wiki :-)
follow-up: 4 comment:3 by , 10 years ago
Replying to mdk:
Is there a possibility to give the mapper a more helpful hint than
Value 'yes' for key 'payment:credit_cards' not in preset.
? Somethink like:Use more detailed 'payment:<credit card>' keys instead!
and perhaps - let me dream - a link to the wiki :-)
This is the default info which is reported for all keys, which are not in the preset. Remember this is only info level, not warning. However we could add an additional warning for payment:credit_cards=yes
. But the above info message will stay there.
comment:4 by , 10 years ago
Replying to Klumbumbus:
This is the default info which is reported for all keys, which are not in the preset. Remember this is only info level, not warning. However we could add an additional warning for
payment:credit_cards=yes
. But the above info message will stay there.
I think this would be helpful. If the mapper fixes the warning, the "cryptic" info will also disapear - mission accomplished.
comment:6 by , 10 years ago
Milestone: | → 15.06 |
---|
follow-up: 13 comment:12 by , 2 years ago
Well, according to the wiki payment:credit_cards=yes is a proper value meaning "common cards used in the region". So there is no need to somehow warn about it or deprecate it.
https://wiki.openstreetmap.org/wiki/Key:payment:*
This is because
payment:credit_cards=yes
is not helpful information. You should usepayment:mastercard=yes
orpayment:visa=yes
, or ...You can use
payment:credit_cards=no
to indicate that there are no credit cards accepted at all.See http://wiki.openstreetmap.org/wiki/Key:payment and the JOSM preset.