@@ -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 | } |
@@ -41,8 +41,7 @@ discard block |
||
| 41 | 41 | require_once CONFIG_PATH . 'config_pagination.php'; |
| 42 | 42 | if(empty($config) || ! is_array($config)){ |
| 43 | 43 | show_error('No configuration found in ' . CONFIG_PATH . 'config_pagination.php'); |
| 44 | - } |
|
| 45 | - else{ |
|
| 44 | + } else{ |
|
| 46 | 45 | if(! empty($overwriteConfig)){ |
| 47 | 46 | $config = array_merge($config, $overwriteConfig); |
| 48 | 47 | } |
@@ -50,8 +49,7 @@ discard block |
||
| 50 | 49 | Config::setAll($config); |
| 51 | 50 | unset($config); |
| 52 | 51 | } |
| 53 | - } |
|
| 54 | - else{ |
|
| 52 | + } else{ |
|
| 55 | 53 | show_error('Unable to find the pagination configuration file'); |
| 56 | 54 | } |
| 57 | 55 | } |
@@ -83,18 +81,15 @@ discard block |
||
| 83 | 81 | $currentUrl = Url::current(); |
| 84 | 82 | if($queryString == ''){ |
| 85 | 83 | $query = '?' . $pageQueryName . '='; |
| 86 | - } |
|
| 87 | - else{ |
|
| 84 | + } else{ |
|
| 88 | 85 | $tab = explode($pageQueryName . '=', $queryString); |
| 89 | 86 | $nb = count($tab); |
| 90 | 87 | if($nb == 1){ |
| 91 | 88 | $query = '?' . $queryString . '&' . $pageQueryName . '='; |
| 92 | - } |
|
| 93 | - else{ |
|
| 89 | + } else{ |
|
| 94 | 90 | if($tab[0] == ''){ |
| 95 | 91 | $query = '?' . $pageQueryName . '='; |
| 96 | - } |
|
| 97 | - else{ |
|
| 92 | + } else{ |
|
| 98 | 93 | $query = '?' . $tab[0] . '' . $pageQueryName . '='; |
| 99 | 94 | } |
| 100 | 95 | } |
@@ -113,20 +108,17 @@ discard block |
||
| 113 | 108 | if($numberOfLink % 2 == 0){ |
| 114 | 109 | $start = $currentPageNumber - ($numberOfLink / 2) + 1; |
| 115 | 110 | $end = $currentPageNumber + ($numberOfLink / 2); |
| 116 | - } |
|
| 117 | - else{ |
|
| 111 | + } else{ |
|
| 118 | 112 | $start = $currentPageNumber - floor($numberOfLink / 2); |
| 119 | 113 | $end = $currentPageNumber + floor($numberOfLink / 2); |
| 120 | 114 | } |
| 121 | 115 | if($start <= 1){ |
| 122 | 116 | $begin = 1; |
| 123 | 117 | $end = $numberOfLink; |
| 124 | - } |
|
| 125 | - else if($start > 1 && $end < $numberOfPage){ |
|
| 118 | + } else if($start > 1 && $end < $numberOfPage){ |
|
| 126 | 119 | $begin = $start; |
| 127 | 120 | $end = $end; |
| 128 | - } |
|
| 129 | - else{ |
|
| 121 | + } else{ |
|
| 130 | 122 | $begin = ($numberOfPage - $numberOfLink) + 1; |
| 131 | 123 | $end = $numberOfPage; |
| 132 | 124 | } |
@@ -138,32 +130,27 @@ discard block |
||
| 138 | 130 | for($i = $begin; $i <= $end; $i++){ |
| 139 | 131 | if($i == $currentPageNumber){ |
| 140 | 132 | $navbar .= $this->config['active_link_open'] . $currentPageNumber . $this->config['active_link_close']; |
| 141 | - } |
|
| 142 | - else{ |
|
| 133 | + } else{ |
|
| 143 | 134 | $navbar .= $this->config['digit_open'] . '<a href="' . $query . $i . '" ' . attributes_to_string($this->config['attributes']) . '>' . $i . '</a>' . $this->config['digit_close']; |
| 144 | 135 | } |
| 145 | 136 | } |
| 146 | 137 | $navbar .= $this->config['next_open'] . '<a href="' . $query . ($currentPageNumber + 1) . '">' . $this->config['next_text'] . '</a>' . $this->config['next_close']; |
| 147 | - } |
|
| 148 | - else if($currentPageNumber > 1 && $currentPageNumber < $numberOfPage){ |
|
| 138 | + } else if($currentPageNumber > 1 && $currentPageNumber < $numberOfPage){ |
|
| 149 | 139 | $navbar .= $this->config['previous_open'] . '<a href="' . $query . ($currentPageNumber - 1) . '">' . $this->config['previous_text'] . '</a>' . $this->config['previous_close']; |
| 150 | 140 | for($i = $begin; $i <= $end; $i++){ |
| 151 | 141 | if($i == $currentPageNumber){ |
| 152 | 142 | $navbar .= $this->config['active_link_open'] . $currentPageNumber . $this->config['active_link_close']; |
| 153 | - } |
|
| 154 | - else{ |
|
| 143 | + } else{ |
|
| 155 | 144 | $navbar .= $this->config['digit_open'] . '<a href="' . $query . $i . '"' . attributes_to_string($this->config['attributes']) . '>' . $i .'</a>' . $this->config['digit_close']; |
| 156 | 145 | } |
| 157 | 146 | } |
| 158 | 147 | $navbar .= $this->config['next_open']."<a href='$query".($currentPageNumber + 1)."'>".$this->config['next_text']."</a>".$this->config['next_close']; |
| 159 | - } |
|
| 160 | - else if($currentPageNumber == $numberOfPage){ |
|
| 148 | + } else if($currentPageNumber == $numberOfPage){ |
|
| 161 | 149 | $navbar .= $this->config['previous_open'] . '<a href="' . $query . ($currentPageNumber - 1) . '">' . $this->config['previous_text'] . '</a>' . $this->config['previous_close']; |
| 162 | 150 | for($i = $begin; $i <= $end; $i++){ |
| 163 | 151 | if($i == $currentPageNumber){ |
| 164 | 152 | $navbar .= $this->config['active_link_open'] . $currentPageNumber . $this->config['active_link_close']; |
| 165 | - } |
|
| 166 | - else{ |
|
| 153 | + } else{ |
|
| 167 | 154 | $navbar .= $this->config['digit_open'] . '<a href="' . $query . $i . '"' . attributes_to_string($this->config['attributes']) . '>' . $i . '</a>' . $this->config['digit_close']; |
| 168 | 155 | } |
| 169 | 156 | } |
@@ -921,20 +921,15 @@ |
||
| 921 | 921 | if(preg_match('#trident/([0-9\.]+);#i', $this->_agent, $aresult)){ |
| 922 | 922 | if($aresult[1] == '3.1'){ |
| 923 | 923 | $this->setVersion('7.0'); |
| 924 | - } |
|
| 925 | - else if($aresult[1] == '4.0'){ |
|
| 924 | + } else if($aresult[1] == '4.0'){ |
|
| 926 | 925 | $this->setVersion('8.0'); |
| 927 | - } |
|
| 928 | - else if($aresult[1] == '5.0'){ |
|
| 926 | + } else if($aresult[1] == '5.0'){ |
|
| 929 | 927 | $this->setVersion('9.0'); |
| 930 | - } |
|
| 931 | - else if($aresult[1] == '6.0'){ |
|
| 928 | + } else if($aresult[1] == '6.0'){ |
|
| 932 | 929 | $this->setVersion('10.0'); |
| 933 | - } |
|
| 934 | - else if($aresult[1] == '7.0'){ |
|
| 930 | + } else if($aresult[1] == '7.0'){ |
|
| 935 | 931 | $this->setVersion('11.0'); |
| 936 | - } |
|
| 937 | - else if($aresult[1] == '8.0'){ |
|
| 932 | + } else if($aresult[1] == '8.0'){ |
|
| 938 | 933 | $this->setVersion('11.0'); |
| 939 | 934 | } |
| 940 | 935 | } |
@@ -55,17 +55,13 @@ |
||
| 55 | 55 | |
| 56 | 56 | if (isset($_SERVER["HTTP_CLIENT_IP"])) { |
| 57 | 57 | $ip = $_SERVER["HTTP_CLIENT_IP"]; |
| 58 | - } |
|
| 59 | - else if (isset($_SERVER["HTTP_X_FORWARDED_FOR"])) { |
|
| 58 | + } else if (isset($_SERVER["HTTP_X_FORWARDED_FOR"])) { |
|
| 60 | 59 | $ip = $_SERVER["HTTP_X_FORWARDED_FOR"]; |
| 61 | - } |
|
| 62 | - else if (isset($_SERVER["HTTP_X_FORWARDED"])) { |
|
| 60 | + } else if (isset($_SERVER["HTTP_X_FORWARDED"])) { |
|
| 63 | 61 | $ip = $_SERVER["HTTP_X_FORWARDED"]; |
| 64 | - } |
|
| 65 | - else if (isset($_SERVER["HTTP_FORWARDED_FOR"])) { |
|
| 62 | + } else if (isset($_SERVER["HTTP_FORWARDED_FOR"])) { |
|
| 66 | 63 | $ip = $_SERVER["HTTP_FORWARDED_FOR"]; |
| 67 | - } |
|
| 68 | - else if (isset($_SERVER["HTTP_FORWARDED"])) { |
|
| 64 | + } else if (isset($_SERVER["HTTP_FORWARDED"])) { |
|
| 69 | 65 | $ip = $_SERVER["HTTP_FORWARDED"]; |
| 70 | 66 | } |
| 71 | 67 | |
@@ -42,8 +42,7 @@ |
||
| 42 | 42 | if(isset($classesMap[$class])){ |
| 43 | 43 | if(file_exists($classesMap[$class])){ |
| 44 | 44 | include_once $classesMap[$class]; |
| 45 | - } |
|
| 46 | - else{ |
|
| 45 | + } else{ |
|
| 47 | 46 | echo 'File for class ' . $class . ' not found'; |
| 48 | 47 | } |
| 49 | 48 | } |
@@ -178,15 +178,13 @@ |
||
| 178 | 178 | function clean_input($str){ |
| 179 | 179 | if(is_array($str)){ |
| 180 | 180 | $str = array_map('clean_input', $str); |
| 181 | - } |
|
| 182 | - else if(is_object($str)){ |
|
| 181 | + } else if(is_object($str)){ |
|
| 183 | 182 | $obj = $str; |
| 184 | 183 | foreach ($str as $var => $value) { |
| 185 | 184 | $obj->$var = clean_input($value); |
| 186 | 185 | } |
| 187 | 186 | $str = $obj; |
| 188 | - } |
|
| 189 | - else{ |
|
| 187 | + } else{ |
|
| 190 | 188 | $str = htmlspecialchars(strip_tags($str), ENT_QUOTES, 'UTF-8'); |
| 191 | 189 | } |
| 192 | 190 | return $str; |