@@ -151,7 +151,7 @@ discard block |
||
| 151 | 151 | $account = $this->get_account(); |
| 152 | 152 | if (!empty($new_password)) { |
| 153 | 153 | //check if the new encrypted password was already used |
| 154 | - if ( !$this->check_password_reuse($new_password, true) |
|
| 154 | + if (!$this->check_password_reuse($new_password, true) |
|
| 155 | 155 | || !$this->check_password_strength($new_password, true)) { |
| 156 | 156 | $this->errstr = "password strength too low"; |
| 157 | 157 | return false; |
@@ -270,7 +270,7 @@ discard block |
||
| 270 | 270 | $password_length = mb_strlen($password); |
| 271 | 271 | |
| 272 | 272 | if ($password_length < $this->_config->get('min_password_length')) { |
| 273 | - if ($show_ui_message){ |
|
| 273 | + if ($show_ui_message) { |
|
| 274 | 274 | midcom::get()->uimessages->add($this->_l10n->get($this->_component), $this->_l10n->get('password too short'), 'error'); |
| 275 | 275 | } |
| 276 | 276 | return false; |
@@ -288,7 +288,7 @@ discard block |
||
| 288 | 288 | } |
| 289 | 289 | |
| 290 | 290 | if ($score < $this->_config->get('min_password_score')) { |
| 291 | - if ($show_ui_message){ |
|
| 291 | + if ($show_ui_message) { |
|
| 292 | 292 | midcom::get()->uimessages->add($this->_l10n->get($this->_component), $this->_l10n->get('password weak'), 'error'); |
| 293 | 293 | } |
| 294 | 294 | return false; |
@@ -432,7 +432,7 @@ discard block |
||
| 432 | 432 | |
| 433 | 433 | public static function get_person_by_formdata(array $data) : ?midcom_db_person |
| 434 | 434 | { |
| 435 | - if ( empty($data['username']) |
|
| 435 | + if (empty($data['username']) |
|
| 436 | 436 | || empty($data['password'])) { |
| 437 | 437 | return null; |
| 438 | 438 | } |
@@ -481,7 +481,7 @@ discard block |
||
| 481 | 481 | * If the maximum number of attempts is reached and the oldest attempt |
| 482 | 482 | * on the stack is within our defined timeframe, we block the account |
| 483 | 483 | */ |
| 484 | - if ( count($attempts) >= $max_attempts |
|
| 484 | + if (count($attempts) >= $max_attempts |
|
| 485 | 485 | && $attempts[$max_attempts - 1] >= (time() - ($timeframe * 60))) { |
| 486 | 486 | $this->disable_account(); |
| 487 | 487 | $stat = false; |