Fixed implicitly nullable params (#6616)

Author: huangdijia

src/AssertableJsonString.php

@@ -256,7 +256,7 @@ public function assertPath($path, $expect)
      * @param null|array $responseData
      * @return $this
      */
-    public function assertStructure(array $structure = null, $responseData = null)
+    public function assertStructure(?array $structure = null, $responseData = null)
     {
         if (is_null($structure)) {
             return $this->assertSimilar($this->decoded);

src/Client.php

@@ -44,7 +44,7 @@ class Client extends Server
 
     protected string $baseUri = 'http://127.0.0.1/';
 
-    public function __construct(ContainerInterface $container, PackerInterface $packer = null, $server = 'http')
+    public function __construct(ContainerInterface $container, ?PackerInterface $packer = null, $server = 'http')
     {
         parent::__construct(
             $container,

src/Concerns/InteractsWithContainer.php

@@ -58,7 +58,7 @@ protected function instance($abstract, $instance)
      * @param string $abstract
      * @return \Mockery\MockInterface
      */
-    protected function mock($abstract, Closure $mock = null)
+    protected function mock($abstract, ?Closure $mock = null)
     {
         return $this->instance($abstract, Mockery::mock(...array_filter(func_get_args())));
     }
@@ -69,7 +69,7 @@ protected function mock($abstract, Closure $mock = null)
      * @param string $abstract
      * @return \Mockery\MockInterface
      */
-    protected function partialMock($abstract, Closure $mock = null)
+    protected function partialMock($abstract, ?Closure $mock = null)
     {
         return $this->instance($abstract, Mockery::mock(...array_filter(func_get_args()))->makePartial());
     }
@@ -80,7 +80,7 @@ protected function partialMock($abstract, Closure $mock = null)
      * @param string $abstract
      * @return \Mockery\MockInterface
      */
-    protected function spy($abstract, Closure $mock = null)
+    protected function spy($abstract, ?Closure $mock = null)
     {
         return $this->instance($abstract, Mockery::spy(...array_filter(func_get_args())));
     }

src/Fluent/AssertableJson.php

@@ -45,7 +45,7 @@ class AssertableJson implements Arrayable
     /**
      * Create a new fluent, assertable JSON data instance.
      */
-    protected function __construct(array $props, string $path = null)
+    protected function __construct(array $props, ?string $path = null)
     {
         $this->path = $path;
         $this->props = $props;
@@ -148,7 +148,7 @@ protected function dotPath(string $key = ''): string
      *
      * @return mixed
      */
-    protected function prop(string $key = null)
+    protected function prop(?string $key = null)
     {
         return Arr::get($this->props, $key);
     }

src/Fluent/Concerns/Debugging.php

@@ -18,7 +18,7 @@ trait Debugging
      *
      * @return $this
      */
-    public function dump(string $prop = null): self
+    public function dump(?string $prop = null): self
     {
         dump($this->prop($prop));
 
@@ -30,7 +30,7 @@ public function dump(string $prop = null): self
      *
      * @return never
      */
-    public function dd(string $prop = null): void
+    public function dd(?string $prop = null): void
     {
         dd($this->prop($prop));
     }
@@ -40,5 +40,5 @@ public function dd(string $prop = null): void
      *
      * @return mixed
      */
-    abstract protected function prop(string $key = null);
+    abstract protected function prop(?string $key = null);
 }

src/Fluent/Concerns/Has.php

@@ -23,7 +23,7 @@ trait Has
      * @param int|string $key
      * @return $this
      */
-    public function count($key, int $length = null): self
+    public function count($key, ?int $length = null): self
     {
         if (is_null($length)) {
             $path = $this->dotPath();
@@ -55,7 +55,7 @@ public function count($key, int $length = null): self
      * @param null|Closure|int $length
      * @return $this
      */
-    public function has($key, $length = null, Closure $callback = null): self
+    public function has($key, $length = null, ?Closure $callback = null): self
     {
         $prop = $this->prop();
 
@@ -198,7 +198,7 @@ abstract protected function interactsWith(string $key): void;
      *
      * @return mixed
      */
-    abstract protected function prop(string $key = null);
+    abstract protected function prop(?string $key = null);
 
     /**
      * Instantiate a new "scope" at the path of the given key.

src/Fluent/Concerns/Interaction.php

@@ -66,5 +66,5 @@ protected function interactsWith(string $key): void
      *
      * @return mixed
      */
-    abstract protected function prop(string $key = null);
+    abstract protected function prop(?string $key = null);
 }

src/Fluent/Concerns/Matching.php

@@ -198,7 +198,7 @@ public function whereContains(string $key, $expected)
      * @param null $value
      * @return $this
      */
-    abstract public function has(string $key, $value = null, Closure $scope = null);
+    abstract public function has(string $key, $value = null, ?Closure $scope = null);
 
     /**
      * Ensures that all properties are sorted the same way, recursively.
@@ -228,5 +228,5 @@ abstract protected function dotPath(string $key = ''): string;
      *
      * @return mixed
      */
-    abstract protected function prop(string $key = null);
+    abstract protected function prop(?string $key = null);
 }

src/Http/TestResponse.php

@@ -355,7 +355,7 @@ public function assertJsonMissingPath(string $path)
      * @param null|array $responseData
      * @return $this
      */
-    public function assertJsonStructure(array $structure = null, $responseData = null)
+    public function assertJsonStructure(?array $structure = null, $responseData = null)
     {
         $this->decodeResponseJson()->assertStructure($structure, $responseData);
 

src/HttpClient.php

@@ -26,7 +26,7 @@ class HttpClient
 
     protected PackerInterface $packer;
 
-    public function __construct(protected ContainerInterface $container, PackerInterface $packer = null, $baseUri = 'http://127.0.0.1:9501')
+    public function __construct(protected ContainerInterface $container, ?PackerInterface $packer = null, $baseUri = 'http://127.0.0.1:9501')
     {
         $this->packer = $packer ?? new JsonPacker();
         $handler = null;