Refactor the node_enabled? method on Geo workers to make it more clear

e9be0cc6 · Douglas Barbosa Alexandre · 222aa91d · e9be0cc6 · e9be0cc6
Commit e9be0cc6 authored May 30, 2017 by Douglas Barbosa Alexandre
Showing with 2 additions and 2 deletions

app/workers/geo_file_download_dispatch_worker.rb app/workers/geo_file_download_dispatch_worker.rb +1 -1

app/workers/geo_repository_sync_worker.rb app/workers/geo_repository_sync_worker.rb +1 -1

No files found.
--- a/app/workers/geo_file_download_dispatch_worker.rb
+++ b/app/workers/geo_file_download_dispatch_worker.rb
@@ -153,7 +153,7 @@ class GeoFileDownloadDispatchWorker
  def node_enabled?
    # Only check every minute to avoid polling the DB excessively
-    unless @last_enabled_check.present? && (Time.now - @last_enabled_check > 1.minute)
+    unless @last_enabled_check.present? && @last_enabled_check > 1.minute.ago
      @last_enabled_check = Time.now
      @current_node_enabled = nil
    end

--- a/app/workers/geo_repository_sync_worker.rb
+++ b/app/workers/geo_repository_sync_worker.rb
@@ -75,7 +75,7 @@ class GeoRepositorySyncWorker
  def node_enabled?
    # Only check every minute to avoid polling the DB excessively
-    unless @last_enabled_check.present? && (Time.now - @last_enabled_check > 1.minute)
+    unless @last_enabled_check.present? && @last_enabled_check > 1.minute.ago
      @last_enabled_check = Time.now
      @current_node_enabled = nil
    end