Allow null id in delete

Author: chadicus

src/Client.php

@@ -272,8 +272,13 @@ public function put($resource, $id, array $data)
      */
     public function startDelete($resource, $id, array $data = null)
     {
-        Util::throwIfNotType(['string' => [$resource, $id]], true);
-        $url = "{$this->_baseUrl}/" . urlencode($resource) . '/' . urlencode($id);
+        Util::throwIfNotType(['string' => [$resource]], true);
+        $url = "{$this->_baseUrl}/" . urlencode($resource);
+        if ($id !== null) {
+            Util::throwIfNotType(['string' => [$id]], true);
+            $url .= '/' . urlencode($id);
+        }
+
         $json = $data !== null ? json_encode($data) : null;
         return $this->_start($url, 'DELETE', $json, ['Content-Type' => 'application/json']);
     }

tests/ClientTest.php

@@ -286,6 +286,25 @@ public function delete()
         $this->assertSame(['Content-Type' => ['application/json']], $response->getResponseHeaders());
     }
 
+    /**
+     * Verify behavior of startDelete when no id is given.
+     *
+     * @test
+     * @covers ::startDelete
+     *
+     * @return void
+     */
+    public function deleteWithoutId()
+    {
+        $adapter = new DeleteAdapter();
+        $authentication = Authentication::createClientCredentials('not under test', 'not under test');
+        $client = new Client($adapter, $authentication, 'baseUrl/v1');
+        $client->startDelete('resource', null, ['foo' => 'bar']);
+        $this->assertSame('baseUrl/v1/resource', $adapter->request->getUrl());
+        $this->assertSame('DELETE', $adapter->request->getMethod());
+        $this->assertSame(json_encode(['foo' => 'bar']), $adapter->request->getBody());
+    }
+
     /**
      * Verfiy delete creates the request body properly
      *