Added support for updating composer plugins

wintercms · Dec 19, 2024 · be09f6c · be09f6c
1 parent f250393
commit be09f6c
Showing 1 changed file with 34 additions and 9 deletions.
diff --git a/modules/system/classes/extensions/PluginManager.php b/modules/system/classes/extensions/PluginManager.php
@@ -107,6 +107,7 @@ class PluginManager extends ExtensionManager implements ExtensionManagerInterfac
 
     /**
      * Initializes the plugin manager
+     * @throws SystemException
      */
     protected function init(): void
     {
@@ -143,6 +144,10 @@ public function list(): array
         );
     }
 
+    /**
+     * @throws SystemException
+     * @throws ApplicationException
+     */
     public function create(string $extension): WinterExtension
     {
         $this->renderComponent(Info::class, sprintf('Running command `create:plugin %s`.', $extension));
@@ -166,6 +171,10 @@ public function create(string $extension): WinterExtension
         return $this->findByIdentifier($extension);
     }
 
+    /**
+     * @throws ApplicationException
+     * @throws SystemException
+     */
     public function install(ExtensionSource|WinterExtension|string $extension): WinterExtension
     {
         // Insure the in memory plugins match those on disk
@@ -278,28 +287,44 @@ public function update(WinterExtension|string $extension): ?bool
 
         // Update the plugin database and version
         if (!($plugin = $this->findByIdentifier($code))) {
-            $this->getOutput()->info(sprintf('Unable to find plugin %s', $code));
+            $this->output->info(sprintf('Unable to find plugin %s', $code));
             return null;
         }
 
-        $this->output->info(sprintf(
-            'Migrating %s (%s) plugin...',
-            Lang::get($plugin->pluginDetails()['name']),
-            $code
-        ));
+        $pluginName = Lang::get($plugin->pluginDetails()['name']);
 
         if (
-            ($composerPackage = $plugin->getComposerPackageName())
+            !$this->getPluginRecord($plugin)->is_frozen
+            && ($composerPackage = $plugin->getComposerPackageName())
             && Composer::updateAvailable($composerPackage)
         ) {
+            $this->output->info(sprintf(
+                'Performing composer update for %s (%s) plugin...',
+                $pluginName,
+                $code
+            ));
+
             Preserver::instance()->store($plugin);
-            $update = Composer::update(package: $composerPackage);
-            dd($update);
+            $update = Composer::update(package: $composerPackage, dryRun: true);
+
+            $versions = $update->getUpgraded()[$composerPackage] ?? null;
+
+            $this->output->{$versions ? 'info' : 'error'}(
+                $versions
+                    ? sprintf('Updated plugin %s (%s) from v%s => v%s', $pluginName, $code, $versions[0], $versions[1])
+                    : sprintf('Failed to update plugin %s (%s)', $pluginName, $code)
+            );
         } elseif (false /* Detect if market */) {
             Preserver::instance()->store($plugin);
             // @TODO: Update files from market
         }
 
+        $this->output->info(sprintf(
+            'Migrating %s (%s) plugin...',
+            Lang::get($plugin->pluginDetails()['name']),
+            $code
+        ));
+
         $this->versionManager->updatePlugin($plugin);
 
         return true;