Opened 11 years ago

Closed 11 years ago

#300 closed defect (fixed)

Maps with commercial layers failed to display in supported browsers except IE.

Reported by: jennyhe Owned by: madair
Priority: P2 Milestone: Future
Component: Widgets Version: 1.1.1
Severity: Major Keywords:
Cc: cclaydon Browser: All
External ID: 1257754 Operating System: All
state: Committed

Description

Steps: Open one flexible web layout with commercial layers in Firefox

Results: Nothing dislayed.

Expected results: Map should display correctly as it shows in IE.

Attachments (6)

error_information.PNG (16.0 KB) - added by jennyhe 11 years ago.
fusion.patch (4.1 KB) - added by liuar 11 years ago.
fusion.js (45.9 KB) - added by liuar 11 years ago.
From_SiteServer_20091130_Packet to Fusion.mgp (42.5 KB) - added by liuar 11 years ago.
ApplicationDefinition.xml (36.2 KB) - added by liuar 11 years ago.
jxlib.uncompressed.js.patch (5.4 KB) - added by chrisclaydon 11 years ago.
Patch for jxlib.uncompressed.js to get things working in Firefox

Download all attachments as: .zip

Change History (19)

comment:1 Changed 11 years ago by zjames

Owner: changed from madair to zjames
Status: newassigned

I'm unable to reproduce this in Firefox 3.0 or 3.5. Can you give more information about your browser version, ApplicationDefinition and any messages displayed in the javascript console?

Changed 11 years ago by jennyhe

Attachment: error_information.PNG added

comment:2 Changed 11 years ago by jennyhe

Version I used is 3.5.3. Please check the attached png file which shows Error Control information.

Changed 11 years ago by liuar

Attachment: fusion.patch added

Changed 11 years ago by liuar

Attachment: fusion.js added

comment:3 Changed 11 years ago by zjames

state: NewApproved

This looks ok, Arthur. Are you a committer or do you want me to put it in trunk?

comment:4 Changed 11 years ago by liuar

Thanks for your review, Zak. I've submitted the code. But there's still a problem of jxLib for Bing(Virtual Earth) in FF 3.5. The error occurs in Jxlib.uncompressed.js Line:17397. Can you reproduce it in your machine?

comment:5 Changed 11 years ago by pagameba

I've committed a change to this code that fixes a javascript error when MapGuide is not present (mapserver users). The call to createSessionId() has been moved inside appDefUrl and extra tests have been added to limit this behaviour to application definitions being loaded from Library:// or Session://.

Please confirm this does not break anything for you as I have only tested without mapguide.

comment:6 Changed 11 years ago by liuar

I've confirmed it, MapGuide works fine. Thanks.
Sorry for the defect in MapServer.

comment:7 Changed 11 years ago by madair

Cc: cclaydon added
Owner: changed from zjames to madair
Status: assignednew

it appears like the src URL for commercial layers API is being included in the AppDef? file. I don't have an updated version of Studio that generates the file file with that in it so can someone please attach and/or send me a smaple AppDef? that includes commercial layers.

Changed 11 years ago by liuar

Attachment: ApplicationDefinition.xml added

comment:8 Changed 11 years ago by liuar

Hi Mike,

I have attached the xml file you needed and a MGP file which contains the data.

Arthur

comment:9 Changed 11 years ago by tomfukushima

Hi Mike, If you still need Studio, Zac should have a DVD that has Studio on it that you can use for this. Tom

comment:10 Changed 11 years ago by tomfukushima

...sorry, I meant Zak, not Zac.

Changed 11 years ago by chrisclaydon

Attachment: jxlib.uncompressed.js.patch added

Patch for jxlib.uncompressed.js to get things working in Firefox

comment:11 Changed 11 years ago by chrisclaydon

I've attached a patch for the jxlib file that should eliminate the errors currently seen in Firefox. (Tested in FF 3.0.15). Obviously these changes should be ported into the JxLib code stream, but I'm afraid that is beyond my level of expertise...

comment:12 Changed 11 years ago by pagameba

Chris, I've reviewed the jxlib patch and committed the changes to the jxlib project, please go ahead and commit your changes to fusion.

comment:13 Changed 11 years ago by liuar

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