Merge pull request #113 from php-http/bugfix/plugin-config

fix plugin instantiation

Author: dbu

CHANGELOG.md

@@ -3,10 +3,14 @@
 
 ## UNRELEASED
 
+### Fixed
+
+- decoder, redirect and retry plugins can now be used and no longer trigger an error because of incorrect constructor arguments.
+
 ### Added
 
 - Support for BatchClient
-- The stopwatch plugin in included by default when using profiling. 
+- The stopwatch plugin in included by default when using profiling.
 
 ### Changed
 

DependencyInjection/Configuration.php

@@ -92,6 +92,7 @@ public function getConfigTreeBuilder()
                     return $v;
                 })
             ->end()
+            ->fixXmlConfig('client')
             ->children()
                 ->arrayNode('main_alias')
                     ->addDefaultsIfNotSet()
@@ -150,7 +151,7 @@ public function getConfigTreeBuilder()
         return $treeBuilder;
     }
 
-    protected function configureClients(ArrayNodeDefinition $root)
+    private function configureClients(ArrayNodeDefinition $root)
     {
         $root->children()
             ->arrayNode('clients')
@@ -196,7 +197,7 @@ protected function configureClients(ArrayNodeDefinition $root)
     /**
      * @param ArrayNodeDefinition $root
      */
-    protected function configurePlugins(ArrayNodeDefinition $root)
+    private function configurePlugins(ArrayNodeDefinition $root)
     {
         $root->children()
             ->arrayNode('plugins')

DependencyInjection/HttplugExtension.php

@@ -145,7 +145,9 @@ private function configurePluginByName($name, Definition $definition, array $con
                 $definition->replaceArgument(0, new Reference($config['cookie_jar']));
                 break;
             case 'decoder':
-                $definition->addArgument($config['use_content_encoding']);
+                $definition->addArgument([
+                    'use_content_encoding' => $config['use_content_encoding'],
+                ]);
                 break;
             case 'history':
                 $definition->replaceArgument(0, new Reference($config['journal']));
@@ -157,16 +159,22 @@ private function configurePluginByName($name, Definition $definition, array $con
                 }
                 break;
             case 'redirect':
-                $definition
-                    ->addArgument($config['preserve_header'])
-                    ->addArgument($config['use_default_for_multiple']);
+                $definition->addArgument([
+                    'preserve_header' => $config['preserve_header'],
+                    'use_default_for_multiple' => $config['use_default_for_multiple'],
+                ]);
                 break;
             case 'retry':
-                $definition->addArgument($config['retry']);
+                $definition->addArgument([
+                    'retries' => $config['retry'],
+                ]);
                 break;
             case 'stopwatch':
                 $definition->replaceArgument(0, new Reference($config['stopwatch']));
                 break;
+
+            default:
+                throw new \InvalidArgumentException(sprintf('Internal exception: Plugin %s is not handled', $name));
         }
     }