Modify

Opened 5 years ago

Closed 5 years ago

#19361 closed defect (fixed)

Preset MappingSerbia

Reported by: Klumbumbus Owned by: Pedja
Priority: normal Milestone:
Component: External preset Version:
Keywords: Cc:

Description

Hi,
your preset https://josm.openstreetmap.de/wiki/PresetsSource?action=diff&version=133 has errors.
For <check /> only the values "on" and "off" are valid, not "yes" and "no".
Also please note that a check has three states: on, off and unset. It is not usefull to set e.g. tunnel=no on every highway=* section that is not a tunnel. So you should remove all these default="no".

Attachments (0)

Change History (11)

comment:1 by anonymous, 5 years ago

Corrected

comment:2 by Pedja, 5 years ago

Corrected

comment:3 by Klumbumbus, 5 years ago

default="yes" is still in

comment:4 by Pedja, 5 years ago

I guess yes is now invalid value?

Last edited 5 years ago by Pedja (previous) (diff)

in reply to:  description comment:5 by Klumbumbus, 5 years ago

Replying to Klumbumbus:

For <check /> only the values "on" and "off" are valid, not "yes" and "no".

comment:7 by anonymous, 5 years ago

Sorry, GitLab is a bit confusing. I think it is visible now.

comment:8 by Klumbumbus, 5 years ago

Next error is empty groups like in line 427-428.
Please test after you make changes. This version and the version before both returned an error when adding the preset to JOSM.

Last edited 5 years ago by Klumbumbus (previous) (diff)

comment:9 by simon04, 5 years ago

You might want to run the XML schema validation locally (here shown using xmllint from http://www.xmlsoft.org/):

$ curl -LO https://gitlab.com/osm-serbia/mappingserbia-josm-presets/raw/master/MappingSerbia.xml
$ curl -LO https://josm.openstreetmap.de/svn/trunk/resources/data/tagging-preset.xsd
$ xmllint --noout --schema tagging-preset.xsd MappingSerbia.xml
MappingSerbia.xml:427: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml:430: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml:433: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml:438: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml:441: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml:444: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml:719: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml:722: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml:731: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml:733: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml:735: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml:737: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml:740: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml:742: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml:744: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml:746: element group: Schemas validity error : Element '{http://josm.openstreetmap.de/tagging-preset-1.0}group': Missing child element(s). Expected is one of ( {http://josm.openstreetmap.de/tagging-preset-1.0}chunk, {http://josm.openstreetmap.de/tagging-preset-1.0}group, {http://josm.openstreetmap.de/tagging-preset-1.0}item, {http://josm.openstreetmap.de/tagging-preset-1.0}separator ).
MappingSerbia.xml fails to validate

comment:10 by anonymous, 5 years ago

Hmm, I uploaded wrong file...

comment:11 by Klumbumbus, 5 years ago

Resolution: fixed
Status: assignedclosed

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Pedja.
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.