alxarafe / alixar

htdocs/product/stock/class/productstockentrepot.class.php

Indentation +2 added lines, -2 removed lines

@@ -547,7 +547,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Renvoi le libelle d'un status donne
      *
@@ -557,7 +557,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         if ($mode == 0) {

htdocs/product/stock/class/productlot.class.php

Indentation +2 added lines, -2 removed lines

@@ -1062,7 +1062,7 @@ 
         return $this->LibStatut(0, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return label of a given status
      *
@@ -1072,7 +1072,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         //global $langs;
 
         //$langs->load('stocks');

Spacing +1 added lines, -1 removed lines

@@ -1030,7 +1030,7 @@
                 $obj = $this->db->fetch_object($result);
                 $this->stats_mo['customers_' . $role] = $obj->nb_customers ? $obj->nb_customers : 0;
                 $this->stats_mo['nb_' . $role] = $obj->nb ? $obj->nb : 0;
-                $this->stats_mo['qty_' . $role] = $obj->qty ? price2num($obj->qty, 'MS') : 0;     // qty may be a float due to the SUM()
+                $this->stats_mo['qty_' . $role] = $obj->qty ? price2num($obj->qty, 'MS') : 0; // qty may be a float due to the SUM()
             } else {
                 $this->error = $this->db->error();
                 $error++;

htdocs/product/stock/class/api_warehouses.class.php

Indentation +5 added lines, -5 removed lines

@@ -16,10 +16,10 @@ 
  * along with this program. If not, see <https://www.gnu.org/licenses/>.
  */
 
- use Luracast\Restler\RestException;
+    use Luracast\Restler\RestException;
 
- require_once DOL_DOCUMENT_ROOT . '/product/stock/class/entrepot.class.php';
- require_once DOL_DOCUMENT_ROOT . '/product/class/product.class.php';
+    require_once DOL_DOCUMENT_ROOT . '/product/stock/class/entrepot.class.php';
+    require_once DOL_DOCUMENT_ROOT . '/product/class/product.class.php';
 
 /**
  * API class for warehouses
@@ -261,7 +261,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Clean sensible object datas
      *
@@ -270,7 +270,7 @@ 
      */
     protected function _cleanObjectDatas($object)
     {
-		// phpcs:enable
+        // phpcs:enable
         $object = parent::_cleanObjectDatas($object);
 
         // Remove the subscriptions because they are handled as a subresource.

htdocs/product/stock/class/entrepot.class.php

Indentation +12 added lines, -12 removed lines

@@ -563,7 +563,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of all warehouses
      *
@@ -572,7 +572,7 @@ 
      */
     public function list_array($status = 1)
     {
-		// phpcs:enable
+        // phpcs:enable
         $liste = array();
 
         $sql = "SELECT rowid, ref as label";
@@ -594,7 +594,7 @@ 
         return $liste;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return number of unique different product into a warehouse
      *
@@ -602,7 +602,7 @@ 
      */
     public function nb_different_products()
     {
-		// phpcs:enable
+        // phpcs:enable
         $ret = array();
 
         $sql = "SELECT count(distinct p.rowid) as nb";
@@ -625,7 +625,7 @@ 
         return $ret;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return stock and value of warehosue
      *
@@ -634,7 +634,7 @@ 
     public function nb_products()
     {
         global $conf;
-		// phpcs:enable
+        // phpcs:enable
         $ret = array();
 
         //For MultiCompany PMP per entity
@@ -684,7 +684,7 @@ 
         return $this->LibStatut($this->statut, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return label of a given status
      *
@@ -694,7 +694,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $statusType = 'status5';
@@ -868,7 +868,7 @@ 
         return 1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return full path to current warehouse
      *
@@ -876,7 +876,7 @@ 
      */
     public function get_full_arbo()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user, $langs, $conf;
 
         $TArbo = array($this->label);
@@ -909,7 +909,7 @@ 
         return implode(' >> ', array_reverse($TArbo));
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return array of children warehouses ids from $id warehouse (recursive function)
      *
@@ -919,7 +919,7 @@ 
      */
     public function get_children_warehouses($id, &$TChildWarehouses)
     {
-		// phpcs:enable
+        // phpcs:enable
 
         $sql = "SELECT rowid
 				FROM " . $this->db->prefix() . "entrepot

Spacing +1 added lines, -1 removed lines

@@ -166,7 +166,7 @@
     /**
      * Warehouse open and any operations are allowed, but warehouse is not included into calculation of stock.
      */
-    const STATUS_OPENEXT_ALL = 3;   // TODO Implement this
+    const STATUS_OPENEXT_ALL = 3; // TODO Implement this
 
 
 

htdocs/product/stock/class/mouvementstock.class.php

Indentation +6 added lines, -6 removed lines

@@ -158,7 +158,7 @@ 
         $this->db = $db;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *  Add a movement of stock (in one direction only).
      *  This is the lowest level method to record a stock change. There is no control if warehouse is open or not.
@@ -188,7 +188,7 @@ 
      */
     public function _create($user, $fk_product, $entrepot_id, $qty, $type, $price = 0, $label = '', $inventorycode = '', $datem = '', $eatby = '', $sellby = '', $batch = '', $skip_batch = false, $id_product_batch = 0, $disablestockchangeforsubproduct = 0, $donotcleanemptylines = 0, $force_update_batch = false)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         require_once DOL_DOCUMENT_ROOT . '/product/class/product.class.php';
@@ -959,7 +959,7 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return Url link of origin object
      *
@@ -969,7 +969,7 @@ 
      */
     public function get_origin($origin_id, $origin_type)
     {
-		// phpcs:enable
+        // phpcs:enable
         $origin = '';
 
         switch ($origin_type) {
@@ -1185,7 +1185,7 @@ 
         return $this->LibStatut($mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the label of the status
      *
@@ -1194,7 +1194,7 @@ 
      */
     public function LibStatut($mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         if ($mode == 0 || $mode == 1) {

Spacing +5 added lines, -5 removed lines

@@ -219,7 +219,7 @@ 
                 'skip_batch'       => &$skip_batch,
                 'id_product_batch' => &$id_product_batch
             );
-            $reshook = $hookmanager->executeHooks('stockMovementCreate', $parameters, $this, $action);    // Note that $action and $object may have been modified by some hooks
+            $reshook = $hookmanager->executeHooks('stockMovementCreate', $parameters, $this, $action); // Note that $action and $object may have been modified by some hooks
 
             if ($reshook < 0) {
                 if (!empty($hookmanager->resPrint)) {
@@ -708,8 +708,8 @@ 
                 $this->price = $obj->price;
                 $this->fk_user_author = $obj->fk_user_author;
                 $this->label = $obj->label;
-                $this->fk_origin = $obj->origin_id;     // For backward compatibility
-                $this->origintype = $obj->origin_type;  // For backward compatibility
+                $this->fk_origin = $obj->origin_id; // For backward compatibility
+                $this->origintype = $obj->origin_type; // For backward compatibility
                 $this->origin_id = $obj->origin_id;
                 $this->origin_type = $obj->origin_type;
                 $this->inventorycode = $obj->inventorycode;
@@ -944,8 +944,8 @@ 
             } else {                    // product_batch record not found
                 $pdluo->fk_product_stock = $vfk_product_stock;
                 $pdluo->qty = $qty;
-                $pdluo->eatby = empty($dluo['eatby']) ? '' : $dluo['eatby'];        // No more used. Now eatby date is store in table of lot, no more into prouct_batch table.
-                $pdluo->sellby = empty($dluo['sellby']) ? '' : $dluo['sellby'];     // No more used. Now sellby date is store in table of lot, no more into prouct_batch table.
+                $pdluo->eatby = empty($dluo['eatby']) ? '' : $dluo['eatby']; // No more used. Now eatby date is store in table of lot, no more into prouct_batch table.
+                $pdluo->sellby = empty($dluo['sellby']) ? '' : $dluo['sellby']; // No more used. Now sellby date is store in table of lot, no more into prouct_batch table.
                 $pdluo->batch = $vbatchnumber;
 
                 $result = $pdluo->create($user, 1);

htdocs/product/class/productfournisseurprice.class.php

Indentation +2 added lines, -2 removed lines

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

htdocs/product/class/product.class.php

Indentation +99 added lines, -99 removed lines

@@ -1023,7 +1023,7 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Check barcode
      *
@@ -1036,7 +1036,7 @@ 
      */
     public function check_barcode($valuetotest, $typefortest)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
         if (isModEnabled('barcode') && getDolGlobalString('BARCODE_PRODUCT_ADDON_NUM')) {
             $module = strtolower($conf->global->BARCODE_PRODUCT_ADDON_NUM);
@@ -1950,7 +1950,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Insert a track that we changed a customer price
      *
@@ -1960,7 +1960,7 @@ 
      */
     private function _log_price($user, $level = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $now = dol_now();
@@ -1989,7 +1989,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Delete a price line
      *
@@ -1999,7 +1999,7 @@ 
      */
     public function log_price_delete($user, $rowid)
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "DELETE FROM " . $this->db->prefix() . "product_price_by_qty";
         $sql .= " WHERE fk_product_price = " . ((int) $rowid);
         $resql = $this->db->query($sql);
@@ -2134,7 +2134,7 @@ 
         return array('pu_ht' => $pu_ht, 'pu_ttc' => $pu_ttc, 'price_min' => $price_min, 'price_base_type' => $price_base_type, 'tva_tx' => $tva_tx, 'tva_npr' => $tva_npr);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Read price used by a provider.
      * We enter as input couple prodfournprice/qty or triplet qty/product_id/fourn_ref.
@@ -2150,7 +2150,7 @@ 
      */
     public function get_buyprice($prodfournprice, $qty, $product_id = 0, $fourn_ref = '', $fk_soc = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $action, $hookmanager;
 
         // Call hook if any
@@ -2948,7 +2948,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Charge tableau des stats OF pour le produit/service
      *
@@ -2957,7 +2957,7 @@ 
      */
     public function load_stats_mo($socid = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user, $hookmanager, $action;
 
         $error = 0;
@@ -3008,7 +3008,7 @@ 
         return 1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Charge tableau des stats OF pour le produit/service
      *
@@ -3017,7 +3017,7 @@ 
      */
     public function load_stats_bom($socid = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user, $hookmanager, $action;
 
         $error = 0;
@@ -3077,7 +3077,7 @@ 
         return 1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Charge tableau des stats propale pour le produit/service
      *
@@ -3086,7 +3086,7 @@ 
      */
     public function load_stats_propale($socid = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user, $hookmanager, $action;
 
         $sql = "SELECT COUNT(DISTINCT p.fk_soc) as nb_customers, COUNT(DISTINCT p.rowid) as nb,";
@@ -3152,7 +3152,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Charge tableau des stats propale pour le produit/service
      *
@@ -3161,7 +3161,7 @@ 
      */
     public function load_stats_proposal_supplier($socid = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user, $hookmanager, $action;
 
         $sql = "SELECT COUNT(DISTINCT p.fk_soc) as nb_suppliers, COUNT(DISTINCT p.rowid) as nb,";
@@ -3206,7 +3206,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Charge tableau des stats commande client pour le produit/service
      *
@@ -3217,7 +3217,7 @@ 
      */
     public function load_stats_commande($socid = 0, $filtrestatut = '', $forVirtualStock = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user, $hookmanager, $action;
 
         $sql = "SELECT COUNT(DISTINCT c.fk_soc) as nb_customers, COUNT(DISTINCT c.rowid) as nb,";
@@ -3333,7 +3333,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Charge tableau des stats commande fournisseur pour le produit/service
      *
@@ -3345,7 +3345,7 @@ 
      */
     public function load_stats_commande_fournisseur($socid = 0, $filtrestatut = '', $forVirtualStock = 0, $dateofvirtualstock = null)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user, $hookmanager, $action;
 
         $sql = "SELECT COUNT(DISTINCT c.fk_soc) as nb_suppliers, COUNT(DISTINCT c.rowid) as nb,";
@@ -3394,7 +3394,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Charge tableau des stats expedition client pour le produit/service
      *
@@ -3406,7 +3406,7 @@ 
      */
     public function load_stats_sending($socid = 0, $filtrestatut = '', $forVirtualStock = 0, $filterShipmentStatus = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user, $hookmanager, $action;
 
         $sql = "SELECT COUNT(DISTINCT e.fk_soc) as nb_customers, COUNT(DISTINCT e.rowid) as nb,";
@@ -3480,7 +3480,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Charge tableau des stats réception fournisseur pour le produit/service
      *
@@ -3492,7 +3492,7 @@ 
      */
     public function load_stats_reception($socid = 0, $filtrestatut = '', $forVirtualStock = 0, $dateofvirtualstock = null)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user, $hookmanager, $action;
 
         $sql = "SELECT COUNT(DISTINCT cf.fk_soc) as nb_suppliers, COUNT(DISTINCT cf.rowid) as nb,";
@@ -3541,7 +3541,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Charge tableau des stats production pour le produit/service
      *
@@ -3554,7 +3554,7 @@ 
      */
     public function load_stats_inproduction($socid = 0, $filtrestatut = '', $forVirtualStock = 0, $dateofvirtualstock = null, $warehouseid = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user, $hookmanager, $action;
 
         $serviceStockIsEnabled = isModEnabled("service") && getDolGlobalString('STOCK_SUPPORTS_SERVICES');
@@ -3668,7 +3668,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Charge tableau des stats contrat pour le produit/service
      *
@@ -3677,7 +3677,7 @@ 
      */
     public function load_stats_contrat($socid = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user, $hookmanager, $action;
 
         $sql = "SELECT COUNT(DISTINCT c.fk_soc) as nb_customers, COUNT(DISTINCT c.rowid) as nb,";
@@ -3742,7 +3742,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Charge tableau des stats facture pour le produit/service
      *
@@ -3751,7 +3751,7 @@ 
      */
     public function load_stats_facture($socid = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user, $hookmanager, $action;
 
         $sql = "SELECT COUNT(DISTINCT f.fk_soc) as nb_customers, COUNT(DISTINCT f.rowid) as nb,";
@@ -3817,7 +3817,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Charge tableau des stats facture recurrentes pour le produit/service
      *
@@ -3826,7 +3826,7 @@ 
      */
     public function load_stats_facturerec($socid = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user, $hookmanager, $action;
 
         $sql = "SELECT COUNT(DISTINCT f.fk_soc) as nb_customers, COUNT(DISTINCT f.rowid) as nb,";
@@ -3891,7 +3891,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Charge tableau des stats facture pour le produit/service
      *
@@ -3900,7 +3900,7 @@ 
      */
     public function load_stats_facture_fournisseur($socid = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user, $hookmanager, $action;
 
         $sql = "SELECT COUNT(DISTINCT f.fk_soc) as nb_suppliers, COUNT(DISTINCT f.rowid) as nb,";
@@ -3944,7 +3944,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return an array formatted for showing graphs
      *
@@ -3955,7 +3955,7 @@ 
      */
     private function _get_stats($sql, $mode, $year = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $tab = array();
 
         $resql = $this->db->query($sql);
@@ -4020,7 +4020,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return nb of units or customers invoices in which product is included
      *
@@ -4033,7 +4033,7 @@ 
      */
     public function get_nb_vente($socid, $mode, $filteronproducttype = -1, $year = 0, $morefilter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
         global $user;
 
@@ -4074,7 +4074,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return nb of units or supplier invoices in which product is included
      *
@@ -4087,7 +4087,7 @@ 
      */
     public function get_nb_achat($socid, $mode, $filteronproducttype = -1, $year = 0, $morefilter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
         global $user;
 
@@ -4127,7 +4127,7 @@ 
         return $this->_get_stats($sql, $mode, $year);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return nb of units in proposals in which product is included
      *
@@ -4140,7 +4140,7 @@ 
      */
     public function get_nb_propal($socid, $mode, $filteronproducttype = -1, $year = 0, $morefilter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
 
         $sql = "SELECT sum(d.qty) as qty, date_format(p.datep, '%Y%m')";
@@ -4179,7 +4179,7 @@ 
         return $this->_get_stats($sql, $mode, $year);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return nb of units in proposals in which product is included
      *
@@ -4192,7 +4192,7 @@ 
      */
     public function get_nb_propalsupplier($socid, $mode, $filteronproducttype = -1, $year = 0, $morefilter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
         global $user;
 
@@ -4232,7 +4232,7 @@ 
         return $this->_get_stats($sql, $mode, $year);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return nb of units in orders in which product is included
      *
@@ -4245,7 +4245,7 @@ 
      */
     public function get_nb_order($socid, $mode, $filteronproducttype = -1, $year = 0, $morefilter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
 
         $sql = "SELECT sum(d.qty) as qty, date_format(c.date_commande, '%Y%m')";
@@ -4284,7 +4284,7 @@ 
         return $this->_get_stats($sql, $mode, $year);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return nb of units in orders in which product is included
      *
@@ -4297,7 +4297,7 @@ 
      */
     public function get_nb_ordersupplier($socid, $mode, $filteronproducttype = -1, $year = 0, $morefilter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
 
         $sql = "SELECT sum(d.qty) as qty, date_format(c.date_commande, '%Y%m')";
@@ -4336,7 +4336,7 @@ 
         return $this->_get_stats($sql, $mode, $year);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return nb of units in orders in which product is included
      *
@@ -4349,7 +4349,7 @@ 
      */
     public function get_nb_contract($socid, $mode, $filteronproducttype = -1, $year = 0, $morefilter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
 
         $sql = "SELECT sum(d.qty) as qty, date_format(c.date_contrat, '%Y%m')";
@@ -4390,7 +4390,7 @@ 
         return $this->_get_stats($sql, $mode, $year);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return nb of units in orders in which product is included
      *
@@ -4403,7 +4403,7 @@ 
      */
     public function get_nb_mos($socid, $mode, $filteronproducttype = -1, $year = 0, $morefilter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
 
         $sql = "SELECT sum(d.qty), date_format(d.date_valid, '%Y%m')";
@@ -4443,7 +4443,7 @@ 
         return $this->_get_stats($sql, $mode, $year);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Link a product/service to a parent product/service
      *
@@ -4458,7 +4458,7 @@ 
     {
         global $user;
 
-		// phpcs:enable
+        // phpcs:enable
         // Clean parameters
         if (!is_numeric($id_pere)) {
             $id_pere = 0;
@@ -4516,7 +4516,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Modify composed product
      *
@@ -4531,7 +4531,7 @@ 
     {
         global $user;
 
-		// phpcs:enable
+        // phpcs:enable
         // Clean parameters
         if (!is_numeric($id_pere)) {
             $id_pere = 0;
@@ -4570,7 +4570,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Remove a link between a subproduct and a parent product/service
      *
@@ -4583,7 +4583,7 @@ 
     {
         global $user;
 
-		// phpcs:enable
+        // phpcs:enable
         if (!is_numeric($fk_parent)) {
             $fk_parent = 0;
         }
@@ -4634,7 +4634,7 @@ 
         return 1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Check if it is a sub-product into a kit
      *
@@ -4644,7 +4644,7 @@ 
      */
     public function is_sousproduit($fk_parent, $fk_child)
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "SELECT fk_product_pere, qty, incdec";
         $sql .= " FROM " . $this->db->prefix() . "product_association";
         $sql .= " WHERE fk_product_pere  = " . ((int) $fk_parent);
@@ -4671,7 +4671,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Add a supplier price for the product.
      *  Note: Duplicate ref is accepted for different quantity only, or for different companies.
@@ -4684,7 +4684,7 @@ 
      */
     public function add_fournisseur($user, $id_fourn, $ref_fourn, $quantity)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $now = dol_now();
@@ -4772,7 +4772,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return list of suppliers providing the product or service
      *
@@ -4780,7 +4780,7 @@ 
      */
     public function list_suppliers()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $list = array();
@@ -4804,7 +4804,7 @@ 
         return $list;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Recopie les prix d'un produit/service sur un autre
      *
@@ -4892,7 +4892,7 @@ 
         return 1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Clone links between products
      *
@@ -4902,7 +4902,7 @@ 
      */
     public function clone_associations($fromId, $toId)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->db->begin();
 
         $sql = 'INSERT INTO ' . $this->db->prefix() . 'product_association (fk_product_pere, fk_product_fils, qty, incdec)';
@@ -4919,7 +4919,7 @@ 
         return 1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Recopie les fournisseurs et prix fournisseurs d'un produit/service sur un autre
      *
@@ -4929,7 +4929,7 @@ 
      */
     public function clone_fournisseurs($fromId, $toId)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->db->begin();
 
         $now = dol_now();
@@ -4965,7 +4965,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Function recursive, used only by get_arbo_each_prod(), to build tree of subproducts into ->res
      *  Define value of this->res
@@ -4980,7 +4980,7 @@ 
      */
     public function fetch_prod_arbo($prod, $compl_path = '', $multiply = 1, $level = 1, $id_parent = 0, $ignore_stock_load = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $tmpproduct = null;
@@ -5033,7 +5033,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Build the tree of subproducts and return it.
      *  this->sousprods must have been loaded by this->get_sousproduits_arbo()
@@ -5044,7 +5044,7 @@ 
      */
     public function get_arbo_each_prod($multiply = 1, $ignore_stock_load = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->res = array();
         if (isset($this->sousprods) && is_array($this->sousprods)) {
             foreach ($this->sousprods as $prod_name => $desc_product) {
@@ -5255,7 +5255,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *     Return tree of all subproducts for product. Tree contains array of array(0=prodid, 1=>qty, 2=>product type, 3=>label, 4=>incdec, 5=>product ref)
      *     Set this->sousprods
@@ -5264,7 +5264,7 @@ 
      */
     public function get_sousproduits_arbo()
     {
-		// phpcs:enable
+        // phpcs:enable
         $parent = array();
 
         foreach ($this->getChildsArbo($this->id) as $keyChild => $valueChild) {    // Warning. getChildsArbo can call getChildsArbo recursively. Starting point is $value[0]=id of product
@@ -5560,7 +5560,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Return label of a given status
      *
@@ -5571,7 +5571,7 @@ 
      */
     public function LibStatut($status, $mode = 0, $type = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $labelStatus = $labelStatusShort = '';
@@ -5670,7 +5670,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Adjust stock in a warehouse for product
      *
@@ -5689,7 +5689,7 @@ 
      */
     public function correct_stock($user, $id_entrepot, $nbpiece, $movement, $label = '', $price = 0, $inventorycode = '', $origin_element = '', $origin_id = null, $disablestockchangeforsubproduct = 0, $extrafields = null)
     {
-		// phpcs:enable
+        // phpcs:enable
         if ($id_entrepot) {
             $this->db->begin();
 
@@ -5729,7 +5729,7 @@ 
         return -1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Adjust stock in a warehouse for product with batch number
      *
@@ -5752,7 +5752,7 @@ 
      */
     public function correct_stock_batch($user, $id_entrepot, $nbpiece, $movement, $label = '', $price = 0, $dlc = '', $dluo = '', $lot = '', $inventorycode = '', $origin_element = '', $origin_id = null, $disablestockchangeforsubproduct = 0, $extrafields = null, $force_update_batch = false)
     {
-		// phpcs:enable
+        // phpcs:enable
         if ($id_entrepot) {
             $this->db->begin();
 
@@ -5792,7 +5792,7 @@ 
         return -1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Load information about stock of a product into ->stock_reel, ->stock_warehouse[] (including stock_warehouse[idwarehouse]->detail_batch for batch products)
      * This function need a lot of load. If you use it on list, use a cache to execute it once for each product id.
@@ -5807,7 +5807,7 @@ 
      */
     public function load_stock($option = '', $includedraftpoforvirtual = null, $dateofvirtualstock = null)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $this->stock_reel = 0;
@@ -5874,7 +5874,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load value ->stock_theorique of a product. Property this->id must be defined.
      *  This function need a lot of load. If you use it on list, use a cache to execute it one for each product id.
@@ -5886,7 +5886,7 @@ 
      */
     public function load_virtual_stock($includedraftpoforvirtual = null, $dateofvirtualstock = null)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $hookmanager, $action;
 
         $stock_commande_client = 0;
@@ -6045,7 +6045,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Move an uploaded file described into $file array into target directory $sdir.
      *
@@ -6055,7 +6055,7 @@ 
      */
     public function add_photo($sdir, $file)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         include_once DOL_DOCUMENT_ROOT . '/core/lib/files.lib.php';
@@ -6092,7 +6092,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return if at least one photo is available
      *
@@ -6101,7 +6101,7 @@ 
      */
     public function is_photo_available($sdir)
     {
-		// phpcs:enable
+        // phpcs:enable
         include_once DOL_DOCUMENT_ROOT . '/core/lib/files.lib.php';
         include_once DOL_DOCUMENT_ROOT . '/core/lib/images.lib.php';
 
@@ -6133,7 +6133,7 @@ 
         return false;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return an array with all photos of product found on disk. There is no sorting criteria.
      *
@@ -6143,7 +6143,7 @@ 
      */
     public function liste_photos($dir, $nbmax = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         include_once DOL_DOCUMENT_ROOT . '/core/lib/files.lib.php';
         include_once DOL_DOCUMENT_ROOT . '/core/lib/images.lib.php';
 
@@ -6194,7 +6194,7 @@ 
         return $tabobj;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Delete a photo and its thumbs
      *
@@ -6203,7 +6203,7 @@ 
      */
     public function delete_photo($file)
     {
-		// phpcs:enable
+        // phpcs:enable
         include_once DOL_DOCUMENT_ROOT . '/core/lib/files.lib.php';
         include_once DOL_DOCUMENT_ROOT . '/core/lib/images.lib.php';
 
@@ -6228,7 +6228,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load size of image file
      *
@@ -6237,7 +6237,7 @@ 
      */
     public function get_image_size($file)
     {
-		// phpcs:enable
+        // phpcs:enable
         $file_osencoded = dol_osencode($file);
         $infoImg = getimagesize($file_osencoded); // Get information on image
         $this->imgWidth = $infoImg[0]; // Largeur de l'image
@@ -6335,7 +6335,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Get a barcode from the module to generate barcode values.
      *  Return value is stored into this->barcode
@@ -6346,7 +6346,7 @@ 
      */
     public function get_barcode($object, $type = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $result = '';
@@ -6452,7 +6452,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return minimum product recommended price
      *
@@ -6460,7 +6460,7 @@ 
      */
     public function min_recommended_price()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $maxpricesupplier = 0;

Spacing +22 added lines, -22 removed lines

@@ -156,7 +156,7 @@ 
      */
     public $price;
 
-    public $price_formated;         // used by takepos/ajax/ajax.php
+    public $price_formated; // used by takepos/ajax/ajax.php
 
     /**
      * Selling price with tax
@@ -165,7 +165,7 @@ 
      */
     public $price_ttc;
 
-    public $price_ttc_formated;     // used by takepos/ajax/ajax.php
+    public $price_ttc_formated; // used by takepos/ajax/ajax.php
 
     /**
      * Minimum price net
@@ -245,7 +245,7 @@ 
     public $packaging;
 
 
-    public $lifetime;   // In seconds
+    public $lifetime; // In seconds
 
     public $qc_frequency;
 
@@ -406,20 +406,20 @@ 
 
     //! Metric of products
     public $weight;
-    public $weight_units;   // scale -3, 0, 3, 6
+    public $weight_units; // scale -3, 0, 3, 6
     public $length;
-    public $length_units;   // scale -3, 0, 3, 6
+    public $length_units; // scale -3, 0, 3, 6
     public $width;
-    public $width_units;    // scale -3, 0, 3, 6
+    public $width_units; // scale -3, 0, 3, 6
     public $height;
-    public $height_units;   // scale -3, 0, 3, 6
+    public $height_units; // scale -3, 0, 3, 6
     public $surface;
-    public $surface_units;  // scale -3, 0, 3, 6
+    public $surface_units; // scale -3, 0, 3, 6
     public $volume;
-    public $volume_units;   // scale -3, 0, 3, 6
+    public $volume_units; // scale -3, 0, 3, 6
 
     public $net_measure;
-    public $net_measure_units;  // scale -3, 0, 3, 6
+    public $net_measure_units; // scale -3, 0, 3, 6
 
     public $accountancy_code_sell;
     public $accountancy_code_sell_intra;
@@ -611,7 +611,7 @@ 
         'import_key'    => array('type' => 'varchar(14)', 'label' => 'ImportId', 'enabled' => 1, 'visible' => -2, 'notnull' => -1, 'index' => 0, 'position' => 1000),
         //'tosell'       =>array('type'=>'integer',      'label'=>'Status',           'enabled'=>1, 'visible'=>1,  'notnull'=>1, 'default'=>'0', 'index'=>1,  'position'=>1000, 'arrayofkeyval'=>array(0=>'Draft', 1=>'Active', -1=>'Cancel')),
         //'tobuy'        =>array('type'=>'integer',      'label'=>'Status',           'enabled'=>1, 'visible'=>1,  'notnull'=>1, 'default'=>'0', 'index'=>1,  'position'=>1000, 'arrayofkeyval'=>array(0=>'Draft', 1=>'Active', -1=>'Cancel')),
-        'mandatory_period' => array('type' => 'integer', 'label' => 'mandatoryperiod', 'enabled' => 1, 'visible' => 1,  'notnull' => 1, 'default' => '0', 'index' => 1,  'position' => 1000),
+        'mandatory_period' => array('type' => 'integer', 'label' => 'mandatoryperiod', 'enabled' => 1, 'visible' => 1, 'notnull' => 1, 'default' => '0', 'index' => 1, 'position' => 1000),
     );
 
     /**
@@ -1176,7 +1176,7 @@ 
             $result = $this->verify(); // We don't check when update called during a create because verify was already done
         } else {
             // we can continue
-            $result = 0;  // @phan-suppress-current-line PhanPluginRedundantAssignment
+            $result = 0; // @phan-suppress-current-line PhanPluginRedundantAssignment
         }
 
         if ($result >= 0) {
@@ -1923,7 +1923,7 @@ 
      */
     private function getArrayForPriceCompare($level = 0)
     {
-        $testExit = array('multiprices','multiprices_ttc','multiprices_base_type','multiprices_min','multiprices_min_ttc','multiprices_tva_tx','multiprices_recuperableonly');
+        $testExit = array('multiprices', 'multiprices_ttc', 'multiprices_base_type', 'multiprices_min', 'multiprices_min_ttc', 'multiprices_tva_tx', 'multiprices_recuperableonly');
 
         foreach ($testExit as $field) {
             if (!isset($this->$field)) {
@@ -2567,8 +2567,8 @@ 
 
         //For MultiCompany
         //PMP per entity & Stocks Sharings stock_reel includes only stocks shared with this entity
-        $separatedEntityPMP = false;    // Set to true to get the AWP from table llx_product_perentity instead of field 'pmp' into llx_product.
-        $separatedStock = false;        // Set to true will count stock from subtable llx_product_stock. It is slower than using denormalized field 'stock', but it is required when using multientity and shared warehouses.
+        $separatedEntityPMP = false; // Set to true to get the AWP from table llx_product_perentity instead of field 'pmp' into llx_product.
+        $separatedStock = false; // Set to true will count stock from subtable llx_product_stock. It is slower than using denormalized field 'stock', but it is required when using multientity and shared warehouses.
         $visibleWarehousesEntities = $conf->entity;
         if (getDolGlobalString('MULTICOMPANY_PRODUCT_SHARING_ENABLED')) {
             if (getDolGlobalString('MULTICOMPANY_PMP_PER_ENTITY_ENABLED')) {
@@ -2760,15 +2760,15 @@ 
                         $sql .= " WHERE entity IN (" . getEntity('productprice') . ")";
                         $sql .= " AND price_level=" . ((int) $i);
                         $sql .= " AND fk_product = " . ((int) $this->id);
-                        $sql .= " ORDER BY date_price DESC, rowid DESC";    // Get the most recent line
-                        $sql .= " LIMIT 1";                                 // Only the first one
+                        $sql .= " ORDER BY date_price DESC, rowid DESC"; // Get the most recent line
+                        $sql .= " LIMIT 1"; // Only the first one
                         $resql = $this->db->query($sql);
                         if ($resql) {
                             $result = $this->db->fetch_array($resql);
 
                             $this->multiprices[$i] = $result ? $result["price"] : null;
                             $this->multiprices_ttc[$i] = $result ? $result["price_ttc"] : null;
-                            $this->multiprices_min[$i] =  $result ? $result["price_min"] : null;
+                            $this->multiprices_min[$i] = $result ? $result["price_min"] : null;
                             $this->multiprices_min_ttc[$i] = $result ? $result["price_min_ttc"] : null;
                             $this->multiprices_base_type[$i] = $result ? $result["price_base_type"] : null;
                             // Next two fields are used only if PRODUIT_MULTIPRICES_USE_VAT_PER_LEVEL is on
@@ -2988,7 +2988,7 @@ 
                 $obj = $this->db->fetch_object($result);
                 $this->stats_mo['customers_' . $role] = $obj->nb_customers ? $obj->nb_customers : 0;
                 $this->stats_mo['nb_' . $role] = $obj->nb ? $obj->nb : 0;
-                $this->stats_mo['qty_' . $role] = $obj->qty ? price2num($obj->qty, 'MS') : 0;     // qty may be a float due to the SUM()
+                $this->stats_mo['qty_' . $role] = $obj->qty ? price2num($obj->qty, 'MS') : 0; // qty may be a float due to the SUM()
             } else {
                 $this->error = $this->db->error();
                 $error++;
@@ -4492,7 +4492,7 @@ 
                 //Addition of a product with the highest rank +1
                 $sql = "INSERT INTO " . $this->db->prefix() . "product_association(fk_product_pere,fk_product_fils,qty,incdec,rang)";
                 $sql .= " VALUES (" . ((int) $id_pere) . ", " . ((int) $id_fils) . ", " . price2num($qty, 'MS') . ", " . price2num($incdec, 'MS') . ", " . ((int) $rank) . ")";
-                if (! $this->db->query($sql)) {
+                if (!$this->db->query($sql)) {
                     dol_print_error($this->db);
                     return -1;
                 } else {
@@ -4613,7 +4613,7 @@ 
                 $sql = "UPDATE " . $this->db->prefix() . "product_association";
                 $sql .= " SET rang = " . ((int) $cpt);
                 $sql .= " WHERE rowid = " . ((int) $objrank->rowid);
-                if (! $this->db->query($sql)) {
+                if (!$this->db->query($sql)) {
                     dol_print_error($this->db);
                     return -1;
                 }
@@ -6680,7 +6680,7 @@ 
      */
     public function getKanbanView($option = '', $arraydata = null)
     {
-        global $langs,$conf;
+        global $langs, $conf;
 
         $selected = (empty($arraydata['selected']) ? 0 : $arraydata['selected']);
 

htdocs/product/class/html.formproduct.class.php

Indentation +4 added lines, -4 removed lines

@@ -232,7 +232,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return full path to current warehouse in $tab (recursive function)
      *
@@ -242,7 +242,7 @@ 
      */
     private function get_parent_path($tab, $final_label = '')
     {
-		//phpcs:enable
+        //phpcs:enable
         if (empty($final_label)) {
             $final_label = $tab['label'];
         }
@@ -523,7 +523,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output a combo box with list of units
      *  Currently the units are not define in the DB
@@ -538,7 +538,7 @@ 
      */
     public function select_measuring_units($name = 'measuring_units', $measuring_style = '', $default = '0', $adddefault = 0, $mode = 0)
     {
-		//phpcs:enable
+        //phpcs:enable
         print $this->selectMeasuringUnits($name, $measuring_style, $default, $adddefault, $mode);
     }
 

htdocs/product/class/propalmergepdfproduct.class.php

Indentation +6 added lines, -6 removed lines

@@ -226,7 +226,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load object in memory from the database
      *
@@ -236,7 +236,7 @@ 
      */
     public function fetch_by_product($product_id, $lang = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf;
 
         $sql = "SELECT";
@@ -404,7 +404,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Delete object in database
      *
@@ -416,7 +416,7 @@ 
      */
     public function delete_by_product($user, $product_id, $lang_id = '', $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
         $error = 0;
 
@@ -452,7 +452,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Delete object in database
      *
@@ -461,7 +461,7 @@ 
      */
     public function delete_by_file($user)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
         $error = 0;