[Geo][Refactoring] (repository|wiki)_sync_due? to contain all the logic directly
def repository_sync_due?(scheduled_time)
never_synced_repository? || repository_sync_needed?(scheduled_time)
end
Neither never_synced_repository?
nor repository_sync_needed?(scheduled_time)
is used now somewhere else so repository_sync_due?
can contain all the logic.
def repository_sync_due?(scheduled_time)
return false unless resync_repository?
return false if repository_retry_at && timestamp < repository_retry_at
return true if last_repository_synced_at.nil?
timestamp > last_repository_synced_at
end
Edited by Valery Sizov