diff --git a/app/models/user.rb b/app/models/user.rb index ed107df41..47761ae88 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -50,7 +50,7 @@ def is_project_admin?() end def project_admin_of?(project) - project && admined_projects.any? {|ap| ap.id == project.id } + !!(project && admined_projects.any? {|ap| ap.id == project.id }) end def most_recent_authentication diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb index bc1b73389..29710b0b0 100644 --- a/spec/models/user_spec.rb +++ b/spec/models/user_spec.rb @@ -514,6 +514,7 @@ expect(user.project_admins.length).to be(0) expect(user.admined_projects.length).to be(0) + expect(user.project_admin_of?(nil)).to be(false) expect(user.project_admin_of?(project1)).to be(false) expect(user.project_admin_of?(project2)).to be(false) end