Ticket #3759 (closed Bug: fixed)

Opened 5 years ago

Last modified 5 years ago

Replace all only searches to the end of the document

Reported by: damo Owned by: garry.yao
Priority: Normal Milestone: CKEditor 3.0
Component: General Version:
Keywords: 3.0RC IBM Review+ Cc:

Description

When using replace all in a document and the cursor was placed in the middle of the document, the search only replaces those instances between the cursor and the end of the document.

Attachments

3759.patch (4.2 KB) - added by garry.yao 5 years ago.
3759_2.patch (1.0 KB) - added by garry.yao 5 years ago.

Change History

Changed 5 years ago by garry.yao

comment:1 Changed 5 years ago by garry.yao

  • Keywords RC Review? added
  • Owner set to garry.yao
  • Status changed from new to assigned

comment:2 Changed 5 years ago by garry.yao

Two updates, now with Match cyclic checked:

  1. The replace action will happen within full document;
  2. The find action will automatically start from document start without an 'alert' stopping.

comment:3 Changed 5 years ago by fredck

  • Keywords 3.0RC added; RC removed

comment:4 Changed 5 years ago by fredck

  • Keywords Review- added; Review? removed

It would be better if this fix could be limited to the reported case, not changing the current "cyclic" behavior for find and replace.

Changed 5 years ago by garry.yao

comment:5 Changed 5 years ago by garry.yao

  • Keywords Review? added; Review- removed

Now the replacement will happen at whole document range independent of cyclic.

comment:6 Changed 5 years ago by martinkou

  • Keywords Review+ added; Review? removed

Please remember to commit #3757 as well as replace is totally broken without it.

comment:7 Changed 5 years ago by garry.yao

  • Status changed from assigned to closed
  • Resolution set to fixed

Fixed with [3707].

Note: See TracTickets for help on using tickets.
© 2003 – 2012 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy