Skip Menu |

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

Report information
The Basics
Id: 105197
Status: resolved
Priority: 0/
Queue: Spreadsheet-Read
People
Owner: Nobody in particular
Requestors: NERDVANA [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: 0.47
Fixed in: 0.61

History Show all quoted text
  Fri Jun 12 18:25:16 2015 NERDVANA [...] cpan.org - Ticket created
Subject: strip => 1 broken when cells => 0
In ReadData($filename, cells => 0, strip => 1), the result still has a hash key for every cell, though they are all undef. It is caused by a backward test on line 209, which uses ss->{cell} if opt->{cell} and ss->{$name} if $opt->{rc}.
  Sat Jun 13 06:47:33 2015 HMBRAND [...] cpan.org - Correspondence added
Thanks for the report. You are completely right. It is now fixed in git and will be in the next release. Adding tests for this also surfaced another buglet, so thanks again.
  Sat Jun 13 06:47:34 2015 The RT System itself - Status changed from 'new' to 'open'
  Sat Jun 13 06:47:39 2015 HMBRAND [...] cpan.org - Status changed from 'open' to 'patched'
  Sat Jun 13 06:47:58 2015 HMBRAND [...] cpan.org - Broken in 0.47 added
  Sat Jun 13 06:48:00 2015 HMBRAND [...] cpan.org - Broken in 0.60 deleted
  Sat Jun 13 09:27:48 2015 HMBRAND [...] cpan.org - Fixed in 0.61 added
  Sat Jun 13 09:27:50 2015 HMBRAND [...] cpan.org - Status changed from 'patched' to 'resolved'