alxarafe / alixar

public/htdocs/eventorganization/conferenceorboothattendee_card.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
 $contextpage = GETPOST('contextpage', 'aZ') ? GETPOST('contextpage', 'aZ') : 'conferenceorboothattendeecard'; // To manage different context of search
 $backtopage = GETPOST('backtopage', 'alpha');
 $backtopageforcancel = GETPOST('backtopageforcancel', 'alpha');
-$lineid   = GETPOSTINT('lineid');
+$lineid = GETPOSTINT('lineid');
 $mode = GETPOST('mode', 'alpha');
 
 $conf_or_booth_id = GETPOSTINT('conforboothid');

public/htdocs/eventorganization/conferenceorboothattendee_list.php

Spacing +11 added lines, -11 removed lines

@@ -46,15 +46,15 @@ 
 $langs->loadLangs(array("eventorganization", "other", "projects", "bills"));
 
 // Get Parameters
-$action      = GETPOST('action', 'aZ09') ? GETPOST('action', 'aZ09') : 'view';   // The action 'add', 'create', 'edit', 'update', 'view', ...
-$massaction  = GETPOST('massaction', 'alpha');   // The bulk action (combo box choice into lists)
-$show_files  = GETPOSTINT('show_files');   // Show files area generated by bulk actions ?
-$confirm     = GETPOST('confirm', 'alpha');   // Result of a confirmation
-$cancel      = GETPOST('cancel', 'alpha');   // We click on a Cancel button
-$toselect    = GETPOST('toselect', 'array');   // Array of ids of elements selected into a list
+$action      = GETPOST('action', 'aZ09') ? GETPOST('action', 'aZ09') : 'view'; // The action 'add', 'create', 'edit', 'update', 'view', ...
+$massaction  = GETPOST('massaction', 'alpha'); // The bulk action (combo box choice into lists)
+$show_files  = GETPOSTINT('show_files'); // Show files area generated by bulk actions ?
+$confirm     = GETPOST('confirm', 'alpha'); // Result of a confirmation
+$cancel      = GETPOST('cancel', 'alpha'); // We click on a Cancel button
+$toselect    = GETPOST('toselect', 'array'); // Array of ids of elements selected into a list
 $contextpage = GETPOST('contextpage', 'aZ') ? GETPOST('contextpage', 'aZ') : str_replace('_', '', basename(dirname(__FILE__)) . basename(__FILE__, '.php')); // To manage different context of search
-$backtopage  = GETPOST('backtopage', 'alpha');   // Go back to a dedicated page
-$optioncss   = GETPOST('optioncss', 'aZ');   // Option for the css output (always '' except when 'print')
+$backtopage  = GETPOST('backtopage', 'alpha'); // Go back to a dedicated page
+$optioncss   = GETPOST('optioncss', 'aZ'); // Option for the css output (always '' except when 'print')
 $mode        = GETPOST('mode', 'aZ');
 
 $id = GETPOSTINT('id');
@@ -95,7 +95,7 @@ 
 
 // Default sort order (if not yet defined by previous GETPOST)
 if (!$sortfield) {
-    reset($object->fields);                 // Reset is required to avoid key() to return null.
+    reset($object->fields); // Reset is required to avoid key() to return null.
     $sortfield = "t." . key($object->fields); // Set here default search field. By default 1st field in definition.
 }
 if (!$sortorder) {
@@ -806,7 +806,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) : '');
 
@@ -898,7 +898,7 @@ 
     } elseif (in_array($val['type'], array('double(24,8)', 'double(6,3)', 'integer', 'real', 'price')) && !in_array($key, array('id', 'rowid', 'ref', 'status')) && $val['label'] != 'TechnicalID' && empty($val['arrayofkeyval'])) {
         $cssforfield .= ($cssforfield ? ' ' : '') . 'right';
     }
-    $cssforfield = preg_replace('/small\s*/', '', $cssforfield);    // the 'small' css must not be used for the title label
+    $cssforfield = preg_replace('/small\s*/', '', $cssforfield); // the 'small' css must not be used for the title label
     if (!empty($arrayfields['t.' . $key]['checked'])) {
         print getTitleFieldOfList($arrayfields['t.' . $key]['label'], 0, $_SERVER['PHP_SELF'], 't.' . $key, '', $param, ($cssforfield ? 'class="' . $cssforfield . '"' : ''), $sortfield, $sortorder, ($cssforfield ? $cssforfield . ' ' : ''), 0, (empty($val['helplist']) ? '' : $val['helplist'])) . "\n";
         $totalarray['nbfield']++;

public/htdocs/ticket/card.php

Braces +6 added lines, -2 removed lines

@@ -192,7 +192,9 @@ 
 
     if (($action == 'add' || ($action == 'update' && $object->status < Ticket::STATUS_CLOSED)) && $permissiontoadd) {
         $ifErrorAction = $action == 'add' ? 'create' : 'edit';
-        if ($action == 'add') $object->track_id = null;
+        if ($action == 'add') {
+            $object->track_id = null;
+        }
         $error = 0;
 
         $fieldsToCheck = [
@@ -344,7 +346,9 @@ 
                 $db->rollback();
                 setEventMessages($object->error, $object->errors, 'errors');
             }
-        } else $action = $ifErrorAction;
+        } else {
+            $action = $ifErrorAction;
+        }
     }
 
     // Mark as Read

Spacing +5 added lines, -5 removed lines

@@ -318,7 +318,7 @@ 
 
                 if (!$error) {
                     // File transfer
-                    $object->copyFilesForTicket('');        // trackid is forced to '' because files were uploaded when no id for ticket exists yet and trackid was ''
+                    $object->copyFilesForTicket(''); // trackid is forced to '' because files were uploaded when no id for ticket exists yet and trackid was ''
                 }
             }
             if (!$error) {
@@ -700,7 +700,7 @@ 
 
     print load_fiche_titre($langs->trans('NewTicket'), '', 'ticket');
 
-    $formticket->trackid = '';      // TODO Use a unique key 'tic' to avoid conflict in upload file feature
+    $formticket->trackid = ''; // TODO Use a unique key 'tic' to avoid conflict in upload file feature
     $formticket->withfromsocid = $socid ? $socid : $user->socid;
     $formticket->withfromcontactid = $contactid ? $contactid : '';
     $formticket->withtitletopic = 1;
@@ -728,7 +728,7 @@ 
 
     print dol_get_fiche_head($head, 'tabTicket', $langs->trans('Ticket'), -1, 'ticket');
 
-    $formticket->trackid = $object->track_id;        // TODO Use a unique key 'tic' to avoid conflict in upload file feature
+    $formticket->trackid = $object->track_id; // TODO Use a unique key 'tic' to avoid conflict in upload file feature
     $formticket->withfromsocid = $object->socid;
     $formticket->withtitletopic = 1;
     //  $formticket->withnotifytiersatcreate = ($notifyTiers ? 1 : (getDolGlobalString('TICKET_CHECK_NOTIFY_THIRDPARTY_AT_CREATION') ? 1 : 0));
@@ -873,9 +873,9 @@ 
         }
 
         if (!$user->socid && getDolGlobalString('TICKET_LIMIT_VIEW_ASSIGNED_ONLY')) {
-            $object->next_prev_filter = "te.fk_user_assign = " . ((int)$user->id);
+            $object->next_prev_filter = "te.fk_user_assign = " . ((int) $user->id);
         } elseif ($user->socid > 0) {
-            $object->next_prev_filter = "te.fk_soc = " . ((int)$user->socid);
+            $object->next_prev_filter = "te.fk_soc = " . ((int) $user->socid);
         }
 
         $head = ticket_prepare_head($object);

public/htdocs/expensereport/ajax/ajaxik.php

Spacing +4 added lines, -4 removed lines

@@ -69,21 +69,21 @@
 
 
 if (empty($fk_expense) || $fk_expense < 0) {
-    $rep->errorMessage =   $langs->transnoentitiesnoconv('ErrorBadValueForParameter', $fk_expense, 'fk_expense');
+    $rep->errorMessage = $langs->transnoentitiesnoconv('ErrorBadValueForParameter', $fk_expense, 'fk_expense');
 } elseif (empty($fk_c_exp_tax_cat) || $fk_c_exp_tax_cat < 0) {
-    $rep->errorMessage =  $langs->transnoentitiesnoconv('ErrorBadValueForParameter', $fk_c_exp_tax_cat, 'fk_c_exp_tax_cat');
+    $rep->errorMessage = $langs->transnoentitiesnoconv('ErrorBadValueForParameter', $fk_c_exp_tax_cat, 'fk_c_exp_tax_cat');
 
     $rep->response_status = 'error';
 } else {
     // @see ndfp.class.php:3576 (method: compute_total_km)
     $expense = new ExpenseReport($db);
     if ($expense->fetch($fk_expense) <= 0) {
-        $rep->errorMessage =  $langs->transnoentitiesnoconv('ErrorRecordNotFound');
+        $rep->errorMessage = $langs->transnoentitiesnoconv('ErrorRecordNotFound');
         $rep->response_status = 'error';
     } else {
         $userauthor = new User($db);
         if ($userauthor->fetch($expense->fk_user_author) <= 0) {
-            $rep->errorMessage =  $langs->transnoentitiesnoconv('ErrorRecordNotFound');
+            $rep->errorMessage = $langs->transnoentitiesnoconv('ErrorRecordNotFound');
             $rep->response_status = 'error';
         } else {
             $expense = new ExpenseReport($db);

public/htdocs/expensereport/card.php

Spacing +2 added lines, -2 removed lines

@@ -153,7 +153,7 @@ 
 }
 $result = restrictedArea($user, 'expensereport', $object->id, 'expensereport');
 
-$permissiontoadd = $user->hasRight('expensereport', 'creer');   // Used by the include of actions_dellink.inc.php
+$permissiontoadd = $user->hasRight('expensereport', 'creer'); // Used by the include of actions_dellink.inc.php
 
 
 /*
@@ -2384,7 +2384,7 @@ 
 
                         // Quantity
                         print '<td class="right">';
-                        print '<input type="text" min="0" class="input_qty right maxwidth50"  name="qty" value="' . dol_escape_htmltag($line->qty) . '" />';  // We must be able to enter decimal qty
+                        print '<input type="text" min="0" class="input_qty right maxwidth50"  name="qty" value="' . dol_escape_htmltag($line->qty) . '" />'; // We must be able to enter decimal qty
                         print '</td>';
 
                         //print '<td class="right">'.$langs->trans('AmountHT').'</td>';

public/htdocs/expensereport/payment/list.php

Spacing +3 added lines, -3 removed lines

@@ -67,7 +67,7 @@ 
 $search_date_endday     = GETPOSTINT('search_date_endday');
 $search_date_endmonth   = GETPOSTINT('search_date_endmonth');
 $search_date_endyear    = GETPOSTINT('search_date_endyear');
-$search_date_start      = dol_mktime(0, 0, 0, $search_date_startmonth, $search_date_startday, $search_date_startyear);  // Use tzserver
+$search_date_start      = dol_mktime(0, 0, 0, $search_date_startmonth, $search_date_startday, $search_date_startyear); // Use tzserver
 $search_date_end        = dol_mktime(23, 59, 59, $search_date_endmonth, $search_date_endday, $search_date_endyear);
 $search_user            = GETPOST('search_user', 'alpha');
 $search_payment_type    = GETPOST('search_payment_type');
@@ -114,7 +114,7 @@ 
     'pndf.amount'           => array('label' => "Amount", 'checked' => 1, 'position' => 70),
 );
 $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
 
 // Initialize technical object to manage hooks of page. Note that conf->hooks_modules contains array of hook context
 $hookmanager->initHooks(array('paymentexpensereportlist'));
@@ -420,7 +420,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/expensereport/list.php

Spacing +4 added lines, -4 removed lines

@@ -117,7 +117,7 @@ 
 $search_date_startendday    = GETPOSTINT('search_date_startendday');
 $search_date_startendmonth  = GETPOSTINT('search_date_startendmonth');
 $search_date_startendyear   = GETPOSTINT('search_date_startendyear');
-$search_date_start          = dol_mktime(0, 0, 0, $search_date_startmonth, $search_date_startday, $search_date_startyear);  // Use tzserver
+$search_date_start          = dol_mktime(0, 0, 0, $search_date_startmonth, $search_date_startday, $search_date_startyear); // Use tzserver
 $search_date_startend       = dol_mktime(23, 59, 59, $search_date_startendmonth, $search_date_startendday, $search_date_startendyear);
 
 $search_date_endday         = GETPOSTINT('search_date_endday');
@@ -126,10 +126,10 @@ 
 $search_date_endendday      = GETPOSTINT('search_date_endendday');
 $search_date_endendmonth    = GETPOSTINT('search_date_endendmonth');
 $search_date_endendyear     = GETPOSTINT('search_date_endendyear');
-$search_date_end            = dol_mktime(0, 0, 0, $search_date_endmonth, $search_date_endday, $search_date_endyear);    // Use tzserver
+$search_date_end            = dol_mktime(0, 0, 0, $search_date_endmonth, $search_date_endday, $search_date_endyear); // Use tzserver
 $search_date_endend         = dol_mktime(23, 59, 59, $search_date_endendmonth, $search_date_endendday, $search_date_endendyear);
 
-$optioncss    = GETPOST('optioncss', 'alpha');
+$optioncss = GETPOST('optioncss', 'alpha');
 
 if ($search_status == '') {
     $search_status = -1;
@@ -588,7 +588,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/expensereport/document.php

Spacing +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
     }
 }
 
-$permissiontoadd = $user->hasRight('expensereport', 'creer');   // Used by the include of actions_dellink.inc.php
+$permissiontoadd = $user->hasRight('expensereport', 'creer'); // Used by the include of actions_dellink.inc.php
 
 
 /*

public/htdocs/expensereport/index.php

Spacing +1 added lines, -1 removed lines

@@ -141,7 +141,7 @@
 $dataseries = dol_sort_array($dataseries, 1, 'desc');
 
 // Merge all entries after the $KEEPNFIRST one into one entry called "Other..." (to avoid to have too much entries in graphic).
-$KEEPNFIRST = 7;    // Keep first $KEEPNFIRST one + 1 with the remain
+$KEEPNFIRST = 7; // Keep first $KEEPNFIRST one + 1 with the remain
 $i = 0;
 if (count($dataseries) > ($KEEPNFIRST + 1)) {
     foreach ($dataseries as $key => $val) {