Skip Menu |
 
rt.cpan.org will be shut down on March 1st, 2021.

This queue is for tickets about the Spreadsheet-XLSX CPAN distribution.

Report information
The Basics
Id: 112599
Status: new
Priority: 0/
Queue: Spreadsheet-XLSX

People
Owner: Nobody in particular
Requestors: LEEJO [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: This module is utterly broken, gut it and make it a wrapper around Spreadsheet::ParseXLSX
Download (untitled) / with headers
text/plain 795b
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;


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.