Add Plugin to all Clients from Service Builder about guzzle

[gimler]

i use the following code in my Service builder class at the moment. I can make a pull request for that when i would be accepted?

    private $plugins = array();

    /**
     * {@inheritdoc}
     */
    public function get($name, $throwAway = false)
    {
        if (!isset($this->builderConfig[$name])) {
            throw new ClientNotFoundException('No client is registered as ' . $name);
        }

        if (!$throwAway && isset($this->clients[$name])) {
            return $this->clients[$name];
        }

        $client = parent::get($name, $throwAway);
        foreach ($this->getPlugins() as $plugin) {
            $client->getEventDispatcher()->addSubscriber($plugin);
        }

        return $client;
    }

    public function addPlugin($plugin)
    {
        $this->plugins[] = $plugin;
    }

    public function getPlugins()
    {
        return $this->plugins;
    }

[mtdowling]

What do you think about making the ServiceBuilder own an EventDispatcher, and make it so that the ServiceBuilder emits an event each time a client is created? Then you could add a listener for this event that decorates the created clients in whatever way you see fit (adds plugins, sets configs, etc).

<?php

$builder->getEventDispatcher()->addListener('service_builder.create_client', function(Event $event) use ($plugins) {
    // Add plugins to the client
    foreach ($plugins as $plugin) {
        $event['client']->getEventDispatcher()->addSubscriber($plugin);
    }

    // Do more stuff other than add plugins
    $event['client']->getConfig()->set('foo', 'bar');
});

I think something like the above would be a bit more flexible, but it does add a bit of code to the caller. What do you think?

[gimler]

nice this is the better way