alxarafe / alixar

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

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

Indentation +12 added lines, -12 removed lines

@@ -45,7 +45,7 @@ 
     public $reception;
 
     /**
-        * Constructor
+     * Constructor
      */
     public function __construct()
     {
@@ -623,15 +623,15 @@ 
     */
 
     /**
-    * Close a reception (Classify it as "Delivered")
-    *
-    * @param    int     $id             Reception ID
-    * @param    int     $notrigger      Disabled triggers
-    *
-    * @url POST    {id}/close
-    *
-    * @return  Object
-    */
+     * Close a reception (Classify it as "Delivered")
+     *
+     * @param    int     $id             Reception ID
+     * @param    int     $notrigger      Disabled triggers
+     *
+     * @url POST    {id}/close
+     *
+     * @return  Object
+     */
     public function close($id, $notrigger = 0)
     {
         if (!DolibarrApiAccess::$user->hasRight('reception', 'creer')) {
@@ -663,7 +663,7 @@ 
         return $this->_cleanObjectDatas($this->reception);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Clean sensible object datas
      *
@@ -672,7 +672,7 @@ 
      */
     protected function _cleanObjectDatas($object)
     {
-		// phpcs:enable
+        // phpcs:enable
         $object = parent::_cleanObjectDatas($object);
 
         unset($object->thirdparty); // id already returned

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

Indentation +8 added lines, -8 removed lines

@@ -1181,7 +1181,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load lines
      *
@@ -1189,7 +1189,7 @@ 
      */
     public function fetch_lines()
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->lines = array();
 
         require_once constant('DOL_DOCUMENT_ROOT') . '/fourn/class/fournisseur.commande.dispatch.class.php';
@@ -1334,7 +1334,7 @@ 
         return $this->LibStatut($this->statut, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return label of a status
      *
@@ -1344,7 +1344,7 @@ 
      */
     public function LibStatut($status, $mode)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         // List of long language codes for status
@@ -1510,7 +1510,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Fetch deliveries method and return an array. Load array this->meths(rowid=>label).
      *
@@ -1518,7 +1518,7 @@ 
      */
     public function fetch_delivery_methods()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $this->meths = array();
 
@@ -1536,7 +1536,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Fetch all deliveries method and return an array. Load array this->listmeths.
      *
@@ -1545,7 +1545,7 @@ 
      */
     public function list_delivery_methods($id = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $this->listmeths = array();

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;