Opened 13 years ago
Closed 13 years ago
#7695 closed enhancement (fixed)
Don't invite to upload data which has not been downloaded
Reported by: | tongro | Owned by: | Don-vip |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Core | Version: | tested |
Keywords: | file save upload | Cc: |
Description
I'm working on a private .osm file, which should not be uploaded, and I save the file. When I then try to exit JOSM, I get invited to upload the data to OSM. This is dangerous.
I have tried the procedure outlined in ticket #4043, but (1) it's quite complicated, and (2) it doesn't seem to work - although the layer icon gets a little warning sign attached to it, exiting the program still results in me being invited to upload the data.
What I think should happen is that JOSM would keep a record of whether the data came from opening a file or from downloading from OSM, or a mixture of both, and then decide whether or not to invite the user to upload an already-saved file.
Attachments (1)
Change History (6)
comment:1 by , 13 years ago
comment:2 by , 13 years ago
Owner: | changed from | to
---|---|
Status: | new → needinfo |
comment:3 by , 13 years ago
That looks much better now, using v.5241.
My test file consists of two layers, one marked as "discourage upload". When I opt to exit the program with pending changes in both layers, the exit dialogue appears, and the options in the "actions to take" column are correct. However, in the "recommended actions" for both layers, both "should be uploaded" and "should be saved" are indicated, which does not match the "discourage upload" settings or the "actions to take" settings. Also (a minor issue) the order of the "recommended actions" (should be saved/uploaded) is the opposite of the "actions to take" options (upload/save), which could cause user error. Screenshot is attached.
The other improvement that seems to be needed is that the enabling of the "discourage upload" option could be made simpler, either integrated into the preferences GUI (though I don't see an obvious place to put it), or maybe enabled by default.
Tony.
comment:4 by , 13 years ago
Owner: | changed from | to
---|---|
Status: | needinfo → assigned |
It should be no longer the case after r5233, see ticket #7684 for more details.
Can you test it with latest version of JOSM please ?