Bug 107628 - Filesave: Table position wrong on RT (filesave and fileopen) in LO, behind chart in DOC and overlapping chart in DOCX (anchor changed?) - position OK in MSO
Summary: Filesave: Table position wrong on RT (filesave and fileopen) in LO, behind ch...
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
(earliest affected) release
Hardware: All All
: medium normal
Assignee: Not Assigned
Keywords: bibisected, bisected, filter:doc, regression
: 107623 (view as bug list)
Depends on:
Blocks: DOC-Tables Regressions-alg_writerframes DOCX-Anchor-and-Text-Wrap DOC-Anchor-and-Text-Wrap
  Show dependency treegraph
Reported: 2017-05-04 15:17 UTC by Timur
Modified: 2022-01-05 14:59 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:
Regression By:

Test ODT with chart and table (27.17 KB, application/vnd.oasis.opendocument.text)
2017-05-04 15:17 UTC, Timur
Test ODT compare with DOC - chart table.png (101.06 KB, image/png)
2017-05-04 15:19 UTC, Timur
details of bibisect in 43max repository (6.35 KB, text/plain)
2017-05-12 03:34 UTC, Terrence Enger
Test compared in LO 6.4+ as ODT/DOC/DOCX and MSO 2016 as DOC/DOCX (139.96 KB, image/png)
2019-10-11 06:38 UTC, Timur

Note You need to log in before you can comment on or make changes to this bug.
Description Timur 2017-05-04 15:17:55 UTC
Created attachment 133073 [details]
Test ODT with chart and table

Open test ODT with chart and table and save as DOC, reopen DOC.
Table position is wrong, it overlapped chart, maybe because anchor changed (or not, Bug 37002 and Bug 49179 are closed).
Looks like it was OK up to 4.1.6 and then wrong since 4.2.0, first table was behind and later in front of chart. 
I guess bibisect would be useful.
Comment 1 Timur 2017-05-04 15:19:29 UTC
Created attachment 133075 [details]
Test ODT compare with DOC - chart table.png
Comment 2 Timur 2017-05-04 15:21:50 UTC
*** Bug 107623 has been marked as a duplicate of this bug. ***
Comment 3 Telesto 2017-05-04 17:07:57 UTC
Setting to NEW. I can confirm, see bug 107623
Comment 4 Terrence Enger 2017-05-12 03:34:01 UTC
Created attachment 133253 [details]
details of bibisect in 43max repository

Working on debian-stretch in the bibisect-43max repository, I conclude

          commit    s-h
          --------  --------
    good  e4444f5a  c7190108
    bad   af102580  6e61ecd0

I am removing keyword bibisectRequest and adding bisected.
Comment 5 Xisco Faulí 2017-05-12 20:12:50 UTC
Regression introduced by:

author	Armin Le Grand <alg@apache.org>	2014-03-19 16:17:02 (GMT)
committer	Miklos Vajna <vmiklos@collabora.co.uk>	2014-03-28 13:31:08 (GMT)
commit 6e61ecd09679a66060f932835622821d39e92f01 (patch)
tree 4ce71adaa6da133cab09660b3efc1341104022a2
parent c7190108f02921868cb617040aebdb2d22c02c1f (diff)
Merge back branch alg_writerframes to trunk

Adding Cc: to Armin Le Grand
Comment 6 Timur 2018-10-10 14:57:11 UTC Comment hidden (obsolete)
Comment 7 QA Administrators 2019-10-11 02:35:30 UTC Comment hidden (obsolete)
Comment 8 Timur 2019-10-11 06:38:04 UTC
Created attachment 154917 [details]
Test compared in LO 6.4+ as ODT/DOC/DOCX and MSO 2016 as DOC/DOCX

(In reply to Timur from comment #6)
> Repro 6.2+. 

Now with 6.4+ I did a proper test, saved ODT as DOC and DOCX and opened those both in LO and MSO. 
Seems that LO opens wrong both round-tripped files (table missing in DOC and overlapping chart in DOCX, which if different from before when reported) while MSO opens table with chart right (MSO shows chart from DOCX wrong but that another issue). 

So this sound now more as fileopen (it's own file) than filesave but nevermind until we have a fix.
Comment 9 Timur 2019-10-11 06:50:32 UTC
Note: another issue with DOC/DOCX is number of footer rows but that's related to bug 94801.
Comment 10 Timur 2020-01-17 09:21:37 UTC Comment hidden (obsolete)
Comment 11 Armin Le Grand 2022-01-05 09:47:27 UTC
This commit was developed for another code base, and not merged by me. For complex changes like this, side-effects are to be expected; sadly I dont't have the cycles to deal with all the fallout. Un-Ccing myself for the while.
Comment 12 Timur 2022-01-05 14:59:00 UTC
Just to note that this regression from 4.2 is still repro in 7.3 master.