Modify

Opened 3 years ago

Closed 3 years ago

Last modified 2 years ago

#20960 closed enhancement (fixed)

[Patch] Warn about construction=* without or conflicting primary tag

Reported by: skyper Owned by: team
Priority: normal Milestone: 22.03
Component: Core validator Version:
Keywords: template_report construction missing conflicting tag Cc:

Description

Inspired by #17607.

What steps will reproduce the problem?

  1. Have a way with only construction=*
  2. Run Validator

What is the expected result?

A warning about missing primary tag

What happens instead?

No warning

OR

What steps will reproduce the problem?

  1. Have a way with railway=rail and construction=rail
  2. Run Validator

What is the expected result?

A warning about conflicting tags

What happens instead?

No warning

Please provide any additional information below. Attach a screenshot if possible.

Not sure if a autofix can be offered but suggestions can be made.
The four primary tags highway, railway, building and landuse use construction=*, atm.

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2021-06-02 23:14:11 +0200 (Wed, 02 Jun 2021)
Revision:17921
Build-Date:2021-06-03 01:31:07
URL:https://josm.openstreetmap.de/svn/trunk

Attachments (2)

josm_20960.patch (3.0 KB ) - added by skyper 3 years ago.
patch
josm_20960v2.patch (9.6 KB ) - added by skyper 3 years ago.
version 2: does not warn about waterway; rearrange some rules to use ordered placeholder numbers

Download all attachments as: .zip

Change History (13)

comment:1 by skyper, 3 years ago

Owner: changed from team to skyper

by skyper, 3 years ago

Attachment: josm_20960.patch added

patch

comment:2 by skyper, 3 years ago

Milestone: 21.11
Owner: changed from skyper to team
Summary: Warn about construction=* without or conflicting primary tag[Patch] Warn about construction=* without or conflicting primary tag

Please find attached patch which updates/adds the rules for construction=*.
I am not sure if waterway=* might be a fifth primary tag where the combination with construction=* is valid.

I gonna work on #20963 which should be fixed together with this ticket.

comment:3 by skyper, 3 years ago

Milestone: 21.11
Owner: changed from team to skyper

Have to little rework the rule and need to find/create a nice icon for building=construction.

comment:4 by skyper, 3 years ago

Version 2 does not warn about waterway=*

Additionally, I rearranged some rules to have harmonized plus ordered placeholder numbers in the throw*. First step of cleaning up.

by skyper, 3 years ago

Attachment: josm_20960v2.patch added

version 2: does not warn about waterway; rearrange some rules to use ordered placeholder numbers

comment:5 by skyper, 3 years ago

Milestone: 22.03
Owner: changed from skyper to team

#20963 has a patch now.

comment:6 by stoecker, 3 years ago

Resolution: fixed
Status: newclosed

In 18397/josm:

fix #20960 - patch by skyper - validator fixes with construction tag

comment:7 by skyper, 3 years ago

Sorry, I mixed up the order in one case, see #22009.

comment:8 by Famlam, 3 years ago

The latest patch seems to lack [!railway] in this line:

area[construction][!building][!highway][!landuse]:closed,

Example: https://www.openstreetmap.org/way/1052080206 , a platform under construction

in reply to:  8 comment:9 by skyper, 3 years ago

Replying to Famlam:

The latest patch seems to lack [!railway] in this line:

area[construction][!building][!highway][!landuse]:closed,

Example: https://www.openstreetmap.org/way/1052080206 , a platform under construction

See #22039

comment:10 by Klumbumbus, 3 years ago

In 18446/josm:

fix #22009, see #20960 - Fix incorrect warning message *:lanes without highway (instead of lanes) (patch by skyper)

comment:11 by Klumbumbus, 2 years ago

In 18463/josm:

fix #22039, see #20960 - Remove warning about construction without highway/railway/waterway/building/landuse as it produces too much false positives

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.