DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG 
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20696>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND 
INSERTED IN THE BUG DATABASE.

http://nagoya.apache.org/bugzilla/show_bug.cgi?id=20696

[PATCH] Workbook.getSheetIndex() needs to ignore case.

           Summary: [PATCH] Workbook.getSheetIndex() needs to ignore case.
           Product: POI
           Version: 2.0-dev
          Platform: Other
        OS/Version: All
            Status: NEW
          Severity: Major
          Priority: Other
         Component: HSSF
        AssignedTo: [EMAIL PROTECTED]
        ReportedBy: [EMAIL PROTECTED]


I have seen several instances of xls files created natively in Excel that have
problems retrieving sheet names by the sheet number.  Apparently, Excel treats
sheet names as case insensitive.  Patch below (created aginst a recent cvs diff)

Index: jakarta-poi/src/java/org/apache/poi/hssf/model/Workbook.java
===================================================================
RCS file:
/home/cvspublic/jakarta-poi/src/java/org/apache/poi/hssf/model/Workbook.java,v
retrieving revision 1.29
diff -r1.29 Workbook.java
516c519
<             if (sheet.equals(name)) {
---
>             if (sheet.equalsIgnoreCase(name)) {

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to