diff --git a/app/controllers/tags_controller.rb b/app/controllers/tags_controller.rb
index 9aa9616d90b1f2be4bf48e5316827fdb04154882..5bf799cfe886f786972a201c41cbecca4638272a 100644
--- a/app/controllers/tags_controller.rb
+++ b/app/controllers/tags_controller.rb
@@ -56,18 +56,11 @@ class TagsController < ApplicationController
@posts = StatusMessage.where(:public => true, :pending => false)
end
- @tag = ActsAsTaggableOn::Tag.where(:name => params[:name]).first
- if @tag
- @posts = @posts.joins("LEFT OUTER JOIN comments ON comments.post_id = posts.id").
- joins("INNER JOIN taggings ON (taggings.tag_id = #{@tag.id} AND
- ((taggable_id = posts.id AND taggable_type = 'Post') OR (taggings.taggable_type = 'Comment' AND taggings.taggable_id = comments.id)))")
+ @posts = @posts.tagged_with(params[:name])
- max_time = params[:max_time] ? Time.at(params[:max_time].to_i) : Time.now
- @posts = @posts.where(StatusMessage.arel_table[:created_at].lt(max_time))
- @posts = @posts.includes({:comments => {:author => :profile}}, :photos).order('posts.created_at DESC').limit(15)
- else
- @posts = []
- end
+ max_time = params[:max_time] ? Time.at(params[:max_time].to_i) : Time.now
+ @posts = @posts.where(StatusMessage.arel_table[:created_at].lt(max_time))
+ @posts = @posts.includes({:comments => {:author => :profile}}, :photos).order('posts.created_at DESC').limit(15)
@posts = PostsFake.new(@posts)
@commenting_disabled = true
diff --git a/app/models/comment.rb b/app/models/comment.rb
index fd416b370d1ef5ee60f23ec112fde87e02979d7c..f0d829dd4a2b15e743e47a4d7a679c99e3caa953 100644
--- a/app/models/comment.rb
+++ b/app/models/comment.rb
@@ -13,11 +13,6 @@ class Comment < ActiveRecord::Base
include Diaspora::Relayable
include Diaspora::Socketable
- include Diaspora::Taggable
-
- acts_as_taggable_on :tags
- extract_tags_from :text
- before_create :build_tags
xml_attr :text
xml_attr :diaspora_handle
diff --git a/app/models/status_message.rb b/app/models/status_message.rb
index d037ec7d7f99f60e3d0f58569bc87f186326cad3..a5aed41b34630af6b2f0535c051f7755ad20323c 100644
--- a/app/models/status_message.rb
+++ b/app/models/status_message.rb
@@ -43,7 +43,7 @@ class StatusMessage < Post
escaped_message = opts[:plain_text] ? self.raw_message: ERB::Util.h(self.raw_message)
mentioned_message = self.format_mentions(escaped_message, opts)
- Diaspora::Taggable.format_tags(mentioned_message, opts)
+ self.format_tags(mentioned_message, opts)
end
def format_mentions(text, opts = {})
diff --git a/app/views/comments/_comment.html.haml b/app/views/comments/_comment.html.haml
index 19e54da98c1e3cabdd0c60221cfe029445893ce3..0809a407b0c9fdb3c9bda3e44a35b0e0cca14926 100644
--- a/app/views/comments/_comment.html.haml
+++ b/app/views/comments/_comment.html.haml
@@ -12,7 +12,7 @@
= person_link(comment.author, :class => "hovercardable")
%span{:class => direction_for(comment.text)}
- = markdownify(Diaspora::Taggable.format_tags(comment.text), :youtube_maps => comment.youtube_titles)
+ = markdownify(comment.text, :youtube_maps => comment.youtube_titles)
%br
%time.timeago{:datetime => comment.created_at}
diff --git a/app/views/people/_person.html.haml b/app/views/people/_person.html.haml
index fb8e759aea124542f50ba87e3fbb238b41f1ed49..b01b479dff1986ab31ace48bc3829d5ecee99b65 100644
--- a/app/views/people/_person.html.haml
+++ b/app/views/people/_person.html.haml
@@ -15,5 +15,5 @@
=person_link(person)
.info
- = Diaspora::Taggable.format_tags(person.profile.tag_string)
+ = person.profile.format_tags(person.profile.tag_string)
diff --git a/app/views/people/contacts.haml b/app/views/people/contacts.haml
index d3a8b47b22697b1a6045857235fafd4131891a3e..0c704398da4658e4532025bae1a729aab800e7c0 100644
--- a/app/views/people/contacts.haml
+++ b/app/views/people/contacts.haml
@@ -47,7 +47,7 @@
= @person.diaspora_handle
.description
- if !@person.profile.tag_string.blank?
- = Diapsora::Taggable.format_tags(@person.profile.tag_string)
+ = @person.profile.format_tags(@person.profile.tag_string)
- if user_signed_in? && @person == current_user.person
%span.hover_edit
= link_to t('people.show.edit'), edit_profile_path
diff --git a/app/views/people/show.html.haml b/app/views/people/show.html.haml
index 32e429da4bedd0ba641c89ff2b16448353568706..f163c814d8d74c285a232b95f2f78e339f9df198 100644
--- a/app/views/people/show.html.haml
+++ b/app/views/people/show.html.haml
@@ -38,7 +38,7 @@
.description
- if !@person.profile.tag_string.blank? && user_signed_in? && (@contact.persisted? || @person == current_user.person || @incoming_request)
- = Diaspora::Taggable.format_tags(@person.profile.tag_string)
+ = @person.profile.format_tags(@person.profile.tag_string)
- if user_signed_in? && @person == current_user.person
%span.hover_edit
= link_to t('.edit'), edit_profile_path
diff --git a/lib/diaspora/taggable.rb b/lib/diaspora/taggable.rb
index 7c3ab66bd24b6c8df4f3537dc6b1a76cbf8cc7b7..2eff354428c06900bae53f82ce7323730c36e7a8 100644
--- a/lib/diaspora/taggable.rb
+++ b/lib/diaspora/taggable.rb
@@ -36,7 +36,7 @@ module Diaspora
unique_matches.values
end
- def self.format_tags(text, opts={})
+ def format_tags(text, opts={})
return text if opts[:plain_text]
regex = /(^|\s)#(#{VALID_TAG_BODY})/
form_message = text.gsub(regex) do |matched_string|
diff --git a/spec/controllers/tags_controller_spec.rb b/spec/controllers/tags_controller_spec.rb
index c412313d01c254ae300b90313f1918e408e23ed8..34d892e5f2afade260a5debabd218333193e1d1a 100644
--- a/spec/controllers/tags_controller_spec.rb
+++ b/spec/controllers/tags_controller_spec.rb
@@ -69,20 +69,6 @@ describe TagsController do
get :show, :name => 'hello'
assigns(:posts).models.should == [stranger_post]
end
-
- it 'displays a post with a comment containing the tag search' do
- bob.post(:status_message, :text => "other post y'all", :to => 'all')
- other_post = bob.post(:status_message, :text => "sup y'all", :to => 'all')
- Factory(:comment, :text => "#hello", :post => other_post)
- get :show, :name => 'hello'
- assigns(:posts).models.should == [other_post]
- response.status.should == 200
- end
-
- it 'succeeds without posts' do
- get :show, :name => 'hellyes'
- response.status.should == 200
- end
end
context "not signed in" do
diff --git a/spec/models/comment_spec.rb b/spec/models/comment_spec.rb
index f815a2c644f18c39f3752ac4e4d0bfc14769e1d5..74f6d787147a6d1dc45199a5bbc26f6045364b67 100644
--- a/spec/models/comment_spec.rb
+++ b/spec/models/comment_spec.rb
@@ -126,11 +126,4 @@ describe Comment do
it_should_behave_like 'it is relayable'
end
- describe 'tags' do
- before do
- @object = Factory.build(:comment)
- end
- it_should_behave_like 'it is taggable'
- end
-
end
diff --git a/spec/shared_behaviors/taggable.rb b/spec/shared_behaviors/taggable.rb
index cb47fe0386770c773cc3e09f6245fa3465fbdcfd..0b1bd95f0e76293c1f66783e6f9c19d8924302a7 100644
--- a/spec/shared_behaviors/taggable.rb
+++ b/spec/shared_behaviors/taggable.rb
@@ -12,7 +12,7 @@ describe Diaspora::Taggable do
def controller
end
- describe '.format_tags' do
+ describe '#format_tags' do
before do
@str = '#what #hey #vöglein'
@object.send(@object.class.field_with_tags_setter, @str)
@@ -21,10 +21,10 @@ describe Diaspora::Taggable do
end
it 'links the tag to /p' do
link = link_to('#vöglein', '/tags/vöglein', :class => 'tag')
- Diaspora::Taggable.format_tags(@str).should include(link)
+ @object.format_tags(@str).should include(link)
end
it 'responds to plain_text' do
- Diaspora::Taggable.format_tags(@str, :plain_text => true).should == @str
+ @object.format_tags(@str, :plain_text => true).should == @str
end
end
describe '#build_tags' do