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

People
Owner: Nobody in particular
Requestors: pat.mariani [...] att.net
Cc:
AdminCc:

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

Attachments
Broadband Intitative Review Input Template (Assign Training).xlsx



Subject: chartsheet fix doesn't account for changed order or deleted sheets
Download (untitled) / with headers
text/plain 1.2k
Perl version 5.8.7 xlsx release .11 solaris .xlsx File attached - loop over last worksheet incorrectly. worksheet named by sheetId (8 in this case) is incorrect after deleted sheets - would be correctly named x1/worksheets/sheet4.xml worksheet description mentions 4 sheets... (apply the sample program to dump the workbook will fail on the last sheet, but get the name correct! since it can not find the data) Moving the sheet order in excel and 're-saving' produces varied results. (name no longer matches data) From workbook.xml [code] - <sheets> <sheet name="Project List" sheetId="1" r:id="rId1" /> <sheet name="Service Areas" sheetId="2" r:id="rId2" /> <sheet name="Communities Chosen" sheetId="3" r:id="rId3" /> <sheet name="CBs" sheetId="8" r:id="rId4" /> </sheets> [/code] Added r:id code back in - the computer guy in me doesn't like the elseif without catching the 'else' !! :) [code] if ($k eq 'name') { $sheet -> {Name} = $v; $sheet -> {Name} = $converter -> convert ($sheet -> {Name}) if $converter; } elsif ($k eq 'sheetId') { ($sheet -> {Id}) = $v; } elsif ($k eq 'r:id') { ($sheet -> {Id}) = $v =~ m{rId(\d+)}; }; [/code]
Subject: Broadband Intitative Review Input Template (Assign Training).xlsx
Download Broadband Intitative Review Input Template (Assign Training).xlsx
application/vnd.openxmlformats-officedocument.spreadsheetml.sheet 850.8k

Message body not shown because it is not plain text.

Subject: Patched in version 0.12
I included your file in the test suite. It wold be very nice to have a smaller one...
Subject: RE: [rt.cpan.org #50236] Patched in version 0.12
Date: Tue, 6 Oct 2009 07:37:10 -0400
To: <bug-Spreadsheet-XLSX [...] rt.cpan.org>
From: "Patrick Mariani" <pat.mariani [...] att.net>
Download (untitled) / with headers
text/plain 431b
I will work on one! I'm sure I can come up with something. Pat Show quoted text
-----Original Message----- From: Dmitry Ovsyanko via RT [mailto:bug-Spreadsheet-XLSX@rt.cpan.org] Sent: Tuesday, October 06, 2009 4:52 AM To: pat.mariani@att.net Subject: [rt.cpan.org #50236] Patched in version 0.12 <URL: https://rt.cpan.org/Ticket/Display.html?id=50236 > I included your file in the test suite. It wold be very nice to have a smaller one...


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.