Item8634: StringifierContrib to skip attachments that it doesn't have the prerequisites to parse

from Item8289:

Anyway, the XLSX.pm shouldn't:

1. use base, that's bad practice, and is probably documented somewhere
2. use SomeOptionalModule - instead, it should try to load them, and disable the XLSX parsing upon failure (maybe with a warning in debug mode)
3. 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

Commenting is disabled while not logged in