Deprecate plugins

Author: sagikazarmark

spec/AddHostPluginSpec.php

@@ -22,14 +22,13 @@ function let(UriInterface $uri)
     function it_is_initializable(UriInterface $uri)
     {
         $uri->getHost()->shouldBeCalled()->willReturn('example.com');
-        $this->beConstructedWith($uri);
+
         $this->shouldHaveType('Http\Client\Plugin\AddHostPlugin');
     }
 
     function it_is_a_plugin(UriInterface $uri)
     {
         $uri->getHost()->shouldBeCalled()->willReturn('example.com');
-        $this->beConstructedWith($uri);
 
         $this->shouldImplement('Http\Client\Plugin\Plugin');
     }

spec/ContentLengthPluginSpec.php

@@ -3,16 +3,20 @@
 namespace spec\Http\Client\Plugin;
 
 use PhpSpec\Exception\Example\SkippingException;
-use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
 use Psr\Http\Message\RequestInterface;
 use Psr\Http\Message\StreamInterface;
+use PhpSpec\ObjectBehavior;
+use Prophecy\Argument;
 
 class ContentLengthPluginSpec extends ObjectBehavior
 {
     function it_is_initializable()
     {
         $this->shouldHaveType('Http\Client\Plugin\ContentLengthPlugin');
+    }
+
+    function it_is_a_plugin()
+    {
         $this->shouldImplement('Http\Client\Plugin\Plugin');
     }
 

spec/HeaderAppendPluginSpec.php

@@ -3,10 +3,10 @@
 namespace spec\Http\Client\Plugin;
 
 use PhpSpec\Exception\Example\SkippingException;
-use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
 use Psr\Http\Message\RequestInterface;
 use Psr\Http\Message\StreamInterface;
+use PhpSpec\ObjectBehavior;
+use Prophecy\Argument;
 
 class HeaderAppendPluginSpec extends ObjectBehavior
 {

spec/HeaderDefaultsPluginSpec.php

@@ -3,10 +3,10 @@
 namespace spec\Http\Client\Plugin;
 
 use PhpSpec\Exception\Example\SkippingException;
-use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
 use Psr\Http\Message\RequestInterface;
 use Psr\Http\Message\StreamInterface;
+use PhpSpec\ObjectBehavior;
+use Prophecy\Argument;
 
 class HeaderDefaultsPluginSpec extends ObjectBehavior
 {

spec/HeaderRemovePluginSpec.php

@@ -3,10 +3,10 @@
 namespace spec\Http\Client\Plugin;
 
 use PhpSpec\Exception\Example\SkippingException;
-use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
 use Psr\Http\Message\RequestInterface;
 use Psr\Http\Message\StreamInterface;
+use PhpSpec\ObjectBehavior;
+use Prophecy\Argument;
 
 class HeaderRemovePluginSpec extends ObjectBehavior
 {

spec/HeaderSetPluginSpec.php

@@ -3,10 +3,10 @@
 namespace spec\Http\Client\Plugin;
 
 use PhpSpec\Exception\Example\SkippingException;
-use PhpSpec\ObjectBehavior;
-use Prophecy\Argument;
 use Psr\Http\Message\RequestInterface;
 use Psr\Http\Message\StreamInterface;
+use PhpSpec\ObjectBehavior;
+use Prophecy\Argument;
 
 class HeaderSetPluginSpec extends ObjectBehavior
 {

src/AddHostPlugin.php

@@ -2,72 +2,11 @@
 
 namespace Http\Client\Plugin;
 
-use Psr\Http\Message\RequestInterface;
-use Psr\Http\Message\UriInterface;
-use Symfony\Component\OptionsResolver\OptionsResolver;
-
 /**
- * Add schema and host to a request. Can be set to overwrite the schema and host if desired.
- *
  * @author Tobias Nyholm <[email protected]>
+ *
+ * @deprecated since version 1.1, to be removed in 2.0. Use {@link \Http\Client\Common\Plugin\AddHostPlugin} instead.
  */
-class AddHostPlugin implements Plugin
+class AddHostPlugin extends \Http\Client\Common\Plugin\AddHostPlugin implements Plugin
 {
-    /**
-     * @var UriInterface
-     */
-    private $host;
-
-    /**
-     * @var bool
-     */
-    private $replace;
-
-    /**
-     * @param UriInterface $host
-     * @param array        $config {
-     *
-     *     @var bool $replace True will replace all hosts, false will only add host when none is specified.
-     * }
-     */
-    public function __construct(UriInterface $host, array $config = [])
-    {
-        if ($host->getHost() === '') {
-            throw new \LogicException('Host can not be empty');
-        }
-
-        $this->host = $host;
-
-        $resolver = new OptionsResolver();
-        $this->configureOptions($resolver);
-        $options = $resolver->resolve($config);
-
-        $this->replace = $options['replace'];
-    }
-
-    /**
-     * {@inheritdoc}
-     */
-    public function handleRequest(RequestInterface $request, callable $next, callable $first)
-    {
-        if ($this->replace || $request->getUri()->getHost() === '') {
-            $uri = $request->getUri()->withHost($this->host->getHost());
-            $uri = $uri->withScheme($this->host->getScheme());
-
-            $request = $request->withUri($uri);
-        }
-
-        return $next($request);
-    }
-
-    /**
-     * @param OptionsResolver $resolver
-     */
-    private function configureOptions(OptionsResolver $resolver)
-    {
-        $resolver->setDefaults([
-            'replace' => false,
-        ]);
-        $resolver->setAllowedTypes('replace', 'bool');
-    }
 }

src/AuthenticationPlugin.php

@@ -3,35 +3,12 @@
 namespace Http\Client\Plugin;
 
 use Http\Message\Authentication;
-use Psr\Http\Message\RequestInterface;
 
 /**
- * Send an authenticated request.
- *
  * @author Joel Wurtz <[email protected]>
+ *
+ * @deprecated since version 1.1, to be removed in 2.0. Use {@link \Http\Client\Common\Plugin\AuthenticationPlugin} instead.
  */
-class AuthenticationPlugin implements Plugin
+class AuthenticationPlugin extends \Http\Client\Common\Plugin\AuthenticationPlugin implements Plugin
 {
-    /**
-     * @var Authentication An authentication system
-     */
-    private $authentication;
-
-    /**
-     * @param Authentication $authentication
-     */
-    public function __construct(Authentication $authentication)
-    {
-        $this->authentication = $authentication;
-    }
-
-    /**
-     * {@inheritdoc}
-     */
-    public function handleRequest(RequestInterface $request, callable $next, callable $first)
-    {
-        $request = $this->authentication->authenticate($request);
-
-        return $next($request);
-    }
 }

src/ContentLengthPlugin.php

@@ -2,34 +2,11 @@
 
 namespace Http\Client\Plugin;
 
-use Http\Message\Encoding\ChunkStream;
-use Psr\Http\Message\RequestInterface;
-
 /**
- * Allow to set the correct content length header on the request or to transfer it as a chunk if not possible.
- *
  * @author Joel Wurtz <[email protected]>
+ *
+ * @deprecated since version 1.1, to be removed in 2.0. Use {@link \Http\Client\Common\Plugin\ContentLengthPlugin} instead.
  */
-class ContentLengthPlugin implements Plugin
+class ContentLengthPlugin extends \Http\Client\Common\Plugin\ContentLengthPlugin implements Plugin
 {
-    /**
-     * {@inheritdoc}
-     */
-    public function handleRequest(RequestInterface $request, callable $next, callable $first)
-    {
-        if (!$request->hasHeader('Content-Length')) {
-            $stream = $request->getBody();
-
-            // Cannot determine the size so we use a chunk stream
-            if (null === $stream->getSize()) {
-                $stream = new ChunkStream($stream);
-                $request = $request->withBody($stream);
-                $request = $request->withAddedHeader('Transfer-Encoding', 'chunked');
-            } else {
-                $request = $request->withHeader('Content-Length', $stream->getSize());
-            }
-        }
-
-        return $next($request);
-    }
 }