alxarafe / alixar

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

htdocs/product/stock/massstockmove.php

Spacing +1 added lines, -1 removed lines

@@ -579,7 +579,7 @@
 $maxfilesizearray = getMaxFileSizeArray();
 $maxmin = $maxfilesizearray['maxmin'];
 if ($maxmin > 0) {
-    print '<input type="hidden" name="MAX_FILE_SIZE" value="' . ($maxmin * 1024) . '">';    // MAX_FILE_SIZE must precede the field type=file
+    print '<input type="hidden" name="MAX_FILE_SIZE" value="' . ($maxmin * 1024) . '">'; // MAX_FILE_SIZE must precede the field type=file
 }
 print '<input type="file" name="userfile" size="20" maxlength="80"> &nbsp; &nbsp; ';
 $out = (!getDolGlobalString('MAIN_UPLOAD_DOC') ? ' disabled' : '');

htdocs/product/stock/stockatdate.php

Spacing +2 added lines, -2 removed lines

@@ -155,7 +155,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) {
@@ -565,7 +565,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 {

htdocs/product/stock/stocktransfer/stocktransfer_list.php

Spacing +3 added lines, -3 removed lines

@@ -75,7 +75,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) {
@@ -345,7 +345,7 @@ 
 // Output page
 // --------------------------------------------------------------------
 
-llxHeader('', $title, $help_url, '', 0, 0, $morejs, $morecss, '', 'bodyforlist');   // Can use also classforhorizontalscrolloftabs instead of bodyforlist for no horizontal scroll
+llxHeader('', $title, $help_url, '', 0, 0, $morejs, $morecss, '', 'bodyforlist'); // Can use also classforhorizontalscrolloftabs instead of bodyforlist for no horizontal scroll
 
 // Example : Adding jquery code
 // print '<script type="text/javascript">
@@ -563,7 +563,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']++;

htdocs/product/stock/stocktransfer/stocktransfer_card.php

Spacing +5 added lines, -5 removed lines

@@ -51,7 +51,7 @@ 
 $confirm    = GETPOST('confirm', 'alpha');
 $cancel     = GETPOST('cancel', 'aZ09');
 $contextpage = GETPOST('contextpage', 'aZ') ? GETPOST('contextpage', 'aZ') : str_replace('_', '', basename(dirname(__FILE__)) . basename(__FILE__, '.php')); // To manage different context of search
-$backtopage = GETPOST('backtopage', 'alpha');                   // if not set, a default page will be used
+$backtopage = GETPOST('backtopage', 'alpha'); // if not set, a default page will be used
 $backtopageforcancel = GETPOST('backtopageforcancel', 'alpha'); // if not set, $backtopage will be used
 $qty = GETPOSTINT('qty');
 $fk_product = GETPOSTINT('fk_product');
@@ -538,28 +538,28 @@ 
         $formconfirm = $form->formconfirm($_SERVER['PHP_SELF'] . '?id=' . $object->id, $langs->trans('ToClone'), $langs->trans('ConfirmCloneAsk', $object->ref), 'confirm_clone', $formquestion, 'yes', 1);
     } elseif ($action == 'destock') { // Destock confirmation
         // Create an array for form
-        $formquestion = array(  'text' => '',
+        $formquestion = array('text' => '',
             0 => array('type' => 'text', 'name' => 'label', 'label' => $langs->trans("Label"), 'value' => $langs->trans('ConfirmDestock', $object->ref), 'size' => 40),
             1 => array('type' => 'text', 'name' => 'inventorycode', 'label' => $langs->trans("InventoryCode"), 'value' => dol_print_date(dol_now(), '%y%m%d%H%M%S'), 'size' => 25)
         );
         $formconfirm = $form->formconfirm($_SERVER['PHP_SELF'] . '?id=' . $object->id, $langs->trans('DestockAllProduct'), '', 'confirm_destock', $formquestion, 'yes', 1);
     } elseif ($action == 'destockcancel') { // Destock confirmation cancel
         // Create an array for form
-        $formquestion = array(  'text' => '',
+        $formquestion = array('text' => '',
             0 => array('type' => 'text', 'name' => 'label', 'label' => $langs->trans("Label"), 'value' => $langs->trans('ConfirmDestockCancel', $object->ref), 'size' => 40),
             1 => array('type' => 'text', 'name' => 'inventorycode', 'label' => $langs->trans("InventoryCode"), 'value' => dol_print_date(dol_now(), '%y%m%d%H%M%S'), 'size' => 25)
         );
         $formconfirm = $form->formconfirm($_SERVER['PHP_SELF'] . '?id=' . $object->id, $langs->trans('DestockAllProductCancel'), '', 'confirm_destockcancel', $formquestion, 'yes', 1);
     } elseif ($action == 'addstock') { // Addstock confirmation
         // Create an array for form
-        $formquestion = array(  'text' => '',
+        $formquestion = array('text' => '',
             0 => array('type' => 'text', 'name' => 'label', 'label' => $langs->trans("Label") . ' :', 'value' => $langs->trans('ConfirmAddStock', $object->ref), 'size' => 40),
             1 => array('type' => 'text', 'name' => 'inventorycode', 'label' => $langs->trans("InventoryCode"), 'value' => dol_print_date(dol_now(), '%y%m%d%H%M%S'), 'size' => 25)
         );
         $formconfirm = $form->formconfirm($_SERVER['PHP_SELF'] . '?id=' . $object->id, $langs->trans('AddStockAllProduct'), '', 'confirm_addstock', $formquestion, 'yes', 1);
     } elseif ($action == 'addstockcancel') { // Addstock confirmation cancel
         // Create an array for form
-        $formquestion = array(  'text' => '',
+        $formquestion = array('text' => '',
             0 => array('type' => 'text', 'name' => 'label', 'label' => $langs->trans("Label") . ' :', 'value' => $langs->trans('ConfirmAddStockCancel', $object->ref), 'size' => 40),
             1 => array('type' => 'text', 'name' => 'inventorycode', 'label' => $langs->trans("InventoryCode"), 'value' => dol_print_date(dol_now(), '%y%m%d%H%M%S'), 'size' => 25)
         );

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

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/entrepot.class.php

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

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/product.class.php

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