[patch] Cannot download only notes

[Zverikk]

1. Open JOSM, open the download dialog.
2. Uncheck "Data" and "GPX", enable "Notes".
3. Select an area and press "Download"

Expected: notes for an area. Instead I've got an error message like "choose either osm data or gpx traces".

[ToeBee]

Yeah, I knew about this. When I initially looked at this logic I thought it would be complicated to implement until the notes feature was enabled for everyone. But looking at it again now I'm not sure why it would be hard. I'll take a closer look tonight.

[skyper]

Replying to ToeBee:

Yeah, I knew about this. When I initially looked at this logic I thought it would be complicated to implement until the notes feature was enabled for everyone. But looking at it again now I'm not sure why it would be hard. I'll take a closer look tonight.

Thanks, I really need it in my area, cause downloading data/gpx is way to much traffic.

[ToeBee]

Yeah, this is simple after all. I think when I was first putting in the checkbox I wasn't instantiating it at all if notes were disabled which would have required the download check to worry about null. But the way I ended up doing it, it just doesn't get added to the GUI if notes are disabled so the isDownloadNotes check just returns false and everything is happy.

The only slight inconsistency now is the message displayed to the user. If notes are disabled, I don't want it to say anything about notes so I'm leaving the error message alone for now. If notes are enabled and nothing is checked, it will still say you need to select either OSM data or GPX data.

[skyper]

This problem effects me even further. On the first time enabled I can not even "comment", "close" or sort notes even after redownloading.

I have to delete the all notes layers in advance and then redownload to fix it.

Repository Root: http://josm.openstreetmap.de/svn
Build-Date: 2014-12-15 02:34:17
Last Changed Author: bastiK
Revision: 7804
Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Relative URL: ^/trunk
URL: http://josm.openstreetmap.de/svn/trunk
Last Changed Date: 2014-12-14 16:20:17 +0100 (Sun, 14 Dec 2014)
Last Changed Rev: 7804

Identification: JOSM/1.5 (7804 en) Linux Debian GNU/Linux 7.7 (wheezy)
Memory Usage: 362 MB / 882 MB (86 MB allocated, but free)
Java version: 1.7.0_65, Oracle Corporation, OpenJDK 64-Bit Server VM
Java package: openjdk-7-jre:amd64-7u71-2.5.3-2~deb7u1
VM arguments: [-Djosm.home=/tmp/.josm_notes2]
Program arguments: [--language=en, --set=osm.notes.enableDownload=true]
Dataset consistency test: No problems found

[skyper]

Replying to skyper:

This problem effects me even further. On the first time enabled I can not even "comment", "close" or sort notes even after redownloading.

I have to delete the all notes layers in advance and then redownload to fix it.

Damn, sorry, wrong ticket, meant #10845.

[bastiK]

In 7808/josm:

applied #10827 - Cannot download only notes (patch by ToeBee)

[aseerel4c26]

Works as expected now (tested with osm.notes.enableDownload =true and =false) tested with Version 7819.