[patch] default preset tourism information translations, route_marker

[frankthetankk]

I am new to creating tickets.

I added details to the 'tourism' presets with 'information' key. Added translation hrefs to key:information. Added label text in english to reflect what is written on the wiki. Under board_type I added geology and qr, as these are starting to be used in the future. Added key 'ref' to a couple categories. Added new item route_marker and moved it above "Other Information Point".

[frankthetankk]

difference file

[skyper]

Some more (minor) technical points:

• please, be careful with white space at the beginning of a line. The indent should be four white spaces.
• adding different named wiki pages for different languages does not work as they will be removed automatically.
• after the introduction of chunk the defaultpresets still need some adoptions, e.g. in advance of adding common keys like ref or operator please have a look at the chunk section on top of the presets.

The missing wiki links where already included but it is always worth checking them.
I do not like the value board_type=qr as

1. it is an abbreviation
2. low numbers
3. wonder what kind of board this should be. Is it about QR-codes or is it a board with only plenty of QR-codes ? Does this value really fit here ?

What I really like and appreciate are the label.

[frankthetankk]

Replying to skyper:

Some more (minor) technical points:

• please, be careful with white space at the beginning of a line. The indent should be four white spaces.

Okay I see. I believe it used an auto indent after pressing enter on the previous line in Editplus. I will use 'space' instead of 'tab'. Which program do you developers use to edit presets or xml files?

• adding different named wiki pages for different languages does not work as they will be removed automatically.

I believe you're referring to line 3240 of the .diff file. I understand what you're saying. I will look into it. After a quick check. It seems the url on the wiki for information=map needs updating. While on the page ​http://wiki.openstreetmap.org/wiki/Key:information and clicking on map for ​http://wiki.openstreetmap.org/wiki/Tag:information%3Dmap I feel the "%3D" should be a "=" so that it redirects to ​http://wiki.openstreetmap.org/wiki/Tag:information=map

• after the introduction of chunk the defaultpresets still need some adoptions, e.g. in advance of adding common keys like ref or operator please have a look at the chunk section on top of the presets.

I remember I tried to figure on the 'chunk' section of code before, but I don't fully understand it yet. In trying to understand your point, are you saying you would rather have for example <reference ref="name_ref" /> instead of <text />?

The missing wiki links where already included but it is always worth checking them.

What do you mean exactly by the wiki links were already included?

I do not like the value board_type=qr as

1. it is an abbreviation
2. low numbers
3. wonder what kind of board this should be. Is it about QR-codes or is it a board with only plenty of QR-codes ? Does this value really fit here ?

What I really like and appreciate are the label.

I see what you mean by qr. Those points outweigh what I figured would be preparing for more tags like this. Do you feel
board_type=geology is still appropriate?

Thank you, and I will work on the patch some more.

[skyper]

Replying to frankthetankk:

Replying to skyper:

• adding different named wiki pages for different languages does not work as they will be removed automatically.

I believe you're referring to line 3240 of the .diff file. I understand what you're saying. I will look into it. After a quick check. It seems the url on the wiki for information=map needs updating. While on the page ​http://wiki.openstreetmap.org/wiki/Key:information and clicking on map for ​http://wiki.openstreetmap.org/wiki/Tag:information%3Dmap I feel the "%3D" should be a "=" so that it redirects to ​http://wiki.openstreetmap.org/wiki/Tag:information=map

It is the same but = is better human readable and utf-8 as stated in the header of the .xml file.

• after the introduction of chunk the defaultpresets still need some adoptions, e.g. in advance of adding common keys like ref or operator please have a look at the chunk section on top of the presets.

I remember I tried to figure on the 'chunk' section of code before, but I don't fully understand it yet. In trying to understand your point, are you saying you would rather have for example <reference ref="name_ref" /> instead of <text />?

Exactly. Chunk is a part which can be included and makes maintaining much easier. Take surface, for example, where we only have to change one line to add a new value instead of 20 or more. It also helps to keep this huge file smaller.

The missing wiki links where already included but it is always worth checking them.

What do you mean exactly by the wiki links were already included?

Oh, sorry, your are right they are still missing

I see what you mean by qr. Those points outweigh what I figured would be preparing for more tags like this. Do you feel
board_type=geology is still appropriate?

+1 for geology but I am kind of subjective here as I already added some.

Thank you, and I will work on the patch some more.

Nice, go ahead and include #10302, if you like to.

[frankthetankk]

Replying to skyper:

I see what you mean by qr. Those points outweigh what I figured would be preparing for more tags like this. Do you feel
board_type=geology is still appropriate?

+1 for geology but I am kind of subjective here as I already added some.

I don't want to step on anyones toes if you guys don't feel it should be in.

Thank you, and I will work on the patch some more.

Nice, go ahead and include #10302, if you like to.

Okay I want to include #10302. Not sure how yet.

[frankthetankk]

tourism/information internal presets

[frankthetankk]

Updated the patch. label text added, added board_type=geology, <reference ref="name_ref_operator" />, missing href links, route_marker.

I want to include #10302 but I dont know how.

[skyper]

Replying to frankthetankk:

Updated the patch. label text added, added board_type=geology, <reference ref="name_ref_operator" />, missing href links, route_marker.

Nice.

I want to include #10302 but I dont know how.

You almost did, if you would have added the link with = instead of %3D.

[frankthetankk]

tourism/information internal presets

[frankthetankk]

Okay I added = instead. How'd I do?

[skyper]

Ticket #10302 has been marked as a duplicate of this ticket.

[bastiK]

I don't feel like route_marker should be included at the moment, as it is poorly documented and usage is low.

The rest is fine, could you please add an updated patch without route_marker?

[frankthetankk]

information keyvalues presets 7381 patch

[frankthetankk]

Replying to bastiK:

I don't feel like route_marker should be included at the moment, as it is poorly documented and usage is low.

The rest is fine, could you please add an updated patch without route_marker?

I added the patch to presets 7381 information_7381_v4.diff​. I left the ref number on Information Terminal and Audioguide. It seems you already added the beginning part of the patch on #10330

[bastiK]

In 7384/josm:

applied #10303 - default preset tourism information translations (patch by frankthetankk)