Item8634: StringifierContrib to skip attachments that it doesn't have the prerequisites to parse
Priority: Enhancement
Current State: Closed
Released In: n/a
Target Release: n/a
from
Item8289:
Anyway, the XLSX.pm shouldn't:
- use base, that's bad practice, and is probably documented somewhere
- use SomeOptionalModule - instead, it should try to load them, and disable the XLSX parsing upon failure (maybe with a warning in debug mode)
- not be in DOS mode
But yeah, you're right, I should just have not asked to index XLSX anyway.
But even if I comment .xslx from the config, it still tries to load the .pm, so it doesn't help.
Did a quick & dirty fix for now, like:
BEGIN {
eval { require Spreadsheet::XLSX; };
return 0 if $@; # Disabled until somebody fixes this loading
}
#use Spreadsheet::XLSX;
--
OlivierRaginel - 25 Sep 2009
i'm going to go with this:
+ try {
+ require Spreadsheet::XLSX;
+ } catch Error with {
+ return '';
+ }
same idea applied to HTML (for
CPAN:HTML::TreeBuilder). everything else is an external program (rather than a CPAN dependency) and already don't cause things to fail to compile, so they don't need any updating.
--
WillNorris - 24 Oct 2010
this fix may deal with the issue on
Item9311
--
WillNorris - 24 Oct 2010