Fix php-curl-class#545: Allow fileless requests to use Content-Type: multipart/form-data.

This change allows making requests that don't contain files to use
Content-Type: multipart/form-data. Previously, requests with array data
not specifying any files were sent using
Content-Type: application/x-www-form-urlencoded.

Author: zachborboa

src/Curl/Curl.php

@@ -176,7 +176,12 @@ interface_exists('JsonSerializable', false) &&
             }
         }
 
-        if (!$binary_data && (is_array($data) || is_object($data))) {
+        if (!$binary_data &&
+            (is_array($data) || is_object($data)) &&
+            (
+                !isset($this->headers['Content-Type']) ||
+                !preg_match('/^multipart\/form-data/', $this->headers['Content-Type'])
+            )) {
             $data = http_build_query($data, '', '&');
         }
 

tests/PHPCurlClass/PHPCurlClassTest.php

@@ -552,6 +552,28 @@ public function testPutFileHandle()
         $this->assertEquals('image/png', $test->curl->response);
     }
 
+    public function testMultipartFormDataContentType()
+    {
+        // Use a PUT request instead of a POST request so the request
+        // multipart/form-data is not automatically parsed and can be tested
+        // against.
+        $test = new Test();
+        $test->curl->setHeader('Content-Type', 'multipart/form-data');
+        $test->server('put', 'PUT', array(
+            'foo' => 'bar',
+        ));
+
+        $this->assertEquals('100-continue', $test->curl->requestHeaders['Expect']);
+        $this->assertStringStartsWith('multipart/form-data; boundary=', $test->curl->requestHeaders['Content-Type']);
+
+        $expected_contains = "\r\n" .
+            'Content-Disposition: form-data; name="foo"' . "\r\n" .
+            "\r\n" .
+            'bar' . "\r\n" .
+            '';
+        $this->assertContains($expected_contains, $test->curl->response);
+    }
+
     public function testPatchRequestMethod()
     {
         $test = new Test();