Skip Menu | You are currently an anonymous guest. | Login | Return to Main | About rt.cpan.org
 

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

X Report information
Id: 14278
Status: resolved
Worked: 10 min
Left: 0 min
Priority: 0/0
Queue: Spreadsheet-ParseExcel

Owner: Nobody
Requestors: BULB <BULB [...] cpan.org>
Cc:
AdminCc:

Severity: Important
Broken in: (no value)
Fixed in: 0.2603



X History Display mode: Brief headersFull headers
#   Tue Aug 23 08:51:52 2005 BULB - Ticket created  
Subject: Drop or at least document the overload on Worksheet
[text/plain 613b]
Hello,

The undocumented numification overload on
Spreadsheet::ParseExcel::Worksheet makes it behave strongly
contraintuitively, as the first sheet evaluates to false in boolean
context. This falsifies the common assumption that
if($book->Worksheet($name)) and if(defined $book->Worksheet($name)) are
equivaltent as the return value is supposed to be an object and such
assumptions holds for majority of Perl object. In addition the overload
does not really seem to be useful. So I suggest removing it, or at
least documenting it.

--
- Jan Hudec <bulb[...]ucw.cz>
#   Fri Jan 23 04:17:36 2009 JMCNAMARA - Correspondence added 10 min  
[text/plain 74b]
Hi,

Removed in Spreadsheet::ParseExcel version 0.48.

Thanks,

John.
--

#   Fri Jan 23 04:17:37 2009 RT_System - Status changed from 'new' to 'open'  
#   Fri Jan 23 04:17:39 2009 JMCNAMARA - Status changed from 'open' to 'resolved'