poi-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 51415] Sheet name cannot exceed 31 characters
Date Thu, 23 Jun 2011 10:49:38 GMT
https://issues.apache.org/bugzilla/show_bug.cgi?id=51415

Yegor Kozlov <yegor@dinom.ru> changed:

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

--- Comment #2 from Yegor Kozlov <yegor@dinom.ru> 2011-06-23 10:49:38 UTC ---
Fixed in r1138819

I fixed workbook.createSheetName(sheetName) to silently truncate the input
argument if it is longer than 31 character. 

The idea to throw IllegalArgumentException is not very good. My concern is
existing usages of POI in production. With version of POI prior to 3.8 existing
code swallowed long sheet names without any exceptions. If we decided to throw
IllegalArgumentException then these system would start to fail which is not
good.

Truncating the input argumnet to 31 character seems a reasonable workaround. 

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


Mime
View raw message