Merge pull request #21 from marcelthole/check-package-name-before-ins…

…tall

Only handle own install or update events

src/ConfiguredMediator.php

@@ -40,6 +40,9 @@ public function uninstall(Composer $composer, IOInterface $io)
 
     public function installOrUpdateFunction(PackageEvent $event): void
     {
+        if (!$this->isDesiredPackageEvent($event, $this->config->package())) {
+            return;
+        }
         $gnuPG = $this->createGnuPG();
         // we do not want to crash if no GnuPG was found
         // but display a noticeable warning to the user

src/OperationPackage.php

@@ -0,0 +1,53 @@
+<?php
+
+declare(strict_types = 1);
+namespace PharIo\ComposerDistributor;
+
+use Composer\Composer;
+use Composer\DependencyResolver\GenericRule;
+use Composer\DependencyResolver\Operation\InstallOperation;
+use Composer\DependencyResolver\Operation\UninstallOperation;
+use Composer\DependencyResolver\Operation\UpdateOperation;
+use Composer\Installer\PackageEvent;
+use Composer\Package\CompletePackage;
+use Composer\Package\Package;
+use Composer\Semver\Constraint\MultiConstraint;
+use RuntimeException;
+
+class OperationPackage {
+
+    public static function createFromEvent(PackageEvent $event, string $pluginName): Package
+    {
+        if (0 >= \version_compare('2.0.0', Composer::VERSION)) {
+            $operation = $event->getOperation();
+
+            switch (true) {
+                case $operation instanceof InstallOperation:
+                case $operation instanceof UninstallOperation:
+                    $package = $operation->getPackage();
+
+                    break;
+                case $operation instanceof UpdateOperation:
+                    $package = $operation->getTargetPackage();
+
+                    break;
+                default:
+                    throw new RuntimeException('No valid operation found');
+            }
+        } else {
+            /** @var GenericRule $rule */
+            $rule = $event->getOperation()->getReason();
+            /** @var MultiConstraint $constraint */
+            $constraint = $rule->getJob()['constraint'];
+
+            if ($rule->getRequiredPackage() !== $pluginName) {
+                throw SomebodyElsesProblem::here($pluginName);
+            }
+
+            /** @var CompletePackage $packages */
+            $package = $event->getInstalledRepo()->findPackage($rule->getRequiredPackage(), $constraint->getPrettyString());
+        }
+
+        return $package;
+    }
+}

src/PackageVersion.php

@@ -91,32 +91,7 @@ public function preRelease() : string
 
     public static function fromPackageEvent(PackageEvent $event, string $pluginName) : self
     {
-        $package = null;
-        if (0 >= version_compare('2.0.0', Composer::VERSION)) {
-            $operation = $event->getOperation();
-            switch (true) {
-                case $operation instanceof InstallOperation:
-                case $operation instanceof UninstallOperation:
-                    $package = $operation->getPackage();
-                    break;
-                case $operation instanceof UpdateOperation:
-                    $package = $operation->getTargetPackage();
-                    break;
-                default:
-                    throw new RuntimeException('No valid operation found');
-            }
-        } else {
-            /** @var GenericRule $rule */
-            $rule = $event->getOperation()->getReason();
-            /** @var MultiConstraint $constraint */
-            $constraint = $rule->getJob()['constraint'];
-            if ($rule->getRequiredPackage() !== $pluginName) {
-                throw SomebodyElsesProblem::here($pluginName);
-            }
-
-            /** @var CompletePackage $packages */
-            $package = $event->getInstalledRepo()->findPackage($rule->getRequiredPackage(), $constraint->getPrettyString());
-        }
+        $package = OperationPackage::createFromEvent($event, $pluginName);
         return new self($package->getName(), $package->getFullPrettyVersion());
     }
 }

src/PluginBase.php

@@ -76,4 +76,11 @@ protected function getComposer(): Composer
         }
         return $this->composer;
     }
+
+    protected function isDesiredPackageEvent(PackageEvent $event, string $pluginName): bool
+    {
+        $package = OperationPackage::createFromEvent($event, $pluginName);
+
+        return $package->getName() === $pluginName;
+    }
 }