From 5f87089b67726ddd6140ff34202e85d1ef41abf1 Mon Sep 17 00:00:00 2001 From: Neil Lalonde Date: Wed, 7 Aug 2019 15:39:48 -0400 Subject: [PATCH] FIX: remove dependency on present? in distributed_mutex lib --- lib/distributed_mutex.rb | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/distributed_mutex.rb b/lib/distributed_mutex.rb index 19f8b5dd4bc..d9a1879710b 100644 --- a/lib/distributed_mutex.rb +++ b/lib/distributed_mutex.rb @@ -83,7 +83,7 @@ class DistributedMutex current_expire_time = redis.get key - if current_expire_time.present? && current_expire_time.to_i > now + if current_expire_time && current_expire_time.to_i > now redis.unwatch got_lock = false @@ -94,7 +94,7 @@ class DistributedMutex redis.expire key, validity end - got_lock = result.present? + got_lock = !result.nil? end [got_lock, expire_time] @@ -109,7 +109,7 @@ class DistributedMutex redis.multi do redis.del key end - return result.present? + return !result.nil? else redis.unwatch return false