Modify ↓
Opened 15 months ago
Last modified 7 months ago
#7508 new defect
[patch?] Empty download of GPS results to 0;0 saved to last download location
| Reported by: | Larry0ua | Owned by: | team |
|---|---|---|---|
| Priority: | normal | 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 (6)
comment:1 Changed 15 months ago by Larry0ua
- Summary changed from Cancel download of GPS results to 0;0 saved to last download location to Empty download of GPS results to 0;0 saved to last download location
Changed 14 months ago by xeen
Changed 14 months ago by xeen
comment:2 Changed 14 months ago by anonymous
- Summary changed from Empty download of GPS results to 0;0 saved to last download location to [patch?] Empty download of GPS results to 0;0 saved to last download location
comment:3 Changed 8 months ago by stoecker
@xeen: Status of this?
comment:4 Changed 7 months ago by Don-vip
#7976 is related.
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.