FIX: Strip links from google indexed bios when the users are new.

app/models/user.rb

@@ -433,13 +433,12 @@ class User < ActiveRecord::Base
   def bio_excerpt
     excerpt = PrettyText.excerpt(bio_cooked, 350)
     return excerpt if excerpt.blank? || has_trust_level?(:basic)
+    PrettyText.strip_links(excerpt)
+  end
 
-    # If the user is not basic, strip links from their bio
+  def bio_processed
-    fragment = Nokogiri::HTML.fragment(excerpt)
+    return bio_cooked if bio_cooked.blank? || has_trust_level?(:basic)
-    fragment.css('a').each do |a|
+    PrettyText.strip_links(bio_cooked)
-      a.replace(a.text)
-    end
-    fragment.to_html
   end
 
   def delete_all_posts!(guardian)

app/views/users/show.html.erb

@@ -1,6 +1,6 @@
 <h2><%= @user.username %></h2>
 
-<p><%= raw @user.bio_cooked %></p>
+<p><%= raw @user.bio_processed %></p>
 
 <p><%= t 'powered_by_html' %></p>
 

lib/pretty_text.rb

@@ -239,6 +239,15 @@ module PrettyText
     ExcerptParser.get_excerpt(html, max_length, options)
   end
 
+  def self.strip_links(string)
+    return string if string.blank?
+
+    # If the user is not basic, strip links from their bio
+    fragment = Nokogiri::HTML.fragment(string)
+    fragment.css('a').each {|a| a.replace(a.text) }
+    fragment.to_html
+  end
+
   protected
 
   def self.ctx_load(*files)

spec/components/pretty_text_spec.rb

@@ -183,6 +183,19 @@ test
 
   end
 
+  describe "strip links" do
+    it "returns blank for blank input" do
+      expect(PrettyText.strip_links("")).to be_blank
+    end
+
+    it "does nothing to a string without links" do
+      expect(PrettyText.strip_links("I'm the <b>batman</b>")).to eq("I'm the <b>batman</b>")
+    end
+
+    it "strips links but leaves the text content" do
+      expect(PrettyText.strip_links("I'm the linked <a href='http://en.wikipedia.org/wiki/Batman'>batman</a>")).to eq("I'm the linked batman")
+    end
+  end
 
   describe "apply cdn" do
     it "should detect bare links to images and apply a CDN" do

spec/models/user_spec.rb

@@ -836,7 +836,7 @@ describe User do
 
   end
 
-  describe "bio_excerpt" do
+  describe "bio link stripping" do
 
     it "returns an empty string with no bio" do
       expect(Fabricate.build(:user).bio_excerpt).to be_blank
@@ -852,11 +852,13 @@ describe User do
 
       it "includes the link if the user is not new" do
         expect(user.bio_excerpt).to eq("im sissy and i love <a href='http://ponycorns.com' rel='nofollow'>http://ponycorns.com</a>")
+        expect(user.bio_processed).to eq("<p>im sissy and i love <a href=\"http://ponycorns.com\" rel=\"nofollow\">http://ponycorns.com</a></p>")
       end
 
       it "removes the link if the user is new" do
         user.trust_level = TrustLevel.levels[:newuser]
         expect(user.bio_excerpt).to eq("im sissy and i love http://ponycorns.com")
+        expect(user.bio_processed).to eq("<p>im sissy and i love http://ponycorns.com</p>")
       end
     end