From: Tom Clegg <tom@curii.com>
Date: Thu, 17 Mar 2022 19:12:46 +0000 (-0400)
Subject: 18691: Add parens to clarify operator precedence.
X-Git-Tag: 2.4.0~36^2~2
X-Git-Url: https://git.arvados.org/arvados.git/commitdiff_plain/74ba49615bbce04863068b095bd103f870c90b4f?ds=sidebyside

18691: Add parens to clarify operator precedence.

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <tom@curii.com>
---

diff --git a/services/api/app/models/group.rb b/services/api/app/models/group.rb
index d2c332b9b1..f56c3281d7 100644
--- a/services/api/app/models/group.rb
+++ b/services/api/app/models/group.rb
@@ -116,7 +116,7 @@ class Group < ArvadosModel
       if !new_record? && !current_user.can?(manage: uuid)
         raise PermissionDeniedError
       end
-      if trash_at || delete_at || !new_record? && TrashedGroup.where(group_uuid: uuid).any?
+      if trash_at || delete_at || (!new_record? && TrashedGroup.where(group_uuid: uuid).any?)
         errors.add(:frozen_by_uuid, "cannot be set on a trashed project")
       end
       if frozen_by_uuid_was.nil?