Modify

Opened 11 years ago

Closed 10 years ago

Last modified 7 years ago

#9106 closed enhancement (fixed)

"building_type without building" warning is strange

Reported by: aceman Owned by: team
Priority: normal Milestone: 13.11 (hotfix)
Component: Core validator Version: latest
Keywords: building_type building Cc:

Description

In the josm.po file there seems to be a "building_type without building" check in the validator. However there is no mention of this tag on the wiki. There is a "building:type", which just redirects to "building".

So I think check should be changed to "building:type without building" and a new check for "building_type=V is undefined, use building=V".
There is also another key that uses the ":type" schema and that is "generator". So you may want to add a new class of checks for these (similarly to *_type).

Also I would propose to make a generic check for "K:V without K", where K is any key.

I propose all this based on reading the josm.po file. If any of these checks are already implemented then I apologize for the duplicate ticket.

Attachments (0)

Change History (9)

in reply to:  description comment:1 by Don-vip, 11 years ago

Replying to aceman:

Also I would propose to make a generic check for "K:V without K", where K is any key.

Doesn't work for all tags, see generator as example.

comment:2 by aceman, 11 years ago

Summary: "building_type without building""building_type without building" warning is strange

Yeah right, those are cases of "K:V=* without X=K". Currently you have "K_type=V without X=K" checks, but only for specific values of X and K. This would probably need data analysis if the rules can be generalized in some way.

comment:3 by aceman, 11 years ago

What about "K:Y=* AND (X=K OR K=*)" then OK, otherwise warn?

comment:4 by skyper, 10 years ago

Keywords: building_type building added

comment:5 by skyper, 10 years ago

Some numbers for building type (taginfo).

We should warn about both types. There are no wiki pages for both tags (building:type=* redirects to building=*).

comment:6 by skyper, 10 years ago

My patch for #8966 disables the check for building_types.

comment:7 by Don-vip, 10 years ago

Resolution: fixed
Status: newclosed

In 6384/josm:

fix #8966, fix #9106, fix #9289, fix #9301, fix #9302 - validator/tagchecker: fix various tests (modified patch by skyper)

comment:8 by Don-vip, 10 years ago

Milestone: 13.11 (6388 hotfix)

comment:9 by stoecker, 7 years ago

Milestone: 13.11 (6388 hotfix)13.11 (hotfix)

Milestone renamed

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.