alxarafe / alixar

public/htdocs/variants/card.php

Spacing +1 added lines, -1 removed lines

@@ -286,7 +286,7 @@
     if ($action != 'editline') {
         print '<div class="tabsAction">' . "\n";
         $parameters = array();
-        $reshook = $hookmanager->executeHooks('addMoreActionsButtons', $parameters, $object, $action);    // Note that $action and $object may have been modified by hook
+        $reshook = $hookmanager->executeHooks('addMoreActionsButtons', $parameters, $object, $action); // Note that $action and $object may have been modified by hook
         if ($reshook < 0) {
             setEventMessages($hookmanager->error, $hookmanager->errors, 'errors');
         }

public/htdocs/variants/combinations.php

Indentation +1 added lines, -1 removed lines

@@ -778,7 +778,7 @@
         <div style="text-align: center">
         <input type="submit" name="create" <?php if (!is_array($productCombination2ValuePairs1)) {
             print ' disabled="disabled"';
-                                           } ?> value="<?php echo $action == 'add' ? $langs->trans('Create') : $langs->trans("Save") ?>" class="button button-save">
+                                            } ?> value="<?php echo $action == 'add' ? $langs->trans('Create') : $langs->trans("Save") ?>" class="button button-save">
         &nbsp;
         <input type="submit" name="cancel" value="<?php echo $langs->trans("Cancel"); ?>" class="button button-cancel">
         </div>

Spacing +3 added lines, -3 removed lines

@@ -30,12 +30,12 @@
 
 $langs->loadLangs(array("products", "other"));
 
-$id = GETPOSTINT('id');                             // ID of the parent Product
+$id = GETPOSTINT('id'); // ID of the parent Product
 $ref = GETPOST('ref', 'alpha');
 
-$combination_id = GETPOSTINT('combination_id');     // ID of the combination
+$combination_id = GETPOSTINT('combination_id'); // ID of the combination
 
-$reference = GETPOST('reference', 'alpha');         // Reference of the variant Product
+$reference = GETPOST('reference', 'alpha'); // Reference of the variant Product
 
 $weight_impact = GETPOSTFLOAT('weight_impact', 2);
 $price_impact_percent = (bool) GETPOST('price_impact_percent');

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

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