diff --git a/app/models/comment.rb b/app/models/comment.rb index 08ec011e765e1e3fbc8e602d44a7ccdbc79b1c45..e3084553fca0b93dfbc7df9a8cbcec33a01d18c7 100644 --- a/app/models/comment.rb +++ b/app/models/comment.rb @@ -25,7 +25,6 @@ class Comment < ActiveRecord::Base serialize :youtube_titles, Hash before_save do - get_youtube_title text self.text.strip! unless self.text.nil? end def diaspora_handle diff --git a/app/models/request.rb b/app/models/request.rb index 775d79361d5229d24f2ffbc339a5521ffc414a29..af97fc7eaff71c7a9598868820369e9a85ad5a1e 100644 --- a/app/models/request.rb +++ b/app/models/request.rb @@ -71,7 +71,7 @@ class Request private def not_already_connected - if sender && recipient && Contact.where(:user_id => self.recipient.owner_id, :person_id => self.sender.id).count > 0 + if sender && recipient && Contact.where(:user_id => self.recipient.owner_id, :person_id => self.sender.id).exists? errors[:base] << 'You have already connected to this person' end end diff --git a/app/models/status_message.rb b/app/models/status_message.rb index 462b6acbe49a12073a78a9975794b3ad2f2ef959..3fb1f9b7faa2baf9b29f79e0a6f2792b49630618 100644 --- a/app/models/status_message.rb +++ b/app/models/status_message.rb @@ -23,9 +23,6 @@ class StatusMessage < Post attr_accessible :text serialize :youtube_titles, Hash - before_save do - get_youtube_title text - end before_create :build_tags diff --git a/lib/youtube_titles.rb b/lib/youtube_titles.rb index 8530e8ad3e27d9b1694a9ec5ec33aad76585ef73..cf90f28b3ca66f616d704f7c8e291f43e0c703c4 100644 --- a/lib/youtube_titles.rb +++ b/lib/youtube_titles.rb @@ -1,4 +1,12 @@ module YoutubeTitles + def self.included(model) + model.class_eval do + before_save do + get_youtube_title text + end + end if model.respond_to?(:before_save) + end + def youtube_title_for video_id http = Net::HTTP.new('gdata.youtube.com', 80) path = "/feeds/api/videos/#{video_id}?v=2"