tika-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (TIKA-2354) Missing many embedded images in .doc files
Date Thu, 04 May 2017 03:03:04 GMT

    [ https://issues.apache.org/jira/browse/TIKA-2354?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15996095#comment-15996095
] 

Hudson commented on TIKA-2354:
------------------------------

UNSTABLE: Integrated in Jenkins build tika-2.x-windows #205 (See [https://builds.apache.org/job/tika-2.x-windows/205/])
TIKA-2354 -- .doc is missing many pictures (tallison: rev 0a55b4a4e29f94e3eec29889117ac2445a9d3577)
* (edit) tika-parser-modules/tika-parser-office-module/src/main/java/org/apache/tika/parser/microsoft/WordExtractor.java


> Missing many embedded images in .doc files
> ------------------------------------------
>
>                 Key: TIKA-2354
>                 URL: https://issues.apache.org/jira/browse/TIKA-2354
>             Project: Tika
>          Issue Type: Bug
>            Reporter: Tim Allison
>            Priority: Blocker
>             Fix For: 2.0, 1.15
>
>         Attachments: 778695.doc
>
>
> On a slightly deeper look at the comparison results between 1.14 and trunk, it looks
like we're missing quite a few embedded images from .doc files.  I initially thought these
could be explained by different handling of macros, but that's not the issue.
> I haven't traced the commit that did it (very likely my fault), but...
> when we call this with a null character run.
> {noformat}
>         // Handle any pictures that we haven't output yet
>         for (Picture p = pictures.nextUnclaimed(); p != null; ) {
>             handlePictureCharacterRun(
>                     null, p, pictures, xhtml
>             );
>             p = pictures.nextUnclaimed();
>         }
> {noformat}
> the null character run then triggers skipping of the picture in this check because {{isRendered(cr)}}
returns false if {{cr}} is {{null}}
> {noformat}
>         if (!isRendered(cr) || picture == null) {
>             // Oh dear, we've run out...
>             // Probably caused by multiple \u0008 images referencing
>             //  the same real image
>             return;
>         }
> {noformat}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message