ACMS-3658: The drs:init:settings command shouldn't be called when DRS…

… updated.
acquia · Mar 22, 2024 · be0d353 · be0d353
1 parent 71b246a
commit be0d353
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/src/Plugin.php b/src/Plugin.php
@@ -5,7 +5,6 @@
 use Composer\Composer;
 use Composer\DependencyResolver\Operation\InstallOperation;
 use Composer\DependencyResolver\Operation\OperationInterface;
-use Composer\DependencyResolver\Operation\UpdateOperation;
 use Composer\EventDispatcher\EventSubscriberInterface;
 use Composer\Installer\PackageEvent;
 use Composer\Installer\PackageEvents;
@@ -119,9 +118,6 @@ protected function getSettingsPackage(OperationInterface $operation): mixed {
     if ($operation instanceof InstallOperation) {
       $package = $operation->getPackage();
     }
-    elseif ($operation instanceof UpdateOperation) {
-      $package = $operation->getTargetPackage();
-    }
     if (isset($package) && $package instanceof PackageInterface && $package->getName() == "acquia/drupal-recommended-settings") {
       return $package;
     }

diff --git a/tests/src/unit/PluginUnitTest.php b/tests/src/unit/PluginUnitTest.php
@@ -140,7 +140,7 @@ public function testGetSettingsPackage(): void {
     $this->plugin->onPostPackageEvent($package_event);
     $method = $this->getReflectionMethod(Plugin::class, "getSettingsPackage");
     $package_name = $method->invokeArgs($this->plugin, [$operation]);
-    $this->assertEquals("acquia/drupal-recommended-settings-1.0", $package_name);
+    $this->assertNull($package_name);
 
     // Assert when any other package is installed.
     $package = new RootPackage("acquia/blt", "14.0", "14.0.0");