Completed
Pull Request — master (#34)
by Stig
04:28 queued 02:13
created
code/EnvironmentChecker.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -257,7 +257,7 @@
 block discarded – undo
257 257
     public static function set_to_email_address($to)
258 258
     {
259 259
         Deprecation::notice('2.0', 'Use config API instead');
260
-        Config::inst()->update('EnvironmentChecker', 'to_email_address',  $to);
260
+        Config::inst()->update('EnvironmentChecker', 'to_email_address', $to);
261 261
     }
262 262
 
263 263
     /**
Please login to merge, or discard this patch.
code/checks/FileAgeCheck.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -89,7 +89,7 @@
 block discarded – undo
89 89
      */
90 90
     public function check()
91 91
     {
92
-        $cutoffTime =  strtotime($this->relativeAge, SS_Datetime::now()->Format('U'));
92
+        $cutoffTime = strtotime($this->relativeAge, SS_Datetime::now()->Format('U'));
93 93
         $files = $this->getFiles();
94 94
         $invalidFiles = array();
95 95
         $validFiles = array();
Please login to merge, or discard this patch.
code/EnvironmentCheckSuite.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -77,7 +77,7 @@  discard block
 block discarded – undo
77 77
 
78 78
             // Existing named checks can be disabled by setting their 'state' to 'disabled'.
79 79
             // This is handy for disabling checks mandated by modules.
80
-            if (!empty($check['state']) && $check['state']==='disabled') {
80
+            if (!empty($check['state']) && $check['state'] === 'disabled') {
81 81
                 continue;
82 82
             }
83 83
             
@@ -130,7 +130,7 @@  discard block
 block discarded – undo
130 130
             } elseif ($checkClass instanceof EnvironmentCheck) {
131 131
                 $output[] = array($checkClass, $checkTitle);
132 132
             } else {
133
-                throw new InvalidArgumentException("Bad EnvironmentCheck: " . var_export($check, true));
133
+                throw new InvalidArgumentException("Bad EnvironmentCheck: ".var_export($check, true));
134 134
             }
135 135
         }
136 136
         return $output;
Please login to merge, or discard this patch.
code/checks/FileWriteableCheck.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
         if ($this->path[0] == '/') {
29 29
             $filename = $this->path;
30 30
         } else {
31
-            $filename = BASE_PATH . DIRECTORY_SEPARATOR . str_replace('/', DIRECTORY_SEPARATOR, $this->path);
31
+            $filename = BASE_PATH.DIRECTORY_SEPARATOR.str_replace('/', DIRECTORY_SEPARATOR, $this->path);
32 32
         }
33 33
         
34 34
         if (file_exists($filename)) {
@@ -59,8 +59,8 @@  discard block
 block discarded – undo
59 59
                         }
60 60
                     }
61 61
                     if ($groupList) {
62
-                        $message .= "	We recommend that you make the file group-writeable and change the group to one of these groups:\n - ". implode("\n - ", $groupList)
63
-                            . "\n\nFor example:\nchmod g+w $filename\nchgrp " . $groupList[0] . " $filename";
62
+                        $message .= "	We recommend that you make the file group-writeable and change the group to one of these groups:\n - ".implode("\n - ", $groupList)
63
+                            . "\n\nFor example:\nchmod g+w $filename\nchgrp ".$groupList[0]." $filename";
64 64
                     } else {
65 65
                         $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 make the file writeable by everyone during the install process.";
66 66
                     }
@@ -72,6 +72,6 @@  discard block
 block discarded – undo
72 72
             return array(EnvironmentCheck::ERROR, $message);
73 73
         }
74 74
 
75
-        return array(EnvironmentCheck::OK,'');
75
+        return array(EnvironmentCheck::OK, '');
76 76
     }
77 77
 }
Please login to merge, or discard this patch.
code/checks/HasClassCheck.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -26,9 +26,9 @@
 block discarded – undo
26 26
     public function check()
27 27
     {
28 28
         if (class_exists($this->className)) {
29
-            return array(EnvironmentCheck::OK, 'Class ' . $this->className.' exists');
29
+            return array(EnvironmentCheck::OK, 'Class '.$this->className.' exists');
30 30
         } else {
31
-            return array(EnvironmentCheck::ERROR, 'Class ' . $this->className.' doesn\'t exist');
31
+            return array(EnvironmentCheck::ERROR, 'Class '.$this->className.' doesn\'t exist');
32 32
         }
33 33
     }
34 34
 }
Please login to merge, or discard this patch.
code/checks/FileAccessibilityAndValidationCheck.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
     public function __construct($path, $fileTypeValidateFunc = 'noVidation', $checkType = null)
61 61
     {
62 62
         $this->path = $path;
63
-        $this->fileTypeValidateFunc = ($fileTypeValidateFunc)? $fileTypeValidateFunc:'noVidation';
63
+        $this->fileTypeValidateFunc = ($fileTypeValidateFunc) ? $fileTypeValidateFunc : 'noVidation';
64 64
         $this->checkType = ($checkType) ? $checkType : self::CHECK_SINGLE;
65 65
     }
66 66
 
@@ -129,7 +129,7 @@  discard block
 block discarded – undo
129 129
                     }
130 130
                 }
131 131
             } else {
132
-                $checkReturn =  array(
132
+                $checkReturn = array(
133 133
                     EnvironmentCheck::ERROR,
134 134
                     sprintf("Invalid file type validation method name passed: %s ", $fileTypeValidateFunc)
135 135
                 );
Please login to merge, or discard this patch.
code/checks/SolrIndexCheck.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@
 block discarded – undo
47 47
         if (!empty($brokenCores)) {
48 48
             return array(
49 49
                 EnvironmentCheck::ERROR,
50
-                'The following indexes are unavailable: ' . implode($brokenCores, ', ')
50
+                'The following indexes are unavailable: '.implode($brokenCores, ', ')
51 51
             );
52 52
         }
53 53
 
Please login to merge, or discard this patch.