fix deprecation for empty parameter for strpos method

Signed-off-by: Markus Opolka <[email protected]>
NETWAYS · Nov 14, 2024 · f00f439 · f00f439
1 parent 699c687
commit f00f439
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/application/controllers/IcingadbimgController.php b/application/controllers/IcingadbimgController.php
@@ -66,7 +66,7 @@ public function init()
         if ($this->hasParam('timerangeto')) {
             $this->timerangeto = urldecode($this->getParam('timerangeto'));
         } else {
-            $this->timerangeto = strpos($this->timerange, '/') ? 'now-' . $this->timerange : "now";
+            $this->timerangeto = strpos($this->timerange ?? '', '/') ? 'now-' . $this->timerange ?? '' : "now";
         }
         $this->cacheTime = $this->hasParam('cachetime') ? $this->getParam('cachetime') : 300;
 

diff --git a/library/Grafana/ProvidedHook/Icingadb/IcingaDbGrapher.php b/library/Grafana/ProvidedHook/Icingadb/IcingaDbGrapher.php
@@ -236,7 +236,7 @@ private function getGraphConf($serviceName, $serviceCommand = null): ?self
             $this->timerange = Url::fromRequest()->hasParam('timerange') ?
                 'now-' . urldecode(Url::fromRequest()->getParam('timerange')) :
                 'now-' . $this->getGraphConfigOption($serviceName, 'timerange', $this->timerange);
-            $this->timerangeto = strpos($this->timerange, '/') ? $this->timerange : $this->timerangeto;
+            $this->timerangeto = strpos($this->timerange ?? '', '/') ? $this->timerange : $this->timerangeto;
         }
 
         $this->height = $this->getGraphConfigOption($serviceName, 'height', $this->height);