From 1d387763157255aee82579421e24a8c11c1bb3c4 Mon Sep 17 00:00:00 2001
From: GeorgeC <gcolon021@gmail.com>
Date: Fri, 6 Sep 2024 09:11:02 -0400
Subject: [PATCH] Fix sub-access rule addition in PrivilegeService

---
 .../hms/dbmi/avillach/auth/service/impl/PrivilegeService.java  | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/pic-sure-auth-services/src/main/java/edu/harvard/hms/dbmi/avillach/auth/service/impl/PrivilegeService.java b/pic-sure-auth-services/src/main/java/edu/harvard/hms/dbmi/avillach/auth/service/impl/PrivilegeService.java
index c27ef5b1..cc02f9cd 100644
--- a/pic-sure-auth-services/src/main/java/edu/harvard/hms/dbmi/avillach/auth/service/impl/PrivilegeService.java
+++ b/pic-sure-auth-services/src/main/java/edu/harvard/hms/dbmi/avillach/auth/service/impl/PrivilegeService.java
@@ -453,7 +453,8 @@ protected void updateAllPrivilegesOnStartup() {
                     subAccessRules.removeIf(subAccessRule -> subAccessRule.getName().startsWith("AR_ALLOW_"));
 
                     // Add the currently allowed query types
-                    accessRule.getSubAccessRule().addAll(this.accessRuleService.getAllowedQueryTypeRules());
+                    subAccessRules.addAll(this.accessRuleService.getAllowedQueryTypeRules());
+                    accessRule.setSubAccessRule(subAccessRules);
                     this.accessRuleService.save(accessRule);
                 });
     }