Modify

Opened 10 years ago

Closed 7 years ago

#10179 closed defect (fixed)

Invalid "Key 'name ' invalid. - Invalid property key"

Reported by: mkoniecz Owned by: team
Priority: normal Milestone: 18.04
Component: Core validator Version:
Keywords: Cc:

Description (last modified by mkoniecz)

Name seems to be perfectly valid key for hotel.

https://api.openstreetmap.org/node/1940976070

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2014-06-20 01:36:37
Last Changed Author: simon04
Revision: 7254
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2014-06-19 11:09:04 +0200 (Thu, 19 Jun 2014)
Last Changed Rev: 7254

Identification: JOSM/1.5 (7254 en) Windows 7 32-Bit
Memory Usage: 243 MB / 247 MB (63 MB allocated, but free)
Java version: 1.7.0_55, Oracle Corporation, Java HotSpot(TM) Client VM
Dataset consistency test: No problems found

Plugin: OpeningHoursEditor (30416)
Plugin: RoadSigns (30495)
Plugin: buildings_tools (30485)
Plugin: continuosDownload (28565)
Plugin: measurement (30416)
Plugin: notes (v0.9.2)
Plugin: turnrestrictions (30454)
Plugin: wikipedia (30449)

Attachments (2)

whitespace.png (15.1 KB ) - added by Klumbumbus 9 years ago.
whitespaces.osm (444 bytes ) - added by Klumbumbus 9 years ago.

Download all attachments as: .zip

Change History (14)

comment:1 by mkoniecz, 10 years ago

Description: modified (diff)

comment:2 by AlfonZ, 10 years ago

There is a space at the end of name key that is causing the trouble.

in reply to:  2 comment:3 by skyper, 10 years ago

Replying to AlfonZ:

There is a space at the end of name key that is causing the trouble.

Well, the warning is correct but JOSM could automatically delete white spaces at the end of the key.

comment:4 by mkoniecz, 10 years ago

It is autofixable.

comment:5 by skyper, 10 years ago

Resolution: duplicate
Status: newclosed

Closed as duplicate of #10180.

comment:6 by mkoniecz, 10 years ago

Not a duplicate, this is about keys, second is about values.

in reply to:  6 comment:7 by skyper, 10 years ago

Replying to Bulwersator:

Not a duplicate, this is about keys, second is about values.

Ok, but it is the same issue and the check for keys is needed but there is no problem with the info about invalid property in key.

comment:8 by Don-vip, 10 years ago

Resolution: duplicate
Status: closedreopened

I agree this is not clear.

comment:9 by skyper, 10 years ago

Still think a similar test like the one for values with an auto-fix option is useful. The autofix on upload only affects modified objects.

comment:10 by Klumbumbus, 9 years ago

I read this and #10180. What exactly is still to do?
See attached example file. Validator gives usefull warnings and no infos.


The only point which could be improved is the wording of the first warning. We could add Value of at the beginning, becasue not the key is wrong but the value.

by Klumbumbus, 9 years ago

Attachment: whitespace.png added

by Klumbumbus, 9 years ago

Attachment: whitespaces.osm added

comment:11 by Klumbumbus, 7 years ago

In 13584/josm:

see #10179 - improve validator warning text. Tag fits better than key as for some warnings the value is the culprit.

comment:12 by Klumbumbus, 7 years ago

Milestone: 18.04
Resolution: fixed
Status: reopenedclosed

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.