Modify ↓
Opened 13 years ago
Closed 12 years ago
#7508 closed defect (worksforme)
[patch?] Empty download of GPS results to 0;0 saved to last download location
Reported by: | Larry0ua | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | Core | Version: | latest |
Keywords: | download | Cc: |
Description
Hi! Here are steps to reproduce the bug:
- Open JOSM
- "Download OSM data"
- uncheck OSM data, check GPS data. Select an area where there is no gps tracks. Press Download.
- Open download dialog once again - it opens in 0;0 location. Should remember previous download location.
Attachments (2)
Change History (7)
comment:1 by , 13 years ago
Summary: | Cancel download of GPS results to 0;0 saved to last download location → Empty download of GPS results to 0;0 saved to last download location |
---|
by , 13 years ago
Attachment: | possible_fix_7508.patch added |
---|
by , 13 years ago
Attachment: | possible_fix_7508.2.patch added |
---|
comment:2 by , 13 years ago
Summary: | Empty download of GPS results to 0;0 saved to last download location → [patch?] Empty download of GPS results to 0;0 saved to last download location |
---|
comment:5 by , 12 years ago
Resolution: | → worksforme |
---|---|
Status: | new → closed |
This bug doesn’t apply anymore. When downloading an empty (gps track wise) region, no gpx layer is created. Re-opening the download dialog will place it at the same position as before.
Note:
See TracTickets
for help on using tickets.
The actual reason for this is the following: GPX Layers do not remember their download bounds, because they are generally not required. When there is no OSM layer and no GPX data to derive the bounds from, MapView zooms in in (0,0). If a map view is visible, the download dialog takes the visible bounds instead of the one it remembered. Thus it appears as if the download dialog remebered (0,0). Closing the empty GPX layer before opening the download dialog will have it display the correct coordinates. I can think of two solutions: if the GPX layer is the first layer, popup a warning message and don’t create a new layer. The other is implemented in the patch: if there is no GPX data and this is the first layer, zoom to the download bounds instead. I’m not very happy with either approach, so I’ll only attach the patch here but won’t commit it for now.