Use find_or_initialize_by (#9504)

6 years ago · 201a9e844a
parent 5548049e71
commit 201a9e844a
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