simplesamlphp / simplesamlphp-module-statistics

lib/LogCleaner.php

Spacing +13 added lines, -13 removed lines

@@ -52,9 +52,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";
     }
 
 
@@ -66,11 +66,11 @@ 
     public function clean(bool $debug = false): array
     {
         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');
@@ -99,7 +99,7 @@ 
             $content = $logparser->parseContent($logline);
 
             if (($i % 10000) == 0) {
-                echo "Read line " . $i . "\n";
+                echo "Read line ".$i."\n";
             }
 
             $trackid = $content[4];
@@ -111,9 +111,9 @@ 
 
             if ($debug) {
                 echo "----------------------------------------\n";
-                echo 'Log line: ' . $logline . "\n";
-                echo 'Date parse [' . substr($logline, 0, $this->statconfig->getValue('datelength', 15)) .
-                    '] to [' . date(DATE_RFC822, $epoch) . ']' . "\n";
+                echo 'Log line: '.$logline."\n";
+                echo 'Date parse ['.substr($logline, 0, $this->statconfig->getValue('datelength', 15)).
+                    '] to ['.date(DATE_RFC822, $epoch).']'."\n";
                 $ret = print_r($content, true);
                 echo htmlentities($ret);
                 if ($i >= 13) {
@@ -150,14 +150,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');
@@ -189,7 +189,7 @@ 
             $content = $logparser->parseContent($logline);
 
             if (($i % 10000) == 0) {
-                echo "Read line " . $i . "\n";
+                echo "Read line ".$i."\n";
             }
 
             $trackid = $content[4];

lib/ACL.php

Spacing +4 added lines, -4 removed lines

@@ -39,7 +39,7 @@ 
 
         foreach ($acl as $rule) {
             if (!is_array($rule)) {
-                throw new Error\Exception('Invalid rule in access control list: ' . var_export($rule, true));
+                throw new Error\Exception('Invalid rule in access control list: '.var_export($rule, true));
             }
             if (count($rule) === 0) {
                 throw new Error\Exception('Empty rule in access control list.');
@@ -48,7 +48,7 @@ 
             $action = array_shift($rule);
             if ($action !== 'allow' && $action !== 'deny') {
                 throw new Error\Exception(
-                    'Invalid action in rule in access control list: ' . var_export($action, true)
+                    'Invalid action in rule in access control list: '.var_export($action, true)
                 );
             }
         }
@@ -65,7 +65,7 @@ 
     {
         $config = Configuration::getOptionalConfig('acl.php');
         if (!$config->hasValue($id)) {
-            throw new Error\Exception('No ACL with id ' . var_export($id, true) . ' in config/acl.php.');
+            throw new Error\Exception('No ACL with id '.var_export($id, true).' in config/acl.php.');
         }
 
         return $config->getArray($id);
@@ -128,7 +128,7 @@ 
             case 'or':
                 return self::opOr($attributes, $rule);
             default:
-                throw new Error\Exception('Invalid ACL operation: ' . var_export($op, true));
+                throw new Error\Exception('Invalid ACL operation: '.var_export($op, true));
         }
     }