Ticket #2026 (closed Bug: duplicate)

Opened 8 months ago

Last modified 7 months ago

IE6 + MediaWiki Extension: Minor edits within an existing line crash IE6

Reported by: rbowmaker Owned by:
Priority: Normal Milestone:
Component: Project : MediaWiki+FCKeditor Version: FCKeditor 2.5.1
Keywords: Pending WorksForMe Cc:

Description

This occurs in both IE6 SP1 and SP2, on both Windows XP and 2000. FCKEditor version is 2.5.1

Steps to reproduce: 1. In IE6, go to FCKEDitor + MediaWiki website sandbox 2. Edit the sandbox page by adding the following line at the start (note the capital 'I'):

blah blah Integration blah blah

3. Save the edit - this will work ok. 4. Edit the page again, and change the capital 'I' to a lower case 'i'. 5. Try to save this edit. Kaboom!

IE6 craches with an access violation.

Attachments

FCKEditorIE6Crash.JPG (146.7 KB) - added by rbowmaker 8 months ago.
Screen dump of crash

Change History

  Changed 8 months ago by w.olchawa

  • keywords Pending WorksForMe added
  • version set to FCKeditor 2.5.1
  • component changed from General to Project : MediaWiki+FCKeditor

Hi

I've tried to reproduce your problem but I only found that an additional <br> is added upon save. IE6 didn't crash at all. IE7 gave the same results. So could you please check the sanbox again or please let me know if I am missing something due to the reproduction.

Thanks in advance.

Changed 8 months ago by rbowmaker

Screen dump of crash

  Changed 8 months ago by rbowmaker

Yes, I can still reproduce the problem in the sandbox using the above steps to reproduce exactly.

IE version is 6.2.2900.2180.xpsp_sp2_gdr.050301-1519, but I can also reproduce it on an earlier IE6 version too.

I tried it on IE7 (on a different PC) too, and got the extra line break.

  Changed 8 months ago by rbowmaker

Perhaps if the extra <br> bug is fixed, that may also eliminate the crashing?

  Changed 8 months ago by w.olchawa

Hi again.

Could you please let us know if it is the IE6 version

6.2.2900.2180.xpsp_sp2_gdr.050301-1519

that you are using? Isn't it 6.0.2900.2180 ?

I've used 6.0.2900.2180.xpsp_sp2_gdr.070227-2254 for the tests and despite the <br> bug everything worked well.

Thanks in advance for the info.

  Changed 8 months ago by rbowmaker

Sorry, the '2' was a typo...

the IE version is 6.0.2900.2180.xpsp_sp2_gdr.050301-1519

follow-up: ↓ 7   Changed 8 months ago by w.olchawa

Hi again.

Thanks for the info. It seems that I'm unable to reproduce the bug because of the different compilations of IE6 that we have. I use a newer version (ver. from 27th February 2007).

If it wouldn't be a problem for you I would suggest upgrading to a newer compilation. But if you are unable to do so I will confirm this bug of course.

Thanks for you patience.

in reply to: ↑ 6   Changed 7 months ago by rbowmaker

Sorry for the delay.

I ran windows update, which installed an unrelated security update (KB944533), which didn't change the IE version at all. However the crashing has now gone away, and I just get the extra <br>. So this might be a hard one to pin down.

However, the extra <br> is still a real problem, and is probably related. Once it occurs, you have to go into WikiText to fix it, which kind of defeats the purpose of the editor. Do you know if a bug has been raised for the extra <br>? I'll see if I can find it.

  Changed 7 months ago by rbowmaker

It looks like this is a duplicate of #1586 That ticket has a patch with it to fix the problem.

  Changed 7 months ago by rbowmaker

The extra <br> problem is covered by several tickets. The main one seems to be #1547 . The suggested fix there is the same as for the crashing problem in #1586 i.e. Don't call Normalize() for IE.

  Changed 7 months ago by w.olchawa

  • status changed from new to closed
  • resolution set to duplicate

The <br> is a different issue tough. Thanks for investigating. I will mark this one as

DUP of #1586

Note: See TracTickets for help on using tickets.