alxarafe / alixar

public/htdocs/user/passwordforgotten.php

Spacing +2 added lines, -2 removed lines

@@ -145,12 +145,12 @@
             $messagewarning .= '</div>';
 
             if ($result <= 0 && $edituser->error == 'USERNOTFOUND') {
-                usleep(20000);  // add delay to simulate setPassword() and send_password() actions delay (0.02s)
+                usleep(20000); // add delay to simulate setPassword() and send_password() actions delay (0.02s)
                 $message .= $messagewarning;
                 $username = '';
             } else {
                 if (empty($edituser->email)) {
-                    usleep(20000);  // add delay to simulate setPassword() and send_password() actions delay (0.02s)
+                    usleep(20000); // add delay to simulate setPassword() and send_password() actions delay (0.02s)
                     $message .= $messagewarning;
                 } else {
                     $newpassword = $edituser->setPassword($user, '', 1);

public/htdocs/user/card.php

Spacing +5 added lines, -5 removed lines

@@ -85,7 +85,7 @@ 
 $dateendvalidity = dol_mktime(0, 0, 0, GETPOSTINT('dateendvaliditymonth'), GETPOSTINT('dateendvalidityday'), GETPOSTINT('dateendvalidityyear'));
 $dateofbirth = dol_mktime(0, 0, 0, GETPOSTINT('dateofbirthmonth'), GETPOSTINT('dateofbirthday'), GETPOSTINT('dateofbirthyear'));
 
-$childids = $user->getAllChildIds(1);   // For later, test on salary visibility
+$childids = $user->getAllChildIds(1); // For later, test on salary visibility
 
 $object = new User($db);
 $extrafields = new ExtraFields($db);
@@ -117,7 +117,7 @@ 
 // Define value to know what current user can do on users
 $canadduser = (!empty($user->admin) || $user->hasRight("user", "user", "write"));
 $canreaduser = (!empty($user->admin) || $user->hasRight("user", "user", "read"));
-$canedituser = (!empty($user->admin) || $user->hasRight("user", "user", "write"));  // edit other user
+$canedituser = (!empty($user->admin) || $user->hasRight("user", "user", "write")); // edit other user
 $candisableuser = (!empty($user->admin) || $user->hasRight("user", "user", "delete"));
 $canreadgroup = $canreaduser;
 $caneditgroup = $canedituser;
@@ -438,7 +438,7 @@ 
                 }
                 $object->gender = GETPOST("gender", 'aZ09');
                 if ($caneditpasswordandsee) {
-                    $object->pass = GETPOST("password", 'none');    // We can keep 'none' for password fields
+                    $object->pass = GETPOST("password", 'none'); // We can keep 'none' for password fields
                 }
                 if ($caneditpasswordandsee || $user->hasRight("api", "apikey", "generate")) {
                     $object->api_key = (GETPOST("api_key", 'alphanohtml')) ? GETPOST("api_key", 'alphanohtml') : $object->api_key;
@@ -546,7 +546,7 @@ 
                         }
                     }
 
-                    $ret = $object->update($user);      // This may include call to setPassword if password has changed
+                    $ret = $object->update($user); // This may include call to setPassword if password has changed
                     if ($ret < 0) {
                         $error++;
                         if ($db->errno() == 'DB_ERROR_RECORD_ALREADY_EXISTS') {
@@ -1944,7 +1944,7 @@ 
                 print '<td>';
                 if (!empty($object->api_key)) {
                     print '<span class="opacitymedium">';
-                    print showValueWithClipboardCPButton($object->api_key, 1, $langs->trans("Hidden"));     // TODO Add an option to also reveal the hash, not only copy paste
+                    print showValueWithClipboardCPButton($object->api_key, 1, $langs->trans("Hidden")); // TODO Add an option to also reveal the hash, not only copy paste
                     print '</span>';
                 }
                 print '</td></tr>';

public/htdocs/user/perms.php

Spacing +4 added lines, -4 removed lines

@@ -224,9 +224,9 @@ 
 // Read the permissions of a user inherited by its groups
 $permsgroupbyentity = array();
 
-$sql = "SELECT DISTINCT gr.fk_id, gu.entity";   // fk_id are permission id and entity is entity of the group
+$sql = "SELECT DISTINCT gr.fk_id, gu.entity"; // fk_id are permission id and entity is entity of the group
 $sql .= " FROM " . MAIN_DB_PREFIX . "usergroup_rights as gr,";
-$sql .= " " . MAIN_DB_PREFIX . "usergroup_user as gu";  // all groups of a user
+$sql .= " " . MAIN_DB_PREFIX . "usergroup_user as gu"; // all groups of a user
 $sql .= " WHERE gr.entity = " . ((int) $entity);
 // The entity on the table gu=usergroup_user should be useless and should never be used because it is already into gr and r.
 // but when using MULTICOMPANY_TRANSVERSE_MODE, we may have inserted record that make rubbish result here due to the duplicate record of
@@ -507,7 +507,7 @@ 
         } else {
             $ishidden = 0;
         }
-        $isexpanded = ! $ishidden;
+        $isexpanded = !$ishidden;
         //var_dump("isexpanded=".$isexpanded);
 
         $permsgroupbyentitypluszero = array();
@@ -531,7 +531,7 @@ 
             } else {
                 $ishidden = 0;
             }
-            $isexpanded = ! $ishidden;
+            $isexpanded = !$ishidden;
             //var_dump('$obj->module='.$obj->module.' isexpanded='.$isexpanded);
 
             // Break detected, we get objMod

public/htdocs/user/group/card.php

Spacing +1 added lines, -1 removed lines

@@ -450,7 +450,7 @@
                 print '<td class="liste_titre right" width="5">&nbsp;</td>';
                 print "</tr>\n";
 
-                $object->fetch($object->id, '', true);  // true to force load of all users, member of the group
+                $object->fetch($object->id, '', true); // true to force load of all users, member of the group
 
                 if (!empty($object->members)) {
                     foreach ($object->members as $useringroup) {

public/htdocs/user/group/list.php

Spacing +2 added lines, -2 removed lines

@@ -66,7 +66,7 @@ 
 $object = new UserGroup($db);
 $extrafields = new ExtraFields($db);
 //$diroutputmassaction = $conf->mymodule->dir_output.'/temp/massgeneration/'.$user->id;
-$hookmanager->initHooks(array($contextpage));   // Note that conf->hooks_modules contains array of activated contexes
+$hookmanager->initHooks(array($contextpage)); // Note that conf->hooks_modules contains array of activated contexes
 
 // Fetch optionals attributes and labels
 $extrafields->fetch_name_optionals_label($object->table_element);
@@ -328,7 +328,7 @@ 
 }
 
 $varpage = empty($contextpage) ? $_SERVER["PHP_SELF"] : $contextpage;
-$htmlofselectarray = $form->multiSelectArrayWithCheckbox('selectedfields', $arrayfields, $varpage, getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN'));  // This also change content of $arrayfields with user setup
+$htmlofselectarray = $form->multiSelectArrayWithCheckbox('selectedfields', $arrayfields, $varpage, getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN')); // This also change content of $arrayfields with user setup
 $selectedfields = ($mode != 'kanban' ? $htmlofselectarray : '');
 $selectedfields .= (count($arrayofmassactions) ? $form->showCheckAddButtons('checkforselect', 1) : '');
 

public/htdocs/user/list.php

Spacing +3 added lines, -3 removed lines

@@ -154,7 +154,7 @@ 
 
 $object->fields = dol_sort_array($object->fields, 'position');
 $arrayfields = dol_sort_array($arrayfields, 'position');
-'@phan-var-force array<string,array{label:string,checked?:int<0,1>,position?:int,help?:string}> $arrayfields';  // dol_sort_array looses type for Phan
+'@phan-var-force array<string,array{label:string,checked?:int<0,1>,position?:int,help?:string}> $arrayfields'; // dol_sort_array looses type for Phan
 
 // Init search fields
 $search_all = trim((GETPOST('search_all', 'alphanohtml') != '') ? GETPOST('search_all', 'alphanohtml') : GETPOST('sall', 'alphanohtml'));
@@ -994,7 +994,7 @@ 
     }
 
     if (empty($obj->country_code)) {
-        $obj->country_code = '';        // TODO Add join in select with country table to get country_code
+        $obj->country_code = ''; // TODO Add join in select with country table to get country_code
     }
 
     // Store properties in $object
@@ -1202,7 +1202,7 @@ 
             if ($api_key) {
                 if ($canreadsecretapi) {
                     print '<span class="opacitymedium">';
-                    print showValueWithClipboardCPButton($object->api_key, 1, dol_trunc($api_key, 3));      // TODO Add an option to also reveal the hash, not only copy paste
+                    print showValueWithClipboardCPButton($object->api_key, 1, dol_trunc($api_key, 3)); // TODO Add an option to also reveal the hash, not only copy paste
                     print '</span>';
                 } else {
                     print '<span class="opacitymedium">' . $langs->trans("Hidden") . '</span>';

public/htdocs/user/bank.php

Spacing +2 added lines, -2 removed lines

@@ -89,8 +89,8 @@
 
 // Define value to know what current user can do on users
 $selfpermission = ($user->id == $id && $user->hasRight('user', 'self', 'creer'));
-$canadduser = (!empty($user->admin) || $user->hasRight('user', 'user', 'creer') || $user->hasRight('hrm', 'write_personal_information', 'write') );
-$canreaduser = (!empty($user->admin) || $user->hasRight('user', 'user', 'lire') || $user->hasRight('hrm', 'read_personal_information', 'read') );
+$canadduser = (!empty($user->admin) || $user->hasRight('user', 'user', 'creer') || $user->hasRight('hrm', 'write_personal_information', 'write'));
+$canreaduser = (!empty($user->admin) || $user->hasRight('user', 'user', 'lire') || $user->hasRight('hrm', 'read_personal_information', 'read'));
 $permissiontoaddbankaccount = ($user->hasRight('salaries', 'write') || $user->hasRight('hrm', 'employee', 'write') || $user->hasRight('user', 'user', 'creer') || $selfpermission);
 $permissiontoreadhr = $user->hasRight('hrm', 'read_personal_information', 'read') || $user->hasRight('hrm', 'write_personal_information', 'write');
 $permissiontowritehr = $user->hasRight('hrm', 'write_personal_information', 'write');

public/htdocs/user/document.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
     || (($user->id != $id) && $user->hasRight("user", "user", "password")));
 }
 
-$permissiontoadd = $caneditfield;   // Used by the include of actions_addupdatedelete.inc.php and actions_linkedfiles
+$permissiontoadd = $caneditfield; // Used by the include of actions_addupdatedelete.inc.php and actions_linkedfiles
 $permtoedit = $caneditfield;
 
 // Security check

public/htdocs/don/admin/donation.php

Spacing +4 added lines, -4 removed lines

@@ -119,10 +119,10 @@ 
 
     if (!$error) {
         setEventMessages($langs->trans("SetupSaved"), null, 'mesgs');
-        $action = '';   // To avoid to execute next actions
+        $action = ''; // To avoid to execute next actions
     } else {
         setEventMessages($langs->trans("Error"), null, 'errors');
-        $action = '';   // To avoid to execute next actions
+        $action = ''; // To avoid to execute next actions
     }
 }
 
@@ -137,10 +137,10 @@ 
 
     if (!$error) {
         setEventMessages($langs->trans("SetupSaved"), null, 'mesgs');
-        $action = '';   // To avoid to execute next actions
+        $action = ''; // To avoid to execute next actions
     } else {
         setEventMessages($langs->trans("Error"), null, 'errors');
-        $action = '';   // To avoid to execute next actions
+        $action = ''; // To avoid to execute next actions
     }
 }