[Bug 64393] New: [PATCH] Corrects relational operators against MissingArgEval

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 64393] New: [PATCH] Corrects relational operators against MissingArgEval

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64393

            Bug ID: 64393
           Summary: [PATCH] Corrects relational operators against
                    MissingArgEval
           Product: POI
           Version: 4.1.2-FINAL
          Hardware: PC
            Status: NEW
          Severity: normal
          Priority: P2
         Component: SS Common
          Assignee: [hidden email]
          Reporter: [hidden email]
  Target Milestone: ---

Created attachment 37204
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=37204&action=edit
Patch to treat MissingArgEval like BlankEval in RelationalOperationEval

I investigated an issue with a formula in the form of:

  =ISERROR(C4,)=0

where the expected value is TRUE when C4 is an error. Further investigation
showed that wherever ISERROR had no second argument, Excel treated it like a
zero when comparing against other numbers. This patch modifies
RelationalOperationEval to treat MissingArgEval the same way it treats
BlankEval.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64393] [PATCH] Corrects relational operators against MissingArgEval

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64393

Dominik Stadler <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 OS|                            |All

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64393] [PATCH] Corrects relational operators against MissingArgEval

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64393

Dominik Stadler <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #1 from Dominik Stadler <[hidden email]> ---
Applied via r1877815 now, thanks for the patch and unit-tests.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]