Bug 54402 - [PATCH] Bug in IFERROR
Summary: [PATCH] Bug in IFERROR
Status: RESOLVED FIXED
Alias: None
Product: POI
Classification: Unclassified
Component: POI Overall (show other bugs)
Version: unspecified
Hardware: Macintosh Mac OS X 10.4
: P2 normal (vote)
Target Milestone: ---
Assignee: POI Developers List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-01-10 14:16 UTC by Johan Karlsteen
Modified: 2013-01-10 15:09 UTC (History)
0 users



Attachments
Bugfix in IFERROR (1.28 KB, application/x-gzip)
2013-01-10 14:16 UTC, Johan Karlsteen
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Johan Karlsteen 2013-01-10 14:16:29 UTC
Created attachment 29841 [details]
Bugfix in IFERROR

I found a bug in IFERROR.

It works good if you have a cell with for example: IFERROR(A1/A2,"error") and A1/A2 is an error but if you have A3=A1/A2 and IFERROR(A3,"error") it will not find the error.

I have attached a patch for this, basically I added the line:
arg = WorkbookEvaluator.dereferenceResult(arg, srcCellRow, srcCellCol);

I do not know if this is the correct way to do this but it works. Also added a test case that fails with the old Iferror.java and works with the new one.
Comment 1 Nick Burch 2013-01-10 15:09:15 UTC
Thanks for this, patch applied (with a minor docs tweak) in r1431403.