Update AssignableItem.php

Thanks to Phind AI :)  

Since my migration to GLPI11, I have random error in APPLIANCE view when a computer was associated

*** Uncaught PHP Exception TypeError: "array_intersect(): Argument #1 ($array) must be of type array, int given" at AssignableItem.php line 63
  Backtrace :
  ./src/Glpi/Features/AssignableItem.php:63          
  ./src/Glpi/Features/AssignableItem.php:63          array_intersect()
  ./src/CommonDBTM.php:2986                          Appliance->canViewItem()
  ./src/CommonDBTM.php:1486                          CommonDBTM->can()
  ./src/Appliance_Item.php:285                       CommonDBTM->getLink()
  ./src/Appliance_Item.php:100                       Appliance_Item::showForItem()
  ./src/CommonGLPI.php:694                           Appliance_Item::displayTabContentForItem()
  ./ajax/common.tabs.php:108                         CommonGLPI::displayStandardTab()
  ...Glpi/Controller/LegacyFileLoadController.php:64 require()
  ./vendor/symfony/http-kernel/HttpKernel.php:181    Glpi\Controller\LegacyFileLoadController->__invoke()
  ./vendor/symfony/http-kernel/HttpKernel.php:76     Symfony\Component\HttpKernel\HttpKernel->handleRaw()
  ./vendor/symfony/http-kernel/Kernel.php:197        Symfony\Component\HttpKernel\HttpKernel->handle()
  ./public/index.php:70                              Symfony\Component\HttpKernel\Kernel->handle()

message was clear as an array was expected, but an INT was provided.

This fix the vew issue

Author: bacus99

src/Glpi/Features/AssignableItem.php

@@ -53,26 +53,30 @@ public static function canView(): bool
     }
 
     /** @see AssignableItemInterface::canViewItem() */
-    public function canViewItem(): bool
-    {
-        if (!parent::canViewItem()) {
-            return false;
-        }
+public function canViewItem(): bool
+{
+    if (!parent::canViewItem()) {
+        return false;
+    }
 
-        $is_assigned = $this->fields['users_id_tech'] === $_SESSION['glpiID']
-            || count(array_intersect($this->fields['groups_id_tech'] ?? [], $_SESSION['glpigroups'] ?? [])) > 0;
-        $is_owned = $this->fields['users_id'] === $_SESSION['glpiID']
-            || count(array_intersect($this->fields['groups_id'] ?? [], $_SESSION['glpigroups'] ?? [])) > 0;
+    // Ensure we have arrays for comparison
+    $tech_groups = (array)($this->fields['groups_id_tech'] ?? []);
+    $own_groups = (array)($this->fields['groups_id'] ?? []);
+    $session_groups = (array)($_SESSION['glpigroups'] ?? []);
 
-        if (!Session::haveRight(static::$rightname, READ)) {
-            return ($is_assigned && Session::haveRight(static::$rightname, READ_ASSIGNED))
-                || ($is_owned && Session::haveRight(static::$rightname, READ_OWNED));
-        }
+    $is_assigned = $this->fields['users_id_tech'] === $_SESSION['glpiID']
+        || count(array_intersect($tech_groups, $session_groups)) > 0;
+    $is_owned = $this->fields['users_id'] === $_SESSION['glpiID']
+        || count(array_intersect($own_groups, $session_groups)) > 0;
 
-        // Has global READ right
-        return true;
+    if (!Session::haveRight(static::$rightname, READ)) {
+        return ($is_assigned && Session::haveRight(static::$rightname, READ_ASSIGNED))
+            || ($is_owned && Session::haveRight(static::$rightname, READ_OWNED));
     }
 
+    return true;
+}
+
     /** @see AssignableItemInterface::canUpdate() */
     public static function canUpdate(): bool
     {