Migration: Fix plugin titles and remove plugins without a corresponding directory

Author: AngelFQC

src/CoreBundle/Migrations/AbstractMigrationChamilo.php

@@ -29,6 +29,7 @@
 use Psr\Log\LoggerInterface;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 use Symfony\Component\Filesystem\Filesystem;
+use Symfony\Component\Finder\Finder;
 use Symfony\Component\HttpFoundation\File\UploadedFile;
 
 abstract class AbstractMigrationChamilo extends AbstractMigration
@@ -437,4 +438,20 @@ protected function getUpdateRootPath(): string
 
         return $this->container->getParameter('kernel.project_dir');
     }
+
+    protected function getPluginDirectoryList(): array
+    {
+        $pluginDir = $this->container->getParameter('kernel.project_dir').'/public/plugin';
+
+        $finder = new Finder();
+        $finder->directories()->in($pluginDir)->depth('== 0');
+
+        $directories = [];
+
+        foreach ($finder as $entry) {
+            $directories[] = $entry->getFilename();
+        }
+
+        return $directories;
+    }
 }

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

@@ -134,20 +134,4 @@ private function insertPluginSettingsByUrl(int $pluginId, array $settingsByUrl):
             );
         }
     }
-
-    private function getPluginDirectoryList(): array
-    {
-        $pluginDir = $this->container->getParameter('kernel.project_dir').'/public/plugin';
-
-        $finder = new Finder();
-        $finder->directories()->in($pluginDir)->depth('== 0');
-
-        $directories = [];
-
-        foreach ($finder as $entry) {
-            $directories[] = $entry->getFilename();
-        }
-
-        return $directories;
-    }
 }

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

@@ -0,0 +1,59 @@
+<?php
+
+/* For licensing terms, see /license.txt */
+
+declare(strict_types=1);
+
+namespace Chamilo\CoreBundle\Migrations\Schema\V200;
+
+use AppPlugin;
+use Chamilo\CoreBundle\Entity\Plugin;
+use Chamilo\CoreBundle\Migrations\AbstractMigrationChamilo;
+use Doctrine\DBAL\Schema\Schema;
+
+class Version20251009111300 extends AbstractMigrationChamilo
+{
+    public function getDescription(): string
+    {
+        return 'Fix plugin titles and remove plugins without a corresponding directory';
+    }
+
+    /**
+     * @inheritDoc
+     */
+    public function up(Schema $schema): void
+    {
+        $directories = $this->getPluginDirectoryList();
+        $idListToDelete = [];
+
+        $pluginRows = $this->connection->executeQuery("SELECT id, title, source FROM plugin")->fetchAllAssociative();
+
+        foreach ($pluginRows as $pluginRow) {
+            $title = str_replace(' ', '', ucwords(str_replace('_', ' ', $pluginRow['title'])));
+
+            if (!\in_array($title, $directories)) {
+                $idListToDelete[] = $pluginRow['id'];
+
+                continue;
+            }
+
+            $source = \in_array($title, AppPlugin::getOfficialPlugins())
+                ? Plugin::SOURCE_OFFICIAL
+                : Plugin::SOURCE_THIRD_PARTY;
+
+            $this->connection->update(
+                'plugin',
+                [
+                    'title' => $title,
+                    'source' => $source,
+                ],
+                ['id' => $pluginRow['id']]
+            );
+        }
+
+        foreach ($idListToDelete as $idToDelete) {
+            $this->connection->delete('access_url_rel_plugin', ['plugin_id' => $idToDelete]);
+            $this->connection->delete('plugin', ['id' => $idToDelete]);
+        }
+    }
+}