source: josm/trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadNotesTask.java@ 10032

Last change on this file since 10032 was 10032, checked in by Don-vip, 9 years ago

fix #12651 - do not create Notes layer when no notes were downloaded

  • Property svn:eol-style set to native
File size: 8.2 KB
Line 
1// License: GPL. For details, see LICENSE file.
2package org.openstreetmap.josm.actions.downloadtasks;
3
4import static org.openstreetmap.josm.tools.I18n.tr;
5import static org.openstreetmap.josm.tools.I18n.trn;
6
7import java.io.IOException;
8import java.net.URL;
9import java.util.List;
10import java.util.concurrent.Future;
11
12import javax.swing.JOptionPane;
13
14import org.openstreetmap.josm.Main;
15import org.openstreetmap.josm.data.Bounds;
16import org.openstreetmap.josm.data.notes.Note;
17import org.openstreetmap.josm.data.osm.NoteData;
18import org.openstreetmap.josm.data.preferences.IntegerProperty;
19import org.openstreetmap.josm.gui.PleaseWaitRunnable;
20import org.openstreetmap.josm.gui.layer.NoteLayer;
21import org.openstreetmap.josm.gui.progress.ProgressMonitor;
22import org.openstreetmap.josm.io.BoundingBoxDownloader;
23import org.openstreetmap.josm.io.OsmApi;
24import org.openstreetmap.josm.io.OsmServerLocationReader;
25import org.openstreetmap.josm.io.OsmServerReader;
26import org.openstreetmap.josm.io.OsmTransferException;
27import org.xml.sax.SAXException;
28
29/** Task for downloading notes */
30public class DownloadNotesTask extends AbstractDownloadTask<NoteData> {
31
32 private static final String PATTERN_API_URL = "https?://.*/api/0.6/notes.*";
33 private static final String PATTERN_DUMP_FILE = "https?://.*/(.*\\.osn(.bz2)?)";
34 /** Property defining the number of notes to be downloaded */
35 public static final IntegerProperty DOWNLOAD_LIMIT = new IntegerProperty("osm.notes.downloadLimit", 1000);
36 /** Property defining number of days a bug needs to be closed to no longer be downloaded */
37 public static final IntegerProperty DAYS_CLOSED = new IntegerProperty("osm.notes.daysClosed", 7);
38
39 private DownloadTask downloadTask;
40
41 /**
42 * Download a specific note by its id.
43 * @param id Note identifier
44 * @param progressMonitor progress monitor
45 * @return the future representing the asynchronous task
46 */
47 public Future<?> download(long id, ProgressMonitor progressMonitor) {
48 final String url = OsmApi.getOsmApi().getBaseUrl() + "notes/" + id;
49 downloadTask = new DownloadRawUrlTask(new OsmServerLocationReader(url), progressMonitor);
50 return Main.worker.submit(downloadTask);
51 }
52
53 @Override
54 public Future<?> download(boolean newLayer, Bounds downloadArea, ProgressMonitor progressMonitor) {
55 downloadTask = new DownloadBoundingBoxTask(new BoundingBoxDownloader(downloadArea), progressMonitor);
56 return Main.worker.submit(downloadTask);
57 }
58
59 @Override
60 public Future<?> loadUrl(boolean newLayer, String url, ProgressMonitor progressMonitor) {
61 if (url.endsWith(".bz2")) {
62 downloadTask = new DownloadBzip2RawUrlTask(new OsmServerLocationReader(url), progressMonitor);
63 } else {
64 downloadTask = new DownloadRawUrlTask(new OsmServerLocationReader(url), progressMonitor);
65 }
66 return Main.worker.submit(downloadTask);
67 }
68
69 @Override
70 public void cancel() {
71 if (downloadTask != null) {
72 downloadTask.cancel();
73 }
74 }
75
76 @Override
77 public String getConfirmationMessage(URL url) {
78 return null;
79 }
80
81 @Override
82 public String getTitle() {
83 return tr("Download OSM Notes");
84 }
85
86 @Override
87 public String[] getPatterns() {
88 return new String[] {PATTERN_API_URL, PATTERN_DUMP_FILE};
89 }
90
91 @Override
92 public boolean isSafeForRemotecontrolRequests() {
93 return true;
94 }
95
96 abstract class DownloadTask extends PleaseWaitRunnable {
97 protected OsmServerReader reader;
98 protected List<Note> notesData;
99
100 DownloadTask(OsmServerReader reader, ProgressMonitor progressMonitor) {
101 super(tr("Downloading Notes"), progressMonitor, false);
102 this.reader = reader;
103 }
104
105 @Override
106 protected void finish() {
107 rememberDownloadedData(new NoteData(notesData));
108 if (isCanceled() || isFailed() || notesData == null || notesData.isEmpty()) {
109 return;
110 }
111 if (Main.isDebugEnabled()) {
112 Main.debug("Notes downloaded: " + notesData.size());
113 }
114
115 List<NoteLayer> noteLayers = null;
116 if (Main.map != null) {
117 noteLayers = Main.map.mapView.getLayersOfType(NoteLayer.class);
118 }
119 if (noteLayers != null && !noteLayers.isEmpty()) {
120 noteLayers.get(0).getNoteData().addNotes(notesData);
121 } else {
122 Main.main.addLayer(new NoteLayer(notesData, tr("Notes")));
123 }
124 }
125
126 @Override
127 protected void cancel() {
128 setCanceled(true);
129 if (reader != null) {
130 reader.cancel();
131 }
132 }
133
134 @Override
135 public abstract void realRun() throws IOException, SAXException, OsmTransferException;
136 }
137
138 class DownloadBoundingBoxTask extends DownloadTask {
139
140 DownloadBoundingBoxTask(OsmServerReader reader, ProgressMonitor progressMonitor) {
141 super(reader, progressMonitor);
142 }
143
144 @Override
145 public void realRun() throws IOException, SAXException, OsmTransferException {
146 if (isCanceled()) {
147 return;
148 }
149 ProgressMonitor subMonitor = progressMonitor.createSubTaskMonitor(ProgressMonitor.ALL_TICKS, false);
150 try {
151 notesData = reader.parseNotes(DOWNLOAD_LIMIT.get(), DAYS_CLOSED.get(), subMonitor);
152 } catch (BoundingBoxDownloader.MoreNotesException e) {
153 notesData = e.notes;
154 JOptionPane.showMessageDialog(Main.parent, "<html>"
155 + trn("{0} note has been downloaded.", "{0} notes have been downloaded.", e.limit, e.limit)
156 + "<br>"
157 + tr("Since the download limit was {0}, there might be more notes to download.", e.limit)
158 + "<br>"
159 + tr("Request a smaller area to make sure that all notes are being downloaded.")
160 + "</html>",
161 tr("More notes to download"), JOptionPane.INFORMATION_MESSAGE);
162 } catch (Exception e) {
163 if (isCanceled())
164 return;
165 if (e instanceof OsmTransferException) {
166 rememberException(e);
167 } else {
168 rememberException(new OsmTransferException(e));
169 }
170 }
171 }
172 }
173
174 class DownloadRawUrlTask extends DownloadTask {
175
176 DownloadRawUrlTask(OsmServerReader reader, ProgressMonitor progressMonitor) {
177 super(reader, progressMonitor);
178 }
179
180 @Override
181 public void realRun() throws IOException, SAXException, OsmTransferException {
182 if (isCanceled()) {
183 return;
184 }
185 ProgressMonitor subMonitor = progressMonitor.createSubTaskMonitor(ProgressMonitor.ALL_TICKS, false);
186 try {
187 notesData = reader.parseRawNotes(subMonitor);
188 } catch (Exception e) {
189 if (isCanceled())
190 return;
191 if (e instanceof OsmTransferException) {
192 rememberException(e);
193 } else {
194 rememberException(new OsmTransferException(e));
195 }
196 }
197 }
198 }
199
200 class DownloadBzip2RawUrlTask extends DownloadTask {
201
202 DownloadBzip2RawUrlTask(OsmServerReader reader, ProgressMonitor progressMonitor) {
203 super(reader, progressMonitor);
204 }
205
206 @Override
207 public void realRun() throws IOException, SAXException, OsmTransferException {
208 if (isCanceled()) {
209 return;
210 }
211 ProgressMonitor subMonitor = progressMonitor.createSubTaskMonitor(ProgressMonitor.ALL_TICKS, false);
212 try {
213 notesData = reader.parseRawNotesBzip2(subMonitor);
214 } catch (Exception e) {
215 if (isCanceled())
216 return;
217 if (e instanceof OsmTransferException) {
218 rememberException(e);
219 } else {
220 rememberException(new OsmTransferException(e));
221 }
222 }
223 }
224 }
225}
Note: See TracBrowser for help on using the repository browser.