Use find_or_initialize_by (#9504)

7 years ago · 351ed12b2f
parent cf6ee4ff24
commit 351ed12b2f
1 changed files with 1 additions and 6 deletions
--- a/app/services/pubsubhubbub/subscribe_service.rb
+++ b/app/services/pubsubhubbub/subscribe_service.rb
@ -58,12 +58,7 @@ class Pubsubhubbub::SubscribeService < BaseService
  end

  def locate_subscription
-    subscription = Subscription.find_by(account: account, callback_url: callback)
-
-    if subscription.nil?
-      subscription = Subscription.new(account: account, callback_url: callback)
-    end
-
+    subscription = Subscription.find_or_initialize_by(account: account, callback_url: callback)
    subscription.domain = domain
    subscription.save!
    subscription