Exit on patch failure should use composer extra, not localPackage

cweagans · Nov 9, 2017 · a4519e8 · a4519e8
1 parent 4b1de2a
commit a4519e8
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/Patches.php b/src/Patches.php
@@ -275,6 +275,10 @@ public function grabPatches()
      */
     public function postInstall(PackageEvent $event)
     {
+        // Check if we should exit in failure.
+        $extra = $this->composer->getPackage()->getExtra();
+        $exitOnFailure = getenv('COMPOSER_EXIT_ON_PATCH_FAILURE') || !empty($extra['composer-exit-on-patch-failure']);
+
         // Get the package object for the current operation.
         $operation = $event->getOperation();
         /** @var PackageInterface $package */
@@ -321,7 +325,7 @@ public function postInstall(PackageEvent $event)
                     $e->getMessage() .
                     '</error>'
                 );
-                if (getenv('COMPOSER_EXIT_ON_PATCH_FAILURE') || !empty($extra['composer-exit-on-patch-failure'])) {
+                if ($exitOnFailure) {
                     throw new \Exception("Cannot apply patch $description ($url)!");
                 }
             }