alxarafe / alixar

public/htdocs/workstation/workstation_card.php

Spacing +5 added lines, -5 removed lines

@@ -90,10 +90,10 @@ 
 
 // Permissions
 $permissiontoread = $user->hasRight('workstation', 'workstation', 'read');
-$permissiontoadd = $user->hasRight('workstation', 'workstation', 'write');      // Used by the include of actions_addupdatedelete.inc.php and actions_lineupdown.inc.php
+$permissiontoadd = $user->hasRight('workstation', 'workstation', 'write'); // Used by the include of actions_addupdatedelete.inc.php and actions_lineupdown.inc.php
 $permissiontodelete = $user->hasRight('workstation', 'workstation', 'delete') || ($permissiontoadd && isset($object->status) && $object->status == $object::STATUS_DISABLED);
-$permissionnote = $user->hasRight('workstation', 'workstation', 'write');      // Used by the include of actions_setnotes.inc.php
-$permissiondellink = $user->hasRight('workstation', 'workstation', 'write');      // Used by the include of actions_dellink.inc.php
+$permissionnote = $user->hasRight('workstation', 'workstation', 'write'); // Used by the include of actions_setnotes.inc.php
+$permissiondellink = $user->hasRight('workstation', 'workstation', 'write'); // Used by the include of actions_dellink.inc.php
 
 $upload_dir = rtrim(getMultidirOutput($object, '', 1), '/');
 
@@ -107,7 +107,7 @@ 
  */
 
 $parameters = array();
-$reshook = $hookmanager->executeHooks('doActions', $parameters, $object, $action);   // Note that $action and $object may have been modified by some hooks
+$reshook = $hookmanager->executeHooks('doActions', $parameters, $object, $action); // Note that $action and $object may have been modified by some hooks
 if ($reshook < 0) {
     setEventMessages($hookmanager->error, $hookmanager->errors, 'errors');
 }
@@ -127,7 +127,7 @@ 
         }
     }
 
-    $triggermodname = 'WORKSTATION_WORKSTATION_MODIFY';    // Name of trigger action code to execute when we modify record
+    $triggermodname = 'WORKSTATION_WORKSTATION_MODIFY'; // Name of trigger action code to execute when we modify record
 
     // Actions cancel, add, update, update_extras, confirm_validate, confirm_delete, confirm_deleteline, confirm_clone, confirm_close, confirm_setdraft, confirm_reopen
     include DOL_DOCUMENT_ROOT . '/core/actions_addupdatedelete.inc.php';

public/htdocs/resource/agenda.php

Spacing +1 added lines, -1 removed lines

@@ -99,7 +99,7 @@
  */
 
 $parameters = array('id' => $id);
-$reshook = $hookmanager->executeHooks('doActions', $parameters, $object, $action);    // Note that $action and $object may have been modified by some hooks
+$reshook = $hookmanager->executeHooks('doActions', $parameters, $object, $action); // Note that $action and $object may have been modified by some hooks
 if ($reshook < 0) {
     setEventMessages($hookmanager->error, $hookmanager->errors, 'errors');
 }

public/htdocs/resource/contact.php

Spacing +1 added lines, -1 removed lines

@@ -45,7 +45,7 @@
 $object = new Dolresource($db);
 
 // Load object
-include DOL_DOCUMENT_ROOT . '/core/actions_fetchobject.inc.php';     // Must be include, not include_once
+include DOL_DOCUMENT_ROOT . '/core/actions_fetchobject.inc.php'; // Must be include, not include_once
 
 // Security check
 if ($user->socid) {

public/htdocs/resource/list.php

Spacing +2 added lines, -2 removed lines

@@ -159,7 +159,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
 
 include DOL_DOCUMENT_ROOT . '/core/actions_changeselectedfields.inc.php';
 
@@ -359,7 +359,7 @@ 
 
 // Output page
 
-llxHeader('', $title, $help_url, '', 0, 0, $morejs, $morecss, '', 'mod-resource page-list bodyforlist');    // Can use also classforhorizontalscrolloftabs instead of bodyforlist for no horizontal scroll
+llxHeader('', $title, $help_url, '', 0, 0, $morejs, $morecss, '', 'mod-resource page-list bodyforlist'); // Can use also classforhorizontalscrolloftabs instead of bodyforlist for no horizontal scroll
 
 $arrayofselected = is_array($toselect) ? $toselect : array();
 

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) : '');