Ticket #2954 (closed Bug: fixed)
Option 'Match cyclic' not working
| Reported by: | garry.yao | Owned by: | martinkou |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.0 |
| Component: | General | Version: | SVN (FCKeditor) - Retired |
| Keywords: | Confirmed IBM Review+ | Cc: | damo |
Description
The config option of Match cyclic which should restart the search from the beginning of document which doesn't work.
Reproducing procedure:
- Open Replace by code sample;
- Make selection as below:
<p>This i^s some <strong>sample text</strong>. You are using <a href="http://www.fckeditor.net/">FCKeditor</a>.</p>
- Open the find/replace dialog, make sure only the match cyclic option is checked, fill in is in 'Find what' input, then click find button.
- Prompted with '...text not found', click find button again.
- Expected result: The document is highlighted as
<p>Th^is^ is some <strong>sample text</strong>. You are using <a href="http://www.fckeditor.net/">FCKeditor</a>.</p>
- Actual result: Again prompted '...text not found'.
Attachments
Change History
comment:1 Changed 4 years ago by martinkou
- Owner set to martinkou
- Keywords Confirmed added
- Status changed from new to assigned
comment:4 Changed 4 years ago by martinkou
- Keywords Review? added
The bug is caused by line 485 of the Find dialog file. It resets the search position to the position of the selection before the dialog is opened after the find reached the end of document. That's wrong. It should reset the search position back to the start of the document instead.
comment:6 Changed 4 years ago by martinkou
- Status changed from assigned to closed
- Resolution set to fixed
Fixed with [3167].
Click here for more info about our SVN system.
Note: See
TracTickets for help on using
tickets.

The bug isn't reproducible outright, but after repeating the procedure a few times does trigger it.