Error when installing preset (chunk containing only list_entry items)

[malcolmh]

I have a preset that produces an error message when I try to install it:

It does not seem to like the fact that I have chunks that contain only list_entry items. However, if I click on "Yes" to accept the install, the preset then works fine, with the chunk invocations working as expected.

[bastiK]

list_entry is normally inside a combo element, so it is a little unusual to put a couple of list_entrys in a chunk and not the entire combo. We could allow this, but then the schema file would no longer be able to catch all the syntax errors in the preset xml.

[skyper]

Replying to bastiK:

list_entry is normally inside a combo element, so it is a little unusual to put a couple of list_entrys in a chunk and not the entire combo. We could allow this, but then the schema file would no longer be able to catch all the syntax errors in the preset xml.

used in combo and multiselection

• on one hand, it is useful but will clutter the presets file even more
• on the other hand a complete syntax check saves quite some time

As copy & paste works on all systems, I am in favour of the syntax check and excluding single "value" attribute definitions from chunk.

[malcolmh]

The reason that I need chunks with only "list_entry" items is that I have many different keys that share a common set of values, so I want to define these once and reference them in combos and multiselects with different keys. The "chunk" facility appears on the face of it to be a generalised macro definition entity, so it should only be the expanded source that is checked for validity in the preset context, i.e. don't check the chunks themselves, but check that all the invocations of the chunks result in valid preset structures.

[Don-vip]

In 7503/josm:

fix #10478 - update preset XSD to allow list_entry in chunk (JOSM already supports it)

[malcolmh]

The latest revision still causes the error at install time. Try installing ​https://raw.githubusercontent.com/OpenSeaMap/josm/master/INT-1-preset.xml as a test case.