You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@poi.apache.org by bu...@apache.org on 2017/06/30 20:13:11 UTC

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

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: dev@poi.apache.org
          Reporter: fanningpj@yahoo.com
  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: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org


[Bug 61244] Remove cell.getCachedFormulaResultTypeEnum()

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=61244

--- Comment #2 from Dominik Stadler <do...@gmx.at> ---
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: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org


[Bug 61244] Make cell.getCachedFormulaResultType() return CellType

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=61244

Dominik Stadler <do...@gmx.at> 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: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org


[Bug 61244] Remove cell.getCachedFormulaResultType()

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=61244

--- Comment #1 from PJ Fanning <fa...@yahoo.com> ---
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: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org


[Bug 61244] Remove cell.getCachedFormulaResultTypeEnum()

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=61244

Dominik Stadler <do...@gmx.at> 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: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org


[Bug 61244] Remove cell.getCachedFormulaResultType()

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=61244

Dominik Stadler <do...@gmx.at> changed:

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

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@poi.apache.org
For additional commands, e-mail: dev-help@poi.apache.org