Merge pull request #691 from JeppeKnockaert/fix-php84-deprecations

Fix PHP 8.4 deprecations

src/ResponseFactory.php

@@ -46,7 +46,7 @@ public function share($key, $value = null): void
         }
     }
 
-    public function getShared(string $key = null, $default = null)
+    public function getShared(?string $key = null, $default = null)
     {
         if ($key) {
             return Arr::get($this->sharedProps, $key, $default);

src/Testing/AssertableInertia.php

@@ -42,7 +42,7 @@ public static function fromTestResponse(TestResponse $response): self
         return $instance;
     }
 
-    public function component(string $value = null, $shouldExist = null): self
+    public function component(?string $value = null, $shouldExist = null): self
     {
         PHPUnit::assertSame($value, $this->component, 'Unexpected Inertia page component.');
 

src/Testing/Concerns/Debugging.php

@@ -4,17 +4,17 @@
 
 trait Debugging
 {
-    public function dump(string $prop = null): self
+    public function dump(?string $prop = null): self
     {
         dump($this->prop($prop));
 
         return $this;
     }
 
-    public function dd(string $prop = null): void
+    public function dd(?string $prop = null): void
     {
         dd($this->prop($prop));
     }
 
-    abstract protected function prop(string $key = null);
+    abstract protected function prop(?string $key = null);
 }

src/Testing/Concerns/Has.php

@@ -38,7 +38,7 @@ public function hasAll($key): self
     /**
      * @return $this
      */
-    public function has(string $key, $value = null, Closure $scope = null): self
+    public function has(string $key, $value = null, ?Closure $scope = null): self
     {
         PHPUnit::assertTrue(
             Arr::has($this->prop(), $key),
@@ -106,7 +106,7 @@ public function misses(string $key): self
         return $this->missing($key);
     }
 
-    abstract protected function prop(string $key = null);
+    abstract protected function prop(?string $key = null);
 
     abstract protected function dotPath(string $key): string;
 

src/Testing/Concerns/Interaction.php

@@ -37,5 +37,5 @@ public function etc(): self
         return $this;
     }
 
-    abstract protected function prop(string $key = null);
+    abstract protected function prop(?string $key = null);
 }

src/Testing/Concerns/Matching.php

@@ -68,7 +68,7 @@ protected function ensureSorted(&$value): void
 
     abstract protected function dotPath(string $key): string;
 
-    abstract protected function prop(string $key = null);
+    abstract protected function prop(?string $key = null);
 
-    abstract public function has(string $key, $value = null, Closure $scope = null);
+    abstract public function has(string $key, $value = null, ?Closure $scope = null);
 }

src/Testing/Concerns/PageObject.php

@@ -8,7 +8,7 @@
 
 trait PageObject
 {
-    public function component(string $value = null, $shouldExist = null): self
+    public function component(?string $value = null, $shouldExist = null): self
     {
         PHPUnit::assertSame($value, $this->component, 'Unexpected Inertia page component.');
 
@@ -23,7 +23,7 @@ public function component(string $value = null, $shouldExist = null): self
         return $this;
     }
 
-    protected function prop(string $key = null)
+    protected function prop(?string $key = null)
     {
         return Arr::get($this->props, $key);
     }

src/Testing/TestResponseMacros.php

@@ -8,7 +8,7 @@ class TestResponseMacros
 {
     public function assertInertia()
     {
-        return function (Closure $callback = null) {
+        return function (?Closure $callback = null) {
             $assert = AssertableInertia::fromTestResponse($this);
 
             if (is_null($callback)) {