Cutlery/calibre
1
0
Fork 0
mirror of https://github.com/kovidgoyal/calibre.git synced 2025-07-08 18:54:09 -04:00
Code Issues Packages Projects Releases Wiki Activity

DOCX Output: Fix some hyperlinks not working in word because the bookmark name is longer than forty characters. Fixes #1688690 [links do not work when converted azw3 to docx; pictures too big](https://bugs.launchpad.net/calibre/+bug/1688690)

Browse Source
This commit is contained in:
Kovid Goyal 2017-05-06 11:05:51 +05:30
parent b2e702667c
commit f273fe407e
No known key found for this signature in database
GPG Key ID: 06BC317B515ACE7C
1 changed files with 16 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
src/calibre/ebooks/docx/writer/links.py
View File

@ -59,7 +59,9 @@ class TOCItem(object):
def sanitize_bookmark_name(base): def sanitize_bookmark_name(base):
return re.sub(r'[^0-9a-zA-Z]', '_', ascii_text(base)) # Max length allowed by Word appears to be 40, we use 32 to leave some
# space for making the name non-unique
return re.sub(r'[^0-9a-zA-Z]', '_', ascii_text(base))[:32].rstrip('_')
class LinksManager(object): class LinksManager(object):
@ -102,6 +104,17 @@ class LinksManager(object):
item, url, tooltip = link item, url, tooltip = link
purl = urlparse(url) purl = urlparse(url)
href = purl.path href = purl.path
def make_link(parent, anchor=None, id=None, tooltip=None):
kw = {}
if anchor is not None:
kw['w_anchor'] = anchor
elif id is not None:
kw['r_id'] = id
if tooltip:
kw['w_tooltip'] = tooltip
return self.namespace.makeelement(parent, 'w:hyperlink', **kw)
if not purl.scheme: if not purl.scheme:
href = item.abshref(href) href = item.abshref(href)
if href in self.document_hrefs: if href in self.document_hrefs:
@ -110,13 +123,13 @@ class LinksManager(object):
bmark = self.anchor_map[key] bmark = self.anchor_map[key]
else: else:
bmark = self.anchor_map[(href, self.top_anchor)] bmark = self.anchor_map[(href, self.top_anchor)]
return self.namespace.makeelement(parent, 'w:hyperlink', w_anchor=bmark, w_tooltip=tooltip or '') return make_link(parent, anchor=bmark, tooltip=tooltip)
else: else:
self.log.warn('Ignoring internal hyperlink with href (%s) pointing to unknown destination' % url) self.log.warn('Ignoring internal hyperlink with href (%s) pointing to unknown destination' % url)
if purl.scheme in {'http', 'https', 'ftp'}: if purl.scheme in {'http', 'https', 'ftp'}:
if url not in self.external_links: if url not in self.external_links:
self.external_links[url] = self.document_relationships.add_relationship(url, self.namespace.names['LINKS'], target_mode='External') self.external_links[url] = self.document_relationships.add_relationship(url, self.namespace.names['LINKS'], target_mode='External')
return self.namespace.makeelement(parent, 'w:hyperlink', r_id=self.external_links[url], w_tooltip=tooltip or '') return make_link(parent, id=self.external_links[url], tooltip=tooltip)
return parent return parent
def process_toc_node(self, toc, level=0): def process_toc_node(self, toc, level=0):