Modify

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#21268 closed enhancement (fixed)

[Patch] Warn about `levels=*` and ignore it in "Presets do not contain property key"

Reported by: skyper Owned by: team
Priority: minor Milestone: 21.08
Component: Core validator Version: latest
Keywords: template_report levels Cc:

Description

What steps will reproduce the problem?

  1. Have a closed way with levels=* among the tags
  2. Run validator

What is the expected result?

One warning about deprecated levels=*

What happens instead?

Two warnings, one about deprecation and additionally Presets do not contain property key - Key 'levels' not in presets

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

levels=* needs to be ignored on the preset test.

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2021-08-26 16:42:55 +0200 (Thu, 26 Aug 2021)
Revision:18182
Build-Date:2021-08-27 01:31:01
URL:https://josm.openstreetmap.de/svn/trunk

Attachments (2)

josm_21268.patch (474 bytes ) - added by skyper 3 years ago.
patch file
josm_21268_v2.patch (1.2 KB ) - added by skyper 3 years ago.
version 2, checks on all building parts and adds general rule

Download all attachments as: .zip

Change History (12)

comment:1 by skyper, 3 years ago

Type: defectenhancement

by skyper, 3 years ago

Attachment: josm_21268.patch added

patch file

comment:2 by skyper, 3 years ago

Summary: Ignore `levels=*` in "Presets do not contain property key"[Patch] Ignore `levels=*` in "Presets do not contain property key"

Please, find attached patch which adds levels to the list of deprecated tags.

comment:3 by skyper, 3 years ago

Milestone: 21.08

comment:4 by skyper, 3 years ago

Milestone: 21.08
Summary: [Patch] Ignore `levels=*` in "Presets do not contain property key"[WIP-Patch] Ignore `levels=*` in "Presets do not contain property key"

Ah, sorry, the deprecated warning only works with building=*, so a more general warning about level=* is needed.

*[levels][!building][!building:part] {
  throwWarning: tr("{0} is unusual, maybe {1} or {2} is meant", "{0.key}", "level", "building:levels");
Last edited 3 years ago by skyper (previous) (diff)

comment:5 by skyper, 3 years ago

The existing rule for building should be changed to warn on all building:part and not only on building:part=yes.

by skyper, 3 years ago

Attachment: josm_21268_v2.patch added

version 2, checks on all building parts and adds general rule

comment:6 by skyper, 3 years ago

Summary: [WIP-Patch] Ignore `levels=*` in "Presets do not contain property key"[Patch] Warn about `levels=*` and Ignore it in "Presets do not contain property key"

I've attached version 2 which checks on all building:part and adds a general warning about levels=*.

comment:7 by Don-vip, 3 years ago

Milestone: 21.08

comment:8 by skyper, 3 years ago

Summary: [Patch] Warn about `levels=*` and Ignore it in "Presets do not contain property key"[Patch] Warn about `levels=*` and ignore it in "Presets do not contain property key"

comment:9 by Don-vip, 3 years ago

Resolution: fixed
Status: newclosed

In 18188/josm:

fix #21268 - Warn about levels=* and ignore it in "Presets do not contain property key" (patch by skyper)

comment:10 by skyper, 3 years ago

Thanks for applying.

I did not add a milestone on purpose or should have better set 21.9 as this introduced a new string to translate.

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.