DO NOT REPLY [Bug 48087] org.apache.poi.xssf.usermodel.XSSFChartSheet completely broken

[bu...@apache.org]

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

--- Comment #1 from Yegor Kozlov <ye...@dinom.ru> 2009-10-30 01:07:18 UTC ---
It is not broken, rather not implemented. 

Initializing of the superclass is skipped intentionally. Chart sheets don't
have rows/columns and most of methods from XSSFSheet don't make sense for
XSSFChartSheet. 

Can you post a use-case that results in NPE? What methods are you calling? 

I agree that NPE is not appropriate. get* methods of the Sheet interface that
don't make sense should either return an empty iterator or false or 0:
At least, the code below should NOT throw NPE:
for(Row row : chartSheet)
for(int i=0; i < chartSheet.getPhysicalNumberOfRows()) 

set* modifiers should either do nothing or throw "Not supported for charts".


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