Ticket #2649 (closed Bug: fixed)

Opened 16 months ago

Last modified 15 months ago

IE: Error on Find dialog with "match whole word"

Reported by: fredck Owned by: martinkou
Priority: Normal Milestone: FCKeditor 2.6.4
Component: General Version: SVN (FCKeditor)
Keywords: Confirmed IE Review+ Cc:

Description

Steps to Reproduce

  1. In the editing area, type "hello you hello".
  2. Open the "Find" dialog.
  3. Check the "Match whole word" option.
  4. Type "hello" in the find field.
  5. Click on "Find".

A JavaScript error is thrown.

It looks like an IE only issue. Works well with Firefox.

Attachments

2649.patch Download (1.1 KB) - added by martinkou 16 months ago.
2649_2.patch Download (1.3 KB) - added by martinkou 16 months ago.

Change History

Changed 16 months ago by martinkou

  • owner set to martinkou
  • status changed from new to assigned

Changed 16 months ago by martinkou

Changed 16 months ago by martinkou

  • keywords Review? added

Changed 16 months ago by martinkou

  • milestone set to FCKeditor 2.6.4

I guess we can put this in 2.6.4, the fix is very simple.

Changed 16 months ago by fredck

  • keywords Review- added; Review? removed

This fix makes no difference for me. I still have the same error.

When opening the debugger, the error happens at line 145 of fck_replace.html because "curNode" is null ('length' is null or not an object).

Changed 16 months ago by martinkou

  • keywords Review? added; Review- removed

Ok, there are actually two possible errors there:

  1. When the search item is at exactly the end of the document, which was fixed in the first patch.
  2. When the search item is at exactly the beginning of the document, which is what's missing in the first patch.

Alright... posting updated patch for problem 2.

Changed 16 months ago by martinkou

Changed 15 months ago by fredck

  • keywords Review+ added; Review? removed

Changed 15 months ago by martinkou

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

Fixed with [2778].

Click here for more info about our SVN system.

Note: See TracTickets for help on using tickets.