Modify

Opened 12 years ago

Closed 12 years ago

#7284 closed defect (fixed)

Download dialog always loads bing attributions

Reported by: skyper Owned by: team
Priority: normal Milestone:
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)

7284.patch (4.9 KB ) - added by simon04 12 years ago.
7284_v2.patch (5.4 KB ) - added by simon04 12 years ago.

Download all attachments as: .zip

Change History (23)

comment:1 by skyper, 12 years ago

Description: modified (diff)

comment:2 by simon04, 12 years ago

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.

by simon04, 12 years ago

Attachment: 7284.patch added

comment:3 by simon04, 12 years ago

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)?

in reply to:  2 comment:4 by skyper, 12 years ago

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 by bastiK, 12 years ago

I'd say with 1 day caching, no one can complain. The original bug remains.

by simon04, 12 years ago

Attachment: 7284_v2.patch added

comment:6 by simon04, 12 years ago

Summary: Download dialog always loads bing attributions[Patch] Download dialog always loads bing attributions

Caching + loading on demand (in 7284_v2.patch) should take care of all. :-)

comment:7 by simon04, 12 years ago

Resolution: fixed
Status: newclosed

Fixed in [o27483].

in reply to:  7 comment:8 by skyper, 12 years ago

Replying to simon04:

Fixed in [o27483].

This was fast !

Thanks

comment:9 by bastiK, 12 years ago

Resolution: fixed
Status: closedreopened

The changeset is ok, but it exposes other problems: #7289

comment:10 by bastiK, 12 years ago

Resolution: fixed
Status: reopenedclosed

comment:11 by bastiK, 12 years ago

Resolution: fixed
Status: closedreopened

JOSM classes cannot be used in the JMapViewer Component.

in reply to:  11 comment:12 by simon04, 12 years ago

Resolution: fixed
Status: reopenedclosed

Replying to bastiK:

JOSM classes cannot be used in the JMapViewer Component.

I realized that too late. Fixed in r4825. Sorry for the mess while fixing this ticket.

comment:13 by bastiK, 12 years ago

See [o27507].

comment:14 by rickmastfan67, 12 years ago

Resolution: fixed
Status: closedreopened

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 by simon04, 12 years ago

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.

in reply to:  13 comment:16 by simon04, 12 years ago

Resolution: fixed
Status: reopenedclosed

See [o27519].

comment:17 by skyper, 12 years ago

Resolution: fixed
Status: closedreopened

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 by skyper, 12 years ago

Summary: [Patch] Download dialog always loads bing attributionsDownload dialog always loads bing attributions

comment:19 by simon04, 12 years ago

The same issue has been reported in #7280.

in reply to:  19 comment:20 by skyper, 12 years ago

Replying to simon04:

The same issue has been reported in #7280.

Then decide by yourself which of the two tickets can be closed and which needs to stay open.

comment:21 by simon04, 12 years ago

Resolution: fixed
Status: reopenedclosed

Let's continue the discussion in #7280.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.