Passed
Push — GENERAL_BUG_REVIEW_240911 ( d4fdcf...6dbc7d )
by Rafael
51:37
created
public/htdocs/salaries/paiement_salary.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -108,7 +108,7 @@
 block discarded – undo
108 108
             // Create a line of payments
109 109
             $paiement = new PaymentSalary($db);
110 110
             $paiement->fk_salary    = $id;
111
-            $paiement->chid         = $id;  // deprecated
111
+            $paiement->chid         = $id; // deprecated
112 112
             $paiement->datep        = $datepaye;
113 113
             $paiement->amounts      = $amounts; // Tableau de montant
114 114
             $paiement->fk_typepayment = GETPOSTINT("paiementtype");
Please login to merge, or discard this patch.
public/htdocs/delivery/card.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -91,10 +91,10 @@
 block discarded – undo
91 91
  */
92 92
 
93 93
 $parameters = array();
94
-$reshook = $hookmanager->executeHooks('doActions', $parameters, $object, $action);       // Note that $action and $object may have been modified by some hooks
94
+$reshook = $hookmanager->executeHooks('doActions', $parameters, $object, $action); // Note that $action and $object may have been modified by some hooks
95 95
 // Delete Link
96 96
 $permissiondellink = $user->hasRight('expedition', 'delivery', 'supprimer'); // Used by the include of actions_dellink.inc.php
97
-include DOL_DOCUMENT_ROOT . '/core/actions_dellink.inc.php';     // Must be include, not include_once
97
+include DOL_DOCUMENT_ROOT . '/core/actions_dellink.inc.php'; // Must be include, not include_once
98 98
 
99 99
 if ($action == 'add') {
100 100
     $db->begin();
Please login to merge, or discard this patch.
public/htdocs/intracommreport/card.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
 $label = (string) GETPOST('label', 'alphanohtml');
51 51
 
52 52
 $exporttype = GETPOSTISSET('exporttype') ? GETPOST('exporttype', 'alphanohtml') : 'deb'; // DEB or DES
53
-$type_declaration = (string) GETPOST('type_declaration', 'alphanohtml');    // 'introduction' or 'expedition'
53
+$type_declaration = (string) GETPOST('type_declaration', 'alphanohtml'); // 'introduction' or 'expedition'
54 54
 
55 55
 $backtopage = GETPOST('backtopage', 'alpha');
56 56
 
@@ -124,8 +124,8 @@  discard block
 block discarded – undo
124 124
 
125 125
 if ($action == 'add' && $permissiontoadd) {
126 126
     $object->label = trim($label);
127
-    $object->exporttype = trim($exporttype);        // 'des' or 'deb'
128
-    $object->type_declaration =  $type_declaration; // 'introduction' or 'expedition'
127
+    $object->exporttype = trim($exporttype); // 'des' or 'deb'
128
+    $object->type_declaration = $type_declaration; // 'introduction' or 'expedition'
129 129
     //$object->subscription = (int) $subscription;
130 130
 
131 131
     // Fill array 'array_options' with data from add form
Please login to merge, or discard this patch.
public/htdocs/intracommreport/list.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
 
142 142
 $object->fields = dol_sort_array($object->fields, 'position');
143 143
 $arrayfields = dol_sort_array($arrayfields, 'position');
144
-'@phan-var-force array<string,array{label:string,checked?:int<0,1>,position?:int,help?:string}> $arrayfields';  // dol_sort_array looses type for Phan
144
+'@phan-var-force array<string,array{label:string,checked?:int<0,1>,position?:int,help?:string}> $arrayfields'; // dol_sort_array looses type for Phan
145 145
 
146 146
 // Security check
147 147
 if ($search_type == '0') {
@@ -433,7 +433,7 @@  discard block
 block discarded – undo
433 433
 }
434 434
 
435 435
 $varpage = empty($contextpage) ? $_SERVER["PHP_SELF"] : $contextpage;
436
-$htmlofselectarray = $form->multiSelectArrayWithCheckbox('selectedfields', $arrayfields, $varpage, getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN'));  // This also change content of $arrayfields with user setup
436
+$htmlofselectarray = $form->multiSelectArrayWithCheckbox('selectedfields', $arrayfields, $varpage, getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN')); // This also change content of $arrayfields with user setup
437 437
 $selectedfields = ($mode != 'kanban' ? $htmlofselectarray : '');
438 438
 $selectedfields .= (count($arrayofmassactions) ? $form->showCheckAddButtons('checkforselect', 1) : '');
439 439
 
Please login to merge, or discard this patch.
public/htdocs/core/ajax/box.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@
 block discarded – undo
45 45
 
46 46
 $boxid = GETPOSTINT('boxid');
47 47
 $boxorder = GETPOST('boxorder');
48
-$zone = GETPOST('zone');        // Can be '0' or '1' or 'pagename'...
48
+$zone = GETPOST('zone'); // Can be '0' or '1' or 'pagename'...
49 49
 $userid = GETPOSTINT('userid');
50 50
 
51 51
 // Security check
Please login to merge, or discard this patch.
public/htdocs/core/ajax/selectobject.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -93,9 +93,9 @@
 block discarded – undo
93 93
     $vartmp = (empty($InfoFieldList[3]) ? '' : $InfoFieldList[3]);
94 94
     $reg = array();
95 95
     if (preg_match('/^.*:(\w*)$/', $vartmp, $reg)) {
96
-        $InfoFieldList[4] = $reg[1];    // take the sort field
96
+        $InfoFieldList[4] = $reg[1]; // take the sort field
97 97
     }
98
-    $InfoFieldList[3] = preg_replace('/:\w*$/', '', $vartmp);    // take the filter field
98
+    $InfoFieldList[3] = preg_replace('/:\w*$/', '', $vartmp); // take the filter field
99 99
 
100 100
     $classname = $InfoFieldList[0];
101 101
     $classpath = $InfoFieldList[1];
Please login to merge, or discard this patch.
public/htdocs/core/ajax/locationincoterms.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@
 block discarded – undo
46 46
 
47 47
 // Security check
48 48
 if (!isModEnabled('incoterm')) {
49
-    httponly_accessforbidden("Module incoterm not enabled");    // This includes the exit.
49
+    httponly_accessforbidden("Module incoterm not enabled"); // This includes the exit.
50 50
 }
51 51
 // There is no other permission on this component. Everybody connected can read content of the incoterm table
52 52
 
Please login to merge, or discard this patch.
public/htdocs/core/ajax/objectonoff.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
 // Load object according to $id and $element
55 55
 $object = fetchObjectByElement($id, $element);
56 56
 if (!is_object($object)) {
57
-    httponly_accessforbidden("Bad value for combination of parameters element/field: Object not found.");   // This includes the exit.
57
+    httponly_accessforbidden("Bad value for combination of parameters element/field: Object not found."); // This includes the exit.
58 58
 }
59 59
 
60 60
 $object->fields[$field] = array('type' => $format, 'enabled' => 1);
@@ -72,7 +72,7 @@  discard block
 block discarded – undo
72 72
 if (!empty($user->socid)) {
73 73
     $socid = $user->socid;
74 74
     if (!empty($object->socid) && $socid != $object->socid) {
75
-        httponly_accessforbidden("Access on object not allowed for this external user.");   // This includes the exit.
75
+        httponly_accessforbidden("Access on object not allowed for this external user."); // This includes the exit.
76 76
     }
77 77
 }
78 78
 
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
 } elseif ($element == 'product' && in_array($field, array('tosell', 'tobuy', 'tobatch'))) { // Special case for products
84 84
     restrictedArea($user, 'produit|service', $object, 'product&product', '', '', 'rowid');
85 85
 } else {
86
-    httponly_accessforbidden("Bad value for combination of parameters element/field: Field not supported.");    // This includes the exit.
86
+    httponly_accessforbidden("Bad value for combination of parameters element/field: Field not supported."); // This includes the exit.
87 87
 }
88 88
 
89 89
 
Please login to merge, or discard this patch.
htdocs/core/modules/supplier_order/mod_commande_fournisseur_muguet.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@
 block discarded – undo
57 57
      */
58 58
     public $name = 'Muguet';
59 59
 
60
-    public $prefix = 'PO';  // PO for "Purchase Order"
60
+    public $prefix = 'PO'; // PO for "Purchase Order"
61 61
 
62 62
 
63 63
     /**
Please login to merge, or discard this patch.