mastofe

commit: e0b5a94a4b98aa99369428af42216b308377c618
parent: 26ec042f38eee5d5c0b2127cd199e27539e28cd0
Author: Matt Jankowski <mjankowski@thoughtbot.com>
Date:   Sun, 16 Apr 2017 10:38:29 -0400

Clean up check that account needs a webfinger update (#1932)


Diffstat:
M
app/services/follow_remote_account_service.rb
6
+++++-

1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/app/services/follow_remote_account_service.rb b/app/services/follow_remote_account_service.rb
@@ -16,7 +16,7 @@ class FollowRemoteAccountService < BaseService
     return Account.find_local(username) if TagManager.instance.local_domain?(domain)
 
     account = Account.find_remote(username, domain)
-    return account unless account&.last_webfingered_at.nil? || 1.day.from_now(account.last_webfingered_at) < Time.now.utc
+    return account unless account_needs_webfinger_update?(account)
 
     Rails.logger.debug "Looking up webfinger for #{uri}"
 
@@ -62,6 +62,10 @@ class FollowRemoteAccountService < BaseService
 
   private
 
+  def account_needs_webfinger_update?(account)
+    account&.last_webfingered_at.nil? || account.last_webfingered_at <= 1.day.ago
+  end
+
   def get_feed(url)
     response = http_client.get(Addressable::URI.parse(url))
     [response.to_s, Nokogiri::XML(response)]