Use isset over array_key_exists

Author: gmponos

src/Client.php

@@ -1,5 +1,5 @@
 <?php
-
+declare(strict_types=1);
 namespace Http\Client\Curl;
 
 use Http\Client\Exception;
@@ -330,10 +330,10 @@ private function createHeaders(RequestInterface $request, array $options)
                 continue;
             }
             if ('content-length' === $header) {
-                if (array_key_exists(CURLOPT_POSTFIELDS, $options)) {
+                if (isset($options[CURLOPT_POSTFIELDS])) {
                     // Small body content length can be calculated here.
                     $values = [strlen($options[CURLOPT_POSTFIELDS])];
-                } elseif (!array_key_exists(CURLOPT_READFUNCTION, $options)) {
+                } elseif (!isset($options[CURLOPT_READFUNCTION])) {
                     // Else if there is no body, forcing "Content-length" to 0
                     $values = [0];
                 }

tests/ClientTest.php

@@ -33,6 +33,27 @@ public function testExpectHeader()
         static::assertContains('Expect:', $headers);
     }
 
+    /**
+     * "Expect" header should be empty.
+     *
+     * @link https://github.com/php-http/curl-client/issues/18
+     */
+    public function testWithNullPostFields()
+    {
+        $client = $this->getMockBuilder(Client::class)->disableOriginalConstructor()
+            ->setMethods(['__none__'])->getMock();
+
+        $createHeaders = new \ReflectionMethod(Client::class, 'createHeaders');
+        $createHeaders->setAccessible(true);
+
+        $request = new Request();
+        $request = $request->withHeader('content-length', 0);
+
+        $headers = $createHeaders->invoke($client, $request, [CURLOPT_POSTFIELDS => null]);
+
+        static::assertContains('Expect:', $headers);
+    }
+
     public function testRewindStream()
     {
         $client = $this->getMockBuilder(Client::class)->disableOriginalConstructor()