From 504674b28849c0cc58ae676e84fa244fb4a5ae36 Mon Sep 17 00:00:00 2001
From: r888888888 <r888888888@gmail.com>
Date: Wed, 3 Feb 2016 23:47:17 -0800
Subject: [PATCH] add rel=nofollow in more places

---
 app/helpers/posts_helper.rb                         | 2 +-
 app/logical/d_text.rb                               | 2 +-
 app/presenters/tag_set_presenter.rb                 | 4 ++--
 app/views/posts/partials/index/_related.html.erb    | 4 ++--
 app/views/posts/partials/show/_information.html.erb | 4 ++--
 public/robots.txt                                   | 1 +
 6 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/app/helpers/posts_helper.rb b/app/helpers/posts_helper.rb
index 6ee3f52fb..67120e1d2 100644
--- a/app/helpers/posts_helper.rb
+++ b/app/helpers/posts_helper.rb
@@ -67,7 +67,7 @@ module PostsHelper
     end
 
     if CurrentUser.is_builder? && !source_search.blank?
-      source_link + "&nbsp;".html_safe + link_to("&raquo;".html_safe, posts_path(:tags => source_search))
+      source_link + "&nbsp;".html_safe + link_to("&raquo;".html_safe, posts_path(:tags => source_search), :rel => "nofollow")
     else
       source_link
     end
diff --git a/app/logical/d_text.rb b/app/logical/d_text.rb
index c40b63343..339916e5a 100644
--- a/app/logical/d_text.rb
+++ b/app/logical/d_text.rb
@@ -120,7 +120,7 @@ class DText
   def self.parse_post_links(str)
     str.gsub(/\{\{([^\}]+)\}\}/) do
       tags = CGI.unescapeHTML($1)
-      %{<a href="/posts?tags=#{u(tags)}">#{h(tags)}</a>}
+      %{<a rel="nofollow" href="/posts?tags=#{u(tags)}">#{h(tags)}</a>}
     end
   end
 
diff --git a/app/presenters/tag_set_presenter.rb b/app/presenters/tag_set_presenter.rb
index 5ffdc4f89..6602e0617 100644
--- a/app/presenters/tag_set_presenter.rb
+++ b/app/presenters/tag_set_presenter.rb
@@ -105,8 +105,8 @@ private
       end
 
       if CurrentUser.user.is_gold? && current_query.present?
-        html << %{<a href="/posts?tags=#{u(current_query)}+#{u(tag)}" class="search-inc-tag">+</a> }
-        html << %{<a href="/posts?tags=#{u(current_query)}+-#{u(tag)}" class="search-exl-tag">&ndash;</a> }
+        html << %{<a rel="nofollow" href="/posts?tags=#{u(current_query)}+#{u(tag)}" class="search-inc-tag">+</a> }
+        html << %{<a rel="nofollow" href="/posts?tags=#{u(current_query)}+-#{u(tag)}" class="search-exl-tag">&ndash;</a> }
       end
     end
 
diff --git a/app/views/posts/partials/index/_related.html.erb b/app/views/posts/partials/index/_related.html.erb
index 01315261c..8c30b27d4 100644
--- a/app/views/posts/partials/index/_related.html.erb
+++ b/app/views/posts/partials/index/_related.html.erb
@@ -1,13 +1,13 @@
 <section id="related-box">
   <h1>Related</h1>
   <ul id="related-list">
-    <li><%= link_to "Deleted posts", posts_path(:tags => "#{params[:tags]} status:deleted") %></li>
+    <li><%= link_to "Deleted posts", posts_path(:tags => "#{params[:tags]} status:deleted"), :rel => "nofollow" %></li>
 
     <% if @post_set.is_tag_subscription? %>
       <li><%= link_to "Edit subscriptions", tag_subscriptions_path %></li>
     <% end %>
 
-    <li><%= link_to "Random post", random_posts_path(:tags => params[:tags]), :id => "random-post" %></li>
+    <li><%= link_to "Random post", random_posts_path(:tags => params[:tags]), :id => "random-post", :rel => "nofollow" %></li>
 
     <li><%= link_to "Mobile version", mobile_posts_path(:tags => params[:tags]) %></li>
 
diff --git a/app/views/posts/partials/show/_information.html.erb b/app/views/posts/partials/show/_information.html.erb
index bc72c3eb4..3ab8e3ef3 100644
--- a/app/views/posts/partials/show/_information.html.erb
+++ b/app/views/posts/partials/show/_information.html.erb
@@ -1,7 +1,7 @@
 <ul>
   <li>ID: <%= post.id %></li>
-  <li>Uploader: <%= link_to_user(post.uploader) + "&nbsp;".html_safe + link_to("&raquo;".html_safe, posts_path(:tags => "user:#{post.uploader.name}")) %></li>
-  <li>Date: <%= link_to time_ago_in_words_tagged(post.created_at), posts_path(:tags => "date:#{post.created_at.to_date}") %></li>
+  <li>Uploader: <%= link_to_user(post.uploader) + "&nbsp;".html_safe + link_to("&raquo;".html_safe, posts_path(:tags => "user:#{post.uploader.name}"), :rel => "nofollow") %></li>
+  <li>Date: <%= link_to time_ago_in_words_tagged(post.created_at), posts_path(:tags => "date:#{post.created_at.to_date}"), :rel => "nofollow" %></li>
   <% if post.approver %>
     <li>Approver: <%= link_to_user(post.approver) %></li>
   <% end %>
diff --git a/public/robots.txt b/public/robots.txt
index 8dfb6a71e..1bb87f005 100644
--- a/public/robots.txt
+++ b/public/robots.txt
@@ -31,6 +31,7 @@ Disallow: /post_flags
 Disallow: /post_sets
 Disallow: /post_versions
 Disallow: /post_votes
+Disallow: /posts.atom
 Disallow: /posts/show_seq
 Disallow: /posts/random
 Disallow: /posts/copy_notes