Merge branch '4088-collection-show-files-filter'

[arvados.git] / services / api / app / models / user.rb

diff --git a/services/api/app/models/user.rb b/services/api/app/models/user.rb
index 3b201b5babd90ee9244263737e499b8a381b35a6..b939d07bf056c00e93383499a47e1bfedadad2c7 100644 (file)
--- a/services/api/app/models/user.rb
+++ b/services/api/app/models/user.rb
@@ -234,13 +234,13 @@ class User < ArvadosModel
   end
 
   def check_auto_admin
-    if not self.uuid.end_with?('anonymouspublic') and
-       User.where("uuid not like '%-000000000000000'").where(:is_admin => true).count == 0
-      if (Rails.configuration.auto_admin_user and self.email == Rails.configuration.auto_admin_user) or
-         Rails.configuration.auto_admin_first_user
-        self.is_admin = true
-        self.is_active = true
-      end
+    return if self.uuid.end_with?('anonymouspublic')
+    if (User.where("email = ?",self.email).where(:is_admin => true).count == 0 and
+        Rails.configuration.auto_admin_user and self.email == Rails.configuration.auto_admin_user) or
+       (User.where("uuid not like '%-000000000000000'").where(:is_admin => true).count == 0 and 
+        Rails.configuration.auto_admin_first_user)
+      self.is_admin = true
+      self.is_active = true
     end
   end