alxarafe / alixar

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/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/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

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>';
                 }

public/htdocs/reception/list.php

Spacing +6 added lines, -6 removed lines

@@ -68,7 +68,7 @@ 
 $search_date_delivery_endday = GETPOSTINT('search_date_delivery_endday');
 $search_date_delivery_endmonth = GETPOSTINT('search_date_delivery_endmonth');
 $search_date_delivery_endyear = GETPOSTINT('search_date_delivery_endyear');
-$search_date_delivery_start = dol_mktime(0, 0, 0, $search_date_delivery_startmonth, $search_date_delivery_startday, $search_date_delivery_startyear);   // Use tzserver
+$search_date_delivery_start = dol_mktime(0, 0, 0, $search_date_delivery_startmonth, $search_date_delivery_startday, $search_date_delivery_startyear); // Use tzserver
 $search_date_delivery_end = dol_mktime(23, 59, 59, $search_date_delivery_endmonth, $search_date_delivery_endday, $search_date_delivery_endyear);
 $search_date_create_startday = GETPOSTINT('search_date_create_startday');
 $search_date_create_startmonth = GETPOSTINT('search_date_create_startmonth');
@@ -76,7 +76,7 @@ 
 $search_date_create_endday = GETPOSTINT('search_date_create_endday');
 $search_date_create_endmonth = GETPOSTINT('search_date_create_endmonth');
 $search_date_create_endyear = GETPOSTINT('search_date_create_endyear');
-$search_date_create_start = dol_mktime(0, 0, 0, $search_date_create_startmonth, $search_date_create_startday, $search_date_create_startyear);   // Use tzserver
+$search_date_create_start = dol_mktime(0, 0, 0, $search_date_create_startmonth, $search_date_create_startday, $search_date_create_startyear); // Use tzserver
 $search_date_create_end = dol_mktime(23, 59, 59, $search_date_create_endmonth, $search_date_create_endday, $search_date_create_endyear);
 $search_billed = GETPOST("search_billed", 'intcomma');
 $search_status = GETPOST('search_status', 'intcomma');
@@ -144,7 +144,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
 
 $error = 0;
 
@@ -336,7 +336,7 @@ 
                 // if the VAT reverse-charge is activated by default in supplier card to resume the information
                 $objecttmp->vat_reverse_charge = $soc->vat_reverse_charge;
 
-                $objecttmp->fk_project          = $rcp->fk_project;
+                $objecttmp->fk_project = $rcp->fk_project;
                 //$objecttmp->multicurrency_code = $rcp->multicurrency_code;
                 if (empty($createbills_onebythird)) {
                     $objecttmp->ref_supplier = $rcp->ref;
@@ -369,7 +369,7 @@ 
                     }
                 }
 
-                $res = $objecttmp->create($user);       // This should create the supplier invoice + links into $objecttmp->linked_objects + add a link to ->origin_id
+                $res = $objecttmp->create($user); // This should create the supplier invoice + links into $objecttmp->linked_objects + add a link to ->origin_id
 
                 //var_dump($objecttmp->error);exit;
                 if ($res > 0) {
@@ -576,7 +576,7 @@ 
             $db->rollback();
 
             $action = 'create';
-            $_GET["origin"] = $_POST["origin"];     // Keep this ?
+            $_GET["origin"] = $_POST["origin"]; // Keep this ?
             $_GET["originid"] = $_POST["originid"]; // Keep this ?
             setEventMessages($object->error, $errors, 'errors');
             $error++;

public/htdocs/reception/class/api_receptions.class.php

Spacing +4 added lines, -4 removed lines

@@ -204,11 +204,11 @@
 
                 $receptionline->fk_product = $line['fk_product'];
                 $receptionline->fk_entrepot = $line['fk_entrepot'];
-                $receptionline->fk_element = $line['fk_element'] ?? $line['origin_id'];             // example: purchase order id.  this->origin is 'supplier_order'
+                $receptionline->fk_element = $line['fk_element'] ?? $line['origin_id']; // example: purchase order id.  this->origin is 'supplier_order'
                 $receptionline->origin_line_id = $line['fk_elementdet'] ?? $line['origin_line_id']; // example: purchase order id
-                $receptionline->fk_elementdet = $line['fk_elementdet'] ?? $line['origin_line_id'];  // example: purchase order line id
-                $receptionline->origin_type = $line['element_type'] ?? $line['origin_type'];        // example 'supplier_order'
-                $receptionline->element_type = $line['element_type'] ?? $line['origin_type'];       // example 'supplier_order'
+                $receptionline->fk_elementdet = $line['fk_elementdet'] ?? $line['origin_line_id']; // example: purchase order line id
+                $receptionline->origin_type = $line['element_type'] ?? $line['origin_type']; // example 'supplier_order'
+                $receptionline->element_type = $line['element_type'] ?? $line['origin_type']; // example 'supplier_order'
                 $receptionline->qty = $line['qty'];
                 //$receptionline->rang = $line['rang'];
                 $receptionline->array_options = $line['array_options'];

public/htdocs/reception/class/reception.class.php

Spacing +4 added lines, -4 removed lines

@@ -734,9 +734,9 @@ 
                 if ($this->origin_object instanceof CommonObject && empty($this->origin_object->lines)) {
                     $res = $this->origin_object->fetch_lines();
                     if ($this->origin_object instanceof CommandeFournisseur) {
-                        $this->commandeFournisseur = $this->origin_object;  // deprecated
+                        $this->commandeFournisseur = $this->origin_object; // deprecated
                     } else {
-                        $this->commandeFournisseur = null;  // deprecated
+                        $this->commandeFournisseur = null; // deprecated
                     }
                     if ($res < 0) {
                         return $res;
@@ -1358,7 +1358,7 @@ 
         if (getDolGlobalInt("STOCK_CALCULATE_ON_RECEPTION_CLOSE")) {
             $this->labelStatus[1]  = 'StatusReceptionValidatedToReceive';
         }
-        $this->labelStatus[2]  = 'StatusReceptionProcessed';
+        $this->labelStatus[2] = 'StatusReceptionProcessed';
 
         // List of short language codes for status
         $this->labelStatusShort[-1] = 'StatusReceptionCanceledShort';
@@ -1468,7 +1468,7 @@ 
         while ($xnbp < $nbp) {
             $line = new CommandeFournisseurDispatch($this->db);
             $line->desc = $langs->trans("Description") . " " . $xnbp;
-            $line->libelle = $langs->trans("Description") . " " . $xnbp;    // deprecated
+            $line->libelle = $langs->trans("Description") . " " . $xnbp; // deprecated
             $line->label = $langs->trans("Description") . " " . $xnbp;
             $line->qty = 10;