Test Failed
Push — 1.0.0-dev ( c68446...90e1c2 )
by nguereza
02:35
created
core/classes/Router.php 1 patch
Indentation   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -277,14 +277,14 @@  discard block
 block discarded – undo
277 277
         }
278 278
 
279 279
         /**
280
-        * Setting the route configuration using the configuration file 
281
-        * and additional configuration from param
282
-        * @param array $overwriteConfig the additional configuration 
283
-        * to overwrite with the existing one
284
-        * @param boolean $useConfigFile whether to use route configuration file
285
-        * 
286
-        * @return object
287
-        */
280
+         * Setting the route configuration using the configuration file 
281
+         * and additional configuration from param
282
+         * @param array $overwriteConfig the additional configuration 
283
+         * to overwrite with the existing one
284
+         * @param boolean $useConfigFile whether to use route configuration file
285
+         * 
286
+         * @return object
287
+         */
288 288
         public function setRouteConfiguration(array $overwriteConfig = array(), $useConfigFile = true) {
289 289
             $route = array();
290 290
             if ($useConfigFile && file_exists(CONFIG_PATH . 'routes.php')) {
@@ -365,7 +365,7 @@  discard block
 block discarded – undo
365 365
             //the URL like http://domain.com/module/controller/method/arg1/arg2
366 366
             if (!$this->controller) {
367 367
                 $this->logger->info('Cannot determine the routing information ' 
368
-                       . 'using the predefined routes configuration, will use the request URI parameters');
368
+                        . 'using the predefined routes configuration, will use the request URI parameters');
369 369
                 //determine route parameters using the route URI param
370 370
                 $this->determineRouteParamsFromRequestUri();
371 371
             }
@@ -478,11 +478,11 @@  discard block
 block discarded – undo
478 478
             }
479 479
         }
480 480
 
481
-         /**
482
-         * Remove the URL suffix and query string values if exists
483
-         * @param  string $uri the route URI to process
484
-         * @return string      the final route uri after processed
485
-         */
481
+            /**
482
+             * Remove the URL suffix and query string values if exists
483
+             * @param  string $uri the route URI to process
484
+             * @return string      the final route uri after processed
485
+             */
486 486
         protected function removeSuffixAndQueryStringFromUri($uri) {
487 487
             $this->logger->debug('Check if URL suffix is enabled in the configuration');
488 488
             //remove url suffix from the request URI
@@ -596,7 +596,7 @@  discard block
 block discarded – undo
596 596
                 $this->logger->info(
597 597
                                     'After loop in predefined routes configuration,'
598 598
                                     . 'the module name is set but the controller is not set,' 
599
-									. 'so we will use module as the controller'
599
+                                    . 'so we will use module as the controller'
600 600
                                 );
601 601
                 $this->controller = $this->module;
602 602
             }
Please login to merge, or discard this patch.
app/config/config.php 1 patch
Indentation   +34 added lines, -34 removed lines patch added patch discarded remove patch
@@ -134,26 +134,26 @@  discard block
 block discarded – undo
134 134
 	*/
135 135
 	
136 136
     /** 
137
-    * The log level
138
-    *
139
-    * The valid log level are: OFF, NONE, DEBUG, INFO, NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY
140
-    *
141
-    * 'OFF' or 'NONE' = do not save log
142
-    * 'EMERGENCY' = enable log for emergency level and above (EMERGENCY)
143
-    * 'ALERT' = enable log for alert level and above (ALERT, EMERGENCY)
144
-    * 'CRITICAL' = enable log for critical level and above (CRITICAL, ALERT, EMERGENCY)
145
-    * 'ERROR' = enable log for error level and above (ERROR, CRITICAL, ALERT, EMERGENCY)
146
-    * 'WARNING' = enable log for warning level and above (WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
147
-    * 'NOTICE' = enable log for notice level and above (NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
148
-    * 'INFO' = enable log for info level and above (INFO, NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
149
-    * 'DEBUG' = enable log for debug level and above (DEBUG, INFO, NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
150
-    *
151
-    * The default value is NONE if the config value is: null, '', 0, false
152
-    * 
153
-    * Note: in production environment it's recommand to set the log level to 'WARNING' or 'ERROR' if not, in small
154
-    * time the log file size will increase very fast and will cost the application performance
155
-    * and also the filesystem usage of your server.
156
-    */
137
+     * The log level
138
+     *
139
+     * The valid log level are: OFF, NONE, DEBUG, INFO, NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY
140
+     *
141
+     * 'OFF' or 'NONE' = do not save log
142
+     * 'EMERGENCY' = enable log for emergency level and above (EMERGENCY)
143
+     * 'ALERT' = enable log for alert level and above (ALERT, EMERGENCY)
144
+     * 'CRITICAL' = enable log for critical level and above (CRITICAL, ALERT, EMERGENCY)
145
+     * 'ERROR' = enable log for error level and above (ERROR, CRITICAL, ALERT, EMERGENCY)
146
+     * 'WARNING' = enable log for warning level and above (WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
147
+     * 'NOTICE' = enable log for notice level and above (NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
148
+     * 'INFO' = enable log for info level and above (INFO, NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
149
+     * 'DEBUG' = enable log for debug level and above (DEBUG, INFO, NOTICE, WARNING, ERROR, CRITICAL, ALERT, EMERGENCY)
150
+     *
151
+     * The default value is NONE if the config value is: null, '', 0, false
152
+     * 
153
+     * Note: in production environment it's recommand to set the log level to 'WARNING' or 'ERROR' if not, in small
154
+     * time the log file size will increase very fast and will cost the application performance
155
+     * and also the filesystem usage of your server.
156
+     */
157 157
     $config['log_level'] = 'NONE';
158 158
 
159 159
 
@@ -180,20 +180,20 @@  discard block
 block discarded – undo
180 180
     $config['log_logger_name'] = array();
181 181
 
182 182
     /**
183
-    * The logger name custom level to use for the log
184
-    * 
185
-    * If this config is set so means the logger level will be used to overwrite 
186
-    * the default log level configuration above. 
187
-    *
188
-    * Example:
189
-    * $config['log_logger_name_level'] = array('MY_LOGGER1' => 'WARNING'); 
190
-    * So if $config['log_level'] = 'ERROR' but all log messages with "MY_LOGGER1" as logger name
191
-    *  will be saved for WARNING message and above
192
-    *  Note: You can also use an regular expression for the logger name.
193
-    *  Example:
194
-    *  $config['log_logger_name_level'] = array('^Class::Con(.*)' => 'info');
195
-    *  So all logger name like "Class::Config", "Class::Cookie", etc. will be match
196
-    */  
183
+     * The logger name custom level to use for the log
184
+     * 
185
+     * If this config is set so means the logger level will be used to overwrite 
186
+     * the default log level configuration above. 
187
+     *
188
+     * Example:
189
+     * $config['log_logger_name_level'] = array('MY_LOGGER1' => 'WARNING'); 
190
+     * So if $config['log_level'] = 'ERROR' but all log messages with "MY_LOGGER1" as logger name
191
+     *  will be saved for WARNING message and above
192
+     *  Note: You can also use an regular expression for the logger name.
193
+     *  Example:
194
+     *  $config['log_logger_name_level'] = array('^Class::Con(.*)' => 'info');
195
+     *  So all logger name like "Class::Config", "Class::Cookie", etc. will be match
196
+     */  
197 197
     $config['log_logger_name_level'] = array();
198 198
 	
199 199
 	
Please login to merge, or discard this patch.
core/classes/Config.php 1 patch
Indentation   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -46,9 +46,9 @@  discard block
 block discarded – undo
46 46
             if ($init) {
47 47
                 $this->init();
48 48
                 //@codeCoverageIgnoreStart
49
-                 if (ENVIRONMENT == 'production' && in_array(strtolower($this->config['log_level']), array('debug', 'info', 'all'))) {
49
+                    if (ENVIRONMENT == 'production' && in_array(strtolower($this->config['log_level']), array('debug', 'info', 'all'))) {
50 50
                     $this->logger->warning('You are in production environment, please set '
51
-                                           . 'log level to WARNING, ERROR, CRITICAL, ALERT, EMERGENCY to increase the application performance');
51
+                                            . 'log level to WARNING, ERROR, CRITICAL, ALERT, EMERGENCY to increase the application performance');
52 52
                 }
53 53
                 //@codeCoverageIgnoreEnd
54 54
             }
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
146 146
             if (empty($this->config['base_url'])) {
147 147
                 if (ENVIRONMENT == 'production') {
148 148
                     $this->logger->warning('Application base URL is not set or invalid, please'
149
-                                           . ' set application base URL to increase the application loading time');
149
+                                            . ' set application base URL to increase the application loading time');
150 150
                 }
151 151
                 $baseUrl = null;
152 152
                 $protocol = 'http';
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
                                                                     );
174 174
                 } else {
175 175
                     $this->logger->warning('Can not determine the application '
176
-                                           . 'base URL automatically, use http://localhost as default');
176
+                                            . 'base URL automatically, use http://localhost as default');
177 177
                     $baseUrl = 'http://localhost/';
178 178
                 }
179 179
                 $this->config['base_url'] = $baseUrl;
@@ -182,11 +182,11 @@  discard block
 block discarded – undo
182 182
         }
183 183
          
184 184
         /**
185
-        * Return the server port using variable
186
-        *
187
-        * @codeCoverageIgnore
188
-        * @return string
189
-        */
185
+         * Return the server port using variable
186
+         *
187
+         * @codeCoverageIgnore
188
+         * @return string
189
+         */
190 190
         protected function getServerPort() {
191 191
             $globals = & class_loader('GlobalVar', 'classes');
192 192
             $serverPort = $globals->server('SERVER_PORT');
Please login to merge, or discard this patch.
core/classes/Log.php 1 patch
Indentation   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
             $this->log(self::DEBUG, $message);
170 170
         } 
171 171
 		
172
-		/**
172
+        /**
173 173
          * Logs with an arbitrary level.
174 174
          *
175 175
          * @param  integer|string $level   the log level in integer or string format,
@@ -187,11 +187,11 @@  discard block
 block discarded – undo
187 187
                 //NOTE: here need put the show_error() "logging" to false 
188 188
                 //to prevent self function loop call
189 189
                 show_error('Invalid config log level [' . $configLogLevel . '], '
190
-                           . 'the value must be one of the following: ' 
191
-                           . implode(', ', array_map('strtoupper', self::$validConfigLevel))
192
-                           , 'Log Config Error', 
193
-                           $logging = false
194
-                       );
190
+                            . 'the value must be one of the following: ' 
191
+                            . implode(', ', array_map('strtoupper', self::$validConfigLevel))
192
+                            , 'Log Config Error', 
193
+                            $logging = false
194
+                        );
195 195
                 return;	
196 196
             }
197 197
 			
@@ -261,7 +261,7 @@  discard block
 block discarded – undo
261 261
         protected function levelCanSaveLog($level) {
262 262
             $result = true;
263 263
             $configLogLevel = get_config('log_level');
264
-             //check if can save log regarding the log level configuration
264
+                //check if can save log regarding the log level configuration
265 265
             $configLevel = self::getLevelValue($configLogLevel);
266 266
             if ($configLevel > $level) {
267 267
                 //can't log
@@ -341,7 +341,7 @@  discard block
 block discarded – undo
341 341
                 //NOTE: here need put the show_error() "logging" to false 
342 342
                 //to prevent self function loop call
343 343
                 show_error('Error : the log dir does not exist or is not writable',
344
-                           'Log directory error', $logging = false);
344
+                            'Log directory error', $logging = false);
345 345
             }
346 346
 			
347 347
             $path = $logSavePath . 'logs-' . date('Y-m-d') . '.log';
Please login to merge, or discard this patch.
core/classes/cache/ApcCache.php 1 patch
Indentation   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
         /**
34 34
          * Construct new ApcCache instance
35 35
          */
36
-	public function __construct() {
36
+    public function __construct() {
37 37
             parent::__construct();
38 38
             if (!$this->isSupported()) {
39 39
                 show_error('The cache for APC[u] driver is not available. Check if APC[u] extension is loaded and enabled.');
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
         public function set($key, $data, $ttl = 0) {
80 80
             $expire = time() + $ttl;
81 81
             $this->logger->debug('Setting cache data for key [' . $key . '], time to live [' . $ttl . '], '
82
-                                 . 'expire at [' . date('Y-m-d H:i:s', $expire) . ']');
82
+                                    . 'expire at [' . date('Y-m-d H:i:s', $expire) . ']');
83 83
             $result = apc_store($key, $data, $ttl);
84 84
             if ($result === false) {
85 85
                 $this->logger->error('Can not save cache data for the key [' . $key . '], return false');
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
                             'mtime'  => $cacheInfos['creation_time'],
128 128
                             'expire' => $cacheInfos['creation_time'] + $cacheInfos['ttl'],
129 129
                             'ttl'    => $cacheInfos['ttl']
130
-                           );
130
+                            );
131 131
             } 
132 132
             $this->logger->info('This cache does not exists skipping');
133 133
             return false;
Please login to merge, or discard this patch.
core/classes/cache/FileCache.php 1 patch
Indentation   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -108,13 +108,13 @@  discard block
 block discarded – undo
108 108
         public function set($key, $data, $ttl = 0) {
109 109
             $expire = time() + $ttl;
110 110
             $this->logger->debug('Setting cache data for key [' . $key . '], '
111
-                           . 'time to live [' . $ttl . '], '
112
-                           . 'expire at [' . date('Y-m-d H:i:s', $expire) . ']');
111
+                            . 'time to live [' . $ttl . '], '
112
+                            . 'expire at [' . date('Y-m-d H:i:s', $expire) . ']');
113 113
             $filePath = $this->getFilePath($key);
114 114
             $handle = fopen($filePath, 'w');
115 115
             if (!is_resource($handle)) {
116 116
                 $this->logger->error('Can not open the file cache '
117
-                                     . '[' . $filePath . '] for the key [' . $key . '], return false');
117
+                                        . '[' . $filePath . '] for the key [' . $key . '], return false');
118 118
                 return false;
119 119
             }
120 120
             flock($handle, LOCK_EX); // exclusive lock, will get released when the file is closed
@@ -191,10 +191,10 @@  discard block
 block discarded – undo
191 191
             if ($data['expire'] > time()) {
192 192
                 $this->logger->info('This cache not yet expired return cache informations');
193 193
                 return array(
194
-                           'mtime' => $data['mtime'],
195
-                           'expire' => $data['expire'],
196
-                           'ttl' => $data['ttl']
197
-                         );
194
+                            'mtime' => $data['mtime'],
195
+                            'expire' => $data['expire'],
196
+                            'ttl' => $data['ttl']
197
+                            );
198 198
             }
199 199
             $this->logger->info('This cache already expired return false');
200 200
             return false;
Please login to merge, or discard this patch.
core/libraries/Email.php 1 patch
Indentation   +23 added lines, -23 removed lines patch added patch discarded remove patch
@@ -395,7 +395,7 @@  discard block
 block discarded – undo
395 395
             $addresses = array();
396 396
             foreach ($pairs as $name => $email) {
397 397
                 if (is_numeric($name)) {
398
-                   $name = null;
398
+                    $name = null;
399 399
                 }
400 400
                 $addresses[] = $this->formatHeader($email, $name);
401 401
             }
@@ -837,10 +837,10 @@  discard block
 block discarded – undo
837 837
             return $headers;
838 838
         }
839 839
 
840
-         /**
841
-         * Get the attachment message for send or the simple message
842
-         * @return string
843
-         */
840
+            /**
841
+             * Get the attachment message for send or the simple message
842
+             * @return string
843
+             */
844 844
         protected function getMessageWithAttachmentForSend() {
845 845
             $message = $this->getWrapMessage();
846 846
             if ($this->hasAttachments()) {
@@ -870,7 +870,7 @@  discard block
 block discarded – undo
870 870
          */
871 871
         protected function sendHelloCommand() {
872 872
             $responseCode = $this->sendCommand('EHLO ' . $this->getSmtpClientHostname())
873
-                                 ->getSmtpResponseCode();
873
+                                    ->getSmtpResponseCode();
874 874
             if ($responseCode !== 250) {
875 875
                 $this->error = $this->smtpResponse;
876 876
                 return false;
@@ -1017,7 +1017,7 @@  discard block
 block discarded – undo
1017 1017
             $message = $this->getMessageWithAttachmentForSend();
1018 1018
             $headers = $this->getHeadersForSend(); 
1019 1019
             $this->logger->info('Sending new mail using mail protocol, the information are listed below: '
1020
-                                  . 'destination: ' . $to . ', headers: ' . $headers . ', message: ' . $message);
1020
+                                    . 'destination: ' . $to . ', headers: ' . $headers . ', message: ' . $message);
1021 1021
             $result = mail($to, $this->subject, $message, $headers, $this->params);
1022 1022
             if (!$result) {
1023 1023
                 $this->error = 'Error when sending mail using mail protocol';
@@ -1025,10 +1025,10 @@  discard block
 block discarded – undo
1025 1025
             return $result;
1026 1026
         }
1027 1027
 
1028
-         /**
1029
-         * Send mail using "smtp" protocol
1030
-         * @return boolean
1031
-         */
1028
+            /**
1029
+             * Send mail using "smtp" protocol
1030
+             * @return boolean
1031
+             */
1032 1032
         protected function sendSmtp() {
1033 1033
             if (!$this->smtpConnection()) {
1034 1034
                 return false;
@@ -1048,7 +1048,7 @@  discard block
 block discarded – undo
1048 1048
             $message = $this->getMessageWithAttachmentForSend();
1049 1049
             $headers = $this->getHeadersForSend();
1050 1050
             $this->logger->info('Sending new mail using SMTP protocol, the information are listed below: '
1051
-                                  . 'destination: ' . $to . ', headers: ' . $headers . ', message: ' . $message);
1051
+                                    . 'destination: ' . $to . ', headers: ' . $headers . ', message: ' . $message);
1052 1052
             
1053 1053
             $recipients = array_merge($this->to, $this->cc, $this->bcc);
1054 1054
             $commands = array(
@@ -1081,17 +1081,17 @@  discard block
 block discarded – undo
1081 1081
             return empty($this->error);
1082 1082
         }
1083 1083
 
1084
-         /**
1085
-         * Return the client hostname for SMTP
1086
-         * 
1087
-         * There are only two legal types of hostname - either a fully
1088
-         * qualified domain name (eg: "mail.example.com") or an IP literal
1089
-         * (eg: "[1.2.3.4]").
1090
-         *
1091
-         * @link    https://tools.ietf.org/html/rfc5321#section-2.3.5
1092
-         * @link    http://cbl.abuseat.org/namingproblems.html
1093
-         * @return string
1094
-         */
1084
+            /**
1085
+             * Return the client hostname for SMTP
1086
+             * 
1087
+             * There are only two legal types of hostname - either a fully
1088
+             * qualified domain name (eg: "mail.example.com") or an IP literal
1089
+             * (eg: "[1.2.3.4]").
1090
+             *
1091
+             * @link    https://tools.ietf.org/html/rfc5321#section-2.3.5
1092
+             * @link    http://cbl.abuseat.org/namingproblems.html
1093
+             * @return string
1094
+             */
1095 1095
         protected function getSmtpClientHostname() {
1096 1096
             $globals = &class_loader('GlobalVar', 'classes');
1097 1097
             if ($globals->server('SERVER_NAME')) {
Please login to merge, or discard this patch.
core/libraries/Assets.php 1 patch
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -44,9 +44,9 @@  discard block
 block discarded – undo
44 44
      */
45 45
     class Assets extends BaseClass {
46 46
 
47
-         /**
48
-         * Construct new instance
49
-         */
47
+            /**
48
+             * Construct new instance
49
+             */
50 50
         public function __construct() {
51 51
             parent::__construct();
52 52
         }
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
          * @see Assets:link
96 96
          */
97 97
         public function css($path) {
98
-           return $this->link($path, 'css');
98
+            return $this->link($path, 'css');
99 99
         }
100 100
 
101 101
         /**
Please login to merge, or discard this patch.
core/libraries/Pagination.php 1 patch
Indentation   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -90,10 +90,10 @@  discard block
 block discarded – undo
90 90
         }
91 91
 
92 92
         /**
93
-        * Set the value of the pagination query string
94
-        * @param string $pQueryString the new value
95
-        * @return object
96
-        */
93
+         * Set the value of the pagination query string
94
+         * @param string $pQueryString the new value
95
+         * @return object
96
+         */
97 97
         public function setPaginationQueryString($pQueryString) {
98 98
             $this->paginationQueryString = $pQueryString;
99 99
             return $this;
@@ -244,15 +244,15 @@  discard block
 block discarded – undo
244 244
                 }
245 245
             }
246 246
             $navbar .= $this->config['next_open'] . '<a href="' . $query . ($currentPageNumber + 1) . '">' 
247
-                       . $this->config['next_text'] . '</a>' . $this->config['next_close'];
247
+                        . $this->config['next_text'] . '</a>' . $this->config['next_close'];
248 248
             return $navbar;
249 249
         }
250 250
 
251 251
 
252
-         /**
253
-         * Build the pagination link for the first page
254
-         * @see Pagination::buildPaginationLinkForFirstAndLastPage
255
-         */
252
+            /**
253
+             * Build the pagination link for the first page
254
+             * @see Pagination::buildPaginationLinkForFirstAndLastPage
255
+             */
256 256
         protected function buildPaginationLinkForFirstPage($begin, $end, $currentPageNumber) {
257 257
             return $this->buildPaginationLinkForFirstAndLastPage($begin, $end, $currentPageNumber, 'first');
258 258
         }
Please login to merge, or discard this patch.