Make the callbacks package hook, and remove symlinks before upgrade a…

…nd uninstall.
horde · Oct 10, 2017 · 372e510 · mrubinsk · Oct 16, 2017 · 372e510
1 parent 8224903
commit 372e510
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 3 deletions.
diff --git a/composer.json b/composer.json
@@ -50,7 +50,9 @@
         "optimize-autoloader": true
     },
     "scripts": {
-        "post-install-cmd": "Horde\\GitTools\\Callbacks::linkHordeRole",
-        "post-update-cmd": "Horde\\GitTools\\Callbacks::linkHordeRole"
+        "post-package-install": "Horde\\GitTools\\Callbacks::linkHordeRole",
+        "pre-package-update": "Horde\\GitTools\\Callbacks::unlinkHordeRole",
+        "post-package-update": "Horde\\GitTools\\Callbacks::linkHordeRole",
+        "pre-package-uninstall": "Horde\\GitTools\\Callbacks::unlinkHordeRole"
     }
 }
diff --git a/lib/Callbacks.php b/lib/Callbacks.php
@@ -18,7 +18,6 @@
 
 class Callbacks
 {
-
     /**
      * Links the Horde_Role files needed to detect "Horde" as a valid role
      * when using PEAR via Composer.
@@ -37,4 +36,17 @@ public static function linkHordeRole(Event $event)
             dirname(__FILE__) . '/../vendor/pear/pear/PEAR/Installer/Role/Horde.xml'
         );
     }
+
+    /**
+     * Unlinks the Horde_Role files needed to detect "Horde" as a valid role
+     * when using PEAR via Composer.
+     *
+     * @param  Event  $event [description]
+     * @return [type]        [description]
+     */
+    public static function unlinkHordeRole(Event $event)
+    {
+        unlink(dirname(__FILE__) . '/../vendor/pear/pear/PEAR/Installer/Role/Horde.php');
+        unlink(dirname(__FILE__) . '/../vendor/pear/pear/PEAR/Installer/Role/Horde.xml');
+    }
 }