From 276603fd833c9fae418f658473a56a334b1942ca Mon Sep 17 00:00:00 2001 From: mbressy Date: Wed, 8 Apr 2026 08:04:55 +0000 Subject: [PATCH 1/2] fix: handle group assignment on uninstall templates --- inc/uninstall.class.php | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/inc/uninstall.class.php b/inc/uninstall.class.php index cac884c..baa02fe 100644 --- a/inc/uninstall.class.php +++ b/inc/uninstall.class.php @@ -213,7 +213,8 @@ private static function doOneUninstall(PluginUninstallModel $model, Transfer $tr } } - if ($item->isField('groups_id')) { + $can_update_groups = $item->isField('groups_id') || method_exists($item, 'prepareGroupFields'); + if ($can_update_groups) { $nbgroup = countElementsInTableForEntity( "glpi_groups", $entity, From f0ceec077feca7c000fafbf8e823ee66ac01119a Mon Sep 17 00:00:00 2001 From: mbressy Date: Wed, 8 Apr 2026 12:28:36 +0000 Subject: [PATCH 2/2] use AssignableItemInterface --- inc/uninstall.class.php | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/inc/uninstall.class.php b/inc/uninstall.class.php index baa02fe..3b48e7a 100644 --- a/inc/uninstall.class.php +++ b/inc/uninstall.class.php @@ -29,6 +29,7 @@ */ use Glpi\Asset\Asset_PeripheralAsset; +use Glpi\Features\AssignableItemInterface; use function Safe\preg_grep; @@ -213,8 +214,10 @@ private static function doOneUninstall(PluginUninstallModel $model, Transfer $tr } } - $can_update_groups = $item->isField('groups_id') || method_exists($item, 'prepareGroupFields'); - if ($can_update_groups) { + if ( + $item->isField('groups_id') + || ($item instanceof AssignableItemInterface) + ) { $nbgroup = countElementsInTableForEntity( "glpi_groups", $entity,