Added the possibility to manage multiple files

Closes #21 and closes #25

Author: stof

README.md

@@ -141,3 +141,27 @@ old parameters unless you've also removed them from the dist version.
 If the old parameter is no longer present (maybe because it has been renamed and
 removed already), no parameters are overwritten. You don't need to remove obsolete
 parameters from the rename map once they have been renamed.
+
+### Managing multiple ignored files
+
+The parameter handler can manage multiple ignored files. To use this feature,
+the ``incenteev-parameters`` extra should contain a JSON array with multiple
+configurations inside it instead of a configuration object:
+
+```json
+{
+    "extra": {
+        "incenteev-parameters": [
+            {
+                "file": "app/config/parameters.yml",
+                "env-map": {}
+            },
+            {
+                "file": "app/config/databases.yml",
+                "dist-file": "app/config/databases.dist.yml",
+                "parameter-key": "config"
+            }
+        ]
+    }
+}
+```

ScriptHandler.php

@@ -14,27 +14,52 @@ public static function buildParameters(Event $event)
     {
         $extras = $event->getComposer()->getPackage()->getExtra();
 
-        if (empty($extras['incenteev-parameters']['file'])) {
+        if (!isset($extras['incenteev-parameters'])) {
+            throw new \InvalidArgumentException('The parameter handler needs to be configured through the extra.incenteev-parameters setting.');
+        }
+
+        $configs = $extras['incenteev-parameters'];
+
+        if (!is_array($configs)) {
+            throw new \InvalidArgumentException('The extra.incenteev-parameters setting must be an array or a configuration object.');
+        }
+
+        if (array_keys($configs) !== range(0, count($configs) - 1)) {
+            $configs = array($configs);
+        }
+
+        foreach ($configs as $config) {
+            if (!is_array($config)) {
+                throw new \InvalidArgumentException('The extra.incenteev-parameters setting must be an array of configuration objects.');
+            }
+
+            self::processFile($config, $event->getIO());
+        }
+    }
+
+    private static function processFile(array $config, IOInterface $io)
+    {
+        if (empty($config['file'])) {
             throw new \InvalidArgumentException('The extra.incenteev-parameters.file setting is required to use this script handler.');
         }
 
-        $realFile = $extras['incenteev-parameters']['file'];
+        $realFile = $config['file'];
 
-        if (empty($extras['incenteev-parameters']['dist-file'])) {
+        if (empty($config['dist-file'])) {
             $distFile = $realFile.'.dist';
         } else {
-            $distFile = $extras['incenteev-parameters']['dist-file'];
+            $distFile = $config['dist-file'];
         }
 
         $keepOutdatedParams = false;
-        if (isset($extras['incenteev-parameters']['keep-outdated'])) {
-            $keepOutdatedParams = (boolean)$extras['incenteev-parameters']['keep-outdated'];
+        if (isset($config['keep-outdated'])) {
+            $keepOutdatedParams = (boolean) $config['keep-outdated'];
         }
 
-        if (empty($extras['incenteev-parameters']['parameter-key'])) {
+        if (empty($config['parameter-key'])) {
             $parameterKey = 'parameters';
         } else {
-            $parameterKey = $extras['incenteev-parameters']['parameter-key'];
+            $parameterKey = $config['parameter-key'];
         }
 
         if (!is_file($distFile)) {
@@ -44,7 +69,6 @@ public static function buildParameters(Event $event)
         $exists = is_file($realFile);
 
         $yamlParser = new Parser();
-        $io = $event->getIO();
 
         $action = $exists ? 'Updating' : 'Creating';
         $io->write(sprintf('<info>%s the "%s" file.</info>', $action, $realFile));
@@ -68,7 +92,7 @@ public static function buildParameters(Event $event)
         $actualParams = (array) $actualValues[$parameterKey];
 
         // Grab values for parameters that were renamed
-        $renameMap = empty($extras['incenteev-parameters']['rename-map']) ? array() : (array) $extras['incenteev-parameters']['rename-map'];
+        $renameMap = empty($config['rename-map']) ? array() : (array) $config['rename-map'];
         $actualParams = array_replace($actualParams, self::getRenameValues($renameMap, $actualParams));
 
         if (!$keepOutdatedParams) {
@@ -80,7 +104,7 @@ public static function buildParameters(Event $event)
             }
         }
 
-        $envMap = empty($extras['incenteev-parameters']['env-map']) ? array() : (array) $extras['incenteev-parameters']['env-map'];
+        $envMap = empty($config['env-map']) ? array() : (array) $config['env-map'];
 
         // Add the params coming from the environment values
         $actualParams = array_replace($actualParams, self::getEnvValues($envMap));

Tests/ScriptHandlerTest.php

@@ -61,11 +61,15 @@ public function provideInvalidConfiguration()
         return array(
             'no extra' => array(
                 array(),
-                'The extra.incenteev-parameters.file setting is required to use this script handler.',
+                'The parameter handler needs to be configured through the extra.incenteev-parameters setting.',
             ),
             'invalid type' => array(
                 array('incenteev-parameters' => 'not an array'),
-                'The extra.incenteev-parameters.file setting is required to use this script handler.',
+                'The extra.incenteev-parameters setting must be an array or a configuration object.',
+            ),
+            'invalid type for multiple file' => array(
+                array('incenteev-parameters' => array('not an array')),
+                'The extra.incenteev-parameters setting must be an array of configuration objects.',
             ),
             'no file' => array(
                 array('incenteev-parameters' => array()),