Bug 89097 - function CELL for external data doesn't work when data file is not opened
Summary: function CELL for external data doesn't work when data file is not opened
Status: RESOLVED WONTFIX
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Calc (show other bugs)
Version:
(earliest affected)
4.3.3.2 release
Hardware: Other All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected
Depends on:
Blocks:
 
Reported: 2015-02-03 19:38 UTC by raal
Modified: 2016-11-12 13:25 UTC (History)
5 users (show)

See Also:
Crash report or crash signature:


Attachments
test case (32.97 KB, application/zip)
2015-02-03 19:38 UTC, raal
Details

Note You need to log in before you can comment on or make changes to this bug.
Description raal 2015-02-03 19:38:56 UTC
Created attachment 113103 [details]
test case

Steps to reproduce
 - unzip files
 - open file fdo43534test.ods, confirm external data file
 - hard recalc 

Actual results
See cells A3:D3 and A4:B4. 
A3:D3  =  F0	F0	F0	F0
A4,B4  = 0	0


Expected results
Results should be 
A3:D3  =  G	F0	P2	F0-
A4,B4  = 1	1



When you first open data file, then formula works:
 - unzip files
 - open file fdo43534data.ods
 - open file fdo43534test.ods, confirm external data file
 - hard recalc 

Tested with LO 4.3.3.2 and current master
Comment 1 Buovjaga 2015-02-07 12:15:59 UTC
It's your lucky day: you stumbled upon a regression :)
3.5.0 works without opening fdo43534data.ods first!

Win 7 Pro 64-bit, LibO Version: 4.4.0.3
Build ID: de093506bcdc5fafd9023ee680b8c60e3e0645d7
Locale: fi_FI

Ubuntu 14.10 64-bit
Version: 4.4.0.3
Build ID: 40m0(Build:3)
Locale: en_US

LibreOffice 3.5.0rc3 
Build ID: 7e68ba2-a744ebf-1f241b7-c506db1-7d53735
Comment 2 Michael Weghorn 2015-02-07 21:59:28 UTC
In the "bibisect-44" repository, it is OK with "oldest", the bug occurs in all other commits.

bibisect result:
 d54656d58d3ed39bbb574e2b4e8d7dfdc96f1003 is the first bad commit
commit d54656d58d3ed39bbb574e2b4e8d7dfdc96f1003
Author: Bjoern Michaelsen <bjoern.michaelsen@canonical.com>
Date:   Fri Oct 17 18:35:47 2014 +0000

    source-hash-0292b4269ec370818bb1ed48222fade636374a4a
    
    commit 0292b4269ec370818bb1ed48222fade636374a4a
    Author:     Thomas Arnhold <thomas@arnhold.org>
    AuthorDate: Thu May 22 11:53:23 2014 +0200
    Commit:     Thomas Arnhold <thomas@arnhold.org>
    CommitDate: Thu May 22 12:50:50 2014 +0200
    
        cppcheck: Exception should be caught by reference
    
        Change-Id: I14b26941a8f3115a5b3893593151c24684b24d60


$ git bisect log
# bad: [4a3091e95fa263d3e2dd81e56e83996f0bb12287] source-hash-2b5b04e1e62914bf0902dfd7943cdc44499c47a6
# good: [812c4a492375ac47b3557fbb32f5637fc89d60d9] source-hash-dea4a3b9d7182700abeb4dc756a24a9e8dea8474
git bisect start 'latest' 'oldest'
# bad: [5d0dfb8e62ae61a240f8313c594d4560e7c8e048] source-hash-0c6cd530de13f80795881f61064f1bf1dcc4ea81
git bisect bad 5d0dfb8e62ae61a240f8313c594d4560e7c8e048
# bad: [c0ef5c9ad28738878bb28206d62a57aa367cad92] source-hash-d47678c990681d0cfc7f2c843bb9aedaeb08511a
git bisect bad c0ef5c9ad28738878bb28206d62a57aa367cad92
# bad: [652c8d1675dc8b174de907d4204dbc2ec98fe325] source-hash-0aa3dee5e88a1494a7a6a8401e084cbdb4324727
git bisect bad 652c8d1675dc8b174de907d4204dbc2ec98fe325
# bad: [0219641f7ec1bee5654f7b8c02d246fa27b03ab1] source-hash-9835fa87ac6ffe43bd9cc85839b2fea1fca2fcad
git bisect bad 0219641f7ec1bee5654f7b8c02d246fa27b03ab1
# bad: [dbe3cc9b9a4b356f550657d57783c89bd293b46b] source-hash-e23806fb815f7e419bb04de11dcb5707c4120eaa
git bisect bad dbe3cc9b9a4b356f550657d57783c89bd293b46b
# bad: [0c6eb6d43e107345f59a3a09bd2c2577302819c2] source-hash-9cff642cd51bc70327789f9dc64349bb9b4cb3ea
git bisect bad 0c6eb6d43e107345f59a3a09bd2c2577302819c2
# bad: [7b9ad53b3ba4d291f49052ad9aee36b5cdcd4ac2] source-hash-9b62a5207745da6b926d2a311e5ddd16da1ae05b
git bisect bad 7b9ad53b3ba4d291f49052ad9aee36b5cdcd4ac2
# bad: [829039bf3db2f38225087d5047e63e83466580e0] source-hash-a1e7915bfec4b9e006a0fc4c6819b63b139af2f0
git bisect bad 829039bf3db2f38225087d5047e63e83466580e0
# bad: [d54656d58d3ed39bbb574e2b4e8d7dfdc96f1003] source-hash-0292b4269ec370818bb1ed48222fade636374a4a
git bisect bad d54656d58d3ed39bbb574e2b4e8d7dfdc96f1003
# first bad commit: [d54656d58d3ed39bbb574e2b4e8d7dfdc96f1003] source-hash-0292b4269ec370818bb1ed48222fade636374a4a
Comment 3 Markus Mohrhard 2015-02-11 21:25:25 UTC
This was discussed with Eike and we are not going to fix this one. We already consider the cell function unnecessary and are not going to support any style based conditions around that function for external files.
Comment 4 raal 2015-02-12 10:35:39 UTC
(In reply to Markus Mohrhard from comment #3)
> This was discussed with Eike and we are not going to fix this one. We
> already consider the cell function unnecessary and are not going to support
> any style based conditions around that function for external files.

In this case function CELL should result #ERROR?  Now function returns bad values.

let it fail for _all_ external references, regardless whether a file is open or not.
Comment 5 Markus Mohrhard 2015-02-13 03:10:12 UTC
(In reply to raal from comment #4)
> (In reply to Markus Mohrhard from comment #3)
> > This was discussed with Eike and we are not going to fix this one. We
> > already consider the cell function unnecessary and are not going to support
> > any style based conditions around that function for external files.
> 
> In this case function CELL should result #ERROR?  Now function returns bad
> values.
> 
> let it fail for _all_ external references, regardless whether a file is open
> or not.

I don't agree. As I don't want to turn this into an edit war and we have the rule that a bug marked as WONT FIX by a developer responsible for that part should stay like that I'm removing the 'regression' keyword to remove that bug from the list.

Please DON'T add the keyword back. This bug is not going to be fixed even if you don't agree with me.
Comment 6 Robinson Tryon (qubit) 2015-12-13 11:12:12 UTC Comment hidden (obsolete)
Comment 7 Xisco Faulí 2016-11-12 13:25:51 UTC
Closing this bug as RESOLVED WONTFIX as per comment 5