Merge pull request #15 from elgentos/php84-compat

PHP 8.4 compatibility

Author: peterjaap

src/Api/ConfigurationInterface.php

@@ -8,10 +8,10 @@ interface ConfigurationInterface
                  CONFIG_RUMVISION_TRACKING_ID   = 'elgentos_rumvision/general/tracking_id',
                  CONFIG_RUMVISION_HOST_NAME     = 'elgentos_rumvision/general/hostname';
 
-    public function isEnabled(int $storeId = null): bool;
+    public function isEnabled(?int $storeId = null): bool;
 
-    public function getTrackingId(int $storeId = null) :string;
+    public function getTrackingId(?int $storeId = null) :string;
 
-    public function getHostName(int $storeId = null) :string;
+    public function getHostName(?int $storeId = null) :string;
 
-}
+}

src/Model/Config.php

@@ -21,7 +21,7 @@ public function __construct(
         private State $appState,
     ) {}
 
-    public function isEnabled(int $storeId = null): bool
+    public function isEnabled(?int $storeId = null): bool
     {
         if ($this->appState->getMode() === State::MODE_DEVELOPER) {
             // Don't run in developer mode
@@ -35,7 +35,7 @@ public function isEnabled(int $storeId = null): bool
         );
     }
 
-    public function getTrackingId(int $storeId = null): string
+    public function getTrackingId(?int $storeId = null): string
     {
         return (string)$this->config->getValue(
             self::CONFIG_RUMVISION_TRACKING_ID,
@@ -44,7 +44,7 @@ public function getTrackingId(int $storeId = null): string
         );
     }
 
-    public function getHostName(int $storeId = null): string
+    public function getHostName(?int $storeId = null): string
     {
         return (string)$this->config->getValue(
             self::CONFIG_RUMVISION_HOST_NAME,