Issue 2323 - Make sure to always resolve links in translatable strings

cms/converters.py

 # coding: utf-8
 
 # This file is part of the Adblock Plus web scripts,
 # Copyright (C) 2006-2015 Eyeo GmbH
 #
 # Adblock Plus is free software: you can redistribute it and/or modify
 # it under the terms of the GNU General Public License version 3 as
 # published by the Free Software Foundation.
 #
 # Adblock Plus is distributed in the hope that it will be useful,
@@ skipping 128 matching lines @@
       result = default
 
     # Insert fixed strings
     for i in range(len(fixed_strings)):
       result = re.sub(r"\{%d\}" % (i + 1), fixed_strings[i], result, 1)
 
     # Insert attributes
     result = escape(result)
     def stringify_attribute((name, value)):
       if name == "href":
-        value = self._params["source"].resolve_link(value, locale)[1] or value

[Wladimir Palant, 2015/04/15 19:13:06]

Note that this step isn't really necessary for Markdown and raw HTML - these
have another processing step that will take care of resolving links. However, it
won't hurt either, and it avoid creating a special case for templates here.

+        link_locale, link = self._params["source"].resolve_link(value, locale)
+        if link:
+          return 'href="%s" hreflang="%s"' % (escape(link), escape(link_locale))
       return '%s="%s"' % (escape(name), escape(value))
 
     for tag in self.whitelist:
       saved = saved_attributes.get(tag, [])
       for attrs in saved:
         attrs = map(stringify_attribute, attrs)
         result = re.sub(
           r"%s([^<>]*?)%s" % (re_escape("<%s>" % tag), re_escape("</%s>" % tag)),
           r'<%s %s>\1</%s>' % (tag, " ".join(attrs), tag),
           result, 1, flags=re.S
@@ skipping 225 matching lines @@
         stack.pop()
       stack[-1]["subitems"].append(item)
       stack.append(item)
     return structured
 
 converters = {
   "html": RawConverter,
   "md": MarkdownConverter,
   "tmpl": TemplateConverter,
 }