Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

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

Maintainer(s)' notes

If you are reporting a bug in Spreadsheet::ParseExcel here are some pointers

1) State the issues as clearly and as concisely as possible. A simple program or Excel test file (see below) will often explain the issue better than a lot of text.

2) Provide information on your system, version of perl and module versions. The following program will generate everything that is required. Put this information in your bug report.

    #!/usr/bin/perl -w

    print "\n    Perl version   : $]";
    print "\n    OS name        : $^O";
    print "\n    Module versions: (not all are required)\n";

    my @modules = qw(
                      Spreadsheet::ParseExcel
                      Scalar::Util
                      Unicode::Map
                      Spreadsheet::WriteExcel
                      Parse::RecDescent
                      File::Temp
                      OLE::Storage_Lite
                      IO::Stringy
                    );

    for my $module (@modules) {
        my $version;
        eval "require $module";

        if (not $@) {
            $version = $module->VERSION;
            $version = '(unknown)' if not defined $version;
        }
        else {
            $version = '(not installed)';
        }

        printf "%21s%-24s\t%s\n", "", $module, $version;
    }

    __END__

3) Upgrade to the latest version of Spreadsheet::ParseExcel (or at least test on a system with an upgraded version). The issue you are reporting may already have been fixed.

4) Create a small example program that demonstrates your problem. The program should be as small as possible. A few lines of codes are worth tens of lines of text when trying to describe a bug.

5) Supply an Excel file that demonstrates the problem. This is very important. If the file is big, or contains confidential information, try to reduce it down to the smallest Excel file that represents the issue. If you don't wish to post a file here then send it to me directly: jmcnamara@cpan.org

6) Say if the test file was created by Excel, OpenOffice, Gnumeric or something else. Say which version of that application you used.

7) If you are submitting a patch you should check with the maintainer whether the issue has already been patched or if a fix is in the works. Patches should be accompanied by test cases.

Asking a question

If you would like to ask a more general question there is the Spreadsheet::ParseExcel Google Group.

Report information
The Basics
Id: 62953
Status: resolved
Priority: 0/
Queue: Spreadsheet-ParseExcel

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

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



Subject: SaveParser omits merged area format
Date: Fri, 12 Nov 2010 14:00:42 +0000
To: bug-Spreadsheet-ParseExcel [...] rt.cpan.org
From: Paul Thornton <prt [...] prt.org>
Download (untitled) / with headers
text/plain 1.6k
I've been trying to track down an issue whereby Spreadsheet::ParseExcel::SaveParser still does not correctly save merged areas. I know this has been discussed in the past, and I've tested with the latest version, but I'm still having problems. I'm not familiar enough with Excel formatting or the inner workings of ParseExcel to offer a patch unfortunately. My template "test-in.xls" was created with Excel 2003 SP3 (version 11.8328.8329). It contains an area which has been merged and set to wrap text, and is then saved. It makes no difference if there is actually some text in the area or not. When the test program "testmerged.pl" is run, a string is written to the cell OK, but in the output file it creates the merge is removed. The wrap text format is retained OK. The output spreadsheet is "test-out.xls" The dump of $cell when running the test shows that the cell has Merged=1 set, so the issue looks to be in SaveParser. If I re-run my test program using the "test-out" as an input, the Merged=1 option is missing from the same dump. Output from the versions program gives: Perl version : 5.008009 OS name : freebsd Module versions: (not all are required) Spreadsheet::ParseExcel 0.58 Scalar::Util 1.22 Unicode::Map 0.112 Spreadsheet::WriteExcel 2.37 Parse::RecDescent 1.963 File::Temp 0.22 OLE::Storage_Lite 0.19 IO::Stringy 2.110 Regards, Paul.
Download testmerged.pl
text/x-perl 880b
#!/usr/bin/perl # Test ParseExcel use Spreadsheet::ParseExcel::SaveParser; use Data::Dumper; $templatefile = "./test-out.xls"; $tmpfile = "./test-out2.xls"; $parser = Spreadsheet::ParseExcel::SaveParser->new(); $template = $parser->Parse($templatefile); $worksheet = $template->worksheet(0); # Write some text into B4 - this is the first cell # of a 4x2 merged area. This cell already exists # in the template as a merged area, we're just # adding the text to it. $cell = $worksheet->get_cell(3,1); $format_id = $cell->{FormatNo}; $worksheet->AddCell(3,1,'This is some text that should wrap and merge into a block from B4-E5', $format_id); print Dumper($cell); # Write some text into a normal cell $worksheet->AddCell(7,1,'After spreadsheet'); $template->SaveAs($tmpfile); print 'Saved file as: ' . $tmpfile . "\n"; exit(0);
Download test-in.xls
application/vnd.ms-excel 13.5k

Message body not shown because it is not plain text.

Download test-out.xls
application/vnd.ms-excel 5.5k

Message body not shown because it is not plain text.

Download (untitled) / with headers
text/plain 276b
Can see if the latest version in the master branch here solves your issue: https://github.com/runrig/spreadsheet-parseexcel I can't tell at a glance if the new test (46_save_parser.t) duplicates your issue. If this fixes your issue I'll close this ticket on the next release.
I'm going to assume this is fixed in 0.60


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.