Bug 129587 - Unselect "Same value for both" - Crash
Summary: Unselect "Same value for both" - Crash
Status: VERIFIED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
7.0.0.0.alpha0+
Hardware: All All
: high major
Assignee: Not Assigned
URL:
Whiteboard: target:7.1.0
Keywords: bibisected, bisected, haveBacktrace, regression
Depends on:
Blocks: Crash
  Show dependency treegraph
 
Reported: 2019-12-23 15:34 UTC by raal
Modified: 2020-06-03 16:39 UTC (History)
7 users (show)

See Also:
Crash report or crash signature:


Attachments
bt with debug symbols (4.43 KB, text/plain)
2019-12-23 16:49 UTC, Julien Nabet
Details
Valgrind trace (63.52 KB, text/x-log)
2019-12-23 16:50 UTC, Julien Nabet
Details

Note You need to log in before you can comment on or make changes to this bug.
Description raal 2019-12-23 15:34:48 UTC
Description:


Version: 6.5.0.0.alpha0+ (x64)
Build ID: 3f680aef65a158cfbc98c8afd1c3628d7f4f7b83
CPU threads: 4; OS: Windows 6.1 Service Pack 1 Build 7601; UI render: default; VCL: win; 

Steps to Reproduce:
open file https://bugs.documentfoundation.org/attachment.cgi?id=156720
double click chart
select Y error bars
Format Y error bars
Unselect "Same value for both"


Actual Results:
Crash

Expected Results:
No crash


Reproducible: Always


User Profile Reset: No



Additional Info:
Comment 1 raal 2019-12-23 15:35:47 UTC
This seems to have begun at the below commit.
Adding Cc: to Muhammet Kara ; Could you possibly take a look at this one? Thanks
 3ebde304b765f931262a9656f8bd7e8dbfbabda9 is the first bad commit
commit 3ebde304b765f931262a9656f8bd7e8dbfbabda9
Author: Norbert Thiebaud <nthiebaud@gmail.com>
Date:   Tue Nov 26 23:09:56 2019 -0800

    source 2d01ed9e8be543460e41e009fa992103a7c8d4c0

author	Muhammet Kara <muhammet.kara@collabora.com>	2019-11-25 21:55:31 +0300
committer	Muhammet Kara <muhammet.kara@collabora.com>	2019-11-26 15:39:56 +0100
commit 2d01ed9e8be543460e41e009fa992103a7c8d4c0 (patch)
tree bd75130e564374647046f03ad4a334c1876d9dd0
parent 9b6b134b3e605236e90aca089867f08d659a6076 (diff)
tdf#94288: Show chart props sidebar on activation
Comment 2 Julien Nabet 2019-12-23 16:49:38 UTC
Created attachment 156758 [details]
bt with debug symbols

On pc Debian x86-64 with master sources updated today, I could reproduce this.
Comment 3 Julien Nabet 2019-12-23 16:50:33 UTC
Created attachment 156759 [details]
Valgrind trace
Comment 4 Julien Nabet 2019-12-23 16:51:56 UTC
Just to be clear, I didn't see where was "Same value for both" but reproduced it by doing:
- click on Y axis
- click on one of the chart line
Comment 5 Xisco Faulí 2019-12-24 15:04:52 UTC
(In reply to Julien Nabet from comment #4)
> Just to be clear, I didn't see where was "Same value for both" but
> reproduced it by doing:
> - click on Y axis
> - click on one of the chart line

That's already reported in bug 129428 ? is it a duplicate ?
Comment 6 Julien Nabet 2019-12-28 16:40:27 UTC
(In reply to Xisco Faulí from comment #5)
> ...
> 
> That's already reported in bug 129428 ? is it a duplicate ?

No idea.
Comment 7 Xisco Faulí 2020-06-03 10:30:36 UTC
Issue fixed by

author	Michael Meeks <michael.meeks@collabora.com>	2020-01-04 18:09:20 +0000
committer	Michael Meeks <michael.meeks@collabora.com>	2020-01-06 19:16:28 +0100
commit 5b77d17c4f1ca734babf962b45c1aa07bdca14e9 (patch)
tree 7ea5542a677748273d2c95cbd3ab1514a4b876f1
parent 0bb6bb7729f159f05100e33797ed667fd9d13ab7 (diff)
sidebar: allow panels to lurk around instead of being disposed.

@Michael, thanks for fixing this issue!!
Comment 8 Commit Notification 2020-06-03 16:39:07 UTC
Xisco Fauli committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/0076010080118e933c95e8d462bb2c3487731974

tdf#129587: sc: Add UItest

It will be available in 7.1.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.