issue2550775 enhancement. Add rel=nofollow to journal links

Author: rouilj

CHANGES.txt

@@ -7,6 +7,9 @@ Entries without name were done by Richard Jones.
 
 Features:
 
+- issue2550775 Added rel=nofollow to links in the journal linking to
+  attachments to allow the admin to delete attachment spam and prevent
+  search engines from increasing the rankings. (John Rouillard)
 - issue2550808 Enhanced the boolean field template function. Now by
   default the labels generated can be clicked on and select the
   corresponding radio button. Also can create a trivalued radiobutton

roundup/cgi/templating.py

@@ -889,7 +889,7 @@ def history(self, direction='descending', dre=re.compile('^\d+$'),
                     pass
                 else:
                     id = self._klass.get(self._nodeid, prop_n, None)
-                    current[prop_n] = '<a href="%s%s">%s</a>'%(
+                    current[prop_n] = '<a rel="nofollow" href="%s%s">%s</a>'%(
                         classname, id, current[prop_n])
 
         # get the journal, sort and reverse
@@ -910,15 +910,15 @@ def history(self, direction='descending', dre=re.compile('^\d+$'),
             if action == 'link' and type(args) == type(()):
                 if len(args) == 3:
                     linkcl, linkid, key = args
-                    arg_s += '<a href="%s%s">%s%s %s</a>'%(linkcl, linkid,
+                    arg_s += '<a rel="nofollow" href="%s%s">%s%s %s</a>'%(linkcl, linkid,
                         linkcl, linkid, key)
                 else:
                     arg_s = str(args)
 
             elif action == 'unlink' and type(args) == type(()):
                 if len(args) == 3:
                     linkcl, linkid, key = args
-                    arg_s += '<a href="%s%s">%s%s %s</a>'%(linkcl, linkid,
+                    arg_s += '<a rel="nofollow" href="%s%s">%s%s %s</a>'%(linkcl, linkid,
                         linkcl, linkid, key)
                 else:
                     arg_s = str(args)
@@ -988,7 +988,7 @@ def history(self, direction='descending', dre=re.compile('^\d+$'),
                                     subml.append('<strike>%s</strike>'%label)
                                 else:
                                     if hrefable:
-                                        subml.append('<a href="%s%s">%s</a>'%(
+                                        subml.append('<a rel="nofollow" href="%s%s">%s</a>'%(
                                             classname, linkid, label))
                                     elif label is None:
                                         subml.append('%s%s'%(classname,
@@ -1015,7 +1015,7 @@ def history(self, direction='descending', dre=re.compile('^\d+$'),
                                 label = None
                         if label is not None:
                             if hrefable:
-                                old = '<a href="%s%s">%s</a>'%(classname,
+                                old = '<a ref="nofollow" href="%s%s">%s</a>'%(classname,
                                     args[k], label)
                             else:
                                 old = label;