Add null to some PHPDoc types

CHANGELOG.md

@@ -15,6 +15,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 * Removed `egeloen/http-adapter` as suggested package since the project is abandoned by @franmomu [#2069](https://github.com/ruflin/Elastica/pull/2069)
 * Removed `0` as valid request data using Analyze API by @franmomu [#2068](https://github.com/ruflin/Elastica/pull/2068)
 ### Fixed
+* Fixed some PHPDoc types adding `null` as possible value by @franmomu [#2070](https://github.com/ruflin/Elastica/pull/2070)
 ### Security
 
 ## [7.1.5](https://github.com/ruflin/Elastica/compare/7.1.5...7.1.4)

src/AbstractUpdateAction.php

@@ -307,9 +307,9 @@ public function hasRetryOnConflict()
      */
     public function setRefresh($refresh = true)
     {
-        \is_bool($refresh) && $refresh = $refresh
-            ? Reindex::REFRESH_TRUE
-            : Reindex::REFRESH_FALSE;
+        if (\is_bool($refresh)) {
+            $refresh = $refresh ? Reindex::REFRESH_TRUE : Reindex::REFRESH_FALSE;
+        }
 
         return $this->setParam(Reindex::REFRESH, $refresh);
     }

src/Connection.php

@@ -341,15 +341,15 @@ public static function create($params = [])
     }
 
     /**
-     * @return string User
+     * @return string|null User
      */
     public function getUsername()
     {
         return $this->hasParam('username') ? $this->getParam('username') : null;
     }
 
     /**
-     * @return string Password
+     * @return string|null Password
      */
     public function getPassword()
     {

src/Query/AbstractGeoDistance.php

@@ -36,14 +36,14 @@ abstract class AbstractGeoDistance extends AbstractQuery
     /**
      * Latitude.
      *
-     * @var float
+     * @var float|null
      */
     protected $_latitude;
 
     /**
      * Longitude.
      *
-     * @var float
+     * @var float|null
      */
     protected $_longitude;
 

src/Reindex.php

@@ -75,7 +75,9 @@ public function run(): Response
 
     public function setWaitForCompletion($value): void
     {
-        \is_bool($value) && $value = $value ? 'true' : 'false';
+        if (\is_bool($value)) {
+            $value = $value ? 'true' : 'false';
+        }
 
         $this->setParam(self::WAIT_FOR_COMPLETION, $value);
     }

src/Request.php

@@ -20,7 +20,7 @@ class Request extends Param
     public const NDJSON_CONTENT_TYPE = 'application/x-ndjson';
 
     /**
-     * @var Connection
+     * @var Connection|null
      */
     protected $_connection;
 

src/Transport/NullTransport.php

@@ -20,7 +20,7 @@ class NullTransport extends AbstractTransport
     /**
      * Response you want to get from the transport.
      *
-     * @var Response Response
+     * @var Response|null Response
      */
     protected $_response;