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: 114521
Status: new
Priority: 0/
Queue: Spreadsheet-XLSX

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

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



Subject: $v might be undefined
Download (untitled) / with headers
text/plain 921b
--- Spreadsheet/XLSX.pm 2015-10-31 03:33:19.000000000 +0100 +++ Spreadsheet/XLSX.pm 2016-05-20 16:28:34.612377628 +0200 @@ -111,7 +111,7 @@ sub _load_workbook { } elsif (length($_) && $parsing_v_tag) { my $v = $s ? $shared_info->{shared_strings}->[$_] : $_; - if ($v eq "</c>") { + if (defined $v && $v eq "</c>") { $v = ""; } my $type = "Text"; @@ -134,7 +134,7 @@ sub _load_workbook { $sheet->{MinRow} = $row if $sheet->{MinRow} > $row; $sheet->{MinCol} = $col if $sheet->{MinCol} > $col; - if ($v =~ /(.*)E\-(.*)/gsm && $type eq "Numeric") { + if (defined $v && $v =~ /(.*)E\-(.*)/gsm && $type eq "Numeric") { $v = $1 / (10**$2); # this handles scientific notation for very small numbers }


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.