Ticket #1586 (closed Bug: fixed)

Opened 7 years ago

Last modified 6 years ago

[IE] JavaScript error on edit

Reported by: Vincent.cabs Owned by:
Priority: Normal Milestone:
Component: Project : MediaWiki+FCKeditor Version: FCKeditor 2.5 Beta
Keywords: Confirmed IE Cc:

Description

Is anyone able to solve that problem?

I have a major crash with IE 6 SP1 and Mediawiki+FCKeditor.

The crash happens when I change a certain bit of an article and when I press "save" or "wikitext". it is very disturbing as I don't know how to debug that.

You can reproduce the crash by following those steps in IE 6:

  1. go on http://mediawiki.fckeditor.net/index.php/TestMyBug : a new page I created

2.. change the title : e.g. remove the ':'

  1. click on 'wikitext' or on the 'save' button

... IE crashes !!

info : Windows 2000, IE 6.0.2800 SP1. works fine with Firefox 2

Attachments

bug FCKeditor.jpg (151.8 KB) - added by Vincent.cabs 7 years ago.
IE crashes on a simple Edit
1586.patch (565 bytes) - added by fredck 7 years ago.
Patch proposal
TC_1586.png (13.1 KB) - added by w.olchawa 6 years ago.

Change History

Changed 7 years ago by Vincent.cabs

IE crashes on a simple Edit

comment:1 follow-up: ↓ 2 Changed 7 years ago by fredck

  • Keywords Pending added; Internet Explorer crash removed
  • Priority changed from High to Normal
  • Milestone FCKeditor 2.5 deleted

I was able to consistently reproduce this problem with IE6. It works well with IE7.

It is not an issue with FCKeditor, but with the MediaWiki plugin. I've debugged it, and found the culprit. I'm attaching a possible patch for it.

I haven't tested it. Could you please confirm that it solved the problem?

Changed 7 years ago by fredck

Patch proposal

comment:2 in reply to: ↑ 1 ; follow-up: ↓ 3 Changed 7 years ago by Vincent.cabs

It works!!!

Thank you sooooo much!

comment:3 in reply to: ↑ 2 ; follow-up: ↓ 4 Changed 7 years ago by Vincent.cabs

Just one question : how did you find the bug ?

comment:4 in reply to: ↑ 3 Changed 7 years ago by fredck

  • Keywords Pending removed
  • Status changed from new to closed
  • Resolution set to fixed

Replying to Vincent.cabs:

Just one question : how did you find the bug ?

Well... some intuition helped :) Let's say that I have a long experience on fighting browsers bugs :)

I've committed the fix to the SVN:
http://svn.wikimedia.org/viewvc/mediawiki?view=rev&revision=27935

comment:5 Changed 6 years ago by w.olchawa

#1935 has been marked as DUP

comment:6 Changed 6 years ago by w.olchawa

  • Status changed from closed to reopened
  • Resolution fixed deleted

As for Vincent.cabs suggestion in #1935 I'm reopening this ticket.

comment:7 follow-up: ↓ 8 Changed 6 years ago by w.olchawa

  • Keywords Pending added

I'm not able to reproduce this bug in IE6, but in WIN SP2.

comment:8 in reply to: ↑ 7 Changed 6 years ago by Vincent.cabs

Replying to w.olchawa:

I'm not able to reproduce this bug in IE6, but in WIN SP2.

Go back to http://mediawiki.fckeditor.net/index.php/TestMyBug. You should be able to reproduce the bug with IE 6. Don't know why. Is there a charset problem ?

comment:9 Changed 6 years ago by w.olchawa

  • Keywords Confimed IE added; Pending removed
  • Summary changed from IE 6 SP1 crashes on edit to [IE] JavaScript erroe on edit

Confirmed in IE6 and IE7. I've attached a screenshot of the error. It's possible that we had a similar bug before but I'm no certain.

Changed 6 years ago by w.olchawa

comment:10 Changed 6 years ago by w.olchawa

  • Summary changed from [IE] JavaScript erroe on edit to [IE] JavaScript error on edit

comment:11 Changed 6 years ago by w.olchawa

  • Keywords Confirmed added; Confimed removed

comment:12 follow-up: ↓ 13 Changed 6 years ago by wwalc

  • Status changed from reopened to closed
  • Resolution set to fixed

I have committed patch provided by fredck (1586.patch) with [1820] and fixed the "object doesn't support this property or method" with [1821].

Other than that, it seems to be a problem to have two SVN repositories...

comment:13 in reply to: ↑ 12 Changed 6 years ago by Vincent.cabs

It works, thanks very much !

comment:14 Changed 6 years ago by w.olchawa

#2026 has been marked as DUP

Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy