Opened 18 months ago

Closed 8 months ago

#5484 closed defect (fixed)

When you move the map, P2 fails to fetch new OSSV Imagery

Reported by: SomeoneElse Owned by: potlatch-dev@…
Priority: minor Milestone:
Component: potlatch2 Version:
Keywords: imagery Cc:


To reproduce:

Run Firefox (I tried it on Ubuntu / KDE and on Windows 7)

Clear recent history in browser (to remove any caching effects)

Browse to e.g.

Edit the map (and login if required)

If the default background layer isn't set to OS OpenData? StreetView?, set it to that.

Note that the OSSV imagery loads OK

Drag the map so that a new bit of imagery is needed. That new bit of imagery does not load.

Whatever iD does to fetch imagery, it doesn't show the same issue.

Change History (7)

comment:1 Changed 18 months ago by Richard

Looking at the console, this appears to be Firefox being sniffy about the fact that OSSV is served over HTTP while P2 has been served over HTTPS. I guess this needs editor-imagery-index to be updated.

comment:3 Changed 18 months ago by SomeoneElse has now been merged.

The problem seems ot occur with http sources that redirect to https. Some other sources at referred to in the editor-imagery-index redirect and some (as yet) do not. A bunch seem to be defined there but not actual exist or be usable, despite trying to fetch tiles in the right zoom range in the right are (e.g some OSM-hosted OS OOC maps). I suspect that many of these are dupicates of e.g. NLS maps.

There are three others - AGRI, Surrey Air Survey and one for the Philippines that show the issue. I can get the first 2 to load and show the same problem, and the fix for them is in here:

I can't get the Philippines imagery to load within P2 (but oddly can externally) so I haven't changed that one.

Presumably editor-layer-index is somehow either canned within P2 or is released only occasionally? The live version used by P2 hasn't been updated yet.

comment:4 Changed 18 months ago by Richard

It pulls it from . I think there might be some sort of build step that's required for that to be updated - not sure.

comment:5 Changed 18 months ago by Tom Hughes

Yes - somebody will need to run make and commit the built files.

comment:6 Changed 18 months ago by Tom Hughes

I've pushed a new build now.

comment:7 Changed 8 months ago by Richard

Resolution: fixed
Status: newclosed
Note: See TracTickets for help on using tickets.