| Subject: | This module is utterly broken, gut it and make it a wrapper around Spreadsheet::ParseXLSX |
1) parses XML using regexps
2) doesn't decode html entities: & > ect
3) doesn't keep number formating metadata
4) doesn't handle dates correctly:
+ returns 01.03.2016 as "Monday, March 01, 2016"
+ which is wrong, 2016 is a leap year so should be Tuesday
5) breaks with any utf-8 data: returns empty column data
6) possibly others: currently 48 issues against it on CPAN
The pod does mention that this is a quick and dirty module to parse .xlsx files - i suggest making the entire module a wrapper around Spreadsheet::ParseXLSX to fix most (if not all) of the current issues and make it much more compatible with Spreadsheet::ParseExcel:
use strict;
use warnings;
use Spreadsheet::ParseXLSX;
sub new {
return Spreadsheet::ParseXLSX->new->parse( $_[1] );
}
1;