Description: Creating a Stock Chart with simple data, saving and reopening is sufficient to corrupt the data. The styles applied to the chart elements were also altered. Steps to Reproduce: 1. Open Writer or Impress (didn't test on Calc) 2. Click 'Insert' -> 'Chart...' 3. Select the chart, right click and open 'Chart Type...' dialog. 4. Select 'Stock Chart 1' as type. 5. Now open the 'Data Table' dialog. 6. There's a single data series with three components: 'Low Values', 'High Values' and 'Close Values'. Click on the label for the data series and then on 'Delete Series'. 7. Click 'Insert Series' twice to insert two data series. These have the same components, plus a fourth 'Open Values'. 8. Optionally, click on 'Delete Row' three times to simplify the next step. 9. Enter data on all cells except the 'Open Values'. Something like 1, 3, 2; 4, 6, 5 will do. 10. Close the dialog, deselect the chart, save the file and close LibreOffice. 11. Reopen the same file. Actual Results: The chart seems clearly broken, with a different style (some data series are now black), names of data series removed, data corrupted and a new anonymous data series. Opening the data table for this chart reveals that those 'Open Values' are no longer there; while the one for the first data series has apparently simply vanished, the second one has pushed into the 'Low Values' of the data series, shifting all cells to the right, so the last cell overflows into a new data series. Expected Results: The chart should contain the same data and have the same styles applied as when it was originally saved. Reproducible: Always User Profile Reset: No OpenGL enabled: Yes Additional Info: In a more complex example, other effects could be seen. Some series were renamed while others weren't, line width was dropped, even some data series were reordered. Version: 7.0.3.1 Build ID: 00(Build:1) CPU threads: 4; OS: Linux 5.4; UI render: default; VCL: gtk3 Locale: en-US (en_US.UTF-8); UI: en-US Ubuntu package version: 1:7.0.3-0ubuntu0.20.04.1 Calc: threaded
Note that this bug report involves specific data in a chart being corrupted. Bug 69698 refers to chart styles. However, this bug seems to also affect styles to some degree, so both issues could be at least related.
Reproduced in LibreOffice 7.0.4.2
Reproduced in LibreOffice 7.1.1.2
Tested and confirmed on LibreOffice 3.3, setting earliest version to "Inherited From OOo"
Changon chart type to 'Stock Chart 2' restores chart data, as does creating the chart as that type in the first place. Also, using the data series without 'Open Values' already present after changing chart type to 'Stock Chart 1' and not creating any new data series avoids this issue. So, the problem seems to be simply that adding new data series to a 'Stock Chart 1' generates data series with 4 components, 'Open Values', while for all other purposes the chart is considered not to have this fourth component. Chart data bleeds into other series and style data, corrupting the chart.
Accordin to the manual, chart type 'Stock Chart 1' should not have an 'Open Values' component at all.
Assigned this to myself.
Aritz Erkiaga committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/commit/c8598f28db8ef5ab5f695cf1af645bb43dbc264d tdf#138556 Don’t add Open Values to stock chart types 1 and 3 It will be available in 7.2.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.
Created attachment 172297 [details] test_stock_chart13_insert_series.log The UITest on my system has failed, see attached log. Is there something wrong with the test code, or something else?
This bug is not present in the last few daily builds, so I'm finally closing it. Regarding the UITest failure you report, I haven't been able to replicate it myself, sorry... I've seen that same error in another testcase in one machine, but it didn't seem to occur in any further checks. I can't pinpoint any particular reason this test code should fail, but I wouldn't be surprised if there was a bug in it, after all I wrote it :') Anyway, could you test it again on your system? I'll ask about the issue in any case, but if you do get this error consistently, then... well, I'll feel a great deal more worried.
(In reply to Aritz Erkiaga from comment #10) I now know why the test fails: ERROR: test_stock_chart13_insert_series (tdf138556.tdf138556) ---------------------------------------------------------------------- Traceback (most recent call last): File "...sw/qa/uitest/chart/tdf138556.py", line 45, in test_stock_chart13_insert_series xToolbar.executeAction( "CLICK", mkPropertyValues({ "POS" : "1" })) tdf138556.com.sun.star.lang.DisposedException: Binary URP bridge disposed during call ...binaryurp/source/bridge.cxx:613 That means that before executing line 45 (i.e, inserting series), the application must have already been disposed (mostly crashed!). And a manual test confirms that on my system when click the insert series icon it crashes, which is exactly bug 142467.
Interesting... So the crash does not depend on this bug, and is not fixed by the patch? But it also wasn't introduced by it?
(In reply to Aritz Erkiaga from comment #12) Yes you are correct. Your patch does not fix the crash, and the crash seems have been there a long time ago. Caolán says he is debugging that crash.
But it was at the time when I am running your UITest script when this crash arises - that been said, you discovered the crash :-)