pkp#10506 Fix call to a removed User Group Collector

Vitaliy-1 · Nov 29, 2024 · 0bc8342 · 0bc8342
1 parent 41c503d
commit 0bc8342
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/classes/security/authorization/UserRolesRequiredPolicy.php b/classes/security/authorization/UserRolesRequiredPolicy.php
@@ -18,7 +18,7 @@
 
 use APP\core\Application;
 use APP\core\Request;
-use APP\facades\Repo;
+use PKP\userGroup\UserGroup;
 
 class UserRolesRequiredPolicy extends AuthorizationPolicy
 {
@@ -53,10 +53,10 @@ public function effect()
         }
         $context = $request->getRouter()->getContext($request);
 
-        $userGroups = Repo::userGroup()->getCollector()
-            ->filterByUserIds([$user->getId()])
-            ->filterByContextIds($context ? [$context->getId(), Application::SITE_CONTEXT_ID] : [Application::SITE_CONTEXT_ID])
-            ->getMany()->toArray();
+        $userGroups = UserGroup::withUserIds([$user->getId()])
+            ->withContextIds($context ? [$context->getId(), Application::SITE_CONTEXT_ID] : [Application::SITE_CONTEXT_ID])
+            ->get()
+            ->toArray();
 
         $roleIds = array_map(fn ($userGroup) => $userGroup->getRoleId(), $userGroups);
         $this->addAuthorizedContextObject(Application::ASSOC_TYPE_USER_ROLES, $roleIds);

diff --git a/classes/user/form/RolesForm.php b/classes/user/form/RolesForm.php
@@ -17,7 +17,6 @@
 namespace PKP\user\form;
 
 use APP\core\Application;
-use APP\facades\Repo;
 use APP\template\TemplateManager;
 use PKP\user\InterestManager;
 use PKP\user\User;
@@ -46,7 +45,6 @@ public function fetch($request, $template = null, $display = false)
 
         $userGroupIds = UserGroup::getIdsByUserId($request->getUser()->getId());
 
-
         $templateMgr->assign('userGroupIds', $userGroupIds);
 
         $userFormHelper = new UserFormHelper();