No warning on closing unsaved edits in layers with upload=never flag

[mdk]

First the use case: In Switzerland we can use GWR address data. Simon Poole has preprocessed files with these data: ​http://qa.poole.ch/addresses/ch/. If you download an .osm file (link 'O', unpack zip) you can load such a file as a separate layer. Now I copy and delete an address from that layer, switch to the data layer and past the address to the building.

I got two problems with layers loaded from file. The first is more for completeness:
If I only load the GWR.osm file and change something, the File.Save menu will be enabled (ok), but when I close the modified layer in the Layers view, it is closed without and warning. But I would expect a warning that I'm about to loose edits.

The second problem is much more annoying:
Download an area into a data layer. Additionally load the GWR.osm file. Edit on both layers. If I now activate the GWR data layer and go to the file menu, Save is disabled! You could save the data with Save as... and hopefully choose the right file to override. The changes are recognized: choose Info about the GWR data layer and you will see your edits in that layer. After uploading the data (from the data layer), the edits in the GWR data file are still not saved, Save is disabled and the edits got lost without warning on closing the layer or JOSM.

Perhaps this has to do with upload='never' flag in the .osm file.

But I would expect, that at least File.Save is enabled for this layer and that I got a warning about unsaved data on deleting the layer or leaving JOSM.

[Don-vip]

Please always attach your status report. Defects without JOSM version are almost useless, I changed the save logic recently and I don't know if you have this change.

[mdk]

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2019-10-19 21:14:00 +0200 (Sat, 19 Oct 2019)
Revision:15466
Build-Date:2019-10-20 01:30:54
URL:https://josm.openstreetmap.de/svn/trunk

Identification: JOSM/1.5 (15466 en) Linux Ubuntu 19.04
Memory Usage: 434 MB / 1984 MB (164 MB allocated, but free)
Java version: 11.0.4+11-post-Ubuntu-1ubuntu219.04, Ubuntu, OpenJDK 64-Bit Server VM
Screen: :0.0 1920x1080
Maximum Screen Size: 1920x1080
Java package: openjdk-11-jre:amd64-11.0.4+11-1ubuntu2~19.04
Java ATK Wrapper package: libatk-wrapper-java:all-0.33.3-21
libcommons-compress-java: libcommons-compress-java:all-1.18-2
libcommons-logging-java: libcommons-logging-java:all-1.2-2
fonts-noto: fonts-noto:-
VM arguments: [-Djosm.restart=true, -Djosm.dir.name=JOSM-latest, -Djava.net.useSystemProxies=true]
Dataset consistency test: No problems found

Plugins:
+ ColumbusCSV (34985)
+ FastDraw (34977)
+ HouseNumberTaggingTool (34977)
+ OpeningHoursEditor (34977)
+ RoadSigns (UNKNOWN)
+ SimplifyArea (34977)
+ buildings_tools (35171)
+ contourmerge (v0.1.4)
+ imagery-xml-bounds (35100)
+ imagery_offset_db (34908)
+ pbf (35033)
+ poly (34991)
+ public_transport (34977)
+ reltoolbox (34977)
+ reverter (35084)
+ terracer (34977)
+ turnrestrictions (34977)
+ utilsplugin2 (35177)
+ wikipedia (1.1.2)

Tagging presets:
+ https://josm.openstreetmap.de/josmfile?page=Presets/OneClick&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Presets/LaneAttributes&preset&zip=1

Map paint styles:
+ https://josm.openstreetmap.de/josmfile?page=Styles/Lane_and_Road_Attributes&zip=1
+ https://josm.openstreetmap.de/josmfile?page=Styles/Maxspeed&style&zip=1

Last errors/warnings:
- W: No configuration settings found.  Using hardcoded default values for all pools.
- W: Cannot start IPv4 remotecontrol https server on port 8112: Keystore was tampered with, or password was incorrect
- W: Cannot start IPv6 remotecontrol https server on port 8112: Keystore was tampered with, or password was incorrect

[Don-vip]

Thanks

[Don-vip]

Replying to mdk:

Download an area into a data layer. Additionally load the GWR.osm file. Edit on both layers. If I now activate the GWR data layer and go to the file menu, Save is disabled!

I can't reproduce this part. Are you sure you didn't forget a step? Save is always enabled for me.

[mdk]

Did you activate the GWR layer in the Layers view? Set the green checkmark "this layer is not currently active (click to active)".

[Don-vip]

Yes. But probably I don't understand the steps as you think you explained them. Can you please share a video?

[mdk]

I'm very sorry, but yesterday I had this problem all the time, but now I can't reproduce it. Yesterday the changeset was created with JOSM/1.5 (15462 en). Today I have 15466. Normally I like the Linux auto update, but in this case ...
Sorry for wasting your time!

[Don-vip]

OK so there's only one problem, the missing save warning for layers with upload=never