Bug 100637 - Wrong result of nested IF() functions in array context
Summary: Wrong result of nested IF() functions in array context
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
Inherited From OOo
Hardware: All All
: medium normal
Assignee: Eike Rathke
URL:
Whiteboard: target:5.3.0
Keywords:
Depends on:
Blocks:
 
Reported: 2016-06-27 09:37 UTC by raal
Modified: 2016-11-23 12:51 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:


Attachments
test case (7.11 KB, application/x-vnd.oasis.opendocument.spreadsheet)
2016-06-27 09:37 UTC, raal
Details

Note You need to log in before you can comment on or make changes to this bug.
Description raal 2016-06-27 09:37:47 UTC
Created attachment 125939 [details]
test case

The bug from https://bz.apache.org/ooo/show_bug.cgi?id=90060

if I input =IF(cond1; IF(cond2;....);....), where cond2 
is non-matrix thing, such as number, the result is wrong.
A test case is attached.
Comment 1 Commit Notification 2016-07-20 14:54:18 UTC
Eike Rathke committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=e54cd3fbf40300416ef337981bd356b88ad44a41

Resolves: tdf#100637 narrow down the JumpMatrix path finish condition

It will be available in 5.3.0.

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

Affected users are encouraged to test the fix and report feedback.
Comment 2 Commit Notification 2016-07-20 15:19:08 UTC
Eike Rathke committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=35595869d19f052cb585c82a1a7e174debb944b0

unit test for tdf#100637 nested IF in array context

It will be available in 5.3.0.

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

Affected users are encouraged to test the fix and report feedback.
Comment 3 Commit Notification 2016-08-03 19:54:03 UTC
Eike Rathke committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=3efcfc63eec7a75545e4f01fd15448744ad651fa

Revert "Resolves: tdf#100637 narrow down the JumpMatrix path finish condition"

It will be available in 5.3.0.

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

Affected users are encouraged to test the fix and report feedback.
Comment 4 Commit Notification 2016-08-03 19:54:08 UTC
Eike Rathke committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=1b6caaeb7a323e91b6f6e2f3612a3bdc0a09fb45

disable unit test for tdf#100637 until fixed again

It will be available in 5.3.0.

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

Affected users are encouraged to test the fix and report feedback.
Comment 5 Commit Notification 2016-08-04 16:59:19 UTC
Eike Rathke committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=e5632d9b2f194a89ec4e60cd99fc050691b14a1e

Resolves: tdf#100637 nested array jump condition needs own JumpMatrix context

It will be available in 5.3.0.

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

Affected users are encouraged to test the fix and report feedback.
Comment 6 Commit Notification 2016-08-04 16:59:24 UTC
Eike Rathke committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=e85bb29a029300bbed18ca0c25bdc9a31773e9cc

re-enable unit test for tdf#100637 nested array IF with scalar

It will be available in 5.3.0.

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

Affected users are encouraged to test the fix and report feedback.
Comment 7 Commit Notification 2016-11-23 12:51:51 UTC
Zdeněk Crhonek committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=d99a6463ab53a5e4de92c1d2fb6b178e3a5bde2a

update IF() test case (add test for tdf#100637)

It will be available in 5.3.0.

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

Affected users are encouraged to test the fix and report feedback.