Modify

Opened 7 years ago

Closed 7 years ago

#15452 closed enhancement (fixed)

[patch] More details are needed for "Tag value longer than allowed"

Reported by: naoliv Owned by: team
Priority: normal Milestone: 18.02
Component: Core validator Version:
Keywords: Cc:

Description

Have any object with a very long fixme tag, for example and validate it.
JOSM says: Tag value longer than allowed - Key 'fixme' invalid

But the user probably doesn't know the maximum number of allowed characters nor he knows how much chars he need to remove to have a valid value again.

Could JOSM, if possible, display length(value)/255 chars (or something like that) together with the message?

JOSM:

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2017-10-17 21:49:06 +0200 (Tue, 17 Oct 2017)
Revision:13016
Build-Date:2017-10-18 01:31:45
URL:http://josm.openstreetmap.de/svn/trunk

Identification: JOSM/1.5 (13016 en) Linux Debian GNU/Linux unstable (sid)
Memory Usage: 444 MB / 3072 MB (190 MB allocated, but free)
Java version: 9-Debian+0-9b181-4, Oracle Corporation, OpenJDK 64-Bit Server VM
Screen: :0.0 1920x1080
Maximum Screen Size: 1920x1080
Java package: openjdk-9-jre:amd64-9~b181-4
Java ATK Wrapper package: libatk-wrapper-java:all-0.33.3-13
VM arguments: [--illegal-access=warn, --add-modules=java.se.ee, -Dawt.useSystemAAFontSettings=on]
Program arguments: [--language=en]
Dataset consistency test: No problems found

Attachments (1)

TagChecker.java.patch (1.3 KB ) - added by Klumbumbus 7 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 by mmd, 7 years ago

(side note: 255 are really characters, not bytes - which is important in the context of utf-8 multibyte chars)

by Klumbumbus, 7 years ago

Attachment: TagChecker.java.patch added

comment:2 by Klumbumbus, 7 years ago

Milestone: 18.02
Summary: More details are needed for "Tag value longer than allowed"[patch] More details are needed for "Tag value longer than allowed"

Patch attached. Please review.

comment:3 by Don-vip, 7 years ago

"255" should be parameterized in case the limit changes or we want to reuse the translated string for something else.

comment:4 by Don-vip, 7 years ago

Resolution: fixed
Status: newclosed

In 13414/josm:

fix #15452 - More details are needed for "Tag value longer than allowed" (patch by Klumbumbus, modified)

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.