silverstripe / silverstripe-environmentcheck

src/EnvironmentCheckSuite.php

Spacing +1 added lines, -1 removed lines

@@ -150,7 +150,7 @@
             } elseif ($checkClass instanceof EnvironmentCheck) {
                 $output[] = [$checkClass, $checkTitle];
             } else {
-                throw new InvalidArgumentException("Bad EnvironmentCheck: " . var_export($check, true));
+                throw new InvalidArgumentException("Bad EnvironmentCheck: ".var_export($check, true));
             }
         }
         return $output;

src/Checks/HasFunctionCheck.php

Spacing +2 added lines, -2 removed lines

@@ -32,8 +32,8 @@
     public function check()
     {
         if (function_exists($this->functionName)) {
-            return [EnvironmentCheck::OK, $this->functionName . '() exists'];
+            return [EnvironmentCheck::OK, $this->functionName.'() exists'];
         }
-        return [EnvironmentCheck::ERROR, $this->functionName . '() doesn\'t exist'];
+        return [EnvironmentCheck::ERROR, $this->functionName.'() doesn\'t exist'];
     }
 }

src/Checks/FileAccessibilityAndValidationCheck.php

Spacing +4 added lines, -4 removed lines

@@ -67,7 +67,7 @@ 
     public function __construct($path, $fileTypeValidateFunc = 'noVidation', $checkType = null)
     {
         $this->path = $path;
-        $this->fileTypeValidateFunc = ($fileTypeValidateFunc)? $fileTypeValidateFunc : 'noVidation';
+        $this->fileTypeValidateFunc = ($fileTypeValidateFunc) ? $fileTypeValidateFunc : 'noVidation';
         $this->checkType = ($checkType) ? $checkType : self::CHECK_SINGLE;
     }
 
@@ -100,7 +100,7 @@ 
                 if ($this->checkType == self::CHECK_SINGLE && count($invalidFiles) < count($files)) {
                     $validFileList = PHP_EOL;
                     foreach ($validFiles as $vf) {
-                        $validFileList .= $vf . PHP_EOL;
+                        $validFileList .= $vf.PHP_EOL;
                     }
                     if ($fileTypeValidateFunc == 'noVidation') {
                         $checkReturn = [
@@ -123,7 +123,7 @@ 
                     } else {
                         $invalidFileList = PHP_EOL;
                         foreach ($invalidFiles as $vf) {
-                            $invalidFileList .= $vf . PHP_EOL;
+                            $invalidFileList .= $vf.PHP_EOL;
                         }
 
                         if ($fileTypeValidateFunc == 'noVidation') {
@@ -144,7 +144,7 @@ 
                     }
                 }
             } else {
-                $checkReturn =  array(
+                $checkReturn = array(
                     EnvironmentCheck::ERROR,
                     sprintf("Invalid file type validation method name passed: %s ", $fileTypeValidateFunc)
                 );

src/Checks/FileAgeCheck.php

Spacing +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
      */
     public function check()
     {
-        $cutoffTime =  strtotime($this->relativeAge, DBDatetime::now()->Format('U'));
+        $cutoffTime = strtotime($this->relativeAge, DBDatetime::now()->Format('U'));
         $files = $this->getFiles();
         $invalidFiles = [];
         $validFiles = [];

src/Checks/HasClassCheck.php

Spacing +2 added lines, -2 removed lines

@@ -32,8 +32,8 @@
     public function check()
     {
         if (class_exists($this->className)) {
-            return [EnvironmentCheck::OK, 'Class ' . $this->className.' exists'];
+            return [EnvironmentCheck::OK, 'Class '.$this->className.' exists'];
         }
-        return [EnvironmentCheck::ERROR, 'Class ' . $this->className.' doesn\'t exist'];
+        return [EnvironmentCheck::ERROR, 'Class '.$this->className.' doesn\'t exist'];
     }
 }

src/Checks/FileWriteableCheck.php

Spacing +3 added lines, -3 removed lines

@@ -34,7 +34,7 @@ 
         if ($this->path[0] == '/') {
             $filename = $this->path;
         } else {
-            $filename = BASE_PATH . DIRECTORY_SEPARATOR . str_replace('/', DIRECTORY_SEPARATOR, $this->path);
+            $filename = BASE_PATH.DIRECTORY_SEPARATOR.str_replace('/', DIRECTORY_SEPARATOR, $this->path);
         }
 
         if (file_exists($filename)) {
@@ -67,8 +67,8 @@ 
                     }
                     if ($groupList) {
                         $message .= "	We recommend that you make the file group-writeable and change the group to "
-                            . "one of these groups:\n - " . implode("\n - ", $groupList)
-                            . "\n\nFor example:\nchmod g+w $filename\nchgrp " . $groupList[0] . " $filename";
+                            . "one of these groups:\n - ".implode("\n - ", $groupList)
+                            . "\n\nFor example:\nchmod g+w $filename\nchgrp ".$groupList[0]." $filename";
                     } else {
                         $message .= "  There is no user-group that contains both the web-server user and the owner "
                             . "of this file.  Change the ownership of the file, create a new group, or temporarily "

src/Checks/SolrIndexCheck.php

Spacing +2 added lines, -2 removed lines

@@ -27,7 +27,7 @@ 
         if (!class_exists(Solr::class)) {
             return [
                 EnvironmentCheck::ERROR,
-                'Class `' . Solr::class . '` not found. Is the fulltextsearch module installed?'
+                'Class `'.Solr::class.'` not found. Is the fulltextsearch module installed?'
             ];
         }
 
@@ -43,7 +43,7 @@ 
         if (!empty($brokenCores)) {
             return [
                 EnvironmentCheck::ERROR,
-                'The following indexes are unavailable: ' . implode($brokenCores, ', ')
+                'The following indexes are unavailable: '.implode($brokenCores, ', ')
             ];
         }
 

src/Checks/CacheHeadersCheck.php

Spacing +4 added lines, -4 removed lines

@@ -123,7 +123,7 @@ 
         $goodTypes = [ValidationResult::TYPE_GOOD, ValidationResult::TYPE_INFO];
         $good = array_filter(
             $this->result->getMessages(),
-            function ($val, $key) use ($goodTypes) {
+            function($val, $key) use ($goodTypes) {
                 if (in_array($val['messageType'], $goodTypes)) {
                     return true;
                 }
@@ -132,14 +132,14 @@ 
             ARRAY_FILTER_USE_BOTH
         );
         if (!empty($good)) {
-            $ret .= "GOOD: " . implode('; ', array_column($good, 'message')) . " ";
+            $ret .= "GOOD: ".implode('; ', array_column($good, 'message'))." ";
         }
 
         // Filter bad messages
         $badTypes = [ValidationResult::TYPE_ERROR, ValidationResult::TYPE_WARNING];
         $bad = array_filter(
             $this->result->getMessages(),
-            function ($val, $key) use ($badTypes) {
+            function($val, $key) use ($badTypes) {
                 if (in_array($val['messageType'], $badTypes)) {
                     return true;
                 }
@@ -148,7 +148,7 @@ 
             ARRAY_FILTER_USE_BOTH
         );
         if (!empty($bad)) {
-            $ret .= "BAD: " . implode('; ', array_column($bad, 'message'));
+            $ret .= "BAD: ".implode('; ', array_column($bad, 'message'));
         }
         return $ret;
     }

src/Checks/SessionCheck.php

Spacing +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
         if ($cookie) {
             return [
                 EnvironmentCheck::ERROR,
-                "Sessions are being set for {$fullURL} : Set-Cookie => " . $cookie,
+                "Sessions are being set for {$fullURL} : Set-Cookie => ".$cookie,
             ];
         }
         return [