Modify

Opened 9 years ago

Closed 9 years ago

Last modified 19 months ago

#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 Klumbumbus, 9 years ago

This is because payment:credit_cards=yes is not helpful information. You should use payment:mastercard=yes or payment: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.

comment:2 by mdk, 9 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 :-)

in reply to:  2 ; comment:3 by Klumbumbus, 9 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.

Last edited 9 years ago by Klumbumbus (previous) (diff)

in reply to:  3 comment:4 by anonymous, 9 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:5 by mdk, 9 years ago

Sorry last comment was mine.

comment:6 by Klumbumbus, 9 years ago

Milestone: 15.06

comment:7 by Don-vip, 9 years ago

Milestone: 15.0615.07

skip milestone 15.06

comment:8 by Klumbumbus, 9 years ago

Resolution: fixed
Status: newclosed

In 8546/josm:

fix #11499 - add warning for inaccurate payment values.

comment:9 by Don-vip, 9 years ago

Milestone: 15.0715.08

Milestone renamed

comment:10 by simon04, 3 years ago

In 17851/josm:

fix #20817, see #11499 - Validator: "payment:credit_cards=yes is inaccurate" as informational warning (patch by mkoniecz)

comment:11 by Klumbumbus, 20 months ago

In 18557/josm:

fix #22367, see #11499 - Don't warn anymore about payment:credit_cards=yes and payment:debit_cards=yes, Reduce warning of payment:electronic_purses=yes and payment:cryptocurrencies=yes to info level

comment:12 by aceman, 19 months 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:*

in reply to:  12 comment:13 by Klumbumbus, 19 months ago

Replying to aceman:

So there is no need to somehow warn about it or deprecate it.

That's what was done in r18557.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.