Passed
Pull Request — dev (#14)
by Rafael
51:23
created
public/htdocs/expensereport/card.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -153,7 +153,7 @@  discard block
 block discarded – undo
153 153
 }
154 154
 $result = restrictedArea($user, 'expensereport', $object->id, 'expensereport');
155 155
 
156
-$permissiontoadd = $user->hasRight('expensereport', 'creer');   // Used by the include of actions_dellink.inc.php
156
+$permissiontoadd = $user->hasRight('expensereport', 'creer'); // Used by the include of actions_dellink.inc.php
157 157
 
158 158
 
159 159
 /*
@@ -2384,7 +2384,7 @@  discard block
 block discarded – undo
2384 2384
 
2385 2385
                         // Quantity
2386 2386
                         print '<td class="right">';
2387
-                        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
2387
+                        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
2388 2388
                         print '</td>';
2389 2389
 
2390 2390
                         //print '<td class="right">'.$langs->trans('AmountHT').'</td>';
Please login to merge, or discard this patch.
public/htdocs/expensereport/payment/list.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
 $search_date_endday     = GETPOSTINT('search_date_endday');
68 68
 $search_date_endmonth   = GETPOSTINT('search_date_endmonth');
69 69
 $search_date_endyear    = GETPOSTINT('search_date_endyear');
70
-$search_date_start      = dol_mktime(0, 0, 0, $search_date_startmonth, $search_date_startday, $search_date_startyear);  // Use tzserver
70
+$search_date_start      = dol_mktime(0, 0, 0, $search_date_startmonth, $search_date_startday, $search_date_startyear); // Use tzserver
71 71
 $search_date_end        = dol_mktime(23, 59, 59, $search_date_endmonth, $search_date_endday, $search_date_endyear);
72 72
 $search_user            = GETPOST('search_user', 'alpha');
73 73
 $search_payment_type    = GETPOST('search_payment_type');
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
     'pndf.amount'           => array('label' => "Amount", 'checked' => 1, 'position' => 70),
115 115
 );
116 116
 $arrayfields = dol_sort_array($arrayfields, 'position');
117
-'@phan-var-force array<string,array{label:string,checked?:int<0,1>,position?:int,help?:string}> $arrayfields';  // dol_sort_array looses type for Phan
117
+'@phan-var-force array<string,array{label:string,checked?:int<0,1>,position?:int,help?:string}> $arrayfields'; // dol_sort_array looses type for Phan
118 118
 
119 119
 // Initialize technical object to manage hooks of page. Note that conf->hooks_modules contains array of hook context
120 120
 $hookmanager->initHooks(array('paymentexpensereportlist'));
@@ -420,7 +420,7 @@  discard block
 block discarded – undo
420 420
 }
421 421
 
422 422
 $varpage = empty($contextpage) ? $_SERVER["PHP_SELF"] : $contextpage;
423
-$htmlofselectarray = $form->multiSelectArrayWithCheckbox('selectedfields', $arrayfields, $varpage, getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN'));  // This also change content of $arrayfields with user setup
423
+$htmlofselectarray = $form->multiSelectArrayWithCheckbox('selectedfields', $arrayfields, $varpage, getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN')); // This also change content of $arrayfields with user setup
424 424
 $selectedfields = ($mode != 'kanban' ? $htmlofselectarray : '');
425 425
 $selectedfields .= (count($arrayofmassactions) ? $form->showCheckAddButtons('checkforselect', 1) : '');
426 426
 
Please login to merge, or discard this patch.
public/htdocs/expensereport/list.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
 $search_date_startendday    = GETPOSTINT('search_date_startendday');
118 118
 $search_date_startendmonth  = GETPOSTINT('search_date_startendmonth');
119 119
 $search_date_startendyear   = GETPOSTINT('search_date_startendyear');
120
-$search_date_start          = dol_mktime(0, 0, 0, $search_date_startmonth, $search_date_startday, $search_date_startyear);  // Use tzserver
120
+$search_date_start          = dol_mktime(0, 0, 0, $search_date_startmonth, $search_date_startday, $search_date_startyear); // Use tzserver
121 121
 $search_date_startend       = dol_mktime(23, 59, 59, $search_date_startendmonth, $search_date_startendday, $search_date_startendyear);
122 122
 
123 123
 $search_date_endday         = GETPOSTINT('search_date_endday');
@@ -126,10 +126,10 @@  discard block
 block discarded – undo
126 126
 $search_date_endendday      = GETPOSTINT('search_date_endendday');
127 127
 $search_date_endendmonth    = GETPOSTINT('search_date_endendmonth');
128 128
 $search_date_endendyear     = GETPOSTINT('search_date_endendyear');
129
-$search_date_end            = dol_mktime(0, 0, 0, $search_date_endmonth, $search_date_endday, $search_date_endyear);    // Use tzserver
129
+$search_date_end            = dol_mktime(0, 0, 0, $search_date_endmonth, $search_date_endday, $search_date_endyear); // Use tzserver
130 130
 $search_date_endend         = dol_mktime(23, 59, 59, $search_date_endendmonth, $search_date_endendday, $search_date_endendyear);
131 131
 
132
-$optioncss    = GETPOST('optioncss', 'alpha');
132
+$optioncss = GETPOST('optioncss', 'alpha');
133 133
 
134 134
 if ($search_status == '') {
135 135
     $search_status = -1;
@@ -588,7 +588,7 @@  discard block
 block discarded – undo
588 588
 }
589 589
 
590 590
 $varpage = empty($contextpage) ? $_SERVER["PHP_SELF"] : $contextpage;
591
-$htmlofselectarray = $form->multiSelectArrayWithCheckbox('selectedfields', $arrayfields, $varpage, getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN'));  // This also change content of $arrayfields with user setup
591
+$htmlofselectarray = $form->multiSelectArrayWithCheckbox('selectedfields', $arrayfields, $varpage, getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN')); // This also change content of $arrayfields with user setup
592 592
 $selectedfields = ($mode != 'kanban' ? $htmlofselectarray : '');
593 593
 $selectedfields .= (count($arrayofmassactions) ? $form->showCheckAddButtons('checkforselect', 1) : '');
594 594
 
Please login to merge, or discard this patch.
public/htdocs/expensereport/document.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -96,7 +96,7 @@
 block discarded – undo
96 96
     }
97 97
 }
98 98
 
99
-$permissiontoadd = $user->hasRight('expensereport', 'creer');   // Used by the include of actions_dellink.inc.php
99
+$permissiontoadd = $user->hasRight('expensereport', 'creer'); // Used by the include of actions_dellink.inc.php
100 100
 
101 101
 
102 102
 /*
Please login to merge, or discard this patch.
public/htdocs/expensereport/index.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -141,7 +141,7 @@
 block discarded – undo
141 141
 $dataseries = dol_sort_array($dataseries, 1, 'desc');
142 142
 
143 143
 // Merge all entries after the $KEEPNFIRST one into one entry called "Other..." (to avoid to have too much entries in graphic).
144
-$KEEPNFIRST = 7;    // Keep first $KEEPNFIRST one + 1 with the remain
144
+$KEEPNFIRST = 7; // Keep first $KEEPNFIRST one + 1 with the remain
145 145
 $i = 0;
146 146
 if (count($dataseries) > ($KEEPNFIRST + 1)) {
147 147
     foreach ($dataseries as $key => $val) {
Please login to merge, or discard this patch.
public/htdocs/public/webportal/tpl/menu.tpl.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -113,7 +113,7 @@
 block discarded – undo
113 113
     'maxTopMenu' => & $maxTopMenu
114 114
 );
115 115
 
116
-$reshook = $hookmanager->executeHooks('PrintTopMenu', $parameters, $context, $context->action);    // Note that $action and $object may have been modified by hook
116
+$reshook = $hookmanager->executeHooks('PrintTopMenu', $parameters, $context, $context->action); // Note that $action and $object may have been modified by hook
117 117
 if ($reshook < 0) {
118 118
     $context->setEventMessages($hookmanager->error, $hookmanager->errors, 'errors');
119 119
 }
Please login to merge, or discard this patch.
public/htdocs/public/webportal/logout.php 1 patch
Braces   +3 added lines, -1 removed lines patch added patch discarded remove patch
@@ -49,7 +49,9 @@
 block discarded – undo
49 49
 }
50 50
 
51 51
 // Not sure this is required
52
-if (isset($_SESSION['webportal_logged_thirdparty_account_id'])) unset($_SESSION['webportal_logged_thirdparty_account_id']);
52
+if (isset($_SESSION['webportal_logged_thirdparty_account_id'])) {
53
+    unset($_SESSION['webportal_logged_thirdparty_account_id']);
54
+}
53 55
 
54 56
 if (GETPOST('noredirect')) {
55 57
     return;
Please login to merge, or discard this patch.
public/htdocs/fourn/ajax/getSupplierPrices.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -115,7 +115,7 @@
 block discarded – undo
115 115
             }
116 116
         }
117 117
 
118
-        $prices[] = array("id" => 'pmpprice', "price" => price2num($price), "label" => $langs->trans("PMPValueShort") . ': ' . price($price, 0, $langs, 0, 0, -1, $conf->currency), "title" => $langs->trans("PMPValueShort") . ': ' . price($price, 0, $langs, 0, 0, -1, $conf->currency));  // For price field, we must use price2num(), for label or title, price()
118
+        $prices[] = array("id" => 'pmpprice', "price" => price2num($price), "label" => $langs->trans("PMPValueShort") . ': ' . price($price, 0, $langs, 0, 0, -1, $conf->currency), "title" => $langs->trans("PMPValueShort") . ': ' . price($price, 0, $langs, 0, 0, -1, $conf->currency)); // For price field, we must use price2num(), for label or title, price()
119 119
     }
120 120
 
121 121
     // Add price for costprice (at end)
Please login to merge, or discard this patch.
public/htdocs/blockedlog/admin/blockedlog_list.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -157,7 +157,7 @@  discard block
 block discarded – undo
157 157
     $previoushash = '';
158 158
     $firstid = '';
159 159
 
160
-    if (! (GETPOSTINT('yeartoexport') > 0)) {
160
+    if (!(GETPOSTINT('yeartoexport') > 0)) {
161 161
         setEventMessages($langs->trans("ErrorFieldRequired", $langs->transnoentitiesnoconv("Year")), null, "errors");
162 162
         $error++;
163 163
     } else {
@@ -581,7 +581,7 @@  discard block
 block discarded – undo
581 581
 
582 582
             // Link to source object
583 583
             print '<td class="tdoverflowmax150"' . (preg_match('/<a/', $object_link) ? '' : 'title="' . dol_escape_htmltag(dol_string_nohtmltag($object_link . ' - ' . $object_link_title)) . '"') . '>';
584
-            print '<!-- object_link -->';   // $object_link can be a '<a href' link or a text
584
+            print '<!-- object_link -->'; // $object_link can be a '<a href' link or a text
585 585
             print $object_link;
586 586
             print '</td>';
587 587
 
Please login to merge, or discard this patch.