Cast args explicitly to int

Icinga · Aug 21, 2023 · c58b2a1 · c58b2a1
1 parent 0e9dc71
commit c58b2a1
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/application/controllers/GraphController.php b/application/controllers/GraphController.php
@@ -152,8 +152,8 @@ protected function supplyImage($object, $checkCommand, $obscuredCheckCommand)
                 $charts[0]
                     ->setFrom($this->graphParams['start'])
                     ->setUntil($this->graphParams['end'])
-                    ->setWidth($this->graphParams['width'])
-                    ->setHeight($this->graphParams['height'])
+                    ->setWidth((int) $this->graphParams['width'])
+                    ->setHeight((int) $this->graphParams['height'])
                     ->setBackgroundColor($this->graphParams['bgcolor'])
                     ->setForegroundColor($this->graphParams['fgcolor'])
                     ->setMajorGridLineColor($this->graphParams['majorGridLineColor'])

diff --git a/application/controllers/MonitoringGraphController.php b/application/controllers/MonitoringGraphController.php
@@ -135,8 +135,8 @@ protected function supplyImage($object, $checkCommand, $obscuredCheckCommand)
                 $charts[0]
                     ->setFrom($this->graphParams['start'])
                     ->setUntil($this->graphParams['end'])
-                    ->setWidth($this->graphParams['width'])
-                    ->setHeight($this->graphParams['height'])
+                    ->setWidth((int) $this->graphParams['width'])
+                    ->setHeight((int) $this->graphParams['height'])
                     ->setBackgroundColor($this->graphParams['bgcolor'])
                     ->setForegroundColor($this->graphParams['fgcolor'])
                     ->setMajorGridLineColor($this->graphParams['majorGridLineColor'])

diff --git a/library/Graphite/Web/Widget/Graphs.php b/library/Graphite/Web/Widget/Graphs.php
@@ -313,8 +313,8 @@ protected function getGraphsList()
                             if ($this->renderInline) {
                                 $chart->setFrom($this->start)
                                     ->setUntil($this->end)
-                                    ->setWidth($actwidth)
-                                    ->setHeight($actheight)
+                                    ->setWidth((int) $actwidth)
+                                    ->setHeight((int) $actheight)
                                     ->setBackgroundColor('white')
                                     ->setForegroundColor('black')
                                     ->setMajorGridLineColor('grey')