@@ -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 | } |
@@ -463,7 +463,7 @@ discard block |
||
| 463 | 463 | $max_attempts = midcom_baseclasses_components_configuration::get($component, 'config')->get('max_password_attempts'); |
| 464 | 464 | $timeframe = midcom_baseclasses_components_configuration::get($component, 'config')->get('password_block_timeframe_min'); |
| 465 | 465 | |
| 466 | - if ( $max_attempts == 0 |
|
| 466 | + if ($max_attempts == 0 |
|
| 467 | 467 | || $timeframe == 0) { |
| 468 | 468 | return $stat; |
| 469 | 469 | } |
@@ -485,7 +485,7 @@ discard block |
||
| 485 | 485 | * If the maximum number of attempts is reached and the oldest attempt |
| 486 | 486 | * on the stack is within our defined timeframe, we block the account |
| 487 | 487 | */ |
| 488 | - if ( count($attempts) >= $max_attempts |
|
| 488 | + if (count($attempts) >= $max_attempts |
|
| 489 | 489 | && $attempts[$max_attempts - 1] >= (time() - ($timeframe * 60))) { |
| 490 | 490 | $this->disable_account(); |
| 491 | 491 | $stat = 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; |