[Bug 61244] New: Remove cell.getCachedFormulaResultType()

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

[Bug 61244] New: Remove cell.getCachedFormulaResultType()

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

            Bug ID: 61244
           Summary: Remove cell.getCachedFormulaResultType()
           Product: POI
           Version: 3.17-dev
          Hardware: PC
                OS: Mac OS X 10.1
            Status: NEW
          Severity: normal
          Priority: P2
         Component: SS Common
          Assignee: [hidden email]
          Reporter: [hidden email]
  Target Milestone: ---

Created attachment 35091
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=35091&action=edit
patch

This method returns an int instead of an enum - some variants are marked for
removal in 3.17 but not all.
I'll attach a patch because I'd like to get a review before proceeding.

--
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 61244] Remove cell.getCachedFormulaResultType()

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

Dominik Stadler <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |enhancement

--
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 61244] Remove cell.getCachedFormulaResultType()

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

--- Comment #1 from PJ Fanning <[hidden email]> ---
Changed method to return CellType and deprecated the
getCachedFormulaResultTypeEnum method.
https://svn.apache.org/repos/asf/poi/trunk@1808678

--
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 61244] Remove cell.getCachedFormulaResultTypeEnum()

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

Dominik Stadler <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED
            Summary|Remove                      |Remove
                   |cell.getCachedFormulaResult |cell.getCachedFormulaResult
                   |Type()                      |TypeEnum()

--
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 61244] Remove cell.getCachedFormulaResultTypeEnum()

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

--- Comment #2 from Dominik Stadler <[hidden email]> ---
I resolved this as I think the main goal of having getCachedFormulaResultType
return CellType is done, the deprecations will lead to removal in future
versions anyway.

--
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 61244] Make cell.getCachedFormulaResultType() return CellType

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

Dominik Stadler <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Remove                      |Make
                   |cell.getCachedFormulaResult |cell.getCachedFormulaResult
                   |TypeEnum()                  |Type() return CellType

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