flack / openpsa

lib/org/openpsa/user/accounthelper.php

Spacing +3 added lines, -3 removed lines

@@ -151,7 +151,7 @@ 
         $account = $this->get_account();
         if (!empty($new_password)) {
             // check if the new encrypted password was already used
-            if (   !$this->check_password_reuse($new_password)
+            if (!$this->check_password_reuse($new_password)
                 || !$this->check_password_strength($new_password)) {
                 return false;
             }
@@ -423,7 +423,7 @@ 
 
     public static function get_person_by_formdata(array $data) : ?midcom_db_person
     {
-        if (   empty($data['username'])
+        if (empty($data['username'])
             || empty($data['password'])) {
             return null;
         }
@@ -472,7 +472,7 @@ 
          * If the maximum number of attempts is reached and the oldest attempt
          * on the stack is within our defined timeframe, we block the account
          */
-        if (   count($attempts) >= $max_attempts
+        if (count($attempts) >= $max_attempts
             && $attempts[$max_attempts - 1] >= (time() - ($timeframe * 60))) {
             $this->disable_account();
             $stat = false;

lib/org/openpsa/user/validator.php

Spacing +3 added lines, -3 removed lines

@@ -58,7 +58,7 @@ 
     {
         $result = $this->is_username_available($fields);
         $accounthelper = $this->get_accounthelper();
-        if (   $fields['password']['switch']
+        if ($fields['password']['switch']
             && !$accounthelper->check_password_strength((string) $fields['password']['password'])) {
             $result = ['password' => $accounthelper->errstr];
         }
@@ -162,9 +162,9 @@ 
         $result = [];
 
         $accounthelper = $this->get_accounthelper(new midcom_db_person($fields["person"]));
-        if (!$accounthelper->check_password_reuse($fields['new_password'])){
+        if (!$accounthelper->check_password_reuse($fields['new_password'])) {
             $result['new_password'] = $accounthelper->errstr;
-        } elseif (!$accounthelper->check_password_strength($fields['new_password'])){
+        } elseif (!$accounthelper->check_password_strength($fields['new_password'])) {
             $result['new_password'] = $accounthelper->errstr;
         }
         return $result ?: true;

Braces +2 added lines, -2 removed lines

@@ -162,9 +162,9 @@
         $result = [];
 
         $accounthelper = $this->get_accounthelper(new midcom_db_person($fields["person"]));
-        if (!$accounthelper->check_password_reuse($fields['new_password'])){
+        if (!$accounthelper->check_password_reuse($fields['new_password'])) {
             $result['new_password'] = $accounthelper->errstr;
-        } elseif (!$accounthelper->check_password_strength($fields['new_password'])){
+        } elseif (!$accounthelper->check_password_strength($fields['new_password'])) {
             $result['new_password'] = $accounthelper->errstr;
         }
         return $result ?: true;

lib/org/openpsa/user/handler/person/account.php

Spacing +3 added lines, -3 removed lines

@@ -44,7 +44,7 @@ 
         $workflow = $this->get_workflow('datamanager', ['controller' => $data['controller']]);
         $response = $workflow->run($request);
 
-        if (   $workflow->get_state() == 'save'
+        if ($workflow->get_state() == 'save'
             && $this->create_account($this->person, $data["controller"]->get_form_values())) {
             midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), sprintf($this->_l10n->get('person %s created'), $this->person->name));
         }
@@ -77,7 +77,7 @@ 
         $person = new midcom_db_person($guid);
         $accounthelper = new org_openpsa_user_accounthelper($person);
         if ($accounthelper->welcome_email()) {
-            midcom::get()->uimessages->add($this->_l10n->get($this->_component), sprintf($this->_l10n->get('password reset and mail to %s sent'), $person->email ));
+            midcom::get()->uimessages->add($this->_l10n->get($this->_component), sprintf($this->_l10n->get('password reset and mail to %s sent'), $person->email));
         } else {
             midcom::get()->uimessages->add($this->_l10n->get($this->_component), $accounthelper->errstr, 'error');
         }
@@ -133,7 +133,7 @@ 
         $accounthelper = new org_openpsa_user_accounthelper($this->person);
 
         // Update account
-        if (   !$accounthelper->set_account($controller->get_form_values()["username"], $password)
+        if (!$accounthelper->set_account($controller->get_form_values()["username"], $password)
             && midcom_connection::get_error() != MGD_ERR_OK) {
             // Failure, give a message
             midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), $this->_l10n->get("failed to update the user account, reason") . ': ' . $accounthelper->errstr, 'error');