Opened 17 years ago
Closed 17 years ago
#2954 closed Bug (fixed)
Option 'Match cyclic' not working
| Reported by: | Garry Yao | Owned by: | Martin Kou |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.0 |
| Component: | General | Version: | SVN (FCKeditor) - Retired |
| Keywords: | Confirmed IBM Review+ | Cc: | Damian |
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 (1)
Change History (7)
comment:1 Changed 17 years ago by
| Keywords: | Confirmed added |
|---|---|
| Owner: | set to Martin Kou |
| Status: | new → assigned |
comment:2 Changed 17 years ago by
| Keywords: | IBM added |
|---|
Changed 17 years ago by
| Attachment: | 2954.patch added |
|---|
comment:4 Changed 17 years ago by
| 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:5 Changed 17 years ago by
| Keywords: | Review+ added; Review? removed |
|---|
comment:6 Changed 17 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Fixed with [3167].
Click here for more info about our SVN system.

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