alxarafe / alixar

Dolibarr/Code/Product/Classes/FormProduct.php

Indentation +4 added lines, -4 removed lines

@@ -233,7 +233,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return full path to current warehouse in $tab (recursive function)
      *
@@ -243,7 +243,7 @@ 
      */
     private function get_parent_path($tab, $final_label = '')
     {
-		//phpcs:enable
+        //phpcs:enable
         if (empty($final_label)) {
             $final_label = $tab['label'];
         }
@@ -524,7 +524,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
@@ -539,7 +539,7 @@ 
      */
     public function select_measuring_units($name = 'measuring_units', $measuring_style = '', $selected = '0', $adddefault = 0, $mode = 0)
     {
-		//phpcs:enable
+        //phpcs:enable
         print $this->selectMeasuringUnits($name, $measuring_style, $selected, $adddefault, $mode);
     }
 

Dolibarr/Code/Product/Classes/Productbatch.php

Spacing +10 added lines, -10 removed lines

@@ -58,18 +58,18 @@ 
     public $fk_product;
 
     // Properties of the lot
-    public $lotid;          // ID in table of the details of properties of each lots
+    public $lotid; // ID in table of the details of properties of each lots
 
     /**
      * @var int|string
      * @deprecated
      */
-    public $sellby = '';    // dlc
+    public $sellby = ''; // dlc
     /**
      * @var int|string
      * @deprecated
      */
-    public $eatby = '';     // dmd/dluo
+    public $eatby = ''; // dmd/dluo
 
 
     /**
@@ -103,15 +103,15 @@ 
         // Insert request
         $sql = "INSERT INTO " . $this->db->prefix() . "product_batch (";
         $sql .= "fk_product_stock,";
-        $sql .= "sellby,";              // no more used
-        $sql .= "eatby,";               // no more used
+        $sql .= "sellby,"; // no more used
+        $sql .= "eatby,"; // no more used
         $sql .= "batch,";
         $sql .= "qty,";
         $sql .= "import_key";
         $sql .= ") VALUES (";
         $sql .= " " . (!isset($this->fk_product_stock) ? 'NULL' : $this->fk_product_stock) . ",";
-        $sql .= " " . (!isset($this->sellby) || dol_strlen($this->sellby) == 0 ? 'NULL' : "'" . $this->db->idate($this->sellby) . "'") . ",";       // no more used
-        $sql .= " " . (!isset($this->eatby) || dol_strlen($this->eatby) == 0 ? 'NULL' : "'" . $this->db->idate($this->eatby) . "'") . ",";          // no more used
+        $sql .= " " . (!isset($this->sellby) || dol_strlen($this->sellby) == 0 ? 'NULL' : "'" . $this->db->idate($this->sellby) . "'") . ","; // no more used
+        $sql .= " " . (!isset($this->eatby) || dol_strlen($this->eatby) == 0 ? 'NULL' : "'" . $this->db->idate($this->eatby) . "'") . ","; // no more used
         $sql .= " " . (!isset($this->batch) ? 'NULL' : "'" . $this->db->escape($this->batch) . "'") . ",";
         $sql .= " " . (!isset($this->qty) ? 'NULL' : $this->qty) . ",";
         $sql .= " " . (!isset($this->import_key) ? 'NULL' : "'" . $this->db->escape($this->import_key) . "'");
@@ -162,7 +162,7 @@ 
         $sql .= " pl.eatby,";
         $sql .= " pl.sellby";
         $sql .= " FROM " . $this->db->prefix() . "product_batch as t";
-        $sql .= " INNER JOIN " . $this->db->prefix() . "product_stock w on t.fk_product_stock = w.rowid";   // llx_product_stock is a parent table so this link does NOT generate duplicate record
+        $sql .= " INNER JOIN " . $this->db->prefix() . "product_stock w on t.fk_product_stock = w.rowid"; // llx_product_stock is a parent table so this link does NOT generate duplicate record
         $sql .= " LEFT JOIN " . $this->db->prefix() . "product_lot as pl on pl.fk_product = w.fk_product and pl.batch = t.batch";
         $sql .= " WHERE t.rowid = " . ((int) $id);
 
@@ -431,7 +431,7 @@ 
                 $this->tms = $this->db->jdate($obj->tms);
                 $this->fk_product_stock = $obj->fk_product_stock;
                 $this->sellby = $this->db->jdate($obj->sellby); // deprecated. do no tuse this data.
-                $this->eatby = $this->db->jdate($obj->eatby);   // deprecated. do not use this data.
+                $this->eatby = $this->db->jdate($obj->eatby); // deprecated. do not use this data.
                 $this->batch = $obj->batch;
                 $this->qty = $obj->qty;
                 $this->import_key = $obj->import_key;
@@ -529,7 +529,7 @@ 
 
                 if ($fk_product > 0) {
                     // Some properties of the lot
-                    $tmp->lotid = $obj->lotid;  // ID in table of the details of properties of each lots
+                    $tmp->lotid = $obj->lotid; // ID in table of the details of properties of each lots
                     $tmp->sellby = $dbs->jdate($obj->sellby ? $obj->sellby : $obj->oldsellby);
                     $tmp->eatby = $dbs->jdate($obj->eatby ? $obj->eatby : $obj->oldeatby);
                 }

Dolibarr/Code/Product/Classes/ProductStockEntrepot.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) {

Dolibarr/Code/Product/Classes/Productlot.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++;

Dolibarr/Code/Product/Classes/PriceExpression.php

Indentation +4 added lines, -4 removed lines

@@ -168,7 +168,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    List all price expressions
      *
@@ -176,7 +176,7 @@ 
      */
     public function list_price_expression()
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "SELECT rowid, title, expression";
         $sql .= " FROM " . $this->db->prefix() . $this->table_element;
         $sql .= " ORDER BY title";
@@ -203,7 +203,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Returns any existing rowid with specified title
      *
@@ -212,7 +212,7 @@ 
      */
     public function find_title($title)
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "SELECT rowid";
         $sql .= " FROM " . $this->db->prefix() . $this->table_element;
         $sql .= " WHERE title = '" . $this->db->escape($title) . "'";

Dolibarr/Code/Product/Classes/ProductCustomerPrice.php

Spacing +1 added lines, -1 removed lines

@@ -256,7 +256,7 @@
         $sql .= " " . (empty($this->localtax2_type) ? "'0'" : "'" . $this->db->escape($this->localtax2_type) . "'") . ",";
         $sql .= " " . (!isset($this->localtax2_tx) ? 'NULL' : (empty($this->localtax2_tx) ? 0 : $this->localtax2_tx)) . ",";
         $sql .= " " . ((int) $user->id) . ",";
-        $sql .=  " " . (!isset($this->price_label) ? 'NULL' : "'" . $this->db->escape($this->price_label) . "'") . ",";
+        $sql .= " " . (!isset($this->price_label) ? 'NULL' : "'" . $this->db->escape($this->price_label) . "'") . ",";
         $sql .= " " . (!isset($this->import_key) ? 'NULL' : "'" . $this->db->escape($this->import_key) . "'");
         $sql .= ")";
 

Dolibarr/Code/Product/Classes/MouvementStock.php

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);

Indentation +6 added lines, -6 removed lines

@@ -166,7 +166,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.
@@ -196,7 +196,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;
 
 
@@ -965,7 +965,7 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return Url link of origin object
      *
@@ -975,7 +975,7 @@ 
      */
     public function get_origin($origin_id, $origin_type)
     {
-		// phpcs:enable
+        // phpcs:enable
         $origin = '';
 
         switch ($origin_type) {
@@ -1188,7 +1188,7 @@ 
         return $this->LibStatut($mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the label of the status
      *
@@ -1197,7 +1197,7 @@ 
      */
     public function LibStatut($mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         if ($mode == 0 || $mode == 1) {

Dolibarr/Code/Product/Classes/Inventory.php

Indentation +2 added lines, -2 removed lines

@@ -648,7 +648,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the status
      *
@@ -658,7 +658,7 @@ 
      */
     public static function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $labelStatus = array();

Spacing +5 added lines, -5 removed lines

@@ -64,10 +64,10 @@ 
      */
     public $picto = 'inventory';
 
-    const STATUS_DRAFT     = 0;     // Draft
-    const STATUS_VALIDATED = 1;     // Inventory is in process
-    const STATUS_RECORDED  = 2;     // Inventory is finisged. Stock movement has been recorded.
-    const STATUS_CANCELED  = 9;     // Canceled
+    const STATUS_DRAFT     = 0; // Draft
+    const STATUS_VALIDATED = 1; // Inventory is in process
+    const STATUS_RECORDED  = 2; // Inventory is finisged. Stock movement has been recorded.
+    const STATUS_CANCELED  = 9; // Canceled
 
     /**
      *  'type' field format ('integer', 'integer:ObjectClass:PathToClass[:AddCreateButtonOrNot[:Filter]]', 'sellist:TableName:LabelFieldName[:KeyFieldName[:KeyFieldParent[:Filter]]]', 'varchar(x)', 'double(24,8)', 'real', 'price', 'text', 'text:none', 'html', 'date', 'datetime', 'timestamp', 'duration', 'mail', 'phone', 'url', 'password')
@@ -107,7 +107,7 @@ 
         'fk_warehouse'       => array('type' => 'integer:Entrepot:product/stock/class/entrepot.class.php', 'label' => 'Warehouse', 'visible' => 1, 'enabled' => 1, 'position' => 30, 'index' => 1, 'help' => 'InventoryForASpecificWarehouse', 'picto' => 'stock', 'css' => 'minwidth300 maxwidth500 widthcentpercentminusx', 'csslist' => 'tdoverflowmax150'),
         'fk_product'         => array('type' => 'integer:Product:product/class/product.class.php', 'label' => 'Product', 'get_name_url_params' => '0::0:-1:0::1', 'visible' => 1, 'enabled' => 1, 'position' => 32, 'index' => 1, 'help' => 'InventoryForASpecificProduct', 'picto' => 'product', 'css' => 'minwidth300 maxwidth500 widthcentpercentminusx', 'csslist' => 'tdoverflowmax150'),
         'categories_product' => array('type' => 'chkbxlst:categorie:label:rowid::type=0:0:', 'label' => 'OrProductsWithCategories', 'visible' => 3, 'enabled' => 1, 'position' => 33, 'help' => '', 'picto' => 'category', 'css' => 'minwidth300 maxwidth500 widthcentpercentminusx'),
-        'date_inventory'     => array('type' => 'date', 'label' => 'DateValue', 'visible' => 1, 'enabled' => '$conf->global->STOCK_INVENTORY_ADD_A_VALUE_DATE', 'position' => 35, 'csslist' => 'nowraponall'),  // This date is not used so disabled by default.
+        'date_inventory'     => array('type' => 'date', 'label' => 'DateValue', 'visible' => 1, 'enabled' => '$conf->global->STOCK_INVENTORY_ADD_A_VALUE_DATE', 'position' => 35, 'csslist' => 'nowraponall'), // This date is not used so disabled by default.
         'date_creation'      => array('type' => 'datetime', 'label' => 'DateCreation', 'enabled' => 1, 'visible' => -2, 'notnull' => 1, 'position' => 500, 'csslist' => 'nowraponall'),
         'tms'                => array('type' => 'timestamp', 'label' => 'DateModification', 'enabled' => 1, 'visible' => -2, 'notnull' => 1, 'position' => 501, 'csslist' => 'nowraponall'),
         'date_validation'    => array('type' => 'datetime', 'label' => 'DateValidation', 'visible' => -2, 'enabled' => 1, 'position' => 502, 'csslist' => 'nowraponall'),

Dolibarr/Code/Product/Classes/PriceGlobalVariableUpdater.php

Indentation +4 added lines, -4 removed lines

@@ -558,7 +558,7 @@ 
         return 1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update next_update into database
      *
@@ -569,7 +569,7 @@ 
      */
     public function update_next_update($next_update, $user = null, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $error = 0;
 
         $this->next_update = $next_update;
@@ -603,7 +603,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update last_status into database
      *
@@ -614,7 +614,7 @@ 
      */
     public function update_status($last_status, $user = null, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $error = 0;
 
         $this->last_status = $last_status;