[Patch] Validator: Fix action done in case of "Don't show again"

[Nakor]

If you try to fix "Duplicate nodes error" with validator plugin with nodes outside of your downloaded areas, a warning message will popup. If you check "Do not ask me anymore" and click "No" (meaning you do NOT want to delete those nodes), validator will still delete them. Then when you try to upload you get a lot of warnings.

[Nakor]

Also it looks like the choice of not asking anymore is stored permanently with now option to deactivate it. Even deinstalling and reinstalling the plugin does not reset the choice. Please provide an option to do so.

[mjulius]

The corresponding setting is message.delete_outside_nodes which you can find in the Advanced Preferences (bottom tab in the Preferences dialog). This is set to "false" when you check that check box in the confirmation dialog. In this case the dialog is not shown anymore and the default action is chosen (which is to delete the nodes) regardless whether you selected Yes or No in the dialog.

Delete the setting or set it to "true" to get the dialog back.

[Nakor]

Thanks for telling me how to get back that check.

It would also be nice to have an option to never modify nodes outside of the download zone.

Thanks, N.

[stoecker]

IN JOSM we let user decide and warn him of possible dangerous actions. When he decides to ignore the warnings, then it's his fault.

[mjulius]

The dialog in question here is not merely a warning. It is a question whether or not he wants to delete nodes outside the download area (among other nodes). This is a decision the user has to make and not just a warning to acknowledge. I think it is perfectly reasonable for the user to expect that his decision will be applied in future if he checks the "Do not show again" checkbox.

This is the scenario here: The user downloads an area and ends up with duplicate nodes outside the download bbox. Then, without selecting anything he clicks on Validate. Validator now lists a bunch of nodes as duplicates - some of which are inside and some of which are outside the download area. There is no easy way to tell.

When the user then highlights the Duplicate nodes error category ind clicks on Fix the here discussed dialog pops up telling the user that he is about to delete some nodes outside the download area and asks whether he wants to delete them. The user - knowing how dangerous that is - clicks on No and validator ignores that node and moves on. When it encounters the next node outside the bbox the dialog pops up again. After the 20th time the user gets tired of this and checks "Do not show again" and klicks No. The user expects that from now on will leave those outside nodes alone without bothering him ever again. And I would call this expectation very reasonable. Instead, validator now deletes all nodes regardless where they are.

This is not a case of ignorance on the user's side.

[stoecker]

Probably this also affects other "Don't show me agains" in JOSM?

[mjulius]

Replying to stoecker:

Probably this also affects other "Don't show me agains" in JOSM?

I think so. I was just about to assign the ticket to Core myself.

The attached patch should fix that.

[stoecker]

(In [2522]) fix #4011 - do not show again now remembers what it should do - patch by mjulius