Bug 150235 - Separator line in columned section
Summary: Separator line in columned section
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
7.2.0.0.beta1+
Hardware: All All
: medium normal
Assignee: Mike Kaganski
URL:
Whiteboard: target:7.5.0 target:7.4.1 target:7.3.6
Keywords: bibisected, bisected, regression
Depends on:
Blocks:
 
Reported: 2022-08-02 18:07 UTC by Richard George
Modified: 2022-08-17 16:41 UTC (History)
3 users (show)

See Also:
Crash report or crash signature:


Attachments
Image of observed behaviour (123.68 KB, image/jpeg)
2022-08-02 18:07 UTC, Richard George
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Richard George 2022-08-02 18:07:32 UTC
Created attachment 181552 [details]
Image of observed behaviour

Two issues that I suspect are connected.

New file.
Define multi-column section with separator line defined at 50% (top or bottom - doesn't matter). Observe that the section is created correctly.

Save and exit.

Load file

The previously defined section now extends to 100% - not 50% as specified. Edit section. Number of columns is retained, BUT the seperator line is set to 'None'.
Comment 1 Richard George 2022-08-02 18:10:06 UTC
BTW - I've no idea what the earliest version this exists with, but I've seen the same issue for a considerable number of versions - certainly from v4, but I suspect it's a long term bug.
Comment 2 raal 2022-08-03 15:26:44 UTC
Confirm with Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: 3ccbfaaf95005a34ca64ad250463ef5ce8842f43
CPU threads: 4; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: cs-CZ (cs_CZ.UTF-8); UI: en-US
Calc: threaded

works in Version 4.1.0.0.alpha0+ (Build ID: efca6f15609322f62a35619619a6d5fe5c9bd5a)
Comment 3 raal 2022-08-03 15:52:15 UTC
Should be bibisectable with linux-64-7.2
Comment 4 Richard George 2022-08-03 16:17:23 UTC
My current versions are 7.3.5.1 (x64) and 7.4.0.1 (parallel installation)
Comment 5 LeroyG 2022-08-04 23:27:44 UTC
And, if after reloading the document, do you choose menu Format - Columns, you will see that the Separator Line Style now is set to None.

Version: 7.2.7.2 (x64) / LibreOffice Community
Build ID: 8d71d29d553c0f7dcbfa38fbfda25ee34cce99a2
CPU threads: 4; OS: Windows 10.0 Build 19043; UI render: default; VCL: win
Locale: es-MX (es_ES); UI: en-US
Calc: CL
Comment 6 raal 2022-08-06 10:53:17 UTC
This seems to have begun at the below commit.
Adding Cc: to Mike Kaganski; Could you possibly take a look at this one?
Thanks
 2ae1375e6189d0b323cb2cb6bfc585c7b258c379 is the first bad commit
commit 2ae1375e6189d0b323cb2cb6bfc585c7b258c379
Author: Jenkins Build User <tdf@pollux.tdf>
Date:   Tue Jun 8 15:27:15 2021 +0200

    source 95ebd24a629b4c8cd62cc20c0701683512cc8fa0

https://git.libreoffice.org/core/+/95ebd24a629b4c8cd62cc20c0701683512cc8fa0
Comment 7 Mike Kaganski 2022-08-06 13:55:58 UTC
https://gerrit.libreoffice.org/c/core/+/137900
Comment 8 Commit Notification 2022-08-06 16:32:42 UTC
Mike Kaganski committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/f1f6c82e613ace836aa78a822688721fb7fb5e80

tdf#150235: use correct types in XMLTextColumnsExport

It will be available in 7.5.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 9 Richard George 2022-08-07 10:54:08 UTC
That's fixed, but there is a further issue with this (might need a new report raising?)

Set the separator line width to ...

0.5 .. comes back as 0.55
0.75 ..  0.8
1.0 .. 0.95

1.25 - 1.75 are ok, then

2.0 ... 2.05
2.25 .. 2.3
2.5 .. 2.45

3.0 is ok

Weird .... gave up after those ...
Comment 10 Mike Kaganski 2022-08-07 11:06:43 UTC
(In reply to Richard George from comment #9)
> That's fixed, but there is a further issue with this (might need a new
> report raising?)

One issue per one report please.
Comment 11 Commit Notification 2022-08-08 09:07:05 UTC
Mike Kaganski committed a patch related to this issue.
It has been pushed to "libreoffice-7-4":

https://git.libreoffice.org/core/commit/a92df2a6771af8cb510e30d9d011325807f79edd

tdf#150235: use correct types in XMLTextColumnsExport

It will be available in 7.4.1.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 12 Commit Notification 2022-08-09 08:49:58 UTC
Mike Kaganski committed a patch related to this issue.
It has been pushed to "libreoffice-7-3":

https://git.libreoffice.org/core/commit/43ba49cd33af6bc8328edb0cb8c289bab266e346

tdf#150235: use correct types in XMLTextColumnsExport

It will be available in 7.3.6.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.
Comment 13 BogdanB 2022-08-17 16:41:23 UTC
Thanks for solving this bug.

Verified. Fixed.

Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: 641d92a73e5b3d0e062e16ed4b42236e1a4796a5
CPU threads: 16; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: ro-RO (ro_RO.UTF-8); UI: en-US
Calc: threaded

Using gdb i am getting this messages, if they are helpful.

warn:stoc:22657:22657:stoc/source/javaloader/javaloader.cxx:381: getJavaVM returned null
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:stoc:22657:22657:stoc/source/javaloader/javaloader.cxx:381: getJavaVM returned null
warn:svl.items:22657:22657:svl/source/items/itempool.cxx:442: old secondary pool: EditEngineItemPool of pool: XOutdevItemPool must be empty.
warn:svl.items:22657:22657:svl/source/items/itempool.cxx:442: old secondary pool: EditEngineItemPool of pool: XOutdevItemPool must be empty.
warn:svl.items:22657:22657:svl/source/items/itempool.cxx:442: old secondary pool: EditEngineItemPool of pool: XOutdevItemPool must be empty.
warn:svl.items:22657:22657:svl/source/items/itempool.cxx:442: old secondary pool: EditEngineItemPool of pool: XOutdevItemPool must be empty.
warn:svl.items:22657:22657:svl/source/items/itempool.cxx:442: old secondary pool: EditEngineItemPool of pool: XOutdevItemPool must be empty.
warn:svl:22657:22657:svl/source/undo/undo.cxx:1070: SfxUndoManager::MarkTopUndoAction(): suspicious call!
warn:sw.core:22657:22657:sw/source/core/docnode/node.cxx:2002: Wrong cond collection, skipping check of Cond Colls.
warn:legacy.osl:22657:22657:svx/source/dialog/rulritem.cxx:480: Wrong MemberId!
warn:legacy.osl:22657:22657:xmloff/source/transform/OOo2Oasis.cxx:1945: duplicate doc handler
warn:sw.core:22657:22657:sw/source/core/attr/format.cxx:216: ~SwFormat: format still has clients on death, but parent format is missing: Frameformat
warn:legacy.osl:22657:22657:svx/source/dialog/rulritem.cxx:480: Wrong MemberId!
warn:legacy.osl:22657:22657:xmloff/source/style/xmlexppr.cxx:907: no prop type specified
warn:legacy.osl:22657:22657:xmloff/source/style/xmlexppr.cxx:907: no prop type specified
warn:legacy.osl:22657:22657:xmloff/source/style/xmlexppr.cxx:907: no prop type specified
warn:legacy.osl:22657:22657:xmloff/source/style/xmlexppr.cxx:907: no prop type specified
warn:legacy.osl:22657:22657:svx/source/dialog/rulritem.cxx:480: Wrong MemberId!
warn:sw.core:22657:22657:sw/source/core/docnode/node.cxx:2002: Wrong cond collection, skipping check of Cond Colls.
warn:stoc:22657:22657:stoc/source/javaloader/javaloader.cxx:381: getJavaVM returned null
warn:legacy.osl:22657:22657:svx/source/dialog/rulritem.cxx:480: Wrong MemberId!
warn:sw.core:22657:22657:sw/source/core/docnode/node.cxx:2002: Wrong cond collection, skipping check of Cond Colls.
warn:stoc:22657:22657:stoc/source/javaloader/javaloader.cxx:381: getJavaVM returned null
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:sw.core:22657:22657:sw/source/core/attr/calbck.cxx:196: a 11SwNumFormat client added as listener to a 12SwCharFormat during client iteration.
warn:legacy.osl:22657:22657:svx/source/dialog/rulritem.cxx:480: Wrong MemberId!