tris/recoll
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Check for newer pdftotext version to avoid double HTML escaping. fixes issue #318

Browse Source
This commit is contained in:
Jean-Francois Dockes 2016-08-05 08:51:34 +02:00
parent 4adf6925b8
commit d6b230043c
1 changed files with 21 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
src/filters/rclpdf.py
View File

@ -88,6 +88,20 @@ class PDFExtractor:
if not self.pdftotext:
self.pdftotext = rclexecm.which("poppler/pdftotext")
# Check if we need to escape portions of text where old
# versions of pdftotext output raw HTML special characters.
self.needescape = True
try:
version = subprocess.check_output([self.pdftotext, "-v"],
stderr=subprocess.STDOUT)
major,minor,rev = version.split()[2].split('.')
# Don't know exactly when this changed but it's fixed in
# jessie 0.26.5
if int(major) > 0 or int(minor) >= 26:
self.needescape = False
except:
pass
# See if we'll try to perform OCR. Need the commands and the
# either the presence of a file in the config dir (historical)
# or a set config variable.
@ -255,13 +269,13 @@ class PDFExtractor:
output += b'<meta http-equiv="Content-Type"' + \
b'content="text/html; charset=UTF-8">\n'
didcs = True
m = re.search(b'''(.*<title>)(.*)(<\/title>.*)''', line)
if not m:
m = re.search(b'''(.*content=")(.*)(".*/>.*)''', line)
if m:
line = m.group(1) + self.em.htmlescape(m.group(2)) + \
m.group(3)
if self.needescape:
m = re.search(b'''(.*<title>)(.*)(<\/title>.*)''', line)
if not m:
m = re.search(b'''(.*content=")(.*)(".*/>.*)''', line)
if m:
line = m.group(1) + self.em.htmlescape(m.group(2)) + \
m.group(3)
# Recoll treats "Subject" as a "title" element
# (based on emails). The PDF "Subject" metadata