@@ -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 | } |