You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@allura.apache.org by gc...@apache.org on 2022/06/23 19:38:20 UTC

[allura] branch gc/8443 created (now 65b3491be)

This is an automated email from the ASF dual-hosted git repository.

gcruz pushed a change to branch gc/8443
in repository https://gitbox.apache.org/repos/asf/allura.git


      at 65b3491be [#8443] included nofollow attr to public links in user profile and activity

This branch includes the following new commits:

     new 65b3491be [#8443] included nofollow attr to public links in user profile and activity

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.



[allura] 01/01: [#8443] included nofollow attr to public links in user profile and activity

Posted by gc...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

gcruz pushed a commit to branch gc/8443
in repository https://gitbox.apache.org/repos/asf/allura.git

commit 65b3491bebd5dba6bc548cd3eb3b0723019b25ef
Author: Guillermo Cruz <gu...@slashdotmedia.com>
AuthorDate: Thu Jun 23 13:38:07 2022 -0600

    [#8443] included nofollow attr to public links in user profile and activity
---
 Allura/allura/ext/user_profile/templates/sections/projects.html | 2 +-
 Allura/allura/ext/user_profile/templates/sections/tools.html    | 4 ++--
 ForgeActivity/forgeactivity/templates/macros.html               | 6 +++---
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/Allura/allura/ext/user_profile/templates/sections/projects.html b/Allura/allura/ext/user_profile/templates/sections/projects.html
index bee6552e9..d5ed052b7 100644
--- a/Allura/allura/ext/user_profile/templates/sections/projects.html
+++ b/Allura/allura/ext/user_profile/templates/sections/projects.html
@@ -42,7 +42,7 @@
                 {{ theme_macros.placeholder_project_icon() }}
             {%- endif -%}
             <span class="project-info">
-                <a href="{{project.url()}}">{{project.name}}</a>
+                <a href="{{project.url()}}" rel="nofollow">{{project.name}}</a>
                 {{project.summary or '&nbsp;'|safe}}
             </span>
             <span class="project-last-updated">
diff --git a/Allura/allura/ext/user_profile/templates/sections/tools.html b/Allura/allura/ext/user_profile/templates/sections/tools.html
index 55aad3530..3c0edd895 100644
--- a/Allura/allura/ext/user_profile/templates/sections/tools.html
+++ b/Allura/allura/ext/user_profile/templates/sections/tools.html
@@ -28,11 +28,11 @@
     <ul>
     {% for tool in c.project.grouped_navbar_entries() %}
         <li>
-            <a href="{{tool.url}}" {{ tool.extra_html_attrs|xmlattr }}>{{ tool.label }}</a>
+            <a href="{{tool.url}}" {{ tool.extra_html_attrs|xmlattr }} rel="nofollow">{{ tool.label }}</a>
             {% if tool.children %}
             <ul>
             {% for child in tool.children %}
-                <li><a href="{{child.url}}" {{ child.extra_html_attrs|xmlattr }}>{{ child.label }}</a></li>
+                <li><a href="{{child.url}}" {{ child.extra_html_attrs|xmlattr }} rel="nofollow">{{ child.label }}</a></li>
             {% endfor %}
             </ul>
             {% endif %}
diff --git a/ForgeActivity/forgeactivity/templates/macros.html b/ForgeActivity/forgeactivity/templates/macros.html
index 4f877324a..4096bb482 100644
--- a/ForgeActivity/forgeactivity/templates/macros.html
+++ b/ForgeActivity/forgeactivity/templates/macros.html
@@ -19,7 +19,7 @@
 
 {% macro activity_obj(o) %}
   {% if o.activity_url %}
-    <a href="{{o.activity_url}}">{{o.activity_name}}</a>
+    <a href="{{o.activity_url}}" rel="nofollow">{{o.activity_name}}</a>
   {% else %}
     {{o.activity_name}}
   {% endif %}
@@ -50,7 +50,7 @@
                         {{ a.verb.capitalize() }} {{ activity_obj(a.obj) }}
                         {% if a.target.activity_name %}on {{ activity_obj(a.target) }}{% endif %}
                         {% if a.obj.project %}on
-                            <a href="{{ a.obj.project.url() }}">{{ a.obj.project.name }}</a>{% endif %}
+                            <a href="{{ a.obj.project.url() }}" rel="nofollow">{{ a.obj.project.name }}</a>{% endif %}
                     </b>
                     {% if a.obj.activity_extras.get('summary') %}
                         <p>
@@ -62,6 +62,6 @@
                 </li>
             {% endfor %}
         </ul>
-        <a class="view-all" href="{{activity_app.url}}">View All</a>
+        <a class="view-all" href="{{activity_app.url}}" rel="nofollow">View All</a>
     {% endif %}
 {%- endmacro %}
\ No newline at end of file