Style fixes

Author: murrant

LibreNMS/Data/Graphing/AbstractGraph.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * AbstractGraph.php
  *
@@ -53,6 +54,7 @@ public function fill(array $vars): self
                 }
             }
         }
+
         return $this;
     }
 }

LibreNMS/Data/Graphing/Builders/MultiLineGraphBuilder.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * MultiLineGraphBuilder.php
  *
@@ -44,42 +45,49 @@ class MultiLineGraphBuilder
     public function unitText(string $unitText): self
     {
         $this->unitText = $unitText;
+
         return $this;
     }
 
     public function units(string $units): self
     {
         $this->units = $units;
+
         return $this;
     }
 
     public function colours(string $colours): self
     {
         $this->colours = $colours;
+
         return $this;
     }
 
     public function scaleMin(float $scaleMin): self
     {
         $this->scaleMin = $scaleMin;
+
         return $this;
     }
 
     public function scaleMax(float $scaleMax): self
     {
         $this->scaleMax = $scaleMax;
+
         return $this;
     }
 
     public function noTotal(bool $noTotal = true): self
     {
         $this->nototal = $noTotal;
+
         return $this;
     }
 
     public function descrLen(int $descrLen): self
     {
         $this->descrLen = $descrLen;
+
         return $this;
     }
 
@@ -101,6 +109,7 @@ public function addDataset(
             'areacolour' => $areaColour,
             'invert' => $invert,
         ];
+
         return $this;
     }
 

LibreNMS/Data/Graphing/Builders/MultiSimplexSeparatedGraphBuilder.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * MultiSimplexSeparatedGraphBuilder.php
  *
@@ -47,60 +48,70 @@ class MultiSimplexSeparatedGraphBuilder
     public function unitText(string $unitText): self
     {
         $this->unitText = $unitText;
+
         return $this;
     }
 
     public function totalUnits(string $totalUnits): self
     {
         $this->totalUnits = $totalUnits;
+
         return $this;
     }
 
     public function colours(string $colours): self
     {
         $this->colours = $colours;
+
         return $this;
     }
 
     public function scaleMin(float $scaleMin): self
     {
         $this->scaleMin = $scaleMin;
+
         return $this;
     }
 
     public function scaleMax(float $scaleMax): self
     {
         $this->scaleMax = $scaleMax;
+
         return $this;
     }
 
     public function divider(float $divider): self
     {
         $this->divider = $divider;
+
         return $this;
     }
 
     public function multiplier(float $multiplier): self
     {
         $this->multiplier = $multiplier;
+
         return $this;
     }
 
     public function textOrig(bool $textOrig = true): self
     {
         $this->textOrig = $textOrig;
+
         return $this;
     }
 
     public function noTotal(bool $noTotal = true): self
     {
         $this->nototal = $noTotal;
+
         return $this;
     }
 
     public function descrLen(int $descrLen): self
     {
         $this->descrLen = $descrLen;
+
         return $this;
     }
 
@@ -116,6 +127,7 @@ public function addDataset(
             'descr' => $description,
             'colour' => $colour,
         ];
+
         return $this;
     }
 

LibreNMS/Data/Graphing/GraphFactory.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * GraphFactory.php
  *
@@ -46,7 +47,7 @@ public function graphFor(string $name, array $vars = []): GraphInterface
         $vars['type'] ??= $name;
 
         // Look for a modern class, e.g. LibreNMS\Data\Graphing\Device\ProcessorSeparateGraph
-        $className = "LibreNMS\\Graphs\\" . ucfirst($type) . "\\" . Str::studly($subtype) . "Graph";
+        $className = 'LibreNMS\\Graphs\\' . ucfirst($type) . '\\' . Str::studly($subtype) . 'Graph';
         if (class_exists($className)) {
             return app($className, ['vars' => $vars]);
         }

LibreNMS/Data/Graphing/LegacyGraph.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * LegacyGraph.php
  *
@@ -47,6 +48,7 @@ class LegacyGraph extends AbstractGraph
 
     /**
      * @param  array<string, scalar>  $vars
+     *
      * @throws InvalidGraph
      */
     public function __construct(
@@ -103,6 +105,7 @@ private function load(): void
 
         if (! $auth) {
             $this->loaded = true;
+
             return;
         }
 
@@ -144,19 +147,21 @@ private function load(): void
     public function authorize(): bool
     {
         $this->load();
+
         return $this->authorized;
     }
 
-
     public function definition(GraphParameters $graph_params): array
     {
         $this->load();
+
         return $this->rrdOptions;
     }
 
     public function getPageTitle(): string
     {
         $this->load();
+
         return $this->pageTitle ?? $this->getGraphTitle();
     }
 
@@ -178,6 +183,7 @@ public function getGraphTitle(): string
     public function getRrdFiles(): array
     {
         $this->load();
+
         return $this->rrdFiles;
     }
 }

LibreNMS/Exceptions/InvalidGraph.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * InvalidGraph.php
  *
@@ -29,8 +30,8 @@
 
 class InvalidGraph extends RrdException
 {
-public function __construct(string $message = "", int $code = 0, ?Throwable $previous = null)
-{
-    parent::__construct($message ?: "Invalid Graph", $code, $previous);
-}
+    public function __construct(string $message = '', int $code = 0, ?Throwable $previous = null)
+    {
+        parent::__construct($message ?: 'Invalid Graph', $code, $previous);
+    }
 }

LibreNMS/Graphs/Device/ProcessorGraph.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * ProcessorGraph.php
  *
@@ -71,6 +72,7 @@ public function getRrdFiles(): array
         foreach ($this->processors as $proc) {
             $files[] = Rrd::name($this->device->hostname, ['processor', $proc->processor_type, $proc->processor_index]);
         }
+
         return $files;
     }
 

LibreNMS/Interfaces/Data/Graphing/GraphInterface.php

@@ -1,4 +1,5 @@
 <?php
+
 /**
  * GraphInterface.php
  *
@@ -30,9 +31,14 @@
 interface GraphInterface
 {
     public function authorize(): bool;
+
     public function validation(): array;
+
     public function definition(GraphParameters $graph_params): array;
+
     public function getPageTitle(): string;
+
     public function getGraphTitle(): string;
+
     public function getRrdFiles(): array;
 }

app/Console/Commands/GraphDetail.php

@@ -24,11 +24,10 @@ public function handle(GraphFactory $graphs): int
         $this->line("Type: $graph->type");
         $this->line("Subtype: $graph->subtype");
         $this->line('Authorized: ' . ($graph->authorize() ? 'true' : 'false'));
-        $this->line("Graph Title: " . $graph->getGraphTitle());
-        $this->line("Page Title: " . substr($graph->getPageTitle(), 0, 80));
-        $this->line("Rrd Files: " . implode(', ', array_map(fn($f) => basename($f), $graph->getRrdFiles())));
-        $this->line("Definition: " . substr(implode(' ', $graph->definition()), 80));
-
+        $this->line('Graph Title: ' . $graph->getGraphTitle());
+        $this->line('Page Title: ' . substr($graph->getPageTitle(), 0, 80));
+        $this->line('Rrd Files: ' . implode(', ', array_map(fn ($f) => basename($f), $graph->getRrdFiles())));
+        $this->line('Definition: ' . substr(implode(' ', $graph->definition()), 80));
 
         return 0;
     }