Bug 55339 - setRowGroupCollapsed not implemented
Summary: setRowGroupCollapsed not implemented
Status: NEW
Alias: None
Product: POI
Classification: Unclassified
Component: SXSSF (show other bugs)
Version: unspecified
Hardware: PC Linux
: P2 enhancement (vote)
Target Milestone: ---
Assignee: POI Developers List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-08-01 10:29 UTC by Ishmeet
Modified: 2016-10-09 06:48 UTC (History)
0 users



Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Ishmeet 2013-08-01 10:29:28 UTC
I am using SXSSF for generating excel sheets.

The method setRowGroupCollapsed has not yet been implemented in SXSSFSheet.class:

    public void setRowGroupCollapsed(int row, boolean collapse)
    {
        //_sh.setRowGroupCollapsed(row, collapse);
        throw new RuntimeException("Not Implemented");
    }

As a result, if I try to collapse rows, the excel sheet generated gives the error message: "Excel found unreadable content".
Comment 1 Ishmeet 2013-08-05 10:52:57 UTC
Note that the method implementation is fine in version 3.8-beta5 (http://grepcode.com/file/repo1.maven.org/maven2/org.apache.poi/poi-ooxml/3.8-beta5/org/apache/poi/xssf/streaming/SXSSFSheet.java#SXSSFSheet):

 public void setRowGroupCollapsed(int row, boolean collapse)
   {
       _sh.setRowGroupCollapsed(row, collapse);
   }

But for some reason, this was commented out in version 3.8:

public void setRowGroupCollapsed(int row, boolean collapse)
{
//_sh.setRowGroupCollapsed(row, collapse);
throw new RuntimeException("Not Implemented");
}

Here is the commit message:
http://svn.apache.org/viewvc/poi/trunk/src/ooxml/java/org/apache/poi/xssf/streaming/SXSSFSheet.java?view=diff&r1=1295057&r2=1295058&pathrev=1295058
Comment 2 Dominik Stadler 2015-03-23 21:22:57 UTC
FYI, in the meantime it was at least brought back for collapsing, but is still not there for uncollapsing:

    public void setRowGroupCollapsed(int row, boolean collapse)
    {
        if (collapse) {
            collapseRow(row);
        } else {
            //expandRow(rowIndex);
            throw new RuntimeException("Not Implemented");
        }
    }
Comment 3 Javen O'Neal 2016-10-09 06:48:53 UTC
(In reply to Ishmeet from comment #1)
> Here is the commit message:
r1295058