Opened 16 years ago
Closed 16 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 16 years ago by
Keywords: | Confirmed added |
---|---|
Owner: | set to Martin Kou |
Status: | new → assigned |
comment:2 Changed 16 years ago by
Keywords: | IBM added |
---|
Changed 16 years ago by
Attachment: | 2954.patch added |
---|
comment:4 Changed 16 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 16 years ago by
Keywords: | Review+ added; Review? removed |
---|
comment:6 Changed 16 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.