Bug 151424 - Spider web chart drawing error since LO Calc version 7.4.1.2
Summary: Spider web chart drawing error since LO Calc version 7.4.1.2
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
7.4.1.2 release
Hardware: x86-64 (AMD64) All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard: target:7.5.0 target:7.4.3
Keywords: bibisected, bisected, regression
: 150951 151735 151989 (view as bug list)
Depends on:
Blocks:
 
Reported: 2022-10-08 13:59 UTC by H077e
Modified: 2022-11-14 13:31 UTC (History)
6 users (show)

See Also:
Crash report or crash signature:


Attachments
incorrectly labeled chart (47.71 KB, image/png)
2022-10-08 13:59 UTC, H077e
Details
correctly labeled chart - version 6.1.5 (44.35 KB, image/png)
2022-10-08 14:01 UTC, H077e
Details
sample Calc file (17.31 KB, application/vnd.oasis.opendocument.spreadsheet)
2022-10-08 14:01 UTC, H077e
Details

Note You need to log in before you can comment on or make changes to this bug.
Description H077e 2022-10-08 13:59:06 UTC
Created attachment 182915 [details]
incorrectly labeled chart

After the upgate to version 7.4.1.2, spider web charts are labeled incorrectly.
Source is the same Calc file.
Before version 7.4.1.2 is the chart correctly labeled.
Comment 1 H077e 2022-10-08 14:01:12 UTC
Created attachment 182916 [details]
correctly labeled chart - version 6.1.5
Comment 2 H077e 2022-10-08 14:01:47 UTC
Created attachment 182917 [details]
sample Calc file
Comment 3 m_a_riosv 2022-10-08 15:42:55 UTC
Reproducible
Version: 7.4.2.1 (x64) / LibreOffice Community
Build ID: 681d65acd9ede00dd724d6716f21cabfdcc95bd2
CPU threads: 4; OS: Windows 10.0 Build 19044; UI render: Skia/Raster; VCL: win
Locale: es-ES (es_ES); UI: en-US Calc: CL
and masster
Version: 7.5.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: 6834fda784f3066a89838cd6cda4fe945f4c7904
CPU threads: 4; OS: Windows 10.0 Build 19044; UI render: Skia/Raster; VCL: win
Locale: es-ES (es_ES); UI: en-US Calc: CL threaded

Regression form
Version: 7.3.6.2 (x64) / LibreOffice Community
Build ID: c28ca90fd6e1a19e189fc16c05f8f8924961e12e
CPU threads: 4; OS: Windows 10.0 Build 19044; UI render: Skia/Vulkan; VCL: win
Locale: es-ES (es_ES); UI: es-ES
Calc: CL
Comment 4 raal 2022-10-08 20:27:36 UTC
This seems to have begun at the below commit.
Adding Cc: to Noel Grandin ; Could you possibly take a look at this one?
Thanks
 81c08fbea6b2e21217f88b82428b4aa07e8d8671 is the first bad commit
commit 81c08fbea6b2e21217f88b82428b4aa07e8d8671
Author: Jenkins Build User <tdf@pollux.tdf>
Date:   Sat Feb 5 22:08:06 2022 +0100

    source 333ccb081b4ab62adce50ca4c93162b9baf984d0

https://git.libreoffice.org/core/+/333ccb081b4ab62adce50ca4c93162b9baf984d0
Comment 5 Commit Notification 2022-10-09 11:25:27 UTC
Noel Grandin committed a patch related to this issue.
It has been pushed to "master":

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

tdf#151424 Spider web chart drawing error

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 6 Julien Nabet 2022-10-09 16:59:26 UTC
On pc Debian x86-64 with master sources updated today (including Noel's patch), I don't reproduce this anymore.
Thank you Noel!
Comment 7 H077e 2022-10-10 11:52:38 UTC
I have tested it.
Yes, is fixed in DEV-Build 7.5.0.0-Alpha0.
Good work and very big thanks!!!

Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: a7683a16e98def4198e1f74a93fecdaacbf639e7
CPU threads: 16; OS: Linux 4.18; UI render: default; VCL: gtk3
Locale: de-DE (de_DE.UTF-8); UI: en-US
Calc: threaded
Comment 8 m_a_riosv 2022-10-10 13:47:28 UTC
And in win10 with
Version: 7.5.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: a7683a16e98def4198e1f74a93fecdaacbf639e7
CPU threads: 4; OS: Windows 10.0 Build 19044; UI render: Skia/Vulkan; VCL: win
Locale: es-ES (es_ES); UI: en-US Calc: CL threaded
Comment 9 Commit Notification 2022-10-10 18:17:50 UTC
Noel Grandin committed a patch related to this issue.
It has been pushed to "libreoffice-7-4":

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

tdf#151424 Spider web chart drawing error

It will be available in 7.4.3.

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 10 Commit Notification 2022-10-13 09:08:24 UTC
Xisco Fauli committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/4ce92931bf406fb467934374af1188927f34a972

tdf#151424: chart2_xshape: Add unittest

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 11 Kevin Suo 2022-10-23 12:34:54 UTC
*** Bug 150951 has been marked as a duplicate of this bug. ***
Comment 12 raal 2022-10-25 16:10:48 UTC
*** Bug 151735 has been marked as a duplicate of this bug. ***
Comment 13 Stéphane Guillou (stragu) 2022-11-14 13:31:51 UTC
*** Bug 151989 has been marked as a duplicate of this bug. ***