Passed
Pull Request — master (#21)
by Tim
07:33 queued 03:26
created
src/AccessCheck.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
         $as->requireAuth();
61 61
 
62 62
         // User logged in with auth source.
63
-        Logger::debug('Statistics auth - valid login with auth source [' . $authsource . ']');
63
+        Logger::debug('Statistics auth - valid login with auth source ['.$authsource.']');
64 64
 
65 65
         // Retrieving attributes
66 66
         $attributes = $as->getAttributes();
@@ -74,12 +74,12 @@  discard block
 block discarded – undo
74 74
             // Check if userid is allowed access..
75 75
             if (in_array($attributes[$useridattr][0], $allowedusers, true)) {
76 76
                 Logger::debug(
77
-                    'Statistics auth - User granted access by user ID [' . $attributes[$useridattr][0] . ']',
77
+                    'Statistics auth - User granted access by user ID ['.$attributes[$useridattr][0].']',
78 78
                 );
79 79
                 return;
80 80
             }
81 81
             Logger::debug(
82
-                'Statistics auth - User denied access by user ID [' . $attributes[$useridattr][0] . ']',
82
+                'Statistics auth - User denied access by user ID ['.$attributes[$useridattr][0].']',
83 83
             );
84 84
         } else {
85 85
             Logger::debug('Statistics auth - no allowedUsers list.');
Please login to merge, or discard this patch.
src/StatDataset.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -78,7 +78,7 @@  discard block
 block discarded – undo
78 78
 
79 79
         $timeresconfig = $timeresconfigs->getOptionalConfigItem($timeres, null);
80 80
         if ($timeresconfig === null) {
81
-            throw new Error\ConfigurationError('Missing \'timeres.' . $timeres . '\' in module configuration.');
81
+            throw new Error\ConfigurationError('Missing \'timeres.'.$timeres.'\' in module configuration.');
82 82
         }
83 83
 
84 84
         $this->timeresconfig = $timeresconfig;
@@ -359,7 +359,7 @@  discard block
 block discarded – undo
359 359
                 'Statistics\FieldPresentation',
360 360
             );
361 361
             if (!class_exists($classname)) {
362
-                throw new Exception('Could not find field presentation plugin [' . $classname . ']: No class found');
362
+                throw new Exception('Could not find field presentation plugin ['.$classname.']: No class found');
363 363
             }
364 364
             $presentationHandler = new $classname($availdelimiters, $fieldpresConfig->getValue('config'), $t);
365 365
 
@@ -400,17 +400,17 @@  discard block
 block discarded – undo
400 400
         $rules = $this->ruleid;
401 401
         foreach ($rules as $rule) {
402 402
             // Get file and extract results.
403
-            $resultFileName = $statdir . '/' . $rule . '-' . $this->timeres . '-' . $this->fileslot . '.stat';
403
+            $resultFileName = $statdir.'/'.$rule.'-'.$this->timeres.'-'.$this->fileslot.'.stat';
404 404
             if (!file_exists($resultFileName)) {
405
-                throw new Exception('Aggregated statitics file [' . $resultFileName . '] not found.');
405
+                throw new Exception('Aggregated statitics file ['.$resultFileName.'] not found.');
406 406
             }
407 407
             if (!is_readable($resultFileName)) {
408
-                throw new Exception('Could not read statitics file [' . $resultFileName . ']. Bad file permissions?');
408
+                throw new Exception('Could not read statitics file ['.$resultFileName.']. Bad file permissions?');
409 409
             }
410 410
             $resultfile = file_get_contents($resultFileName);
411 411
             $newres = unserialize($resultfile);
412 412
             if (empty($newres)) {
413
-                throw new Exception('Aggregated statistics in file [' . $resultFileName . '] was empty.');
413
+                throw new Exception('Aggregated statistics in file ['.$resultFileName.'] was empty.');
414 414
             }
415 415
             $resarray[] = $newres;
416 416
         }
Please login to merge, or discard this patch.
src/ACL.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -74,7 +74,7 @@  discard block
 block discarded – undo
74 74
     {
75 75
         $config = Configuration::getOptionalConfig('acl.php');
76 76
         if (!$config->hasValue($id)) {
77
-            throw new Error\Exception('No ACL with id ' . var_export($id, true) . ' in config/acl.php.');
77
+            throw new Error\Exception('No ACL with id '.var_export($id, true).' in config/acl.php.');
78 78
         }
79 79
 
80 80
         return $config->getArray($id);
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
             case 'or':
138 138
                 return self::opOr($attributes, $rule);
139 139
             default:
140
-                throw new Error\Exception('Invalid ACL operation: ' . var_export($op, true));
140
+                throw new Error\Exception('Invalid ACL operation: '.var_export($op, true));
141 141
         }
142 142
     }
143 143
 
Please login to merge, or discard this patch.