Bug 20696 - [PATCH] Workbook.getSheetIndex() needs to ignore case.
Summary: [PATCH] Workbook.getSheetIndex() needs to ignore case.
Status: RESOLVED FIXED
Alias: None
Product: POI
Classification: Unclassified
Component: HSSF (show other bugs)
Version: 2.0-dev
Hardware: Other All
: P3 major (vote)
Target Milestone: ---
Assignee: POI Developers List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2003-06-12 00:57 UTC by Eric Ladner
Modified: 2004-11-16 19:05 UTC (History)
0 users


Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Eric Ladner 2003-06-12 00:57:04 UTC 
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)) {
Comment 1 Avik Sengupta 2003-07-01 14:55:07 UTC 
Seems fine.. committed.