Bug 142718 - DOCX: Writer try to show thousands pages instead only three
Summary: DOCX: Writer try to show thousands pages instead only three
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
(earliest affected) release
Hardware: All All
: high major
Assignee: Not Assigned
Keywords: bibisected, filter:docx, regression
Depends on:
Blocks: DOCX-Images
  Show dependency treegraph
Reported: 2021-06-08 14:33 UTC by Roman Kuznetsov
Modified: 2021-06-21 03:29 UTC (History)
4 users (show)

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

DOCS example (1.59 MB, application/vnd.openxmlformats-officedocument.wordprocessingml.document)
2021-06-08 14:34 UTC, Roman Kuznetsov
Bibisect log (8.51 KB, text/plain)
2021-06-08 17:54 UTC, Telesto

Note You need to log in before you can comment on or make changes to this bug.
Description Roman Kuznetsov 2021-06-08 14:33:41 UTC
This one follows from bug 96840

DOCX: Writer try to show thousands pages instead only three

Steps to Reproduce:
1. Try open a DOCX from attach and try scroll down with mouse wheel
2. Look at status bar =>  It shows thousands pages instead pair
3. Look at the document => All pages are empty

Actual Results:
Writer try to show thousands pages instead only three

Expected Results:
Writer just shows only three pages with images

Reproducible: Always

User Profile Reset: No

Additional Info:
Version: (x64) / LibreOffice Community
Build ID: 033fdd5aab3c44af26c7b53393afd2a7a8a056cd
CPU threads: 4; OS: Windows 6.1 Service Pack 1 Build 7601; UI render: Skia/Raster; VCL: win
Locale: ru-RU (ru_RU); UI: en-US
Calc: CL
Comment 1 Roman Kuznetsov 2021-06-08 14:34:38 UTC
Created attachment 172707 [details]
DOCS example
Comment 2 Telesto 2021-06-08 17:32:59 UTC
Repro with
Version: (x64) / LibreOffice Community
Build ID: 3b57ebb445df8a2bc3d916ea79f8af45e20e4e62
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: nl-NL
Calc: CL

and with

and with
Build ID: 8a35821d8636a03b8bf4e15b48f59794652c68ba

but not with
Comment 3 Telesto 2021-06-08 17:54:11 UTC
Created attachment 172708 [details]
Bibisect log

Bibisect will be hard.. 

There is period where Libo doesn't launch (the skipped commits in bibisect). Followed by period of an empty page (marked as bad). And when the image shows again, the page counter starts running.. 

But well, the first ' skipped commit'  seems could possible be it:

author	Oliver-Rainer Wittmann <orw@apache.org>	2014-07-15 12:15:37 +0000
committer	Caolán McNamara <caolanm@redhat.com>	2014-07-16 09:01:02 +0100
commit 133d6d8d4dec497276c3a9b0ba6761cffd322f81 (patch)
tree 26282ae68d27e44ff6bf7ad22023272cf4d1bf54
parent 7dd5f9308895644ae6ece56253c892046cc6ed5e (diff)
Resolves: #i121443# register text frames and graphics...
on the correct page frame

- trigger formatting of selected object when moving view to the selected object
or when graphical horizontal line has been inserted.
Comment 4 Timur 2021-06-09 09:00:50 UTC
In 4.3 there were changes: 3 pages OK, LO doesn't load because of 9f308fbc02439e25f8932314a9374c205ebdbc4c, 1 or 2 empty pages, page counting NOK.
So page counting is bad fix to empty pages. 

For 1 or 2 empty pages, the first bad commit should be in range:
To test, it's 64e710e1dd3fb676c7def4119e2b5a674cb799d2 and 43b3580df87b2bf3467d0efaf21f3e39d2e03f63 in bibisect.
To me these seem suspicious:

author	Tomaž Vajngerl <tomaz.vajngerl@collabora.com>	2014-03-05 17:59:51 +0100
commit 85b7de13f8a8417a9dd642fe00569c45c5907a1a (patch)
writerfilter: add reading of wp14:anchorId

author	Tomaž Vajngerl <tomaz.vajngerl@collabora.com>	2014-03-05 18:43:04 +0100
commit ca6e60d4c014e7471acc7a7601148fe751634f27 (patch)
writerfilter: handle anchorId and put it into the FrameGrabBag

author	Tomaž Vajngerl <tomaz.vajngerl@collabora.com>	2014-03-05 18:44:52 +0100
commit 7452e530c01193de2dedf54b2f91e797f341f19e (patch)
oox & writerfilter: add "anchorId" token and update model.xml

For page counting, the first bad commit 8acaddcdf3f1e2b79f8435ffc89e7880a59c6a1d is one of two sources:

author	Caolán McNamara <caolanm@redhat.com>	2014-03-06 17:23:19 +0000
commit af523f604df0da28687604d42c24f6dd3765f31a (patch)
convert acceleration config page to .ui

author	Caolán McNamara <caolanm@redhat.com>	2014-03-07 08:52:34 +0000
commit f4f2ba8dd15fde198556d37cd0e1577b84985282 (patch)
callcatcher: update unused code
Comment 5 Timur 2021-06-18 09:22:57 UTC
Hi Xisco, I did a bibisect here, please confirm and see who should be called in.