OpenLayers OpenLayers

Ticket #467 (closed bug: fixed)

Opened 2 years ago

Last modified 2 years ago

isEquivalentURL() not complete for IE

Reported by: euzuro Assigned to:
Priority: major Milestone: 2.3 Release
Component: general Version: 2.3 RC1
Keywords: Cc:
State:

Description

as can be seen by loading

http://www.openlayers.org/~euzuro/foo/bar/test.html

in ff and ie, all anchors are not created equal.

looks like if we really want to support relative URLs in IE, we're going to either come up with another trick or else write a parser. :-(

Attachments

urls.patch (6.1 kB) - added by euzuro on 01/12/07 05:37:03.
this might be a fix. seems to pass tests tests on ie6, ff, moz, and opera

Change History

01/10/07 19:47:00 changed by euzuro

FF:

a.href = barf/barf/fly.html
a.pathname = /~euzuro/foo/bar/barf/barf/fly.html
a.host = www.openlayers.org

a.href = http://www.openlayers.org/~euzuro/foo/bar/barf/barf/fly.html
a.pathname = /~euzuro/foo/bar/barf/barf/fly.html
a.host = www.openlayers.org

windowpath: /~euzuro/foo/bar/test.html

IE:

a.href = barf/barf/fly.html
a.pathname = 
a.host = 

a.href = http://www.openlayers.org/~euzuro/foo/bar/barf/barf/fly.html
a.pathname = ~euzuro/foo/bar/barf/barf/fly.html
a.host = www.openlayers.org:80

windowpath: /~euzuro/foo/bar/test.html

01/10/07 19:50:31 changed by euzuro

the above comment, IE is IE6

here is IE7:

 a.href = barf/barf/fly.html
 a.pathname = barf/barf/fly.html
 
 b.href = http://www.openlayers.org/~euzuro/foo/bar/barf/barf/fly.html
 b.pathname = ~euzuro/foo/bar/barf/barf/fly.html

01/11/07 13:01:10 changed by tschaub

It might be more productive to figure out why checkImgURL is needed at all (instead of duplicating the browser's path parser). Maybe that is well understood by others, but it seems a bit voodoo to me (and potentially a memory management issue).

01/12/07 05:37:03 changed by euzuro

  • attachment urls.patch added.

this might be a fix. seems to pass tests tests on ie6, ff, moz, and opera

01/12/07 05:37:25 changed by euzuro

  • keywords set to review.

01/19/07 10:21:55 changed by crschmidt

  • keywords changed from review to pullup.

Applied in r2162. Marking for pullup to 2.3.

01/22/07 22:15:12 changed by crschmidt

  • keywords deleted.
  • status changed from new to closed.
  • resolution set to fixed.