diff options
| author | Dominik Stadler <centic@apache.org> | 2024-07-07 05:03:15 +0000 |
|---|---|---|
| committer | Dominik Stadler <centic@apache.org> | 2024-07-07 05:03:15 +0000 |
| commit | 164c239a52ac44383c4cbad9cbf607a9e93b7f57 (patch) | |
| tree | 399c67b7cca9ced5e19ab0477525b48be8e57b5f /poi-ooxml | |
| parent | d090c6b77ab8162b6cd2fcd7f57b67e694a53c68 (diff) | |
| download | poi-164c239a52ac44383c4cbad9cbf607a9e93b7f57.tar.gz poi-164c239a52ac44383c4cbad9cbf607a9e93b7f57.zip | |
HSSFWorkbook.getSheet(): Return first found sheet
We do not need to loop over all sheets always but should
be able to return the first found sheet.
This may change semantics for cases where there are
multiple sheets where name only differs in case, but
the JavaDoc did not state which one will be returned.
All three implementations (HSSF, XSSF, SXSSF)
now behave the same way.
Closes #653
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1918982 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to 'poi-ooxml')
| -rw-r--r-- | poi-ooxml/src/main/java/org/apache/poi/xssf/streaming/SXSSFWorkbook.java | 3 | ||||
| -rw-r--r-- | poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java | 3 |
2 files changed, 6 insertions, 0 deletions
diff --git a/poi-ooxml/src/main/java/org/apache/poi/xssf/streaming/SXSSFWorkbook.java b/poi-ooxml/src/main/java/org/apache/poi/xssf/streaming/SXSSFWorkbook.java index da64a880f2..401529839b 100644 --- a/poi-ooxml/src/main/java/org/apache/poi/xssf/streaming/SXSSFWorkbook.java +++ b/poi-ooxml/src/main/java/org/apache/poi/xssf/streaming/SXSSFWorkbook.java @@ -808,6 +808,9 @@ public class SXSSFWorkbook implements Workbook { /** * Get sheet with the given name * + * If there are multiple matches, the first sheet from the list + * of sheets is returned. + * * @param name of the sheet * @return Sheet with the name provided or <code>null</code> if it does not exist */ diff --git a/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java b/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java index 7c03b27fa4..3703f0dd4b 100644 --- a/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java +++ b/poi-ooxml/src/main/java/org/apache/poi/xssf/usermodel/XSSFWorkbook.java @@ -1208,6 +1208,9 @@ public class XSSFWorkbook extends POIXMLDocument implements Workbook, Date1904Su /** * Get sheet with the given name (case insensitive match) * + * If there are multiple matches, the first sheet from the list + * of sheets is returned. + * * @param name of the sheet * @return XSSFSheet with the name provided or {@code null} if it does not exist */ |