verifying that the item implements AssignableItem interface before unsetting the groups_id field

Author: TarekRemo

inc/commoninjectionlib.class.php

@@ -28,7 +28,9 @@
  * -------------------------------------------------------------------------
  */
 use Glpi\Exception\Http\HttpException;
+use Glpi\Features\AssignableItemInterface;
 
+use function Safe\class_implements;
 use function Safe\preg_match;
 use function Safe\preg_replace;
 
@@ -965,7 +967,8 @@ private function setValueForItemtype($itemtype, $field, $value, $fromdb = false)
                 if (isForeignKeyField($field) || (str_contains($field, 'is_')) || (method_exists($injectionClass, 'isNullable') && !$injectionClass->isNullable($field))) {
                     //If the field concernes groupds, unseting it instead of setting it to 0 in order to avoid associating the item to a non existing group (id 0)
                     $group_fields = ['groups_id_tech', 'groups_id', 'groups_id_normal'];
-                    if (in_array($field, $group_fields)) {
+                    $implemented_classes = class_implements($itemtype);
+                    if (in_array($field, $group_fields) && $implemented_classes && in_array(AssignableItemInterface::class, $implemented_classes)) {
                         unset($this->values[$itemtype][$field]);
                     } else {
                         // If the field is an id, we set it to 0