From: radhika <radhika@curoverse.com>
Date: Thu, 4 Jun 2015 16:51:26 +0000 (-0400)
Subject: 6203: Corrected one dumb switched order of if conditions that caused 5s lag!!
X-Git-Tag: 1.1.0~1558^2~15
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/b39d4108e3a78f3bfebc4cf934bfc202eb1f2cf0

6203: Corrected one dumb switched order of if conditions that caused 5s lag!!
---

diff --git a/services/api/app/models/collection.rb b/services/api/app/models/collection.rb
index 4f274dcaff..422a7ede73 100644
--- a/services/api/app/models/collection.rb
+++ b/services/api/app/models/collection.rb
@@ -103,7 +103,7 @@ class Collection < ArvadosModel
 
       # Remove any permission signatures from the manifest.
       self[:manifest_text] = self.class.munge_manifest_locators!(self[:manifest_text]) do |match|
-        if not in_old_manifest[match[1]] && !cleared_replication_confirmed
+        if not cleared_replication_confirmed  and not in_old_manifest[match[1]]
           self.replication_confirmed_at = nil
           self.replication_confirmed = nil
           cleared_replication_confirmed = true