Passed
Push — master ( a33fc1...2109ce )
by Andreas
12:24
created
lib/org/openpsa/user/accounthelper.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -151,7 +151,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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;
Please login to merge, or discard this patch.