Merge pull request #18 from QuintenJustus/fix/support-php-81

Support PHP 8.1
cjmellor · Jul 31, 2023 · c1e657d · c1e657d
2 parents 87ebcb8 + 9198f11
commit c1e657d
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 4 deletions.
diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml
@@ -13,7 +13,7 @@ jobs:
     strategy:
       fail-fast: true
       matrix:
-        php: [ 8.2 ]
+        php: [ 8.1, 8.2 ]
         laravel: [ 10 ]
         stability: [ prefer-lowest, prefer-stable ]
 

diff --git a/composer.json b/composer.json
@@ -18,7 +18,7 @@
         }
     ],
     "require": {
-        "php": "^8.2",
+        "php": "^8.1",
         "illuminate/contracts": "^10.0",
         "laravel/pint": "^1.10",
         "spatie/laravel-package-tools": "^1.15"

diff --git a/src/Concerns/GiveExperience.php b/src/Concerns/GiveExperience.php
@@ -23,9 +23,13 @@ trait GiveExperience
     public function addPoints(
         int $amount,
         int $multiplier = null,
-        string $type = AuditType::Add->value,
+        string $type = null,
         string $reason = null
     ): Experience {
+        if ($type === null) {
+            $type = AuditType::Add->value;
+        }
+
         /**
          * If the Multiplier Service is enabled, apply the Multipliers.
          */

diff --git a/src/Services/MultiplierService.php b/src/Services/MultiplierService.php
@@ -4,7 +4,7 @@
 
 use Illuminate\Support\Collection;
 
-readonly class MultiplierService
+class MultiplierService
 {
     public function __construct(
         private Collection $multipliers,