Commits

Anonymous committed ec37f2b

Fixed #12183 -- Made the urlize filter insert the nofollow attribute properly when an http: URL occurs after a mailto: URL. Thanks eronen for the report.

Comments (0)

Files changed (2)

django/utils/html.py

     trim_url = lambda x, limit=trim_url_limit: limit is not None and (len(x) > limit and ('%s...' % x[:max(0, limit - 3)])) or x
     safe_input = isinstance(text, SafeData)
     words = word_split_re.split(force_unicode(text))
-    nofollow_attr = nofollow and ' rel="nofollow"' or ''
     for i, word in enumerate(words):
         match = None
         if '.' in word or '@' in word or ':' in word:
             lead, middle, trail = match.groups()
             # Make URL we want to point to.
             url = None
+            nofollow_attr = ' rel="nofollow"' if nofollow else ''
             if middle.startswith('http://') or middle.startswith('https://'):
                 url = smart_urlquote(middle)
             elif middle.startswith('www.') or ('@' not in middle and \

tests/regressiontests/defaultfilters/tests.py

             u'<a href="http://en.wikipedia.org/wiki/Caf%C3%A9" rel="nofollow">'
             u'http://en.wikipedia.org/wiki/Café</a>')
 
+        # Check urlize adds nofollow properly - see #12183
+        self.assertEqual(urlize('foo@bar.com or www.bar.com'),
+            u'<a href="mailto:foo@bar.com">foo@bar.com</a> or '
+            u'<a href="http://www.bar.com" rel="nofollow">www.bar.com</a>')
+
         # Check urlize handles IDN correctly - see #13704
         self.assertEqual(urlize('http://c✶.ws'),
             u'<a href="http://xn--c-lgq.ws" rel="nofollow">http://c✶.ws</a>')