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-1980) HTML head tags found after first script not parsed by HtmlParser (regression)
Date Fri, 12 Aug 2016 16:50:20 GMT

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

Hudson commented on TIKA-1980:
------------------------------

SUCCESS: Integrated in Tika-trunk #1091 (See [https://builds.apache.org/job/Tika-trunk/1091/])
fix for TIKA-1980 contributed by naegelejd (jnaegele: rev 9024f12e0fbef206330b6317a537354fafbe9cd7)
* tika-core/src/main/java/org/apache/tika/sax/XHTMLContentHandler.java
* tika-parsers/src/test/resources/test-documents/head.html
* tika-parsers/src/test/java/org/apache/tika/parser/html/HtmlParserTest.java
TIKA-1980 - via Joseph Naegele. This closes #121 (tallison: rev 5495ffcf35bb702d6cdb28b4fa3bdc7df34e55e9)
* tika-parsers/src/test/resources/test-documents/head.html
* tika-parsers/src/test/resources/test-documents/testHTML_head.html
* tika-parsers/src/test/java/org/apache/tika/parser/html/HtmlParserTest.java


> HTML head tags found after first script not parsed by HtmlParser (regression)
> -----------------------------------------------------------------------------
>
>                 Key: TIKA-1980
>                 URL: https://issues.apache.org/jira/browse/TIKA-1980
>             Project: Tika
>          Issue Type: Bug
>          Components: parser
>            Reporter: Joseph Naegele
>            Assignee: Tim Allison
>             Fix For: 2.0, 1.14
>
>         Attachments: head.html, testAllHeadElements.patch
>
>
> TIKA-1938 introduced a regression where HTML elements found within {{<head>}} after
the first {{<script>}} element are "dropped".
> I've attached a patch that adds a JUnit test to {{tika-parsers/src/test/java/org/apache/tika/parser/html/HtmlParserTest.java}},
and the corresponding test resource, which belongs at {{tika-parsers/src/test/resources/test-documents/head.html}}.
The new test document is needed because it contains {{<script>}} elements before the
{{<title>}} element and before some of the {{<meta>}} tags.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message