[Bug 61532] New: XSSFFormulaEvaluator.evaluateFormulaCell() replaces the formula and changes the cell type

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

[Bug 61532] New: XSSFFormulaEvaluator.evaluateFormulaCell() replaces the formula and changes the cell type

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

            Bug ID: 61532
           Summary: XSSFFormulaEvaluator.evaluateFormulaCell() replaces
                    the formula and changes the cell type
           Product: POI
           Version: 3.14-FINAL
          Hardware: PC
            Status: NEW
          Severity: major
          Priority: P2
         Component: SXSSF
          Assignee: [hidden email]
          Reporter: [hidden email]
  Target Milestone: ---

XSSFFormulaEvaluator.evaluateFormulaCell(), at least when used with a formula
of type VLOOKUP in an SXSSFWorkbook, behaves the way the documentation says
that evaluateInCell() is supposed to work; it replaces the actual value of the
cell with the results of running the formula, and changes the cell type to
whatever the type of the result is. So, the two methods behave identically.

The correct behavior, according to the documentation, is to simply update the
result value cache.

--
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 61532] XSSFFormulaEvaluator.evaluateFormulaCell() replaces the formula and changes the cell type

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

Nick Burch <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 OS|                            |All
             Status|NEW                         |NEEDINFO

--- Comment #1 from Nick Burch <[hidden email]> ---
Are you able to try with Apache POI 3.17, released last week, and see how that
behaves?

If the problem remains, would you be able to write a short junit unit test (or
failing that short standalone program) demonstrating the issue, so we can
reproduce it to take a look?

--
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 61532] XSSFFormulaEvaluator.evaluateFormulaCell() replaces the formula and changes the cell type

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=61532

Dominik Stadler <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |RESOLVED
         Resolution|---                         |DUPLICATE

--- Comment #2 from Dominik Stadler <[hidden email]> ---


*** This bug has been marked as a duplicate of bug 61148 ***

--
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 61532] XSSFFormulaEvaluator.evaluateFormulaCell() replaces the formula and changes the cell type

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=61532

Dominik Stadler <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|DUPLICATE                   |---

--
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 61532] XSSFFormulaEvaluator.evaluateFormulaCell() replaces the formula and changes the cell type

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=61532

Dominik Stadler <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |NEEDINFO

--
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 61532] XSSFFormulaEvaluator.evaluateFormulaCell() replaces the formula and changes the cell type

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=61532

--- Comment #3 from Dave Neuer <[hidden email]> ---
I don't believe this is a duplicate of the linked issue; the description of
that issue is different, though they may share an underlying cause.

I'll work on a small test case demonstrating the issue in the coming days.

--
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 61532] XSSFFormulaEvaluator.evaluateFormulaCell() replaces the formula and changes the cell type

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=61532

Dominik Stadler <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEEDINFO                    |RESOLVED
         Resolution|---                         |DUPLICATE

--- Comment #4 from Dominik Stadler <[hidden email]> ---
I have now tested it as far as I can and added a unit-test via r1808932 which
shows that the two methods behave as documented and expected after fixing bug
61148.

Please verify it with a build which contains these changes and reopen the bug
if it is still not fixed for you with some special combination of
formula/call-order.

*** This bug has been marked as a duplicate of bug 61148 ***

--
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]