Opened 5 years ago

Last modified 5 years ago

#21041 closed defect

waylength not evaluated in preset validator — at Version 1

Reported by: Famlam Owned by: team
Priority: normal Milestone: 21.06
Component: Core Version: latest
Keywords: template_report validator presets Cc: simon04

Description (last modified by Famlam)

What steps will reproduce the problem?

  1. Enable "Run data validator on user input
  2. Add the following mapcss validator rule (to reproduce this problem easily)
    way[highway][waylength()<1] {
            throwWarning: tr("This way is less than 1 m");
    }
    
  3. Select any existing highway (clearly larger than 1 m). I chose way 24511833
  4. Click the preset corresponding to the highway setting (in this case the preset for tertiary roads)

What is the expected result?

Validator in presets dialog knows way length

What happens instead?

Validator in preset dialog doesn't know waylength (but pretends it knows and that it's short ;) )
https://josm.openstreetmap.de/raw-attachment/ticket/21041/Waylength%20error.png

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

This feature was introduced in #19078

Edit: it also triggers if you use [waylength()=0] instead of [waylength()<1] so apparently it thinks the way is exactly 0m long.

URL:https://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2021-06-02 22:03:39 +0200 (Wed, 02 Jun 2021)
Build-Date:2021-06-02 20:11:30
Revision:17919
Relative:URL: ^/trunk

Identification: JOSM/1.5 (17919 nl) Windows 10 64-Bit
OS Build number: Windows 10 Home 2009 (19043)
Memory Usage: 534 MB / 1820 MB (164 MB allocated, but free)
Java version: 1.8.0_291-b10, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Look and Feel: com.sun.java.swing.plaf.windows.WindowsLookAndFeel
Screen: \Display0 1920×1080 (scaling 1.00×1.00)
Maximum Screen Size: 1920×1080
Best cursor sizes: 16×16→32×32, 32×32→32×32
System property file.encoding: Cp1252
System property sun.jnu.encoding: Cp1252
Locale info: nl_NL
Numbers with default locale: 1234567890 -> 1234567890
Dataset consistency test: No problems found

Plugins:
+ OpeningHoursEditor (35640)
+ SimplifyArea (35640)
+ imagery_offset_db (35640)
+ pt_assistant (1ff2e15)
+ reverter (35732)
+ tageditor (35640)
+ turnlanes-tagging (288)
+ undelete (35640)
+ utilsplugin2 (35691)

Tagging presets:
+ http://mijndev.openstreetmap.nl/~allroads/JOSM/Presets/NL-Fiets.zip

Map paint styles:
- https://josm.openstreetmap.de/josmfile?page=Styles/Lane_and_Road_Attributes&zip=1
- https://josm.openstreetmap.de/josmfile?page=Styles/PublicTransport&zip=1
+ %UserProfile%\Documents\tijdelijke bestanden\josm-eigen.mappaint.mapcss
+ https://josm.openstreetmap.de/josmfile?page=Styles/Sidewalks&zip=1

Validator rules:
+ %UserProfile%\Documents\tijdelijke bestanden\josm-eigen.validator.mapcss

Last errors/warnings:
- 00007.469 E: Lokaliseren van afbeelding 'bus.png' mislukt

Change History (2)

by Famlam, 5 years ago

Attachment: Waylength error.png added

comment:1 by Famlam, 5 years ago

Description: modified (diff)
Note: See TracTickets for help on using tickets.