alxarafe / alixar

public/htdocs/product/stock/stats/commande_fournisseur.php

Spacing +2 added lines, -2 removed lines

@@ -40,8 +40,8 @@
 
 $id = GETPOSTINT('id');
 $ref = GETPOST('ref', 'alpha');
-$batch      = GETPOST('batch', 'alpha');
-$objectid  = GETPOSTINT('productid');
+$batch = GETPOST('batch', 'alpha');
+$objectid = GETPOSTINT('productid');
 
 // Security check
 $fieldvalue = (!empty($id) ? $id : (!empty($ref) ? $ref : ''));

public/htdocs/product/stock/stats/expedition.php

Spacing +2 added lines, -2 removed lines

@@ -40,8 +40,8 @@
 
 $id = GETPOSTINT('id');
 $ref = GETPOST('ref', 'alpha');
-$batch      = GETPOST('batch', 'alpha');
-$objectid  = GETPOSTINT('productid');
+$batch = GETPOST('batch', 'alpha');
+$objectid = GETPOSTINT('productid');
 
 // Security check
 $fieldvalue = (!empty($id) ? $id : (!empty($ref) ? $ref : ''));

public/htdocs/product/stock/stats/reception.php

Spacing +2 added lines, -2 removed lines

@@ -40,8 +40,8 @@
 
 $id = GETPOSTINT('id');
 $ref = GETPOST('ref', 'alpha');
-$batch      = GETPOST('batch', 'alpha');
-$objectid  = GETPOSTINT('productid');
+$batch = GETPOST('batch', 'alpha');
+$objectid = GETPOSTINT('productid');
 
 // Security check
 $fieldvalue = (!empty($id) ? $id : (!empty($ref) ? $ref : ''));

public/htdocs/product/stock/list.php

Spacing +3 added lines, -3 removed lines

@@ -452,7 +452,7 @@ 
     //'builddoc'=>img_picto('', 'pdf', 'class="pictofixedwidth"').$langs->trans("PDFMerge"),
 );
 //if ($user->rights->stock->supprimer) $arrayofmassactions['predelete']=img_picto('', 'delete', 'class="pictofixedwidth"').$langs->trans("Delete");
-if (GETPOSTINT('nomassaction') || in_array($massaction, array('presend', 'predelete','preaffecttag'))) {
+if (GETPOSTINT('nomassaction') || in_array($massaction, array('presend', 'predelete', 'preaffecttag'))) {
     $arrayofmassactions = array();
 }
 if (isModEnabled('category') && $user->hasRight('stock', 'creer')) {
@@ -530,7 +530,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) : '');
 
@@ -647,7 +647,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/product/stock/stockatdate.php

Spacing +5 added lines, -7 removed lines

@@ -157,7 +157,7 @@ 
     if ($productid > 0) {
         $sql .= " AND ps.fk_product = " . ((int) $productid);
     }
-    if (! empty($search_fk_warehouse)) {
+    if (!empty($search_fk_warehouse)) {
         $sql .= " AND ps.fk_entrepot IN (" . $db->sanitize(implode(",", $search_fk_warehouse)) . ")";
     }
     if ($search_ref) {
@@ -373,8 +373,7 @@ 
 
     // Lines of title
     print implode(";", ($mode == 'future') ?
-        array('"Product Reference"', '"Label"', '"Current Stock"', '"' . $stocklabel . '"', '"Virtual Stock"') :
-        array('"Product Reference"', '"Label"', '"' . $stocklabel . '"', '"Estimated Stock Value"', '"Estimate Sell Value"', '"Movements"', '"Current Stock"')) . "\r\n";
+        array('"Product Reference"', '"Label"', '"Current Stock"', '"' . $stocklabel . '"', '"Virtual Stock"') : array('"Product Reference"', '"Label"', '"' . $stocklabel . '"', '"Estimated Stock Value"', '"Estimate Sell Value"', '"Movements"', '"Current Stock"')) . "\r\n";
 } else {
     llxHeader('', $title, $helpurl, '', 0, 0, '', '', '', 'mod-product page-stock_stockatdate');
 
@@ -604,7 +603,7 @@ 
 
         if ($mode == 'future') {
             $prod->load_stock('warehouseopen,warehouseinternal,nobatch', 0, $dateendofday);
-            $stock = $prod->stock_theorique;        // virtual stock at a date
+            $stock = $prod->stock_theorique; // virtual stock at a date
             $prod->load_stock('warehouseopen,warehouseinternal,nobatch', 0);
             $virtualstock = $prod->stock_theorique; // virtual stock in infinite future
         } else {
@@ -712,7 +711,7 @@ 
                         $url .= '&search_date_startyear=' . GETPOSTINT('dateyear');
                     }
                     if (count($search_fk_warehouse) > 1) {
-                        $url = '';  // Do not show link, multi warehouse as filter not managed yet by target page
+                        $url = ''; // Do not show link, multi warehouse as filter not managed yet by target page
                     } else {
                         foreach ($search_fk_warehouse as $val) {
                             $url .= ($val > 0 ? '&search_warehouse=' . ((int) $val) : '');
@@ -767,8 +766,7 @@ 
         '',
         $productid > 0 ? price2num($totalcurrentstock, 'MS') : '',
         '',
-        price(price2num($totalvirtualstock, 'MS'))) :
-        array(
+        price(price2num($totalvirtualstock, 'MS'))) : array(
         '"' . $langs->trans("Totalforthispage") . '"',
         '',
         '',

public/htdocs/product/stock/productlot_card.php

Spacing +6 added lines, -6 removed lines

@@ -153,7 +153,7 @@ 
 
     $backurlforlist = dol_buildpath('/product/stock/productlot_list.php', 1);
 
-    if ($action == 'seteatby' && $user->hasRight('stock', 'creer') && ! GETPOST('cancel', 'alpha')) {
+    if ($action == 'seteatby' && $user->hasRight('stock', 'creer') && !GETPOST('cancel', 'alpha')) {
         $newvalue = dol_mktime(12, 0, 0, GETPOSTINT('eatbymonth'), GETPOSTINT('eatbyday'), GETPOSTINT('eatbyyear'));
 
         // check parameters
@@ -178,7 +178,7 @@ 
         }
     }
 
-    if ($action == 'setsellby' && $user->hasRight('stock', 'creer') && ! GETPOST('cancel', 'alpha')) {
+    if ($action == 'setsellby' && $user->hasRight('stock', 'creer') && !GETPOST('cancel', 'alpha')) {
         $newvalue = dol_mktime(12, 0, 0, GETPOSTINT('sellbymonth'), GETPOSTINT('sellbyday'), GETPOSTINT('sellbyyear'));
 
         // check parameters
@@ -203,7 +203,7 @@ 
         }
     }
 
-    if ($action == 'seteol_date' && $user->hasRight('stock', 'creer') && ! GETPOST('cancel', 'alpha')) {
+    if ($action == 'seteol_date' && $user->hasRight('stock', 'creer') && !GETPOST('cancel', 'alpha')) {
         $newvalue = dol_mktime(12, 0, 0, GETPOSTINT('eol_datemonth'), GETPOSTINT('eol_dateday'), GETPOSTINT('eol_dateyear'));
         $result = $object->setValueFrom('eol_date', $newvalue, '', null, 'date', '', $user, 'PRODUCTLOT_MODIFY');
         if ($result < 0) {
@@ -214,7 +214,7 @@ 
         }
     }
 
-    if ($action == 'setmanufacturing_date' && $user->hasRight('stock', 'creer') && ! GETPOST('cancel', 'alpha')) {
+    if ($action == 'setmanufacturing_date' && $user->hasRight('stock', 'creer') && !GETPOST('cancel', 'alpha')) {
         $newvalue = dol_mktime(12, 0, 0, GETPOSTINT('manufacturing_datemonth'), GETPOSTINT('manufacturing_dateday'), GETPOSTINT('manufacturing_dateyear'));
         $result = $object->setValueFrom('manufacturing_date', $newvalue, '', null, 'date', '', $user, 'PRODUCTLOT_MODIFY');
         if ($result < 0) {
@@ -225,7 +225,7 @@ 
         }
     }
 
-    if ($action == 'setscrapping_date' && $user->hasRight('stock', 'creer') && ! GETPOST('cancel', 'alpha')) {
+    if ($action == 'setscrapping_date' && $user->hasRight('stock', 'creer') && !GETPOST('cancel', 'alpha')) {
         $newvalue = dol_mktime(12, 0, 0, GETPOSTINT('scrapping_datemonth'), GETPOSTINT('scrapping_dateday'), GETPOSTINT('scrapping_dateyear'));
         $result = $object->setValueFrom('scrapping_date', $newvalue, '', null, 'date', '', $user, 'PRODUCTLOT_MODIFY');
         if ($result < 0) {
@@ -247,7 +247,7 @@ 
         }
     } */
 
-    if ($action == 'setqc_frequency' && $user->hasRight('stock', 'creer') && ! GETPOST('cancel', 'alpha')) {
+    if ($action == 'setqc_frequency' && $user->hasRight('stock', 'creer') && !GETPOST('cancel', 'alpha')) {
         $result = $object->setValueFrom('qc_frequency', GETPOST('qc_frequency'), '', null, 'int', '', $user, 'PRODUCT_MODIFY');
         if ($result < 0) { // Prévoir un test de format de durée
             setEventMessages($object->error, null, 'errors');

public/htdocs/product/stock/class/api_stockmovements.class.php

Indentation +3 added lines, -3 removed lines

@@ -156,7 +156,7 @@ 
      * $price Can be set to update AWP (Average Weighted Price) when you make a stock increase
      * $dlc Eat-by date. Will be used if lot does not exists yet and will be created.
      * $dluo Sell-by date. Will be used if lot does not exists yet and will be created.
-         *
+     *
      * @param int $product_id Id product id {@min 1} {@from body} {@required true}
      * @param int $warehouse_id Id warehouse {@min 1} {@from body} {@required true}
      * @param float $qty Qty to add (Use negative value for a stock decrease) {@from body} {@required true}
@@ -278,7 +278,7 @@ 
 
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Clean sensible object datas
      *
@@ -287,7 +287,7 @@ 
      */
     protected function _cleanObjectDatas($object)
     {
-		// phpcs:enable
+        // phpcs:enable
         $object = parent::_cleanObjectDatas($object);
 
         // Remove useless data

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

Indentation +12 added lines, -12 removed lines

@@ -598,7 +598,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of all warehouses
      *
@@ -607,7 +607,7 @@ 
      */
     public function list_array($status = 1)
     {
-		// phpcs:enable
+        // phpcs:enable
         $liste = array();
 
         $sql = "SELECT rowid, ref as label";
@@ -629,7 +629,7 @@ 
         return $liste;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return number of unique different product into a warehouse
      *
@@ -637,7 +637,7 @@ 
      */
     public function nb_different_products()
     {
-		// phpcs:enable
+        // phpcs:enable
         $ret = array();
 
         $sql = "SELECT count(distinct p.rowid) as nb";
@@ -660,7 +660,7 @@ 
         return $ret;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return stock and value of warehosue
      *
@@ -669,7 +669,7 @@ 
     public function nb_products()
     {
         global $conf;
-		// phpcs:enable
+        // phpcs:enable
         $ret = array();
 
         //For MultiCompany PMP per entity
@@ -719,7 +719,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
      *
@@ -729,7 +729,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $statusType = 'status5';
@@ -903,7 +903,7 @@ 
         return 1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return full path to current warehouse
      *
@@ -911,7 +911,7 @@ 
      */
     public function get_full_arbo()
     {
-		// phpcs:enable
+        // phpcs:enable
         $TArbo = array($this->label);
 
         $protection = 100; // We limit depth of warehouses to 100
@@ -944,7 +944,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)
      *
@@ -954,7 +954,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
 
 
 

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

Indentation +6 added lines, -6 removed lines

@@ -162,7 +162,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.
@@ -192,7 +192,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 constant('DOL_DOCUMENT_ROOT') . '/product/class/product.class.php';
@@ -963,7 +963,7 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return Url link of origin object
      *
@@ -973,7 +973,7 @@ 
      */
     public function get_origin($origin_id, $origin_type)
     {
-		// phpcs:enable
+        // phpcs:enable
         $origin = '';
 
         switch ($origin_type) {
@@ -1189,7 +1189,7 @@ 
         return $this->LibStatut($mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the label of the status
      *
@@ -1198,7 +1198,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);