alxarafe / alixar

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

public/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) {

public/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++;

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

Indentation +5 added lines, -5 removed lines

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

public/htdocs/product/stock/stocktransfer/stocktransfer_card.php

Spacing +5 added lines, -5 removed lines

@@ -52,7 +52,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');
@@ -556,28 +556,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)
         );

public/htdocs/product/stock/stocktransfer/stocktransfer_list.php

Spacing +4 added lines, -4 removed lines

@@ -77,7 +77,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) {
@@ -347,7 +347,7 @@ 
 // Output page
 // --------------------------------------------------------------------
 
-llxHeader('', $title, $help_url, '', 0, 0, $morejs, $morecss, '', 'bodyforlist mod-product page-stock-stocktransfer_stocktransfer_list');   // Can use also classforhorizontalscrolloftabs instead of bodyforlist for no horizontal scroll
+llxHeader('', $title, $help_url, '', 0, 0, $morejs, $morecss, '', 'bodyforlist mod-product page-stock-stocktransfer_stocktransfer_list'); // Can use also classforhorizontalscrolloftabs instead of bodyforlist for no horizontal scroll
 
 // Example : Adding jquery code
 // print '<script type="text/javascript">
@@ -475,7 +475,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) : '');
 
@@ -566,7 +566,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/stocktransfer/class/stocktransfer.class.php

Indentation +2 added lines, -2 removed lines

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