Opened 17 months ago
Closed 17 months ago
#7284 closed defect (fixed)
Download dialog always loads bing attributions
| Reported by: | skyper | Owned by: | team |
|---|---|---|---|
| Priority: | normal | Component: | Core |
| Version: | latest | Keywords: | bing attribution download dialog |
| Cc: |
Description (last modified by skyper)
If I open the download dialog, the bing attributions are always loaded, even though I only use mapnik.
This is happening for month now, I did only forget to fill a ticket.
Attachments (2)
Change History (23)
comment:1 Changed 17 months ago by skyper
- Description modified (diff)
comment:2 follow-up: ↓ 4 Changed 17 months ago by simon04
Changed 17 months ago by simon04
comment:3 Changed 17 months ago by simon04
Patch attached to cache the attribution data for 1 week. Are there any legal issues with this approach (e.g., is loading the attribution data required for Bing-internal statistics)?
comment:4 in reply to: ↑ 2 Changed 17 months ago by skyper
Replying to simon04:
In the download dialog, you can select Bing as layer. I don't see any defect here.
Maybe I was not clear:
It is loaded without selecting the bing layer. It is not needed for all other layers and should not be loaded if not needed.
comment:5 Changed 17 months ago by bastiK
I'd say with 1 day caching, no one can complain. The original bug remains.
Changed 17 months ago by simon04
comment:6 Changed 17 months ago by simon04
- Summary changed from Download dialog always loads bing attributions to [Patch] Download dialog always loads bing attributions
Caching + loading on demand (in 7284_v2.patch) should take care of all. :-)
comment:7 follow-up: ↓ 8 Changed 17 months ago by simon04
- Resolution set to fixed
- Status changed from new to closed
Fixed in [o27483].
comment:8 in reply to: ↑ 7 Changed 17 months ago by skyper
comment:9 Changed 17 months ago by bastiK
- Resolution fixed deleted
- Status changed from closed to reopened
The changeset is ok, but it exposes other problems: #7289
comment:10 Changed 17 months ago by bastiK
- Resolution set to fixed
- Status changed from reopened to closed
comment:11 follow-up: ↓ 12 Changed 17 months ago by bastiK
- Resolution fixed deleted
- Status changed from closed to reopened
JOSM classes cannot be used in the JMapViewer Component.
comment:12 in reply to: ↑ 11 Changed 17 months ago by simon04
- Resolution set to fixed
- Status changed from reopened to closed
comment:13 follow-up: ↓ 16 Changed 17 months ago by bastiK
See [o27507].
comment:14 Changed 17 months ago by rickmastfan67
- Resolution fixed deleted
- Status changed from closed to reopened
In continuation of ticket:7289#comment:10, the fixes don't seem to have fully fixed the problems here on my end.
As of right now in r4831, while looking in the Java Console, I never see the message stating that the Bing Attribution is fully loaded post-r4806 (r4806 was the last build I saw it in). In r4831, every time I download some data from OSM, I'm greeted with a "Error: null" message when I turn on the Bing Imagery layer (as long as I didn't have previous imagery from that area). However, if I delete the Bing layer and then hit the Bing button again, the imagery will start to download (but I don't get the "Bing Attribution has been successfully loaded message in the Java Console).
Also, it seems that post-r4806 can successfully download the Bing Attribution ONLY if it doesn't exist in your cache. If it exists, then you'll get the "Error: null" message on your first attempt of loading imagery.
Now, I don't think it's a problem with the attribution file itself because the most current copy that I have (downloaded on 01/21/12), works just fine in r4806. So, something was changed between r4806 and r4818 that is causing this problem for me (my gut says something in r4813 broke it, it's that or r4818).
Repository Root: http://josm.openstreetmap.de/svn Build-Date: 2012-01-21 02:32:22 Last Changed Author: bastiK Revision: 4831 Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b URL: http://josm.openstreetmap.de/svn/trunk Last Changed Date: 2012-01-20 22:00:13 +0100 (Fri, 20 Jan 2012) Last Changed Rev: 4831 Identification: JOSM/1.5 (4831 en) Memory Usage: 189 MB / 2730 MB (97 MB allocated, but free) Java version: 1.6.0_30, Sun Microsystems Inc., Java HotSpot(TM) 64-Bit Server VM Operating system: Windows 7 Dataset consistency test: No problems found Plugin: OpeningHoursEditor (27355) Plugin: buildings_tools (27355) Plugin: imageryadjust (27494) Plugin: licensechange (27472) Plugin: mapdust (27355) Plugin: measurement (27289) Plugin: openstreetbugs (27355) Plugin: restart (27289) Plugin: reverter (27393) Plugin: turnrestrictions (27355) Plugin: undelete (27395) Plugin: utilsplugin2 (27516)
comment:15 Changed 17 months ago by simon04
The reason that "Successfully loaded Bing attribution data." isn't shown is that the message is only printed, if the attribution data is loaded from the web. Instead, if it comes from the cache, no message is shown.
The "Error: null" errors are of course not expected … I'm investigating.
comment:16 in reply to: ↑ 13 Changed 17 months ago by simon04
- Resolution set to fixed
- Status changed from reopened to closed
See [o27519].
comment:17 Changed 17 months ago by skyper
- Resolution fixed deleted
- Status changed from closed to reopened
It seems to work but I get strange messages on the console:
Repository Root: http://josm.openstreetmap.de/svn Build-Date: 2012-01-30 02:32:23 Last Changed Author: stoecker Revision: 4878 Repository UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b URL: http://josm.openstreetmap.de/svn/trunk Last Changed Date: 2012-01-30 00:02:54 +0100 (Mon, 30 Jan 2012) Last Changed Rev: 4878 Warnung: Fehlende Einstellungsdatei "$HOME/.josm/preferences.xml". Datei mit Standardeinstellungen wird erstellt. GET http://api.openstreetmap.org/api/capabilities... OK Successfully loaded Bing attribution data. Bing: attribution data is not yet loaded. Bing: attribution data is not yet loaded. Bing: attribution data is not yet loaded. Bing: attribution data is not yet loaded. Bing: attribution data is not yet loaded. Bing: attribution data is not yet loaded. Bing: attribution data is not yet loaded. Bing: attribution data is not yet loaded. Bing: attribution data is not yet loaded. Bing: attribution data is not yet loaded. Bing: attribution data is not yet loaded. GET http://api.openstreetmap.org/api/0.6/map?bbox=....
comment:18 Changed 17 months ago by skyper
- Summary changed from [Patch] Download dialog always loads bing attributions to Download dialog always loads bing attributions
comment:19 follow-up: ↓ 20 Changed 17 months ago by simon04
The same issue has been reported in #7280.
comment:20 in reply to: ↑ 19 Changed 17 months ago by skyper
comment:21 Changed 17 months ago by simon04
- Resolution set to fixed
- Status changed from reopened to closed
Let's continue the discussion in #7280.



In the download dialog, you can select Bing as layer. I don't see any defect here.
Instead, we could cache the attribution data to avoid regular downloads.