simplesamlphp / simplesamlphp-module-statistics

hooks/hook_sanitycheck.php

Spacing +7 added lines, -7 removed lines

@@ -17,7 +17,7 @@ 
     try {
         $statconfig = Configuration::getConfig('module_statistics.php');
     } catch (Exception $e) {
-        $hookinfo['errors'][] = '[statistics] Could not get configuration: ' . $e->getMessage();
+        $hookinfo['errors'][] = '[statistics] Could not get configuration: '.$e->getMessage();
         return;
     }
 
@@ -25,19 +25,19 @@ 
     $inputfile = $statconfig->getValue('inputfile');
 
     if (file_exists($statdir)) {
-        $hookinfo['info'][] = '[statistics] Statistics dir [' . $statdir . '] exists';
+        $hookinfo['info'][] = '[statistics] Statistics dir ['.$statdir.'] exists';
         if (is_writable($statdir)) {
-            $hookinfo['info'][] = '[statistics] Statistics dir [' . $statdir . '] is writable';
+            $hookinfo['info'][] = '[statistics] Statistics dir ['.$statdir.'] is writable';
         } else {
-            $hookinfo['errors'][] = '[statistics] Statistics dir [' . $statdir . '] is not writable';
+            $hookinfo['errors'][] = '[statistics] Statistics dir ['.$statdir.'] is not writable';
         }
     } else {
-        $hookinfo['errors'][] = '[statistics] Statistics dir [' . $statdir . '] does not exist';
+        $hookinfo['errors'][] = '[statistics] Statistics dir ['.$statdir.'] does not exist';
     }
 
     if (file_exists($inputfile)) {
-        $hookinfo['info'][] = '[statistics] Input file [' . $inputfile . '] exists';
+        $hookinfo['info'][] = '[statistics] Input file ['.$inputfile.'] exists';
     } else {
-        $hookinfo['errors'][] = '[statistics] Input file [' . $inputfile . '] does not exist';
+        $hookinfo['errors'][] = '[statistics] Input file ['.$inputfile.'] does not exist';
     }
 }

hooks/hook_cron.php

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
             $aggregator->store($results);
         }
     } catch (Exception $e) {
-        $message = 'Loganalyzer threw exception: ' . $e->getMessage();
+        $message = 'Loganalyzer threw exception: '.$e->getMessage();
         Logger::warning($message);
         $croninfo['summary'][] = $message;
     }

hooks/hook_configpage.php

Spacing +1 added lines, -1 removed lines

@@ -11,7 +11,7 @@
  *
  * @param \SimpleSAML\XHTML\Template &$template The template that we should alter in this hook.
  */
-function statistics_hook_configpage(Template &$template): void
+function statistics_hook_configpage(Template&$template): void
 {
     $template->data['links'][] = [
         'href' => Module::getModuleURL('statistics/'),

src/Graph/GoogleCharts.php

Spacing +18 added lines, -18 removed lines

@@ -53,7 +53,7 @@ 
         foreach ($datasets as $dataset) {
             $setstr[] = self::extEncode($dataset);
         }
-        return 'e:' . join(',', $setstr);
+        return 'e:'.join(',', $setstr);
     }
 
 
@@ -94,31 +94,31 @@ 
      */
     public function show(array $axis, array $axispos, array $datasets, array $maxes): string
     {
-        $labeld = '&chxt=x,y' . '&chxr=0,0,1|1,0,' . $maxes[0];
+        $labeld = '&chxt=x,y'.'&chxr=0,0,1|1,0,'.$maxes[0];
         if (count($datasets) > 1) {
             if (count($datasets) !== count($maxes)) {
                 throw new \Exception('Incorrect number of max calculations for graph plotting.');
             }
-            $labeld = '&chxt=x,y,r' . '&chxr=0,0,1|1,0,' . $maxes[0] . '|2,0,' . $maxes[1];
+            $labeld = '&chxt=x,y,r'.'&chxr=0,0,1|1,0,'.$maxes[0].'|2,0,'.$maxes[1];
         }
 
-        $url = 'https://chart.apis.google.com/chart?' .
+        $url = 'https://chart.apis.google.com/chart?'.
             // Dimension of graph. Default is 800x350
-            'chs=' . $this->x . 'x' . $this->y .
+            'chs='.$this->x.'x'.$this->y.
 
             // Dateset values
-            '&chd=' . $this->encodedata($datasets) .
+            '&chd='.$this->encodedata($datasets).
 
             // Fill area...
-            '&chco=ff5c00,cca600' .
-            '&chls=1,1,0|1,6,3' .
+            '&chco=ff5c00,cca600'.
+            '&chls=1,1,0|1,6,3'.
 
             // chart type is linechart
-            '&cht=lc' .
-            $labeld .
-            '&chxl=0:|' . $this->encodeaxis($axis) . #.$'|1:||top' .
-            '&chxp=0,' . join(',', $axispos) .
-            '&chg=' . (2400 / (count($datasets[0]) - 1)) . ',-1,3,3'; // lines
+            '&cht=lc'.
+            $labeld.
+            '&chxl=0:|'.$this->encodeaxis($axis).#.$'|1:||top' .
+            '&chxp=0,'.join(',', $axispos).
+            '&chg='.(2400 / (count($datasets[0]) - 1)).',-1,3,3'; // lines
 
         return $url;
     }
@@ -131,18 +131,18 @@ 
      */
     public function showPie(array $axis, array $datasets): string
     {
-        $url = 'https://chart.apis.google.com/chart?' .
+        $url = 'https://chart.apis.google.com/chart?'.
 
         // Dimension of graph. Default is 800x350
-        'chs=' . $this->x . 'x' . $this->y .
+        'chs='.$this->x.'x'.$this->y.
 
         // Dateset values.
-        '&chd=' . $this->encodedata([$datasets]) .
+        '&chd='.$this->encodedata([$datasets]).
 
         // chart type is linechart
-        '&cht=p' .
+        '&cht=p'.
 
-        '&chl=' . $this->encodeaxis($axis);
+        '&chl='.$this->encodeaxis($axis);
 
         return $url;
     }

src/Ruleset.php

Spacing +2 added lines, -2 removed lines

@@ -51,7 +51,7 @@ 
          * Walk through file lists, and get available [rule][fileslot]...
          */
         if (!is_dir($statdir)) {
-            throw new Exception('Statisics output directory [' . $statdir . '] does not exist.');
+            throw new Exception('Statisics output directory ['.$statdir.'] does not exist.');
         }
         $filelist = scandir($statdir);
         $this->available = [];
@@ -65,7 +65,7 @@ 
             }
         }
         if (empty($this->available)) {
-            throw new Exception('No aggregated statistics files found in [' . $statdir . ']');
+            throw new Exception('No aggregated statistics files found in ['.$statdir.']');
         }
 
         /**

src/DateHandlerMonth.php

Spacing +1 added lines, -1 removed lines

@@ -57,6 +57,6 @@
     {
         $month = ($from % 12) + 1;
         $year = 2000 + intval(floor($from / 12));
-        return $year . '-' . $month;
+        return $year.'-'.$month;
     }
 }

src/Aggregator.php

Spacing +14 added lines, -14 removed lines

@@ -62,9 +62,9 @@ 
      */
     public function dumpConfig(): void
     {
-        echo 'Statistics directory   : ' . $this->statdir . "\n";
-        echo 'Input file             : ' . $this->inputfile . "\n";
-        echo 'Offset                 : ' . $this->offset . "\n";
+        echo 'Statistics directory   : '.$this->statdir."\n";
+        echo 'Input file             : '.$this->inputfile."\n";
+        echo 'Offset                 : '.$this->offset."\n";
     }
 
 
@@ -73,7 +73,7 @@ 
     public function debugInfo(): void
     {
         // 1024*1024=1048576
-        echo 'Memory usage           : ' . number_format(memory_get_usage() / 1048576, 2) . " MB\n";
+        echo 'Memory usage           : '.number_format(memory_get_usage() / 1048576, 2)." MB\n";
     }
 
 
@@ -81,7 +81,7 @@ 
      */
     public function loadMetadata(): void
     {
-        $filename = $this->statdir . '/.stat.metadata';
+        $filename = $this->statdir.'/.stat.metadata';
         $metadata = null;
         if (file_exists($filename)) {
             $metadata = unserialize(file_get_contents($filename));
@@ -107,7 +107,7 @@ 
         $this->metadata['memory'] = memory_get_usage();
         $this->metadata['lastrun'] = time();
 
-        $filename = $this->statdir . '/.stat.metadata';
+        $filename = $this->statdir.'/.stat.metadata';
         file_put_contents($filename, serialize($this->metadata), LOCK_EX);
     }
 
@@ -122,17 +122,17 @@ 
         $this->loadMetadata();
 
         if (!is_dir($this->statdir)) {
-            throw new Exception('Statistics module: output dir does not exist [' . $this->statdir . ']');
+            throw new Exception('Statistics module: output dir does not exist ['.$this->statdir.']');
         }
 
         if (!file_exists($this->inputfile)) {
-            throw new Exception('Statistics module: input file does not exist [' . $this->inputfile . ']');
+            throw new Exception('Statistics module: input file does not exist ['.$this->inputfile.']');
         }
 
         $file = fopen($this->inputfile, 'r');
 
         if ($file === false) {
-            throw new Exception('Statistics module: unable to open file [' . $this->inputfile . ']');
+            throw new Exception('Statistics module: unable to open file ['.$this->inputfile.']');
         }
 
         $logparser = new LogParser(
@@ -177,14 +177,14 @@ 
             $action = trim($content[5]);
 
             if ($this->fromcmdline && ($i % 10000) == 0) {
-                echo "Read line " . $i . "\n";
+                echo "Read line ".$i."\n";
             }
 
             if ($debug) {
                 echo "----------------------------------------\n";
-                echo 'Log line: ' . $logline . "\n";
-                echo 'Date parse [' . substr($logline, 0, $this->statconfig->getOptionalValue('datelength', 15)) .
-                    '] to [' . date(DATE_RFC822, $epoch) . ']' . "\n";
+                echo 'Log line: '.$logline."\n";
+                echo 'Date parse ['.substr($logline, 0, $this->statconfig->getOptionalValue('datelength', 15)).
+                    '] to ['.date(DATE_RFC822, $epoch).']'."\n";
                 $ret = print_r($content, true);
                 echo htmlentities($ret);
                 if ($i >= 13) {
@@ -355,7 +355,7 @@ 
                         }
                     }
 
-                    $filename = $this->statdir . '/' . $rulename . '-' . $tres . '-' . $fileno . '.stat';
+                    $filename = $this->statdir.'/'.$rulename.'-'.$tres.'-'.$fileno.'.stat';
                     if (file_exists($filename)) {
                         $previousData = unserialize(file_get_contents($filename));
                         $filledresult = $this->cummulateData($previousData, $filledresult);

src/Controller/Statistics.php

Spacing +2 added lines, -2 removed lines

@@ -181,7 +181,7 @@ 
                     $data = $dataset->getDebugData();
                     foreach ($data as $de) {
                         if (isset($de[1])) {
-                            echo '"' . $de[0] . '",' . $de[1] . "\n";
+                            echo '"'.$de[0].'",'.$de[1]."\n";
                         }
                     }
                     exit;
@@ -239,7 +239,7 @@ 
             }
         }
         if ($type === 'get') {
-            return Module::getModuleURL("statistics/") . '?' . http_build_query($vars, '', '&');
+            return Module::getModuleURL("statistics/").'?'.http_build_query($vars, '', '&');
         }
         return $vars;
     }

src/LogCleaner.php

Spacing +13 added lines, -13 removed lines

@@ -50,9 +50,9 @@ 
      */
     public function dumpConfig(): void
     {
-        echo 'Statistics directory   : ' . $this->statdir . "\n";
-        echo 'Input file             : ' . $this->inputfile . "\n";
-        echo 'Offset                 : ' . $this->offset . "\n";
+        echo 'Statistics directory   : '.$this->statdir."\n";
+        echo 'Input file             : '.$this->inputfile."\n";
+        echo 'Offset                 : '.$this->offset."\n";
     }
 
 
@@ -64,11 +64,11 @@ 
     public function clean(bool $debug = false): array
     {
         if (!is_dir($this->statdir)) {
-            throw new Exception('Statistics module: output dir does not exist [' . $this->statdir . ']');
+            throw new Exception('Statistics module: output dir does not exist ['.$this->statdir.']');
         }
 
         if (!file_exists($this->inputfile)) {
-            throw new Exception('Statistics module: input file does not exist [' . $this->inputfile . ']');
+            throw new Exception('Statistics module: input file does not exist ['.$this->inputfile.']');
         }
 
         $file = fopen($this->inputfile, 'r');
@@ -97,7 +97,7 @@ 
             $content = $logparser->parseContent($logline);
 
             if (($i % 10000) == 0) {
-                echo "Read line " . $i . "\n";
+                echo "Read line ".$i."\n";
             }
 
             $trackid = $content[4];
@@ -109,9 +109,9 @@ 
 
             if ($debug) {
                 echo "----------------------------------------\n";
-                echo 'Log line: ' . $logline . "\n";
-                echo 'Date parse [' . substr($logline, 0, $this->statconfig->getOptionalValue('datelength', 15)) .
-                    '] to [' . date(DATE_RFC822, $epoch) . ']' . "\n";
+                echo 'Log line: '.$logline."\n";
+                echo 'Date parse ['.substr($logline, 0, $this->statconfig->getOptionalValue('datelength', 15)).
+                    '] to ['.date(DATE_RFC822, $epoch).']'."\n";
                 $ret = print_r($content, true);
                 echo htmlentities($ret);
                 if ($i >= 13) {
@@ -147,14 +147,14 @@ 
      */
     public function store(array $todelete, string $outputfile): void
     {
-        echo "Preparing to delete [" . count($todelete) . "] trackids\n";
+        echo "Preparing to delete [".count($todelete)."] trackids\n";
 
         if (!is_dir($this->statdir)) {
-            throw new Exception('Statistics module: output dir do not exists [' . $this->statdir . ']');
+            throw new Exception('Statistics module: output dir do not exists ['.$this->statdir.']');
         }
 
         if (!file_exists($this->inputfile)) {
-            throw new Exception('Statistics module: input file do not exists [' . $this->inputfile . ']');
+            throw new Exception('Statistics module: input file do not exists ['.$this->inputfile.']');
         }
 
         $file = fopen($this->inputfile, 'r');
@@ -186,7 +186,7 @@ 
             $content = $logparser->parseContent($logline);
 
             if (($i % 10000) == 0) {
-                echo "Read line " . $i . "\n";
+                echo "Read line ".$i."\n";
             }
 
             $trackid = $content[4];