Bug 54402

Summary: [PATCH] Bug in IFERROR
Product: POI Reporter: Johan Karlsteen <johan.karlsteen>
Component: POI OverallAssignee: POI Developers List <dev>
Status: RESOLVED FIXED    
Severity: normal    
Priority: P2    
Version: unspecified   
Target Milestone: ---   
Hardware: Macintosh   
OS: Mac OS X 10.4   
Attachments: Bugfix in IFERROR

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.