Fix - Resolve SQL alias errors and undefined array keys (#295)

* Fix - Resolve SQL alias errors and undefined array keys

* Remove useless fallback

* Update CHANGELOG

Author: RomainLvr

CHANGELOG.md

@@ -9,6 +9,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Fixed
 
+ - Fix SQL alias errors and undefined array keys
  - Fix missing `delay` to where clause
  - Fix weekly backlog calculation using wrong week reference point (#294)
  - Fix the SQL query for the backlog graph (#290)

inc/helpdeskplus.class.php

@@ -159,13 +159,13 @@ public function __construct($config = [])
 
         if (isset($mr_values['groups_assign_id'])) {
             $this->criteria_group_assign = [
-                Group_Ticket::getTable() . '.groups_id' => $mr_values['groups_assign_id'],
+                'gt.groups_id' => $mr_values['groups_assign_id'],
             ];
         }
 
         if (isset($mr_values['groups_request_id'])) {
-            $this->criteria_group_assign = [
-                Group_Ticket::getTable() . '.groups_id' => $mr_values['groups_request_id'],
+            $this->criteria_group_request = [
+                'gtr.groups_id' => $mr_values['groups_request_id'],
             ];
         }
 
@@ -1338,7 +1338,7 @@ public function reportVstackbarRespectedSlasByGroup($config = [])
                     Ticket::getTable() . '.status'  => array_merge(Ticket::getSolvedStatusArray(), Ticket::getClosedStatusArray()),
                     SLA::getTable() . '.id' => $_SESSION['mreporting_values']['slas'],
                 ],
-                'GROUPBY' => ['glpi_groups_tickets.groups_id', 'respected_sla'],
+                'GROUPBY' => [Group_Ticket::getTable() . '.groups_id', 'respected_sla'],
             ];
             $query['WHERE']['AND'] = $delay;