Merge pull request php-curl-class#558 from zachborboa/master

Add tests for content-length with DELETE requests

Author: zachborboa

src/Curl/Curl.php

@@ -265,7 +265,9 @@ public function delete($url, $query_parameters = array(), $data = array())
 
         $this->setUrl($url, $query_parameters);
         $this->setOpt(CURLOPT_CUSTOMREQUEST, 'DELETE');
-        $this->setOpt(CURLOPT_POSTFIELDS, $this->buildPostData($data));
+        if (!empty($data)) {
+            $this->setOpt(CURLOPT_POSTFIELDS, $this->buildPostData($data));
+        }
         return $this->exec();
     }
 

tests/PHPCurlClass/PHPCurlClassTest.php

@@ -622,6 +622,22 @@ public function testDeleteRequestBody()
         $this->assertEquals('{"get":{"foo":"bar"},"delete":{"wibble":"wubble"}}', $test->curl->rawResponse);
     }
 
+    public function testDeleteContentLengthSetWithBody()
+    {
+        $request_body = 'a=1&b=2&c=3';
+        $test = new Test();
+        $test->server('request_method', 'DELETE', array(), $request_body);
+        $this->assertEquals(strlen($request_body), $test->curl->requestHeaders['content-length']);
+    }
+
+    public function testDeleteContentLengthUnsetWithoutBody()
+    {
+        $request_body = array();
+        $test = new Test();
+        $test->server('request_method', 'DELETE', array(), $request_body);
+        $this->assertFalse(isset($test->curl->requestHeaders['content-length']));
+    }
+
     public function testHeadRequestMethod()
     {
         $test = new Test();