@@ -103,8 +103,7 @@ |
||
103 | 103 | $this->header = array(); |
104 | 104 | if(function_exists('apache_request_headers')){ |
105 | 105 | $this->header = apache_request_headers(); |
106 | - } |
|
107 | - else if(function_exists('getallheaders')){ |
|
106 | + } else if(function_exists('getallheaders')){ |
|
108 | 107 | $this->header = getallheaders(); |
109 | 108 | } |
110 | 109 | } |
@@ -60,8 +60,7 @@ discard block |
||
60 | 60 | while(($module = readdir($moduleDir)) !== false){ |
61 | 61 | if($module != '.' && $module != '..' && preg_match('/^([a-z0-9-_]+)$/i', $module) && is_dir(MODULE_PATH . $module)){ |
62 | 62 | static::$list[] = $module; |
63 | - } |
|
64 | - else{ |
|
63 | + } else{ |
|
65 | 64 | $logger->info('Skipping [' .$module. '], may be this is not a directory or does not exists or is invalid name'); |
66 | 65 | } |
67 | 66 | } |
@@ -70,8 +69,7 @@ discard block |
||
70 | 69 | |
71 | 70 | if(static::hasModule()){ |
72 | 71 | $logger->info('The application contains the module below [' . implode(', ', static::getModuleList()) . ']'); |
73 | - } |
|
74 | - else{ |
|
72 | + } else{ |
|
75 | 73 | $logger->info('The application contains no module skipping'); |
76 | 74 | } |
77 | 75 | } |
@@ -118,8 +116,7 @@ discard block |
||
118 | 116 | $autoloads['languages'] = array_merge($autoloads['languages'], $autoload['languages']); |
119 | 117 | } |
120 | 118 | unset($autoload); |
121 | - } |
|
122 | - else{ |
|
119 | + } else{ |
|
123 | 120 | show_error('No autoload configuration found in autoload.php for module [' .$module. ']'); |
124 | 121 | } |
125 | 122 | } |
@@ -145,8 +142,7 @@ discard block |
||
145 | 142 | if(! empty($route) && is_array($route)){ |
146 | 143 | $routes = array_merge($routes, $route); |
147 | 144 | unset($route); |
148 | - } |
|
149 | - else{ |
|
145 | + } else{ |
|
150 | 146 | show_error('No routing configuration found in [' .$file. '] for module [' . $module . ']'); |
151 | 147 | } |
152 | 148 | } |
@@ -175,8 +171,7 @@ discard block |
||
175 | 171 | if(file_exists($filePath)){ |
176 | 172 | $logger->info('Found controller [' . $class . '] in module [' .$module. '], the file path is [' .$filePath. ']'); |
177 | 173 | return $filePath; |
178 | - } |
|
179 | - else{ |
|
174 | + } else{ |
|
180 | 175 | $logger->info('Controller [' . $class . '] does not exist in the module [' .$module. ']'); |
181 | 176 | return false; |
182 | 177 | } |
@@ -202,8 +197,7 @@ discard block |
||
202 | 197 | if(file_exists($filePath)){ |
203 | 198 | $logger->info('Found model [' . $class . '] in module [' .$module. '], the file path is [' .$filePath. ']'); |
204 | 199 | return $filePath; |
205 | - } |
|
206 | - else{ |
|
200 | + } else{ |
|
207 | 201 | $logger->info('Model [' . $class . '] does not exist in the module [' .$module. ']'); |
208 | 202 | return false; |
209 | 203 | } |
@@ -228,8 +222,7 @@ discard block |
||
228 | 222 | if(file_exists($filePath)){ |
229 | 223 | $logger->info('Found configuration [' . $configuration . '] in module [' .$module. '], the file path is [' .$filePath. ']'); |
230 | 224 | return $filePath; |
231 | - } |
|
232 | - else{ |
|
225 | + } else{ |
|
233 | 226 | $logger->info('Configuration [' . $configuration . '] does not exist in the module [' .$module. ']'); |
234 | 227 | return false; |
235 | 228 | } |
@@ -255,8 +248,7 @@ discard block |
||
255 | 248 | if(file_exists($filePath)){ |
256 | 249 | $logger->info('Found helper [' . $helper . '] in module [' .$module. '], the file path is [' .$filePath. ']'); |
257 | 250 | return $filePath; |
258 | - } |
|
259 | - else{ |
|
251 | + } else{ |
|
260 | 252 | $logger->info('Helper [' . $helper . '] does not exist in the module [' .$module. ']'); |
261 | 253 | return false; |
262 | 254 | } |
@@ -282,8 +274,7 @@ discard block |
||
282 | 274 | if(file_exists($filePath)){ |
283 | 275 | $logger->info('Found library [' . $class . '] in module [' .$module. '], the file path is [' .$filePath. ']'); |
284 | 276 | return $filePath; |
285 | - } |
|
286 | - else{ |
|
277 | + } else{ |
|
287 | 278 | $logger->info('Library [' . $class . '] does not exist in the module [' .$module. ']'); |
288 | 279 | return false; |
289 | 280 | } |
@@ -311,8 +302,7 @@ discard block |
||
311 | 302 | if(file_exists($filePath)){ |
312 | 303 | $logger->info('Found view [' . $view . '] in module [' .$module. '], the file path is [' .$filePath. ']'); |
313 | 304 | return $filePath; |
314 | - } |
|
315 | - else{ |
|
305 | + } else{ |
|
316 | 306 | $logger->info('View [' . $view . '] does not exist in the module [' .$module. ']'); |
317 | 307 | return false; |
318 | 308 | } |
@@ -339,8 +329,7 @@ discard block |
||
339 | 329 | if(file_exists($filePath)){ |
340 | 330 | $logger->info('Found language [' . $language . '] in module [' .$module. '], the file path is [' .$filePath. ']'); |
341 | 331 | return $filePath; |
342 | - } |
|
343 | - else{ |
|
332 | + } else{ |
|
344 | 333 | $logger->info('Language [' . $language . '] does not exist in the module [' .$module. ']'); |
345 | 334 | return false; |
346 | 335 | } |
@@ -137,8 +137,7 @@ discard block |
||
137 | 137 | if(! in_array(strtolower($this->logger), $configLoggerName)){ |
138 | 138 | return; |
139 | 139 | } |
140 | - } |
|
141 | - else if(strtolower($this->logger) !== strtolower($configLoggerName)){ |
|
140 | + } else if(strtolower($this->logger) !== strtolower($configLoggerName)){ |
|
142 | 141 | return; |
143 | 142 | } |
144 | 143 | } |
@@ -218,20 +217,15 @@ discard block |
||
218 | 217 | //the default value is NONE, so means no need test for NONE |
219 | 218 | if($level == 'fatal'){ |
220 | 219 | $value = self::FATAL; |
221 | - } |
|
222 | - else if($level == 'error'){ |
|
220 | + } else if($level == 'error'){ |
|
223 | 221 | $value = self::ERROR; |
224 | - } |
|
225 | - else if($level == 'warning' || $level == 'warn'){ |
|
222 | + } else if($level == 'warning' || $level == 'warn'){ |
|
226 | 223 | $value = self::WARNING; |
227 | - } |
|
228 | - else if($level == 'info'){ |
|
224 | + } else if($level == 'info'){ |
|
229 | 225 | $value = self::INFO; |
230 | - } |
|
231 | - else if($level == 'debug'){ |
|
226 | + } else if($level == 'debug'){ |
|
232 | 227 | $value = self::DEBUG; |
233 | - } |
|
234 | - else if($level == 'all'){ |
|
228 | + } else if($level == 'all'){ |
|
235 | 229 | $value = self::ALL; |
236 | 230 | } |
237 | 231 | return $value; |
@@ -248,17 +242,13 @@ discard block |
||
248 | 242 | //the default value is NONE, so means no need test for NONE |
249 | 243 | if($level == self::FATAL){ |
250 | 244 | $value = 'FATAL'; |
251 | - } |
|
252 | - else if($level == self::ERROR){ |
|
245 | + } else if($level == self::ERROR){ |
|
253 | 246 | $value = 'ERROR'; |
254 | - } |
|
255 | - else if($level == self::WARNING){ |
|
247 | + } else if($level == self::WARNING){ |
|
256 | 248 | $value = 'WARNING'; |
257 | - } |
|
258 | - else if($level == self::INFO){ |
|
249 | + } else if($level == self::INFO){ |
|
259 | 250 | $value = 'INFO'; |
260 | - } |
|
261 | - else if($level == self::DEBUG){ |
|
251 | + } else if($level == self::DEBUG){ |
|
262 | 252 | $value = 'DEBUG'; |
263 | 253 | } |
264 | 254 | //no need for ALL |
@@ -69,8 +69,7 @@ |
||
69 | 69 | $this->dompdf->render(); |
70 | 70 | if($stream){ |
71 | 71 | $this->dompdf->stream($filename); |
72 | - } |
|
73 | - else{ |
|
72 | + } else{ |
|
74 | 73 | return $this->dompdf->output(); |
75 | 74 | } |
76 | 75 | } |
@@ -150,8 +150,7 @@ |
||
150 | 150 | foreach ($emails as $name => $email) { |
151 | 151 | if(is_numeric($name)){ |
152 | 152 | $this->setTo($email); |
153 | - } |
|
154 | - else{ |
|
153 | + } else{ |
|
155 | 154 | $this->setTo($email, $name); |
156 | 155 | } |
157 | 156 | } |
@@ -93,8 +93,7 @@ |
||
93 | 93 | $logger->info('Delete cookie item ['.$item.']'); |
94 | 94 | unset($_COOKIE[$item]); |
95 | 95 | return true; |
96 | - } |
|
97 | - else{ |
|
96 | + } else{ |
|
98 | 97 | $logger->warning('Cookie item ['.$item.'] to be deleted does not exists'); |
99 | 98 | return false; |
100 | 99 | } |
@@ -487,8 +487,7 @@ discard block |
||
487 | 487 | } |
488 | 488 | chdir($destination_directory); |
489 | 489 | } |
490 | - } |
|
491 | - else{ |
|
490 | + } else{ |
|
492 | 491 | $this->logger->warning('Can not create the upload directory [' .$destination_directory. ']'); |
493 | 492 | } |
494 | 493 | } |
@@ -640,8 +639,7 @@ discard block |
||
640 | 639 | // set original filename if not have a new name |
641 | 640 | if (empty($this->filename)) { |
642 | 641 | $this->filename = $this->file_array[$this->input]['name']; |
643 | - } |
|
644 | - else{ |
|
642 | + } else{ |
|
645 | 643 | // Replace %s for extension in filename |
646 | 644 | // Before: /[\w\d]*(.[\d\w]+)$/i |
647 | 645 | // After: /^[\s[:alnum:]\-\_\.]*\.([\d\w]+)$/iu |
@@ -223,8 +223,7 @@ discard block |
||
223 | 223 | if(! Security::validateCSRF()){ |
224 | 224 | show_error('Invalide data, Cross Site Request Forgery do his job, the data to validate is corrupted.'); |
225 | 225 | } |
226 | - } |
|
227 | - else{ |
|
226 | + } else{ |
|
228 | 227 | $this->logger->info('CSRF is not enabled in configuration or not set manully, no need to check it'); |
229 | 228 | } |
230 | 229 | } |
@@ -234,8 +233,7 @@ discard block |
||
234 | 233 | foreach ($this->getData() as $inputName => $inputVal) { |
235 | 234 | if(is_array($this->data[$inputName])){ |
236 | 235 | $this->data[$inputName] = array_map('trim', $this->data[$inputName]); |
237 | - } |
|
238 | - else{ |
|
236 | + } else{ |
|
239 | 237 | $this->data[$inputName] = trim($this->data[$inputName]); |
240 | 238 | } |
241 | 239 | |
@@ -604,8 +602,7 @@ discard block |
||
604 | 602 | if ($inputVal == '' && $callbackReturn == true) { |
605 | 603 | $this->_setError($inputName, $ruleName, $this->_getLabel($inputName)); |
606 | 604 | } |
607 | - } |
|
608 | - else if($inputVal == '') { |
|
605 | + } else if($inputVal == '') { |
|
609 | 606 | $this->_setError($inputName, $ruleName, $this->_getLabel($inputName)); |
610 | 607 | } |
611 | 608 | } |
@@ -664,8 +661,7 @@ discard block |
||
664 | 661 | $this->_setError($inputName, $ruleName . ',post:key', array($this->_getLabel($inputName), $this->_getLabel(str_replace('post:', '', $doNotEqual)))); |
665 | 662 | continue; |
666 | 663 | } |
667 | - } |
|
668 | - else{ |
|
664 | + } else{ |
|
669 | 665 | if ($inputVal == $doNotEqual) { |
670 | 666 | $this->_setError($inputName, $ruleName . ',string', array($this->_getLabel($inputName), $doNotEqual)); |
671 | 667 | continue; |
@@ -222,8 +222,7 @@ |
||
222 | 222 | } |
223 | 223 | $str .= '</tr>'; |
224 | 224 | $str .= '</thead>'; |
225 | - } |
|
226 | - else{ |
|
225 | + } else{ |
|
227 | 226 | //no need check for footer |
228 | 227 | $use_footer = false; |
229 | 228 | } |