Bug 841720 - [Project-wide search replace] Uninvolved entries are mistakely wiped out when undo in search both mode
Summary: [Project-wide search replace] Uninvolved entries are mistakely wiped out when...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Zanata
Classification: Retired
Component: Component-Logic
Version: 1.7
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ---
: 1.7
Assignee: David Mason
QA Contact: Ding-Yi Chen
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-07-20 01:50 UTC by Ding-Yi Chen
Modified: 2012-09-13 00:09 UTC (History)
1 user (show)

Fixed In Version: 1.7.0-alpha-2-SNAPSHOT (20120720-1208)
Story Points: ---
Clone Of:
Environment:
Last Closed: 2012-09-11 05:11:19 UTC
Embargoed:


Attachments (Terms of Use)

Description Ding-Yi Chen 2012-07-20 01:50:20 UTC
Description of problem:

After performed project-wid search and replace in "search both mode", undo the replace will mistakely wipe out the uninvolved entries, which do not have the search term in target.

Version-Release number of selected component (if applicable):
Zanata version 1.7-SNAPSHOT (20120720-0026)

How reproducible:
Always

Steps to Reproduce:
1. Visit a document.
2. Find two rows  (row A and row B), that contains same word. Suppose the word is "TERM"
3. Click at row A, copy from source, but remove "TERM", save as approved
4. Click at row B, copy from source, save as approved
5. Click search and replace, and select "search both".
6. Search "TERM", click "select all",  and relace with "WORD"
   You should see that row B is replaced, but row A is untouched.
7. Click on notification panel, click "undo"
  
Actual results:
Content of row A is wiped out.

Expected results:
Content of row A is kept.

Additional info:

Comment 1 David Mason 2012-07-20 02:08:22 UTC
Prevented system trying to undo text flows that were not changed.

See https://github.com/zanata/zanata/commit/f8f003e9dce5e15baf653746f08fc47875afc1a2

Comment 2 Ding-Yi Chen 2012-07-20 04:31:18 UTC
VERIFIED with Zanata version 1.7.0-alpha-2-SNAPSHOT (20120720-1208)


Note You need to log in before you can comment on or make changes to this bug.