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 2015/11/02 03:09:13 UTC

[Bug 58570] New: SXSSFCell.setAsActiveCell silently does nothing

https://bz.apache.org/bugzilla/show_bug.cgi?id=58570

            Bug ID: 58570
           Summary: SXSSFCell.setAsActiveCell silently does nothing
           Product: POI
           Version: 3.14-dev
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: SXSSF
          Assignee: dev@poi.apache.org
          Reporter: onealj@apache.org

Created attachment 33245
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=33245&action=edit
SXSSFCell.setAsActiveCell should throw an exception until it's fully
implemented

Right now SXSSFCell silently does nothing. It should either log that it's a
non-operation or throw some exception that the setAsActiveCell is not relevant
or not supported for SXSSFWorkbooks, until someone can determine a better
solution.

-- 
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 58570] SXSSFCell.setAsActiveCell silently does nothing

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

Javen O'Neal <on...@apache.org> changed:

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

--- Comment #2 from Javen O'Neal <on...@apache.org> ---
In r1717900:
Add CellAddress Sheet.getActiveCell() (previously only available in XSSFSheet,
backwards-compatibility note: previously returned a String)
Add void Sheet.setActiveCell(CellAddress)
Add Cell.getAddress() to support this (CellAddress(Cell) could also be used)
Add unit tests for Cell.setAsActiveCell()
Add unit tests for Sheet.getActiveCell and Sheet.setActiveCell

In summary, SXSSFCell.setAsActiveCell is now implemented!

Future public methods that are not implemented should be annotated with
@NotImplemented per comment 1.

Updated docs in r1717901

-- 
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 58570] SXSSFCell.setAsActiveCell silently does nothing

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

--- Comment #1 from Javen O'Neal <on...@apache.org> ---
Added NotImplemented decorator and made method throw
RuntimeException(NotImplemented) in r1717164 [1].

This didn't break the Jenkins build, which also means this isn't unit tested.

http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/xssf/streaming/SXSSFCell.java?annotate=1717164&diff_format=h#l569

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