PHP 8.4 deprecation (#981)

Author: erikn69

.github/workflows/run-tests.yml

@@ -12,7 +12,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php: [7.4, 8.0, 8.1, 8.2, 8.3]
+        php: [7.4, 8.0, 8.1, 8.2, 8.3, 8.4]
         laravel: [ 11.*, 10.*, 9.*, 8.*, 7.* ]
         include:
           - laravel: 11.*
@@ -38,14 +38,20 @@ jobs:
             php: 8.0
           - laravel: 9.*
             php: 7.4
+          - laravel: 9.*
+            php: 8.4
           - laravel: 8.*
             php: 8.3
+          - laravel: 8.*
+            php: 8.4
           - laravel: 7.*
             php: 8.1
           - laravel: 7.*
             php: 8.2
           - laravel: 7.*
             php: 8.3
+          - laravel: 7.*
+            php: 8.4
 
     name: PHP${{ matrix.php }} - Laravel${{ matrix.laravel }} - ${{ matrix.dependency-version }}
 

src/Events/Audited.php

@@ -34,9 +34,9 @@ class Audited
      *
      * @param \OwenIt\Auditing\Contracts\Auditable   $model
      * @param \OwenIt\Auditing\Contracts\AuditDriver $driver
-     * @param \OwenIt\Auditing\Contracts\Audit       $audit
+     * @param \OwenIt\Auditing\Contracts\Audit|null  $audit
      */
-    public function __construct(Auditable $model, AuditDriver $driver, Audit $audit = null)
+    public function __construct(Auditable $model, AuditDriver $driver, ?Audit $audit = null)
     {
         $this->model = $model;
         $this->driver = $driver;

src/Exceptions/AuditableTransitionException.php

@@ -16,7 +16,7 @@ class AuditableTransitionException extends AuditingException
     /**
      * {@inheritdoc}
      */
-    public function __construct($message = '', array $incompatibilities = [], $code = 0, Throwable $previous = null)
+    public function __construct($message = '', array $incompatibilities = [], $code = 0, ?Throwable $previous = null)
     {
         parent::__construct($message, $code, $previous);
 

tests/Unit/AuditableTest.php

@@ -488,14 +488,14 @@ public function itReturnsTheAuditData()
      *
      * @param string $guard
      * @param string $driver
-     * @param int $id
-     * @param string $type
+     * @param int|null $id
+     * @param string|null $type
      */
     public function itReturnsTheAuditDataIncludingUserAttributes(
         string $guard,
         string $driver,
-        int $id = null,
-        string $type = null
+        ?int $id = null,
+        ?string $type = null
     ) {
         $this->app['config']->set('audit.user.guards', [
             $guard,