DO NOT REPLY [Bug 51456] New: POI web documentation ambiguity

[bu...@apache.org]

https://issues.apache.org/bugzilla/show_bug.cgi?id=51456

             Bug #: 51456
           Summary: POI web documentation ambiguity
           Product: POI
           Version: unspecified
          Platform: All
        OS/Version: Windows CE
            Status: NEW
          Severity: normal
          Priority: P2
         Component: POI Overall
        AssignedTo: dev@poi.apache.org
        ReportedBy: dsheil@vartmp.com
    Classification: Unclassified


In the documentation for POI text extraction -

http://poi.apache.org/spreadsheet/quick-guide.html#TextExtraction

One of the lines of code is:

CellReference cellRef = new CellReference(row.getRowNum(), cell.getCellNum());

This assumes the Cell class has a getCellNum() method call.  But only one
implementing class for cell uses a getCellNum() method - HSSFCell.  Neither
XSSFCell nor SXSSFCell do.

As that documentation page is a "Busy Developers' Guide" for those who "want to
use HSSF and XSSF read and write spreadsheets in a hurry", I think that the
manner in which it currently is displayed has to be changed in some fashion. 
Either it should be noted this will only extract text for HSSF, or the code
should handle both HSSF and XSSF, or two code examples, one for each case
should be there.  As it currently stands, it only serves to confuse those
looking at it, especially if they're having other unrelated problems with POI. 
There are a number of options for how it can be changed, but any one of those
options would be better than the confusing example which exists now.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- 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

DO NOT REPLY [Bug 51456] POI web documentation ambiguity

[bu...@apache.org]

https://issues.apache.org/bugzilla/show_bug.cgi?id=51456

Yegor Kozlov <ye...@dinom.ru> changed:

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

--- Comment #1 from Yegor Kozlov <ye...@dinom.ru> 2011-07-01 11:44:29 UTC ---
Fixed in r1141906

The correct method is getColumnIndex().

Yegor

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- 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