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