Remove deprecated containerCopy() and containerCopyStream() methods

Use `containerArchive()` and `containerArchiveStream()` instead.

Author: clue

README.md

@@ -243,7 +243,7 @@ The following API endpoints resolve with a string in the [TAR file format](https
 
 ```php
 $client->containerExport($container);
-$client->containerCopy($container, $config);
+$client->containerArchive($container, $path);
 ```
 
 Keep in mind that this means the whole string has to be kept in memory.
@@ -257,14 +257,14 @@ a [`Stream`](https://github.com/reactphp/stream) instance instead:
 
 ```php
 $stream = $client->containerExportStream($image);
-$stream = $client->containerCopyStream($image, $config);
+$stream = $client->containerArchiveStream($container, $path);
 ```
 
 Accessing individual files in the TAR file format string or stream is out of scope
 for this library.
 Several libraries are available, one that is known to work is [clue/reactphp-tar](https://github.com/clue/reactphp-tar).
 
-See also the [copy example](examples/copy.php) and the [export example](examples/export.php).
+See also the [archive example](examples/archive.php) and the [export example](examples/export.php).
 
 #### JSON streaming
 

src/Client.php

@@ -712,89 +712,6 @@ public function containerRemove($container, $v = false, $force = false)
         )->then(array($this->parser, 'expectEmpty'));
     }
 
-    /**
-     * [deprecated] Copy files or folders of container id
-     *
-     * This resolves with a string in the TAR file format containing all files
-     * specified in the given $path.
-     *
-     * Keep in mind that this means the whole string has to be kept in memory.
-     * For bigger containers it's usually a better idea to use a streaming approach,
-     * see containerCopyStream() for more details.
-     *
-     * Accessing individual files in the TAR file format string is out of scope
-     * for this library. Several libraries are available, one that is known to
-     * work is clue/reactphp-tar (see links).
-     *
-     * @param string $container container ID
-     * @param string $resource  path to file or directory to copy
-     * @return PromiseInterface Promise<string> tar stream
-     * @link https://docs.docker.com/engine/api/v1.22/#copy-files-or-folders-from-a-container
-     * @link https://github.com/clue/reactphp-tar
-     * @deprecated 0.3.0 Deprecated in Docker Engine API v1.20 (Docker v1.8) and removed in Docker Engine API v1.24 (Docker v1.12), use `containerArchive()` instead
-     * @see self::containerArchive()
-     * @see self::containerCopyStream()
-     */
-    public function containerCopy($container, $path)
-    {
-        return $this->postJson(
-            $this->uri->expand(
-                '/containers/{container}/copy',
-                array(
-                    'container' => $container
-                )
-            ),
-            array(
-                'Resource' => $path
-            )
-        )->then(array($this->parser, 'expectPlain'));
-    }
-
-    /**
-     * [Deprecated] Copy files or folders of container id
-     *
-     * This returns a stream in the TAR file format containing all files
-     * specified in the given $path.
-     *
-     * This works for (any number of) files of arbitrary sizes as only small chunks have to
-     * be kept in memory.
-     *
-     * Accessing individual files in the TAR file format stream is out of scope
-     * for this library. Several libraries are available, one that is known to
-     * work is clue/reactphp-tar (see links).
-     *
-     * The resulting stream is a well-behaving readable stream that will emit
-     * the normal stream events.
-     *
-     * @param string $container container ID
-     * @param string $path      path to file or directory to copy
-     * @return ReadableStreamInterface tar stream
-     * @link https://docs.docker.com/engine/api/v1.22/#copy-files-or-folders-from-a-container
-     * @link https://github.com/clue/reactphp-tar
-     * @deprecated 0.3.0 Deprecated in Docker Engine API v1.20 (Docker v1.8) and removed in Docker Engine API v1.24 (Docker v1.12), use `containerArchiveStream()` instead
-     * @see self::containerArchiveStream()
-     * @see self::containerCopy()
-     */
-    public function containerCopyStream($container, $path)
-    {
-        return $this->streamingParser->parsePlainStream(
-            $this->browser->withOptions(array('streaming' => true))->post(
-                $this->uri->expand(
-                    '/containers/{container}/copy',
-                    array(
-                        'container' => $container
-                    )
-                ),
-                array(
-                    'Content-Type' => 'application/json'
-                ),
-                $this->json(array(
-                    'Resource' => $path
-                ))
-            )
-        );
-    }
-
     /**
      * Get a tar archive of a resource in the filesystem of container id.
      *

tests/ClientTest.php

@@ -417,24 +417,6 @@ public function testContainerResize()
         $this->expectPromiseResolveWith('', $this->client->containerResize(123, 800, 600));
     }
 
-    public function testContainerCopy()
-    {
-        $data = 'tar stream';
-        $this->expectRequestFlow('post', '/containers/123/copy', $this->createResponse($data), 'expectPlain');
-
-        $this->expectPromiseResolveWith($data, $this->client->containerCopy('123', 'file.txt'));
-    }
-
-    public function testContainerCopyStream()
-    {
-        $stream = $this->getMockBuilder('React\Stream\ReadableStreamInterface')->getMock();
-
-        $this->expectRequest('post', '/containers/123/copy', $this->createResponse(''));
-        $this->streamingParser->expects($this->once())->method('parsePlainStream')->will($this->returnValue($stream));
-
-        $this->assertSame($stream, $this->client->containerCopyStream('123', 'file.txt'));
-    }
-
     public function testContainerArchive()
     {
         $data = 'tar stream';