search

monitorjbl / excel-streaming-reader

An easy-to-use implementation of a streaming Excel reader using Apache POI
Apache License 2.0
953 stars 344 forks source link

Handle Cells without defined style. #90

Closed alvaroAndresCarral closed 7 years ago

alvaroAndresCarral commented 7 years ago

If the Cell has no defined style, return the default style of the Workbook.

For example if a cell is formatted under the "General" category, its style instead of being General it ended up being 'Null'. This PR adds defines the Cell style to the Workbooks default as Excel does.

monitorjbl commented 7 years ago

Could you make a unit test for this case? Nothing fancy, just a quick test on a Cell that has no style.

alvaroAndresCarral commented 7 years ago

Added unit test.

monitorjbl commented 7 years ago

Awesome, thanks for the submission!