[Patch needs review] Shift-R is broken

[hjart]

Shift-R (copy tags from previous selection) has been broken for a while (I usually run latest). Now when you create a new object (a node or line) and tag it, then create a second object and try to copy tags from the first object to the second using Shift-R, instead tags from whatever object was selected before the creation of the first object is copied. To correctly tag the second object the first object has to be selected a second time before using Shift-R. This makes the function a lot less useful than it used to be.

[maxerickson@…]

Same or similar issue with newly added tags not getting picked up.

[michael2402]

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

[stoecker]

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

[MKnight]

Bug is between 12040 and 12057 somewhere here: https://josm.openstreetmap.de/changeset?sfp_email=&sfph_mail=&reponame=josm&new=12057%40%2F&old=12040%40%2F&sfp_email=&sfph_mail=

would be nice if there are all versions (not only the published) available at the website in the future ... than its possible for Daus and/or not programmers like me, to find when a bug _exactly_ was born ... and give the Information here for easier and/or faster fixing.

[Don-vip]

Michael, can you please have a look to see if it's a core regression or if the plugin needs to be updated to latest selection events changes?

[kre3d]

Started occurring after [12051], which causes the updateEnabledState callback of JosmAction to no longer trigger when the tags of the selection have changed.

[AndrewBuck]

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

[Don-vip]

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

[Klumbumbus]

What's the status of this ticket?

[Don-vip]

I began to look at it this week but don't understand how it works yet. I hope to fix it this week-end.

[kre3d]

I've been using a build with [12051] reverted with no problems.

FWIW other actions that depend on the tags of the selected object are also affected by the regression in [12051], e.g. Update Multipolygon. Prior to [12051], selecting a multipolygon then deleting the type=multipolygon tag would disable the Update Multipolygon action. After [12051], the action will remain enabled until the object is reselected.

[AlaskaDave]

2017-12-19 Please, please, fix this. It has always been a huge timesaver for me. I miss this feature tremendously and I'm sure others do as well.

Thanking you profusely in advance,

AlaskaDave

[hjart]

Yes, can someone please have a look at this. I'm still trying to use this all the time every day :-(

[GerdP]

@team: Problem is that the plugin does not react on a TagsChangedEvent.
Open problems with this patch:

• I am not 100% sure that I understood what the FireMode.IMMEDIATELY means.
• It is also not clear to me what should happen when one selects multiple objects and adds or removes a tag.
• When you undo something this might also fire a TagsChangedEvent and thus the next Shift+R might add unexpected tags.

Don't know how to improve this further.

[GerdP]

With v2 the 3rd problem (undo) seems to be solved, now it only reacts when the tags of a selected object are changed.

[GerdP]

I still try to fix this. I can't remember exactly how it behaved.
What should happen when you

• first select an object with e.g. highway=service + service=driveway
• next select another way with tag highway=path + surface=unpaved
• next press Shift+R. My understanding is that the 2nd way shoud be changed to highway=service + service=driveway + surface=unpaved
• next select another way with tag highway=unclassfied and press Shift+R again. Should it be changed to

highway=highway=service + service=driveway or
highway=service + service=driveway + surface=unpaved?

[GerdP]

OK, I've tried with JOSM r12040 and plugin version 33212. It collects the tags of the selected objects, so the result for my example is highway=service + service=driveway + surface=unpaved.
I'll try to get this behaviour back again.

[AlaskaDave]

It's been so long since it was working that I've forgotten the details but I do remember that the last way you added or modified was somehow remembered. You could then apply those same tags to the next object(s) you selected by typing [Shift]-R. Those tags would remain in the buffer and could be applied to any number of other objects whether new objects or existing ones. Once you change a tag or add a new tag that would update the buffer contents so that afterward the new tag set would be applied to subsequent objects by typing [Shift]-R

Thanks for working on this. I miss the [Shift]-R function every single day.

[GerdP]

requires JOSM r14518 or newer

[GerdP]

I think with 14862-v3.patch Shift+R works fine again but it requires JOSM r14518 or never.
@team: Do I commit a change to a plugin which requires an untested version of JOSM?
Alternative would be to double the code in the plugin until the new version is tested.

[stoecker]

Set the required JOSM version in build.xml. The rest is automatic.

[GerdP]

The wiki is not clear about this. What happens when you execute tested version of JOSM and update the plugins? Will it ignore the newer plugin or will it first update the plugin and next tell the user to update JOSM to an untested version?

[stoecker]

JOSM will always download the last version with compatible API (downwards) if the entries are correct. Development thus always continues without caring for tested versions.

[GerdP]

fixed with [o34767:34768]
Shift+R pastes the tags of the last edited or selected object(s). If multiple objects were selected, only the common tags are pasted.

[AlaskaDave]

Okay, that all sounds great! Thank you very much, Gerd.

Now, how do I use the patched plugin? I tried updating plugins but JOSM told me they were all up-to-date.

[GerdP]

Replying to AlaskaDave:

Okay, that all sounds great! Thank you very much, Gerd.

Now, how do I use the patched plugin? I tried updating plugins but JOSM told me they were all up-to-date.

That is good. See comment:23

[AlaskaDave]

OMG! It works again. [Shift]-R WORKS AGAIN!

I'm deliriously happy.

Dave