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.
Owner: |
Nobody in particular
|
Requestors: |
cap10morgan [...] gmail.com
|
Cc: |
|
AdminCc: |
|
|
Severity: |
Normal |
Broken in: |
0.49 |
Fixed in: |
(no value)
|
spreadsheet-parseexcel-bug.zip
|
Wed Feb 04 21:12:13 2009
cap10morgan [...] gmail.com - Ticket created
When CellHandler is set in the constructor, subsequent calls to row_range return erroneous
results. This appears to be due to the fact that it assumes you're parsing the entire worksheet
before calling row_range, which is not the case when using the CellHandler callback.
This is on Mac OS X Leopard using Perl 5.8.8 from MacPorts.
Thu Feb 05 04:47:48 2009
jmcnamara [...] cpan.org - Correspondence added
On Wed Feb 04 21:12:13 2009, cap10morgan wrote:
Show quoted text> When CellHandler is set in the constructor, subsequent calls to
> row_range return erroneous
> results.
Hi,
Thanks for that.
Could you post an example spreadsheet and program. See the following for
some information on what to include to help debugging:
http://rt.cpan.org/Public/Dist/Display.html?Name=Spreadsheet-ParseExcel
John.
--
Thu Feb 05 04:47:49 2009
The RT System itself - Status changed from 'new' to 'open'
Thu Feb 05 12:05:18 2009
cap10morgan [...] gmail.com - Correspondence added
On Thu Feb 05 04:47:48 2009, JMCNAMARA wrote:
Show quoted text> On Wed Feb 04 21:12:13 2009, cap10morgan wrote:
> > When CellHandler is set in the constructor, subsequent calls to
> > row_range return erroneous
> > results.
>
> Hi,
>
> Thanks for that.
>
> Could you post an example spreadsheet and program. See the following for
> some information on what to include to help debugging:
>
>
http://rt.cpan.org/Public/Dist/Display.html?Name=Spreadsheet-ParseExcel
>
> John.
Sure, no problem. Just unzip the attachment and run ./spreadsheet-parseexcel-bug.pl
spreadsheet-parseexcel-bug.xls to see the bug in action. Let me know if you need anything
else to help squash it. :)
Message body not shown because it is not plain text.
Thu Feb 05 12:23:36 2009
jmcnamara [...] cpan.org - Correspondence added
On Thu Feb 05 12:05:18 2009, cap10morgan wrote:
Show quoted text> run ./spreadsheet- parseexcel-bug.pl
> spreadsheet-parseexcel-bug.xls to see the bug in action. Let me know
> if you need anything else to help squash it. :)
Hi,
Hmmm, I'd like to argue that it isn't a bug and that it probably isn't
wise to mix post-parsing functions into the parsing phase.
However, it will probably just be quicker to fix it. :-)
Look out for a fix in the next release.
John.
--
Thu Feb 05 13:01:44 2009
cap10morgan [...] gmail.com - Correspondence added
On Thu Feb 05 12:23:36 2009, JMCNAMARA wrote:
Show quoted text> On Thu Feb 05 12:05:18 2009, cap10morgan wrote:
> > run ./spreadsheet- parseexcel-bug.pl
> > spreadsheet-parseexcel-bug.xls to see the bug in action. Let me know
> > if you need anything else to help squash it. :)
>
> Hi,
>
> Hmmm, I'd like to argue that it isn't a bug and that it probably isn't
> wise to mix post-parsing functions into the parsing phase.
>
> However, it will probably just be quicker to fix it. :-)
>
> Look out for a fix in the next release.
>
> John.
Yeah, that makes sense. If it helps, my use case is that before I switched to the event-driven
model (i.e. CellHandler), I was using the row_range values to create a progress bar to give
some visual feedback to the user. Obviously that's mostly necessary when dealing with large
spreadsheets, but large spreadsheets are also when you most need to use CellHandler and
NotSetCell. :)
Glad to hear there will be a fix coming. Thanks!
Tue Mar 11 13:14:08 2014
DOUGW [...] cpan.org - Correspondence added
On Thu Feb 05 12:05:18 2009, cap10morgan wrote:
Show quoted text>
> Sure, no problem. Just unzip the attachment and run ./spreadsheet-
> parseexcel-bug.pl
> spreadsheet-parseexcel-bug.xls to see the bug in action. Let me know
> if you need anything
> else to help squash it. :)
MaxRow is updated as the worksheet is parsed, so I don't see this getting fixed. If you can devise a way to tell what the MaxRow is before parsing the worksheet data, let me know and I'll fix it, otherwise I'll probably reject this issue eventually.