Ticket #277 (closed task: fixed)

Opened 9 months ago

Last modified 2 months ago

IE crashes on loggingpane (bug /w fix)

Reported by: anonymous Assigned to: cederberg@gmail.com
Priority: low Milestone: MochiKit 1.4
Component: component1 Version:
Severity: minor Keywords:
Cc:

Description

http://trac.mochikit.com/browser/mochikit/trunk/MochiKit/LoggingPane.js#L78 

var url = win.location.href.split("?")[0].replace(/[#:\/.><&-]/g, "_");
var name = uid + "_" + url;
var nwin = win.open("", name, "dependent,resizable,height=200");

IE7 gives an invalid argument error on the third line when when the url has a "%20" in it.
e.g "www.example.com/cool%20page.html".
Regexpression in first line changed below: (%20) 

var url = win.location.href.split("?")[0].replace(/[#:\/.><&-(%20)]/g, "_");

Big thanks to everyone who contributes here.

Change History

05/19/08 13:06:19 changed by cederberg@gmail.com

  • owner changed from somebody to cederberg@gmail.com.
  • status changed from new to assigned.
  • milestone set to MochiKit 1.4.

Thanks for the patch. Moving to MochiKit 1.4 milestone.

05/20/08 01:51:10 changed by cederberg@gmail.com

  • status changed from assigned to closed.
  • resolution set to fixed.

Fixed in [1377] by replacing all % in the URL with _ (similar to provided patch, but more generic).