Issue 121897 - Multiple undo/redo operation erase first letter on nearby indexed words
Summary: Multiple undo/redo operation erase first letter on nearby indexed words
Status: CLOSED FIXED
Alias: None
Product: Writer
Classification: Application
Component: editing (show other issues)
Version: 3.4.1
Hardware: All All
: P3 Blocker (vote)
Target Milestone: 4.1.0
Assignee: zhengfan
QA Contact: edoardopanfili
URL:
Keywords:
Depends on: 123389
Blocks:
  Show dependency tree
 
Reported: 2013-03-14 13:52 UTC by info
Modified: 2017-05-20 10:35 UTC (History)
7 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: 3.4.1
Developer Difficulty: ---


Attachments
Collection of 6 screenshots and two .odt files. (435.96 KB, application/zip)
2013-03-14 13:52 UTC, info
no flags Details
STEP1 - preview of original text (index_err.odt) (71.54 KB, image/png)
2013-03-19 15:53 UTC, info
no flags Details
STEP2 - text selection is made ready for moving (72.21 KB, image/png)
2013-03-19 15:54 UTC, info
no flags Details
STEP3 - selected text is now on new location (74.20 KB, image/png)
2013-03-19 15:54 UTC, info
no flags Details
STEP4 - selected text is moved back by using 'Undo' option (74.83 KB, image/png)
2013-03-19 15:54 UTC, info
no flags Details
STEP5 - text is moved once again using 'Redo' option (74.03 KB, image/png)
2013-03-19 15:55 UTC, info
no flags Details
STEP6 - preview of saved text with visible damages (index_err_final.odt) (75.55 KB, image/png)
2013-03-19 15:55 UTC, info
no flags Details
Original text document used for screenshots (11.12 KB, application/octet-stream)
2013-03-19 15:56 UTC, info
no flags Details
Saved text that contain errors (11.16 KB, application/octet-stream)
2013-03-19 15:56 UTC, info
no flags Details
STEP1 - add new index (75.66 KB, image/png)
2013-03-25 15:51 UTC, info
no flags Details
STEP2 - define new index entry (76.46 KB, image/png)
2013-03-25 15:52 UTC, info
no flags Details
STEP3 - new index has been added (67.95 KB, image/png)
2013-03-25 15:53 UTC, info
no flags Details
Original text reproduced (12.49 KB, application/vnd.oasis.opendocument.text)
2013-03-25 19:15 UTC, Edwin Sharp
no flags Details
File derived from index_no_problem.odt (11.51 KB, application/octet-stream)
2013-03-26 15:00 UTC, info
no flags Details
Simplified Sample file (8.45 KB, application/vnd.oasis.opendocument.text)
2013-05-10 05:30 UTC, zhengfan
no flags Details

Note You need to log in before you can comment on or make changes to this issue.
Description info 2013-03-14 13:52:06 UTC
Created attachment 80405 [details]
Collection of 6 screenshots and two .odt files.

Finally, after more than two years I catch predictable pattern how to reproduce this serious problem if you are going to add index tags in your text. Bug is continuously present at least from version 3.2, and it is still here. 

Please find in attachment, 6 step how to reproduce it and original (index_err.odt) and modified document (index_err_final.odt).

I'll really appreciate anyone how will take a time to inspect what is going on, since it cost me until now lots of suffering, pain and time for proof reading.

Many thanks!
Comment 1 info 2013-03-19 15:53:40 UTC
Created attachment 80426 [details]
STEP1 - preview of original text (index_err.odt)
Comment 2 info 2013-03-19 15:54:06 UTC
Created attachment 80427 [details]
STEP2 - text selection is made ready for moving
Comment 3 info 2013-03-19 15:54:26 UTC
Created attachment 80428 [details]
STEP3 - selected text is now on new location
Comment 4 info 2013-03-19 15:54:46 UTC
Created attachment 80429 [details]
STEP4 - selected text is moved back by using 'Undo' option
Comment 5 info 2013-03-19 15:55:05 UTC
Created attachment 80430 [details]
STEP5 - text is moved once again using 'Redo' option
Comment 6 info 2013-03-19 15:55:34 UTC
Created attachment 80431 [details]
STEP6 - preview of saved text with visible damages (index_err_final.odt)
Comment 7 info 2013-03-19 15:56:11 UTC
Created attachment 80432 [details]
Original text document used for screenshots
Comment 8 info 2013-03-19 15:56:36 UTC
Created attachment 80433 [details]
Saved text that contain errors
Comment 9 Edwin Sharp 2013-03-25 12:30:47 UTC
Please describe how exactly the indexes were defined.
Comment 10 info 2013-03-25 15:51:30 UTC
Created attachment 80458 [details]
STEP1 - add new index
Comment 11 info 2013-03-25 15:52:06 UTC
Created attachment 80459 [details]
STEP2 - define new index entry
Comment 12 info 2013-03-25 15:53:09 UTC
Created attachment 80460 [details]
STEP3 - new index has been added
Comment 13 info 2013-03-25 15:54:18 UTC
(In reply to comment #9)
> Please describe how exactly the indexes were defined.

Please find in attachment new added screenshots.
Comment 14 Edwin Sharp 2013-03-25 19:15:04 UTC
Created attachment 80463 [details]
Original text reproduced

Rewritten using Rev. 1400866, the phenomena described by the originator can not be reproduced.
Comment 15 Edwin Sharp 2013-03-25 19:16:29 UTC
Undo redo operations won't cause the alleged bug.
Comment 16 info 2013-03-26 14:56:47 UTC
Thanks elish for quick response. I'm asking you a little more attention. I forgot to highlight an important detail: changes in indexed words become visible next time when you open saved changes. You cannot see any change (and that's why I had so many trouble to identify this problem) if you did not close a file after save and open it again.
If that still cannot help you to reproduce this problem please let me know which version you are using since I tried it again with your sample document using v.3.4.1.
Comment 17 info 2013-03-26 15:00:17 UTC
Created attachment 80466 [details]
File derived from index_no_problem.odt
Comment 18 Edwin Sharp 2013-03-26 15:11:59 UTC
This is strange.
I did close and reopen index_no_problem.odt before attaching here.
From index_no_problem2.odt I see the word models is problem, unlike original file where words mirror and better - the bug is inconsistent.
I am using Rev. 1400866.
Comment 19 hanya 2013-03-26 15:45:39 UTC
I can reproduce the problem too on Rev. 1457731.
In my opinion, 
<text:alphabetical-index-mark text:string-value="Mirror"/> bites the following 
character but not happen with 
the parts wrapped with text:alphabetical-index-mark-start and end tags.

In the attached screen shot, last "Simulation" part is correct even the 
index mark is there just before the word. Maybe this is because it was in 
the modified range.

So, it seems non ranged index mark has the problem?
Comment 20 info 2013-04-09 15:45:55 UTC
Please let me know do you need any additional assistance from my side since I can reproduce that bug every time.
Comment 21 zhengfan 2013-05-10 04:57:50 UTC
Seems it is reproduce-able on my local env: Win XP, AOO 4.0 dev snapshot.
Comment 22 zhengfan 2013-05-10 05:30:25 UTC
Created attachment 80658 [details]
Simplified Sample file
Comment 23 zhengfan 2013-05-14 07:57:20 UTC
I will take this issue.
Comment 24 SVN Robot 2013-05-17 03:36:27 UTC
"zhengfan" committed SVN revision 1483640 into trunk:
i121897, change the hints clearing method from RstAttr to clearSwpHhints.
Comment 25 zhengfan 2013-05-17 07:11:01 UTC
Fixed in R1483640.
Comment 26 edoardopanfili 2013-07-09 07:08:09 UTC
verified using info@envox.hr description
(error reproducible on openoffice 3.4)

all works fine on 

OS X 10.8.4
AOO400m3(Build:9702)  -  Rev. 1499347
2013-07-03 14:06:53 (Wed, 03 Jul 2013)

an on windows 8
AOO400m3(Build:9702)  -  Rev. 1499347
2013-07-03 15:11:47 (Mi, 03 Jul 2013)
Comment 27 Oliver-Rainer Wittmann 2013-09-30 14:26:17 UTC
closing in order to be able to reopen it
Comment 28 Oliver-Rainer Wittmann 2013-09-30 14:28:50 UTC
reopening.

the fix triggers bug 123389.
Thus, I reverted the fix and provide an alternative solution which does not cause bug 123389. The solution has been committed for bug 123389.
Thus, further verification needed for the next release
Comment 29 Oliver-Rainer Wittmann 2013-09-30 14:29:32 UTC
solved by alternative solution - see bug 123389
Comment 30 liuping 2014-02-27 09:47:00 UTC
Verify AOO410m1(Build:9750)  -  Rev. 1570848
 on windows7 , Pass