fix group assignment logic

pluginsGLPI · Feb 21, 2024 · 1ac9eee · 1ac9eee
1 parent ac57edc
commit 1ac9eee
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 10 deletions.
diff --git a/inc/group_group.class.php b/inc/group_group.class.php
@@ -128,28 +128,31 @@ function manageGroup($groups_id) {
    }
 
    function getGroups($ticket_id, $removeAlreadyAssigned = true) {
-      $groups = $user_groups = $ticket_groups = [];
+      $groups = $user_groups = [];
 
       // get groups for user connected
       $tmp_user_groups  = Group_User::getUserGroups($_SESSION['glpiID']);
       foreach ($tmp_user_groups as $current_group) {
          $user_groups[$current_group['id']] = $current_group['id'];
-         $groups[$current_group['id']] = $current_group['id'];
+
+         if (!$_SESSION['plugins']['escalade']['config']['use_filter_assign_group']) {
+            $groups[$current_group['id']] = $current_group['id'];
+         }
       }
 
       // get groups already assigned in the ticket
       if ($ticket_id > 0) {
          $ticket = new Ticket();
          $ticket->getFromDB($ticket_id);
          foreach ($ticket->getGroups(CommonITILActor::ASSIGN) as $current_group) {
-            $ticket_groups[$current_group['groups_id']] = $current_group['groups_id'];
+            $groups[$current_group['groups_id']] = $current_group['groups_id'];
          }
       }
 
       // To do an escalation, the user must be in a group currently assigned to the ticket
       // or no group is assigned to the ticket
       // TODO : matching with "view all tickets (yes/no) option in profile user"
-      if (!empty($ticket_groups) && count(array_intersect($ticket_groups, $user_groups)) == 0) {
+      if (!empty($groups) && count(array_intersect($groups, $user_groups)) == 0) {
          return [];
       }
 

diff --git a/setup.php b/setup.php
@@ -138,12 +138,10 @@ function plugin_init_escalade() {
       ];
       $PLUGIN_HOOKS['item_add']['escalade']['User'] = 'plugin_escalade_item_add_user';
 
-       //filter group feature
-       if ($escalade_config['use_filter_assign_group']) {
-          $PLUGIN_HOOKS[Hooks::FILTER_ACTORS]['escalade'] = [
-             'PluginEscaladeTicket', 'filter_actors',
-          ];
-       }
+      //filter group feature
+      $PLUGIN_HOOKS[Hooks::FILTER_ACTORS]['escalade'] = [
+         'PluginEscaladeTicket', 'filter_actors',
+      ];
 
       // == Interface links ==
       if (Session::haveRight('config', UPDATE)) {