Support getter autowiring

dunglas · dunglas · commit 4d6652c3e066 · 2017-01-09T21:24:36.000+01:00
diff --git a/DependencyInjection/Configuration.php b/DependencyInjection/Configuration.php
@@ -30,6 +30,10 @@ public function getConfigTreeBuilder()
         $treeBuilder->root('dunglas_action')
             ->fixXmlConfig('directory', 'directories')
             ->children()
+                ->scalarNode('methods')
+                    ->defaultValue(['__construct', 'get*', 'set*'])
+                    ->info('The list of methods to autowire.')
+                ->end()
                 ->arrayNode('directories')
                     ->info('List of directories relative to the kernel root directory containing classes.')
                     ->prototype('scalar')->end()
diff --git a/DependencyInjection/DunglasActionExtension.php b/DependencyInjection/DunglasActionExtension.php
@@ -38,7 +38,7 @@ public function load(array $configs, ContainerBuilder $container)
             $directoryList = array_merge($directoryList, $directories);
 
             foreach ($classes as $class) {
-                $this->registerClass($container, $class, $config['tags']);
+                $this->registerClass($container, $class, $config['tags'], $config['methods']);
             }
         }
 
@@ -119,16 +119,17 @@ private function getClasses($directory)
      * @param ContainerBuilder $container
      * @param string           $className
      * @param array            $tags
+     * @param string[]         $methods
      */
-    private function registerClass(ContainerBuilder $container, $className, array $tags)
+    private function registerClass(ContainerBuilder $container, $className, array $tags, array $methods)
     {
         if ($container->has($className)) {
             return;
         }
 
         $definition = $container->register($className, $className);
         if (method_exists($definition, 'setAutowiredMethods')) {
-            $definition->setAutowiredMethods(['__construct', 'set*']);
+            $definition->setAutowiredMethods($methods);
         } else {
             $definition->setAutowired(true);
         }

Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ public function load(array $configs, ContainerBuilder $container)`
`38`	`38`	`$directoryList = array_merge($directoryList, $directories);`
`39`	`39`
`40`	`40`	`foreach ($classes as $class) {`
`41`		`- $this->registerClass($container, $class, $config['tags']);`
	`41`	`+ $this->registerClass($container, $class, $config['tags'], $config['methods']);`
`42`	`42`	`}`
`43`	`43`	`}`
`44`	`44`
`@@ -119,16 +119,17 @@ private function getClasses($directory)`
`119`	`119`	`* @param ContainerBuilder $container`
`120`	`120`	`* @param string $className`
`121`	`121`	`* @param array $tags`
	`122`	`+ * @param string[] $methods`
`122`	`123`	`*/`
`123`		`- private function registerClass(ContainerBuilder $container, $className, array $tags)`
	`124`	`+ private function registerClass(ContainerBuilder $container, $className, array $tags, array $methods)`
`124`	`125`	`{`
`125`	`126`	`if ($container->has($className)) {`
`126`	`127`	`return;`
`127`	`128`	`}`
`128`	`129`
`129`	`130`	`$definition = $container->register($className, $className);`
`130`	`131`	`if (method_exists($definition, 'setAutowiredMethods')) {`
`131`		`- $definition->setAutowiredMethods(['__construct', 'set*']);`
	`132`	`+ $definition->setAutowiredMethods($methods);`
`132`	`133`	`} else {`
`133`	`134`	`$definition->setAutowired(true);`
`134`	`135`	`}`