The patch for add functions including:
Using pdftotext from poppler to parse pdf file.
Using catdoc to parse doc file.
Using catppt to parse ppt file.
Using xls3csv to parse xls file.
Using ebooklib from http://sourceforge.net/projects/oplsepublibrary/ to parse epub file.
(Put lib files in /sites/all/libraries/eBookLib/)
Attention: the lib from http://sourceforge.net/projects/oplsepublibrary/ is bugy and less maintained for ages, so I attach the lib with my fix here.

TODO:
add encoding process
Maybe better command exist checking

CommentFileSizeAuthor
#3 apachesolr_attachments.patch7.76 KBshenzhuxi
apachesolr_attachments.patch5.9 KBshenzhuxi
eBookLib.tar_.gz7.22 KBshenzhuxi

Comments

jpmckinney’s picture

jpmckinney commented
Status: Active » Needs work

On some systems, these libraries maybe installed to /usr/local/bin, /opt/bin, $HOME/bin, etc.

The patch should account for that.

jpmckinney’s picture

jpmckinney commented
Version: 6.x-1.0-beta2 » 6.x-1.x-dev
shenzhuxi’s picture

shenzhuxi commented
StatusFileSize
new apachesolr_attachments.patch7.76 KB

Options are added

jpmckinney’s picture

jpmckinney commented
Status: Needs work » Needs review

Disregard my previous comment - I had misread the patch. The patch does in fact allow for changing the path to the libraries.

jpmckinney’s picture

jpmckinney commented
Status: Needs review » Needs work

The second patch is better, but why do you include the following changes?

-        if (empty($node->language)) {
-          // 'und' is the language-neutral code in Drupal 7.
-          $document->language = 'und';
-        }
-        else {
-          $document->language = $node->language;
-        }

-        // Shortened project name because the watchdog limits type to 16 characters.
-        watchdog('ApacheSolrAttach', 'Could not extract any indexable text from %filepath', array('%filepath' => $file->filepath), WATCHDOG_WARNING);
+        watchdog('Apache Solr Attachments', 'Could not extract any indexable text from %filepath', array('%filepath' => $file->filepath), WATCHDOG_WARNING);