Skip Menu |

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

Report information
The Basics
Id: 81909
Status: resolved
Priority: 0/
Queue: Spreadsheet-WriteExcel-Styler
People
Owner: Nobody in particular
Requestors: stephenclouse [...] gmail.com
Cc: jmcnamara [...] cpan.org
AdminCc:
Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Tue Dec 11 13:52:50 2012 stephenclouse [...] gmail.com - Ticket created
Subject: Support Excel::Writer::XLSX
Right now an error is thrown if !$workbook->isa('Spreadsheet::WriteExcel') Spreadsheet::WriteExcel has been superseded by Excel::Writer::XLSX, according to its author. The two are fully API-compatible, so all that's really necessary for Styler to work is adjusting the isa check.
  Wed Dec 12 13:41:54 2012 DAMI [...] cpan.org - Cc JMCNAMARA added
  Wed Dec 12 14:31:57 2012 DAMI [...] cpan.org - Correspondence added
RT-Send-CC: jmcnamara [...] cpan.org
Le Mar 11 Déc 2012 13:52:50, SCLOUSE a écrit : Show quoted text
> Right now an error is thrown if !$workbook->isa
('Spreadsheet::WriteExcel') Show quoted text
> > Spreadsheet::WriteExcel has been superseded by Excel::Writer::XLSX, > according to its author. The two are fully API-compatible, so all > that's really necessary for Styler to work is adjusting the isa check.
Hi, Thanks for the suggestion; fixed in v1.02 (now we just check if the workbook object received as argument has an 'add_format' method). I hesitated to also rename the module so that it would sit in the new Excel::* namespace, but for now I left it as is. @John : if you are inclined to it, feel free to borrow the code and incorporate it directly within Excel::Writer. This would be probably more convenient for users, and then I would mark my module as obsolete. Best regards, Laurent Dami
  Wed Dec 12 14:31:59 2012 The RT System itself - Status changed from 'new' to 'open'
  Wed Dec 12 14:31:59 2012 DAMI [...] cpan.org - Status changed from 'open' to 'resolved'