Merge pull request #332 from leon38/fix-dependency-injection

dbu · web-flow · commit 1fddc9490f4b · 2019-04-12T13:08:49.000+02:00
Fix error on undefined variable when a client is named default in config
diff --git a/src/DependencyInjection/HttplugExtension.php b/src/DependencyInjection/HttplugExtension.php
@@ -127,6 +127,7 @@ private function configureClients(ContainerBuilder $container, array $config)
                 $default = $first;
             } else {
                 $default = 'default';
+                $serviceId = 'httplug.client.'.$default;
             }
 
             // Autowiring alias for special clients, if they are enabled on the default client

Original file line number	Diff line number	Diff line change
`@@ -127,6 +127,7 @@ private function configureClients(ContainerBuilder $container, array $config)`
`127`	`127`	`$default = $first;`
`128`	`128`	`} else {`
`129`	`129`	`$default = 'default';`
	`130`	`+ $serviceId = 'httplug.client.'.$default;`
`130`	`131`	`}`
`131`	`132`
`132`	`133`	`// Autowiring alias for special clients, if they are enabled on the default client`