Merge pull request chamilo#6811 from christianbeeznest/ras-22987

User: purge extra fields on delete + orphan cleanup migration - refs #22987

Author: christianbeeznest

src/CoreBundle/Migrations/Schema/V200/Version20250926070900.php

@@ -0,0 +1,40 @@
+<?php
+
+declare(strict_types=1);
+
+/* For licensing terms, see /license.txt */
+
+namespace Chamilo\CoreBundle\Migrations\Schema\V200;
+
+use Chamilo\CoreBundle\Migrations\AbstractMigrationChamilo;
+use Doctrine\DBAL\Schema\Schema;
+
+final class Version20250926070900 extends AbstractMigrationChamilo
+{
+    public function getDescription(): string
+    {
+        return 'Purge orphan user extra fields (values + tags) where user no longer exists.';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->addSql("
+            DELETE v FROM extra_field_values v
+            INNER JOIN extra_field f ON f.id = v.field_id
+            LEFT JOIN `user` u ON u.id = v.item_id
+            WHERE f.item_type = 1 AND u.id IS NULL
+        ");
+
+        $this->addSql("
+            DELETE r FROM extra_field_rel_tag r
+            INNER JOIN extra_field f ON f.id = r.field_id
+            LEFT JOIN `user` u ON u.id = r.item_id
+            WHERE f.item_type = 1  AND u.id IS NULL
+        ");
+    }
+
+    public function down(Schema $schema): void
+    {
+        // No-op (data purge).
+    }
+}

src/CoreBundle/Repository/Node/UserRepository.php

@@ -166,6 +166,21 @@ public function deleteUser(User $user, bool $destroy = false): void
                     $this->reassignUserResourcesToFallbackSQL($user, $fallbackUser, $connection);
                 }
 
+                // Remove extra fields
+                $connection->executeStatement(
+                    'DELETE v FROM extra_field_values v
+                INNER JOIN extra_field f ON f.id = v.field_id
+                WHERE f.item_type = :userType AND v.item_id = :uid',
+                    ['userType' => ExtraField::USER_FIELD_TYPE, 'uid' => $user->getId()]
+                );
+
+                $connection->executeStatement(
+                    'DELETE r FROM extra_field_rel_tag r
+                INNER JOIN extra_field f ON f.id = r.field_id
+                WHERE f.item_type = :userType AND r.item_id = :uid',
+                    ['userType' => ExtraField::USER_FIELD_TYPE, 'uid' => $user->getId()]
+                );
+
                 // Remove group relationships
                 $connection->executeStatement(
                     'DELETE FROM usergroup_rel_user WHERE user_id = :userId',