[patch OK] Issue with bridge preset

[Nakor]

Select a way.
Choose the bridge preset.
Click Apply preset.

ACTUAL RESULT:
Nothing happens

EXPECTED RESULT:
bridge=yes should be added to the way

[AlfonZ]

Selected way has to have some other tag filled. Applying bridge preset to a way with no tags works as expected.

Changeset r3581 changed bridge preset from Key to Combo.
The difference being, that Key's value is being applied always, Combo's default value is being applied only if all items in selection have no tags.
Check's behaviour looks similar to Combo's, Text's to Key's.

There is a comment in the source code "use default only in case it is a totally new entry", so this behaviour might be correct.

Changeset r3481 is related to this as well.

[anonymous]

Using the bridge preset used to set the bridge=yes tag. It is pretty annoying to have to do an extra click to set bridge=yes.

Can we please go back to previous behavior?

[stoecker]

Oneclick presets has a bridge presets for this.

[Nakor]

I do not understand why empty ways get bridge=yes set automatically but not others.

In the US we had the TIGER import. TIGER import does not know anything about bridges, my typical scenario is split a way where there is a bridge, set it has a bridge and depending on the situation add a layer tag.

Before I would open the bridge preset dialog, set the layer if needed and apply. Now I need to additionally set bridge=yes. Using the one-click preset is not a solution as I would need to go back and change the layer if needed.

So can we either:

1) have the default bridge=yes apply to ALL ways (empty or not)

2) keep the current behavior by default and have an option to force behavior described in 1)

Thanks,

N.

[stoecker]

This is a security measure, so calling a preset does not overwrite data unwanted. Maybe we need to find a solution to specially enable it. I will have a look, but it may take time. As said, you can use the oneclick preset. It opens no dialog at all, but only sets bridge=yes.

[Nakor]

What about making the security measure smarter i.e. does not overwrite an existing tag (i.e. do nothing if there is a bridge=no for instance) but add a non-existing key (i.e. set bridge=yes if there is no bridge key)?

Issue with one-click is I still have to go back edit the layer (and it also forces me to go through the menu unless there is a way to put it in the toolbar).

[AlfonZ]

Enabled filling of default value for Combo

[AlfonZ]

It looks to me that the security measures you are talking about have already been added in r591 and r949.
Default value (in the case of bridge bridge=yes) will be applied only if all selected ways have no bridge tag.

If there is existing bridge tag, its value will show up in preset dialog, default value will not overwrite it.

In the case of selecting two or more ways with different bridge values, <different> will show up in preset dialog and user can leave it as it is or change it.

Is there a case I missed?

Please see attached patch.

[stoecker]

Sorry, but there is a reason why the values aren't taken always. They would apply always when calling a dialog and this is unwanted. This is only wanted for the key settings, not for additional or optional entries.

I don't understand what troubles you have with oneclick. This is a normal presets file, so it works exactly like all the other presets including the possibility to add it to the toolbar.

[AlfonZ]

Geometries for example how default works for Combos

[AlfonZ]

I still don't see why filling default value in preset dialog (in case of no prior values) is a bad thing. It will show up in dialog and user can review it. It won't be set without user supervision.

Please try following:
Open attached DefaultCombo-Address.osm

1. Select first way.
2. Open Annotation/Addresses preset, fill it up (e.g. HouseNumber=1, StreetName=Foo, CityName=Bar, CountryCode=QQ) and apply.
3. Select second way. It has no tags.
4. Open Annotation/Addresses preset. StreetName, CityName and CountryCode is already filled up due to use_last_as_default="true", which is as intended.
5. Select third way. It has some building tags.
6. Open Annotation/Addresses preset. StreetName, CityName is already filled, CountryCode is not. The difference being that StreetName and CityName are Texts, CountryCode is Combo.

As I understand it, the intent was that CountryCode will be filled up as well. Otherwise you'll be forced to select correct country for each address you entered; or use other tricks to accomplish that.

My other point is, that there is difference between how default works for Text and Combo. They should be unified, IMHO. Default for Checks behaves even more differently, but they are more complicated.

[Nakor]

I will explain one more time the issue with one-click: I use the one click, then I need to edit the layer field; with the previous behavior I would just open, the bridge preset, fill layer if needed and apply.
And this is not talking about the current inconsistency between the bridge preset and the tunnel preset.

Is there any reason why the following proposition cannot be applied (besides having someone coding it)?

not overwrite an existing tag (i.e. do nothing if there is a bridge=no for instance) but add a non-existing key (i.e. set bridge=yes if there is no bridge key)

[stoecker]

Due to changes in the past the possibilities have been increased (i.e. the use_last_ad_default you mention, which is also potentially dangerous). The combo boxes are used for lots of presets and for lots of optional parameters and they should not be set when not really checked. When activate them always, then we add lots of unnecessary and also most often wrong data to the database. I don't say this topic shouldn't be fixed, but to output the values of combo boxes always is not the way to go.

Probably way to go is have a "set_always" tag.

[AlfonZ]

patch for filling default values controlled by property

[AlfonZ]

First, some statistics from the internal presets.
There is total of 410 Combos that can be summed up to 128 unique Combos. 117 out of 129 have default="", only 11 have non-empty default or use_last_as_default:

• Highways/Streets/Motorway,Motorway Link,Trunk,Trunk Link/lanes
• Highways/Streets/Bridge/bridge
• Highways/Streets/Bridge/layer
• Highways/Streets/Tunnel/layer
• Transport/Car,Motorcycle/Parking/parking
• Sports/Sport (Ball)/Golf/leisure
• Man Made/Power/Power Line/power
• Facilities/Tourism/Information Board/board_type
• Facilities/Health/Pharmacy/dispensing
• Annotation/Addresses/addr:country
• Annotation/Address Interpolation/addr:interpolation

Anyways, I have prepared another patch, see attached DefaultValuesFilling-property.diff

It unifies when is the default value applied in Texts, Combos and Checks, controlled by taggingpreset.fill-default-for-tagged-primitives property (feel free to change the name to more appropriate).

When the property value is true, check for (other) tags in selected item(s) is skipped.
This changes the current behaviour for Texts and Checks, which were using default value even for tagged items (see Address example above).

[Nakor]

Can we get this fixed?

I see that for instance the prking preset automatically applies parking=surface. How is this different from bridge automatically applying bridge=yes?

Thanks in advance

[stoecker]

I had not yet the time to test your patch. Applying default values for text and check probably is a bug and it sounds as you patch fixes it. But as said I had no time for it yet.

[Nakor]

Patch updated against 4018

[Nakor]

I have been using since without any problem so far. I attached a version updated against 4018. Please commit.

[Fabi2]

See #5903 for the real reason, why this does not work any longer. It was a side effect of fixging the escaping of <combo> (#5892).

[Fabi2]

Sorry does not see, that this was reported long before.

[stoecker]

In [4050/josm]:

fix #5564 - patch by Nakor - unify handling of default values in presets