alxarafe / alixar

public/htdocs/variants/list.php

Spacing +4 added lines, -4 removed lines

@@ -143,7 +143,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
 
 $permissiontoread = $user->hasRight('variants', 'read');
 $permissiontoadd = $user->hasRight('variants', 'write');
@@ -324,7 +324,7 @@ 
 //}
 // Add where from hooks
 $parameters = array();
-$reshook = $hookmanager->executeHooks('printFieldListGroupBy', $parameters, $object);    // Note that $action and $object may have been modified by hook
+$reshook = $hookmanager->executeHooks('printFieldListGroupBy', $parameters, $object); // Note that $action and $object may have been modified by hook
 $sql .= $hookmanager->resPrint;
 $sql = preg_replace("/,\s*$/", "", $sql);
 
@@ -513,7 +513,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) : '');
 
@@ -617,7 +617,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/imports/emptyexample.php

Spacing +2 added lines, -2 removed lines

@@ -101,8 +101,8 @@
 
 
 // List of targets fields
-$headerlinefields = array();    // Array of fields (label to show)
-$contentlinevalues = array();   // Array of example values
+$headerlinefields = array(); // Array of fields (label to show)
+$contentlinevalues = array(); // Array of example values
 $i = 0;
 foreach ($fieldstarget as $code => $label) {
     $withoutstar = preg_replace('/\*/', '', $fieldstarget[$code]);

public/htdocs/imports/import.php

Spacing +7 added lines, -7 removed lines

@@ -145,7 +145,7 @@ 
 $endatlinenb        = (GETPOST('endatlinenb') ? GETPOST('endatlinenb') : '');
 $updatekeys         = (GETPOST('updatekeys', 'array') ? GETPOST('updatekeys', 'array') : array());
 $separator          = (GETPOST('separator', 'nohtml') ? GETPOST('separator', 'nohtml', 3) : '');
-$enclosure          = (GETPOST('enclosure', 'nohtml') ? GETPOST('enclosure', 'nohtml') : '"');  // We must use 'nohtml' and not 'alphanohtml' because we must accept "
+$enclosure          = (GETPOST('enclosure', 'nohtml') ? GETPOST('enclosure', 'nohtml') : '"'); // We must use 'nohtml' and not 'alphanohtml' because we must accept "
 $charset            = GETPOST('charset', 'aZ09');
 $separator_used     = str_replace('\t', "\t", $separator);
 
@@ -615,7 +615,7 @@ 
     $maxfilesizearray = getMaxFileSizeArray();
     $maxmin = $maxfilesizearray['maxmin'];
     if ($maxmin > 0) {
-        print '<input type="hidden" name="MAX_FILE_SIZE" value="' . ($maxmin * 1024) . '">';    // MAX_FILE_SIZE must precede the field type=file
+        print '<input type="hidden" name="MAX_FILE_SIZE" value="' . ($maxmin * 1024) . '">'; // MAX_FILE_SIZE must precede the field type=file
     }
     print '<input type="file" name="userfile" size="20" maxlength="80"> &nbsp; &nbsp; ';
     $out = (!getDolGlobalString('MAIN_UPLOAD_DOC') ? ' disabled' : '');
@@ -734,7 +734,7 @@ 
     //var_dump($_SESSION["dol_array_match_file_to_database_select"]);
     $serialized_array_match_file_to_database = isset($_SESSION["dol_array_match_file_to_database_select"]) ? $_SESSION["dol_array_match_file_to_database_select"] : '';
     $fieldsarray = explode(',', $serialized_array_match_file_to_database);
-    $array_match_file_to_database = array();        // Same than $fieldsarray but with mapped value only  (col1 => 's.fielda', col2 => 's.fieldb'...)
+    $array_match_file_to_database = array(); // Same than $fieldsarray but with mapped value only  (col1 => 's.fielda', col2 => 's.fieldb'...)
     foreach ($fieldsarray as $elem) {
         $tabelem = explode('=', $elem, 2);
         $key = $tabelem[0];
@@ -1088,7 +1088,7 @@ 
     foreach ($fieldstarget as $code => $line) {
         //var_dump($line);
 
-        $tmparray = explode('|', $line["label"]);   // If label of field is several translation keys separated with |
+        $tmparray = explode('|', $line["label"]); // If label of field is several translation keys separated with |
         $labeltoshow = '';
         foreach ($tmparray as $tmpkey => $tmpval) {
             $labeltoshow .= ($labeltoshow ? ' ' . $langs->trans('or') . ' ' : '') . $langs->transnoentities($tmpval);
@@ -1118,7 +1118,7 @@ 
     //var_dump($fieldssource);
     //var_dump($fieldstarget);
 
-    $modetoautofillmapping = 'session';     // Use setup in session
+    $modetoautofillmapping = 'session'; // Use setup in session
     if ($initialloadofstep4) {
         $modetoautofillmapping = 'guess';
     }
@@ -1330,7 +1330,7 @@ 
         // - Then we set to disabled all fields that are selected
         print 'function setOptionsToDisabled() {' . "\n";
         print '		console.log("Remove the disabled flag everywhere");' . "\n";
-        print '		$("select.targetselectchange").not($( this )).find(\'option\').prop("disabled", false);' . "\n";  // Enable all options
+        print '		$("select.targetselectchange").not($( this )).find(\'option\').prop("disabled", false);' . "\n"; // Enable all options
         print '		arrayofselectedvalues = [];' . "\n";
 
         print '		$("select.targetselectchange").each(function(){' . "\n";
@@ -1347,7 +1347,7 @@ 
         print '     $.each(arrayofselectedvalues, function(key, value) {' . "\n"; // Loop on each selected value
         print '         if (value != -1) {' . "\n";
         print '     		console.log("Process key="+key+" value="+value+" to disable.");' . "\n";
-        print '				$("select.targetselectchange").find(\'option[value="\'+value+\'"]:not(:selected)\').prop("disabled", true);' . "\n";  // Set to disabled except if currently selected
+        print '				$("select.targetselectchange").find(\'option[value="\'+value+\'"]:not(:selected)\').prop("disabled", true);' . "\n"; // Set to disabled except if currently selected
         print '         }' . "\n";
         print '     });' . "\n";
         print '}' . "\n";

public/htdocs/imports/class/import.class.php

Indentation +4 added lines, -4 removed lines

@@ -157,7 +157,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load description int this->array_import_module, this->array_import_fields, ... of an importable dataset
      *
@@ -167,7 +167,7 @@ 
      */
     public function load_arrays($user, $filter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf;
 
         dol_syslog(get_class($this) . "::load_arrays user=" . $user->id . " filter=" . $filter);
@@ -287,7 +287,7 @@ 
 
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Build an import example file.
      *  Arrays this->array_export_xxx are already loaded for required datatoexport
@@ -300,7 +300,7 @@ 
      */
     public function build_example_file($model, $headerlinefields, $contentlinevalues, $datatoimport)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $indice = 0;

public/htdocs/webhook/target_list.php

Spacing +12 added lines, -12 removed lines

@@ -39,15 +39,15 @@ 
 $langs->loadLangs(array('other'));
 
 // 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
-$contextpage = GETPOST('contextpage', 'aZ') ? GETPOST('contextpage', 'aZ') : 'targetlist';    // 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')
+$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') : 'targetlist'; // 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')
 $mode        = GETPOST('mode', 'aZ');
 if (empty($mode)) {
     $mode = 'modulesetup';
@@ -82,7 +82,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) {
@@ -518,7 +518,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) : '');
 
@@ -612,7 +612,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/webhook/class/target.class.php

Indentation +2 added lines, -2 removed lines

@@ -783,7 +783,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the status
      *
@@ -793,7 +793,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
             global $langs;
 

public/htdocs/webhook/target_card.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
 global $conf, $db, $hookmanager, $langs, $user;
 
 // Load translation files required by the page
-$langs->loadLangs(array('other','admin'));
+$langs->loadLangs(array('other', 'admin'));
 
 // Get parameters
 $id = GETPOSTINT('id');
@@ -47,7 +47,7 @@ 
 $contextpage = GETPOST('contextpage', 'aZ') ? GETPOST('contextpage', 'aZ') : 'targetcard'; // To manage different context of search
 $backtopage = GETPOST('backtopage', 'alpha');
 $backtopageforcancel = GETPOST('backtopageforcancel', 'alpha');
-$lineid   = GETPOSTINT('lineid');
+$lineid = GETPOSTINT('lineid');
 
 // Initialize technical objects
 $object = new Target($db);

public/htdocs/reception/card.php

Indentation +1 added lines, -1 removed lines

@@ -966,7 +966,7 @@
             /**
              * @var array $suffix2numAsked map HTTP query parameter suffixes (like '1_0') to line indices so that
              *                             extrafields from HTTP query can be assigned to the correct dispatch line
-            */
+             */
             $suffix2numAsked = array();
             $dispatchLines = array();
 

Spacing +7 added lines, -7 removed lines

@@ -343,7 +343,7 @@ 
 
         // Loop lines to calculate $totalqty
         for ($i = 1; $i <= $num; $i++) {
-            $idl = "idl" . $i;    // id line source
+            $idl = "idl" . $i; // id line source
 
             //$sub_qty = array();
             //$subtotalqty = 0;
@@ -351,7 +351,7 @@ 
             //$j = 0;
             //$batch = "batchl".$i."_0";
             //$stockLocation = "ent1".$i."_0";
-            $qty = "qtyl" . $i;   // qty
+            $qty = "qtyl" . $i; // qty
 
             //reception line for product with no batch management and no multiple stock location
             if (GETPOST($qty, 'alpha') > 0) {
@@ -365,7 +365,7 @@ 
 
         if ($totalqty > 0) {  // There is at least one thing to ship
             for ($i = 1; $i <= $num; $i++) {
-                $idl = "idl" . $i;    // id line source
+                $idl = "idl" . $i; // id line source
                 $lineToTest = '';
                 $lineId = GETPOSTINT($idl);
                 foreach ($objectsrc->lines as $linesrc) {
@@ -1144,7 +1144,7 @@ 
 
                     print '<input type="hidden" name="productl' . $indiceAsked . '" value="' . $line->fk_product . '">';
 
-                    if (! array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
+                    if (!array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
                         print '<input type="hidden" name="productid' . $indiceAsked . '" value="' . $line->fk_product . '">';
 
                         // Show product and description
@@ -1166,7 +1166,7 @@ 
                     print '</td>';
                 } else {
                     print "<td>";
-                    if (! array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
+                    if (!array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
                         if ($type == 1) {
                             $text = img_object($langs->trans('Service'), 'service');
                         } else {
@@ -1195,7 +1195,7 @@ 
 
                 // Qty in source purchase order line
                 print '<td class="center">';
-                if (! array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
+                if (!array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
                     print $line->qty;
                 }
                 print '<input type="hidden" name="fk_commandefournisseurdet' . $indiceAsked . '" value="' . $line->id . '">';
@@ -1207,7 +1207,7 @@ 
                 // Qty already received
                 print '<td class="center">';
                 $quantityDelivered = $objectsrc->receptions[$line->id];
-                if (! array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
+                if (!array_key_exists($line->id, $arrayofpurchaselinealreadyoutput)) { // Add test to avoid to show qty twice
                     print $quantityDelivered;
                 }
                 print '<input name="qtydelivered' . $indiceAsked . '" id="qtydelivered' . $indiceAsked . '" type="hidden" value="' . $quantityDelivered . '">';

public/htdocs/reception/dispatch.php

Spacing +2 added lines, -2 removed lines

@@ -442,7 +442,7 @@ 
     print '<center>';
 
     print '<br>';
-    $disabled = 0;  // This is used to disable or not the bulk selection of target warehouse. No reason to have it disabled so forced to 0.
+    $disabled = 0; // This is used to disable or not the bulk selection of target warehouse. No reason to have it disabled so forced to 0.
 
     if ($object->statut == Reception::STATUS_DRAFT || ($object->statut == Reception::STATUS_VALIDATED && !getDolGlobalString('STOCK_CALCULATE_ON_RECEPTION'))) {
         require_once constant('DOL_DOCUMENT_ROOT') . '/product/class/html.formproduct.class.php';
@@ -546,7 +546,7 @@ 
                 print '<td class="right">' . $langs->trans("SupplierRef") . '</td>';
                 print '<td class="right">' . $langs->trans("QtyOrdered") . '</td>';
                 if ($object->status == Reception::STATUS_DRAFT) {
-                    print '<td class="right">' . $langs->trans("QtyToReceive");   // Qty to dispatch (sum for all lines of batch detail if there is)
+                    print '<td class="right">' . $langs->trans("QtyToReceive"); // Qty to dispatch (sum for all lines of batch detail if there is)
                 } else {
                     print '<td class="right">' . $langs->trans("QtyDispatchedShort") . '</td>';
                 }