Fix tag item association

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 ### Fixed
 
 - Avoid a per-tag database lookup when rendering the tag column in item lists
+- Fix tag associate item
 
 ## [2.14.6] - 2026-08-04
 

ajax/add_item_to_tag.php

@@ -0,0 +1,70 @@
+<?php
+
+/**
+ * -------------------------------------------------------------------------
+ * Tag plugin for GLPI
+ * -------------------------------------------------------------------------
+ *
+ * LICENSE
+ *
+ * This file is part of Tag.
+ *
+ * Tag is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * Tag is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with Tag. If not, see <http://www.gnu.org/licenses/>.
+ * -------------------------------------------------------------------------
+ * @copyright Copyright (C) 2014-2026 by Teclib'.
+ * @license   GPLv2 https://www.gnu.org/licenses/gpl-2.0.html
+ * @link      https://github.com/pluginsGLPI/tag
+ * -------------------------------------------------------------------------
+ */
+
+use Glpi\Exception\Http\BadRequestHttpException;
+use Glpi\Exception\Http\AccessDeniedHttpException;
+
+Session::checkLoginUser();
+
+if (!isset($_POST['plugin_tag_tags_id'], $_POST['itemtype'], $_POST['items_id'])) {
+    throw new BadRequestHttpException(__s('Missing parameters', 'tag'));
+}
+
+$tag = new PluginTagTag();
+if (!$tag->getFromDB($_POST['plugin_tag_tags_id']) || !$tag->can($tag->getID(), UPDATE)) {
+    throw new AccessDeniedHttpException(__s('You do not have permission to update this tag', 'tag'));
+}
+
+$itemtype = $_POST['itemtype'];
+if (!is_a($itemtype, CommonDBTM::class, true) || !PluginTagTag::canItemtype($itemtype)) {
+    throw new BadRequestHttpException(__s('Invalid item type', 'tag'));
+}
+
+$item = new $itemtype();
+if (!$item->getFromDB($_POST['items_id']) || !$item->canUpdateItem()) {
+    throw new AccessDeniedHttpException(__s('You do not have permission to update this item', 'tag'));
+}
+
+$tag_item = new PluginTagTagItem();
+$found = $tag_item->find([
+    'plugin_tag_tags_id' => $tag->getID(),
+    'items_id'           => $item->getID(),
+    'itemtype'           => $itemtype,
+]);
+
+if (count($found) === 0) {
+    $tag_item->add([
+        'plugin_tag_tags_id' => $tag->getID(),
+        'items_id'           => $item->getID(),
+        'itemtype'           => $itemtype,
+    ]);
+}
+
+Html::back();

inc/tagitem.class.php

@@ -197,7 +197,7 @@ public static function showForTag(PluginTagTag $tag)
         if ($canedit) {
             echo "<div class='firstbloc'>";
             echo "<form name='tagitem_form{$rand}' id='tagitem_form{$rand}' method='post'
-               action='" . Toolbox::getItemTypeFormURL('PluginTagTag') . "'>";
+               action='" . plugin_tag_geturl() . "/ajax/add_item_to_tag.php'>";
 
             echo "<table class='tab_cadre_fixe'>";
             echo "<tr class='tab_bg_2'><th colspan='2'>" . __s('Add an item') . "</th></tr>";

tests/TagTestCase.php

@@ -46,7 +46,7 @@ protected function logOut()
         $_SESSION['glpi_currenttime'] = $ctime;
     }
 
-    public function loginAs(array $credentials): int
+    public function loginAs(array $credentials, int $rights = CREATE | UPDATE | PURGE): int
     {
         global $DB;
 
@@ -59,7 +59,7 @@ public function loginAs(array $credentials): int
         $DB->update(
             'glpi_profilerights',
             [
-                'rights' => CREATE | UPDATE | PURGE,
+                'rights' => $rights,
             ],
             [
                 'profiles_id' => $user_profile,
@@ -72,14 +72,14 @@ public function loginAs(array $credentials): int
         return $user->getID();
     }
 
-    public function createTag(string $tagName): int
+    public function createTag(string $tagName, array $typeMenu = ['Ticket']): int
     {
         $tag = new PluginTagTag();
         $tag->add(
             [
                 'name' => $tagName,
                 'is_active' => 1,
-                'type_menu' => ['Ticket'],
+                'type_menu' => $typeMenu,
             ],
         );
         $this->assertGreaterThan(0, $tag->getID());

tests/Units/TagItemTest.php

@@ -30,17 +30,20 @@
 
 namespace GlpiPlugin\Tag\Tests\Units;
 
+use Computer;
 use GlpiPlugin\Tag\Tests\TagTestCase;
+use PluginTagTagItem;
 use Ticket;
 
 final class TagItemTest extends TagTestCase
 {
+    private const TECH_USER = ['login' => 'tech', 'pass' => 'tech'];
+
     public function testTagsFromTicket(): void
     {
         $tagID1 = $this->createTag('TicketTag1');
         $tagID2 = $this->createTag('TicketTag2');
 
-
         $ticket = new Ticket();
         $ticket->add([
             'name' => 'Ticket add Tag',
@@ -56,4 +59,23 @@ public function testTagsFromTicket(): void
         $this->isItemTagged($ticket, $tagID2);
     }
 
+    public function testTagAssociationCreatesLink(): void
+    {
+        $this->loginAs(self::TECH_USER);
+
+        $tag = $this->createTag('MyTag', ['Computer']);
+        $computer = $this->createItem(Computer::class, [
+            'name' => 'Computer to tag',
+            'entities_id' => 0,
+        ]);
+
+        $tagItem = new PluginTagTagItem();
+        $tagItem->add([
+            'plugin_tag_tags_id' => $tag,
+            'itemtype' => Computer::class,
+            'items_id' => $computer->getID(),
+        ]);
+
+        $this->isItemTagged($computer, $tag);
+    }
 }