Opened 12 years ago

Closed 12 years ago

#379 closed defect (fixed)

Normal layer with consume maps failed to display in Overview map wdiget

Reported by: jennyhe Owned by: madair
Priority: P2 Milestone: 2.0
Component: Widgets Version: SVN
Severity: Critical Keywords:
Cc: madair@… Browser: All
External ID: 1313287 Operating System: All
state: New

Description

Steps:

  1. Open one flexible web layout with consume maps by IE.

Results: Overview map is blank.

Expected results: The normal layer should be displayed in that widget.

Attachments (1)

OverviewMapFix.patch (6.7 KB ) - added by chrisclaydon 12 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 by liuar, 12 years ago

Cc: madair@… added; arthur.liu@… removed
Severity: MajorCritical

Hi Mike,

I think this defect is result from http://trac.osgeo.org/fusion/changeset/2068.[[BR]] In this submission, current map's baselayer will be used as the overview map. If user chooses google map as baselayer, then the google layer will not displayed at all.
I remember that OpenLayers' overviewMap control has some problems in displaying consume maps. At least we cannot display combined Google layer and MapGuide layer in the overview map at the same time. Am I right? at

If it won't break current overview map behavior, I would suggest to rollback this submission. What's your opinion?

Regards,
Arthur

comment:2 by madair, 12 years ago

Milestone: Future2.0
Status: newassigned
Version: 1.1.1SVN

Arthur,

I think you have the incorrect changeset number there, but in any case I would like to have the OverviewMap projection issue resolved in the 2.0 release. There has been some discussion on the list about this which I will try and see if it fixes the issue

by chrisclaydon, 12 years ago

Attachment: OverviewMapFix.patch added

comment:4 by chrisclaydon, 12 years ago

I have attached a patch that should resolve the issue. Please review.

comment:5 by chrisclaydon, 12 years ago

Resolution: fixed
Status: assignedclosed

The patch was reviewed by Mike Adair, and submitted with the following revision:

http://trac.osgeo.org/fusion/changeset/2139

The changes are actually pretty minor - I've moved the code that creates the Fusion layer object from the initializeWidget method into the mapWidgetLoaded method. This preserves the intent of changeset 2068, which seeks to re-initialize the overview when the main map is reloaded, but enforces that the underlying map definition has loaded before attempting to create an OpenLayers layer that corresponds to it.

Note: See TracTickets for help on using tickets.