alxarafe / alixar

public/htdocs/fourn/class/fournisseur.commande.class.php

Indentation +14 added lines, -14 removed lines

@@ -606,7 +606,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Load array lines
      *
@@ -615,7 +615,7 @@ 
      */
     public function fetch_lines($only_product = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
 
         $this->lines = array();
 
@@ -880,7 +880,7 @@ 
         return $this->LibStatut($this->statut, $mode, $this->billed);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return label of a status
      *
@@ -891,7 +891,7 @@ 
      */
     public function LibStatut($status, $mode = 0, $billed = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $hookmanager;
 
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
@@ -1389,7 +1389,7 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Cancel an approved order.
      *  The cancellation is done after approval
@@ -1400,7 +1400,7 @@ 
      */
     public function Cancel($user, $idwarehouse = -1)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf;
 
         $error = 0;
@@ -2537,7 +2537,7 @@ 
         return $ret;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set a delivery in database for this supplier order
      *
@@ -2549,7 +2549,7 @@ 
      */
     public function Livraison($user, $date, $type, $comment)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $result = 0;
@@ -2651,7 +2651,7 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set delivery date
      *
@@ -2663,7 +2663,7 @@ 
      */
     public function set_date_livraison($user, $delivery_date, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         return $this->setDeliveryDate($user, $delivery_date, $notrigger);
     }
 
@@ -2723,7 +2723,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set the id projet
      *
@@ -2734,7 +2734,7 @@ 
      */
     public function set_id_projet($user, $id_projet, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if ($user->hasRight("fournisseur", "commande", "creer") || $user->hasRight("supplier_order", "creer")) {
             $error = 0;
 
@@ -3242,7 +3242,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load indicators for dashboard (this->nbtodo and this->nbtodolate)
      *
@@ -3252,7 +3252,7 @@ 
      */
     public function load_board($user, $mode = 'opened')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $sql = "SELECT c.rowid, c.date_creation as datec, c.date_commande, c.fk_statut, c.date_livraison as delivery_date, c.total_ht";

Spacing +15 added lines, -15 removed lines

@@ -520,7 +520,7 @@ 
             $this->ref_supplier = $obj->ref_supplier;
             $this->socid = $obj->fk_soc;
             $this->fourn_id = $obj->fk_soc;
-            $this->statut = $obj->status;   // deprecated
+            $this->statut = $obj->status; // deprecated
             $this->status = $obj->status;
             $this->billed = $obj->billed;
             $this->user_author_id = $obj->user_author_id;
@@ -551,7 +551,7 @@ 
             $this->fk_project = $obj->fk_project;
             $this->cond_reglement_id = $obj->fk_cond_reglement;
             $this->cond_reglement_code = $obj->cond_reglement_code;
-            $this->cond_reglement = $obj->cond_reglement_label;         // deprecated
+            $this->cond_reglement = $obj->cond_reglement_label; // deprecated
             $this->cond_reglement_label = $obj->cond_reglement_label;
             $this->cond_reglement_doc = $obj->cond_reglement_doc;
             $this->fk_account = $obj->fk_account;
@@ -684,10 +684,10 @@ 
                     $sqlsearchpackage .= ' WHERE entity IN (' . getEntity('product_fournisseur_price') . ")";
                     $sqlsearchpackage .= " AND fk_product = " . ((int) $objp->fk_product);
                     $sqlsearchpackage .= " AND ref_fourn = '" . $this->db->escape($objp->ref_supplier) . "'";
-                    $sqlsearchpackage .= " AND quantity <= " . ((float) $objp->qty);  // required to be qualified
-                    $sqlsearchpackage .= " AND (packaging IS NULL OR packaging = 0 OR packaging <= " . ((float) $objp->qty) . ")";  // required to be qualified
+                    $sqlsearchpackage .= " AND quantity <= " . ((float) $objp->qty); // required to be qualified
+                    $sqlsearchpackage .= " AND (packaging IS NULL OR packaging = 0 OR packaging <= " . ((float) $objp->qty) . ")"; // required to be qualified
                     $sqlsearchpackage .= " AND fk_soc = " . ((int) $this->socid);
-                    $sqlsearchpackage .= " ORDER BY packaging ASC";     // Take the smaller package first
+                    $sqlsearchpackage .= " ORDER BY packaging ASC"; // Take the smaller package first
                     $sqlsearchpackage .= " LIMIT 1";
 
                     $resqlsearchpackage = $this->db->query($sqlsearchpackage);
@@ -1623,10 +1623,10 @@ 
                         $line->date_end,
                         $line->array_options,
                         $line->fk_unit,
-                        $line->multicurrency_subprice,  // pu_ht_devise
-                        $line->origin,     // origin
-                        $line->origin_id,  // origin_id
-                        $line->rang,       // rang
+                        $line->multicurrency_subprice, // pu_ht_devise
+                        $line->origin, // origin
+                        $line->origin_id, // origin_id
+                        $line->rang, // rang
                         $line->special_code
                     );
                     if ($result < 0) {
@@ -2000,7 +2000,7 @@ 
             $this->db->begin();
 
             $product_type = $type;
-            $label = '';    // deprecated
+            $label = ''; // deprecated
 
             if ($fk_product > 0) {
                 if (getDolGlobalString('SUPPLIER_ORDER_WITH_PREDEFINED_PRICES_ONLY')) { // Not the common case
@@ -2143,7 +2143,7 @@ 
             $this->line->total_localtax2 = $total_localtax2;
             $this->line->total_ttc = $total_ttc;
             $this->line->product_type = $type;
-            $this->line->special_code   = (!empty($special_code) ? $special_code : 0);
+            $this->line->special_code = (!empty($special_code) ? $special_code : 0);
             $this->line->origin = $origin;
             $this->line->origin_id = $origin_id;
             $this->line->fk_unit = $fk_unit;
@@ -3350,7 +3350,7 @@ 
         global $conf, $langs;
 
         if (!dol_strlen($modele)) {
-            $modele = '';   // No doc template/generation by default
+            $modele = ''; // No doc template/generation by default
 
             if (!empty($this->model_pdf)) {
                 $modele = $this->model_pdf;
@@ -3892,10 +3892,10 @@ 
                     $sqlsearchpackage .= ' WHERE entity IN (' . getEntity('product_fournisseur_price') . ")";
                     $sqlsearchpackage .= " AND fk_product = " . ((int) $objp->fk_product);
                     $sqlsearchpackage .= " AND ref_fourn = '" . $this->db->escape($objp->ref_supplier) . "'";
-                    $sqlsearchpackage .= " AND quantity <= " . ((float) $objp->qty);  // required to be qualified
-                    $sqlsearchpackage .= " AND (packaging IS NULL OR packaging = 0 OR packaging <= " . ((float) $objp->qty) . ")";  // required to be qualified
+                    $sqlsearchpackage .= " AND quantity <= " . ((float) $objp->qty); // required to be qualified
+                    $sqlsearchpackage .= " AND (packaging IS NULL OR packaging = 0 OR packaging <= " . ((float) $objp->qty) . ")"; // required to be qualified
                     $sqlsearchpackage .= " AND fk_soc = " . ((int) $objp->socid);
-                    $sqlsearchpackage .= " ORDER BY packaging ASC";     // Take the smaller package first
+                    $sqlsearchpackage .= " ORDER BY packaging ASC"; // Take the smaller package first
                     $sqlsearchpackage .= " LIMIT 1";
 
                     $resqlsearchpackage = $this->db->query($sqlsearchpackage);

public/htdocs/fourn/class/fournisseur.facture.class.php

Indentation +16 added lines, -16 removed lines

@@ -1012,7 +1012,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load this->lines
      *
@@ -1020,7 +1020,7 @@ 
      */
     public function fetch_lines()
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->lines = array();
 
         $sql = 'SELECT f.rowid, f.ref as ref_supplier, f.description as line_desc, f.date_start, f.date_end, f.pu_ht, f.pu_ttc, f.qty, f.remise_percent, f.vat_src_code, f.tva_tx';
@@ -1307,7 +1307,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Add a discount line into an invoice (as an invoice line) using an existing absolute discount (Consume the discount)
      *
@@ -1316,7 +1316,7 @@ 
      */
     public function insert_discount($idremise)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         include_once DOL_DOCUMENT_ROOT . '/core/lib/price.lib.php';
@@ -1542,7 +1542,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Tag invoice as a paid invoice
      *
@@ -1555,7 +1555,7 @@ 
      */
     public function set_paid($user, $close_code = '', $close_note = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         dol_syslog(get_class($this) . "::set_paid is deprecated, use setPaid instead", LOG_NOTICE);
         return $this->setPaid($user, $close_code, $close_note);
     }
@@ -1620,7 +1620,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Tag the invoice as not fully paid + trigger call BILL_UNPAYED
      *  Function used when a direct debit payment is refused,
@@ -1633,7 +1633,7 @@ 
      */
     public function set_unpaid($user)
     {
-		// phpcs:enable
+        // phpcs:enable
         dol_syslog(get_class($this) . "::set_unpaid is deprecated, use setUnpaid instead", LOG_NOTICE);
         return $this->setUnpaid($user);
     }
@@ -2568,7 +2568,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of replaceable invoices
      *  Status valid or abandoned for other reason + not paid + no payment + not already replaced
@@ -2579,7 +2579,7 @@ 
      */
     public function list_replacable_supplier_invoices($socid = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $return = array();
@@ -2617,7 +2617,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of qualifying invoices for correction by credit note
      *  Invoices that respect the following rules are returned:
@@ -2629,7 +2629,7 @@ 
      */
     public function list_qualified_avoir_supplier_invoices($socid = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $return = array();
@@ -2671,7 +2671,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load indicators for dashboard (this->nbtodo and this->nbtodolate)
      *
@@ -2680,7 +2680,7 @@ 
      */
     public function load_board($user)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $sql = 'SELECT ff.rowid, ff.date_lim_reglement as datefin, ff.fk_statut as status, ff.total_ht, ff.total_ttc';
@@ -4431,7 +4431,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Mise a jour de l'objet ligne de commande en base
      *
@@ -4439,7 +4439,7 @@ 
      */
     public function update_total()
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->db->begin();
 
         // Mise a jour ligne en base

Spacing +17 added lines, -18 removed lines

@@ -227,7 +227,7 @@ 
     /**
      * @var int ID
      */
-    public $fk_account;     // default bank account
+    public $fk_account; // default bank account
 
     /**
      * @var int Transport mode id
@@ -463,7 +463,7 @@ 
             $this->location_incoterms = $_facrec->location_incoterms;
 
             // Clean parameters
-            if (! $this->type) {
+            if (!$this->type) {
                 $this->type = self::TYPE_STANDARD;
             }
             if (!empty(GETPOST('ref_supplier'))) {
@@ -477,7 +477,7 @@ 
 
             $this->array_options = $_facrec->array_options;
 
-            if (! $this->mode_reglement_id) {
+            if (!$this->mode_reglement_id) {
                 $this->mode_reglement_id = 0;
             }
             $this->status = self::STATUS_DRAFT;
@@ -726,7 +726,7 @@ 
             /*
              * Insert lines of template invoices
              */
-            if (! $error && $this->fac_rec > 0 && $_facrec instanceof FactureFournisseurRec) {
+            if (!$error && $this->fac_rec > 0 && $_facrec instanceof FactureFournisseurRec) {
                 foreach ($_facrec->lines as $i => $val) {
                     if ($_facrec->lines[$i]->fk_product) {
                         $prod = new Product($this->db);
@@ -753,7 +753,7 @@ 
                     $buyprice = empty($_facrec->lines[$i]->buyprice) ? 0 : $_facrec->lines[$i]->buyprice;
 
                     // If buyprice not defined from template invoice, we try to guess the best value
-                    if (! $buyprice && $_facrec->lines[$i]->fk_product > 0) {
+                    if (!$buyprice && $_facrec->lines[$i]->fk_product > 0) {
                         require_once constant('DOL_DOCUMENT_ROOT') . '/fourn/class/fournisseur.product.class.php';
                         $producttmp = new ProductFournisseur($this->db);
                         $producttmp->fetch($_facrec->lines[$i]->fk_product);
@@ -1004,7 +1004,7 @@ 
 
                 $this->extraparams = isset($obj->extraparams) ? (array) json_decode($obj->extraparams, true) : array();
 
-                $this->socid  = $obj->socid;
+                $this->socid = $obj->socid;
 
                 // Retrieve all extrafield
                 // fetch optionals attributes and labels
@@ -2278,7 +2278,7 @@ 
             // Multicurrency
             $supplierinvoiceline->fk_multicurrency = $this->fk_multicurrency;
             $supplierinvoiceline->multicurrency_code = $this->multicurrency_code;
-            $supplierinvoiceline->multicurrency_subprice    = ($this->type == self::TYPE_CREDIT_NOTE ? -abs($pu_ht_devise) : $pu_ht_devise); // For credit note, unit price always negative, always positive otherwise
+            $supplierinvoiceline->multicurrency_subprice = ($this->type == self::TYPE_CREDIT_NOTE ? -abs($pu_ht_devise) : $pu_ht_devise); // For credit note, unit price always negative, always positive otherwise
 
             $supplierinvoiceline->multicurrency_total_ht = (($this->type == self::TYPE_CREDIT_NOTE || $qty < 0) ? -abs($multicurrency_total_ht) : $multicurrency_total_ht); // For credit note and if qty is negative, total is negative
             $supplierinvoiceline->multicurrency_total_tva = (($this->type == self::TYPE_CREDIT_NOTE || $qty < 0) ? -abs($multicurrency_total_tva) : $multicurrency_total_tva); // For credit note and if qty is negative, total is negative
@@ -2444,7 +2444,7 @@ 
 
         $line->qty = ($this->type == self::TYPE_CREDIT_NOTE ? abs((float) $qty) : $qty); // For credit note, quantity is always positive and unit price negative
         $line->subprice = ($this->type == self::TYPE_CREDIT_NOTE ? -abs($pu_ht) : $pu_ht); // For credit note, unit price always negative, always positive otherwise
-        $line->pu_ht = $line->subprice;  // deprecated
+        $line->pu_ht = $line->subprice; // deprecated
         $line->pu_ttc = ($this->type == self::TYPE_CREDIT_NOTE ? -abs($pu_ttc) : $pu_ttc); // For credit note, unit price always negative, always positive otherwise
 
         $line->remise_percent = $remise_percent;
@@ -2729,7 +2729,7 @@ 
 
             while ($obj = $this->db->fetch_object($resql)) {
                 $facturestatic->date_echeance = $this->db->jdate($obj->datefin);
-                $facturestatic->statut = $obj->status;  // For backward compatibility
+                $facturestatic->statut = $obj->status; // For backward compatibility
                 $facturestatic->status = $obj->status;
 
                 $response->nbtodo++;
@@ -3164,15 +3164,15 @@ 
         // Load source object
         $object->fetch($fromid);
         $object->id = 0;
-        $object->statut = self::STATUS_DRAFT;   // For backward compatibility
+        $object->statut = self::STATUS_DRAFT; // For backward compatibility
         $object->status = self::STATUS_DRAFT;
 
         $object->fetch_thirdparty(); // We need it to recalculate VAT localtaxes according to main sale taxes and vendor
 
         // Clear fields
         $object->ref_supplier       = (empty($this->ref_supplier) ? $langs->trans("CopyOf") . ' ' . $object->ref_supplier : $this->ref_supplier);
-        $object->author             = $user->id;  // FIXME? user_validation_id is replacement for author
-        $object->user_validation_id = 0;  // FIXME?  user_validation_id is replacement for author
+        $object->author             = $user->id; // FIXME? user_validation_id is replacement for author
+        $object->user_validation_id = 0; // FIXME?  user_validation_id is replacement for author
         $object->fk_facture_source  = 0;
         $object->date_creation      = '';
         $object->date_validation    = '';
@@ -3475,14 +3475,14 @@ 
         if (!empty($paymentmode) && $paymentmode != 'all') {
             $sql .= ", " . MAIN_DB_PREFIX . "c_paiement as cp";
         }
-        $sql .= " WHERE f.paye = 0";    // Only unpaid
-        $sql .= " AND f.fk_statut = " . self::STATUS_VALIDATED;   // Only validated status
+        $sql .= " WHERE f.paye = 0"; // Only unpaid
+        $sql .= " AND f.fk_statut = " . self::STATUS_VALIDATED; // Only validated status
         if ($datetouse == 'invoicedate') {
             $sql .= " AND f.datef = '" . $this->db->idate($tmpidate, 'gmt') . "'";
         } else {
             $sql .= " AND f.date_lim_reglement = '" . $this->db->idate($tmpidate, 'gmt') . "'";
         }
-        $sql .= " AND f.entity IN (" . getEntity('supplier_invoice', 0) . ")";  // One batch process only one company (no sharing)
+        $sql .= " AND f.entity IN (" . getEntity('supplier_invoice', 0) . ")"; // One batch process only one company (no sharing)
         if (!empty($paymentmode) && $paymentmode != 'all') {
             $sql .= " AND f.fk_mode_reglement = cp.id AND cp.code = '" . $this->db->escape($paymentmode) . "'";
         }
@@ -3702,7 +3702,7 @@ 
                                 $actioncomm->create($user);
                             }
 
-                            $this->db->commit();    // We always commit
+                            $this->db->commit(); // We always commit
                         }
 
                         if ($errormesg) {
@@ -4377,8 +4377,7 @@ 
         $sql .= " " . ($this->fk_parent_line > 0 ? "'" . $this->db->escape($this->fk_parent_line) . "'" : "null") . ",";
         $product_label
             = !empty($this->product_label)
-              ? $this->product_label :
-                (!empty($this->label) ? $this->label : null);
+              ? $this->product_label : (!empty($this->label) ? $this->label : null);
         $sql .= " " . (!empty($product_label) ? "'" . $this->db->escape($product_label) . "'" : "null") . ",";
         $sql .= " '" . $this->db->escape($this->desc ? $this->desc : $this->description) . "',";
         $sql .= " '" . $this->db->escape($this->ref_supplier) . "',";

public/htdocs/fourn/class/api_supplier_orders.class.php

Indentation +2 added lines, -2 removed lines

@@ -712,7 +712,7 @@ 
         );
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Clean sensible object datas
      *
@@ -721,7 +721,7 @@ 
      */
     protected function _cleanObjectDatas($object)
     {
-		// phpcs:enable
+        // phpcs:enable
         $object = parent::_cleanObjectDatas($object);
 
         unset($object->rowid);

public/htdocs/fourn/class/fournisseur.class.php

Indentation +4 added lines, -4 removed lines

@@ -161,7 +161,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Create a supplier category
      *
@@ -171,7 +171,7 @@ 
      */
     public function CreateCategory($user, $name)
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "INSERT INTO " . MAIN_DB_PREFIX . "categorie (label,visible,type)";
         $sql .= " VALUES ";
         $sql .= " ('" . $this->db->escape($name) . "',1,1)";
@@ -188,7 +188,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the suppliers list
      *
@@ -196,7 +196,7 @@ 
      */
     public function ListArray()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
         global $user;
 

public/htdocs/fourn/class/paiementfourn.class.php

Indentation +4 added lines, -4 removed lines

@@ -596,7 +596,7 @@ 
         return $this->LibStatut($this->statut, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the label of a given status
      *
@@ -606,7 +606,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $langs->load('compta');
@@ -893,7 +893,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load the third party of object, from id into this->thirdparty
      *
@@ -902,7 +902,7 @@ 
      */
     public function fetch_thirdparty($force_thirdparty_id = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         require_once constant('DOL_DOCUMENT_ROOT') . '/fourn/class/fournisseur.facture.class.php';
 
         if (empty($force_thirdparty_id)) {

Spacing +1 added lines, -1 removed lines

@@ -143,7 +143,7 @@
                 $this->note_private         = $obj->note;
                 $this->type_code            = $obj->payment_code;
                 $this->type_label           = $obj->payment_type;
-                $this->fk_paiement           = $obj->fk_paiement;
+                $this->fk_paiement = $obj->fk_paiement;
                 $this->statut               = $obj->statut;
 
                 $error = 1;

public/htdocs/fourn/class/fournisseur.product.class.php

Indentation +16 added lines, -16 removed lines

@@ -228,7 +228,7 @@ 
         $this->reputations = array('-1' => '', 'FAVORITE' => $langs->trans('Favorite'), 'NOTTHGOOD' => $langs->trans('NotTheGoodQualitySupplier'), 'DONOTORDER' => $langs->trans('DoNotOrderThisProductToThisSupplier'));
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Remove all prices for this couple supplier-product
      *
@@ -237,7 +237,7 @@ 
      */
     public function remove_fournisseur($id_fourn)
     {
-		// phpcs:enable
+        // phpcs:enable
         $ok = 1;
 
         $this->db->begin();
@@ -262,7 +262,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Remove a price for a couple supplier-product
      *
@@ -271,7 +271,7 @@ 
      */
     public function remove_product_fournisseur_price($rowid)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
 
         $error = 0;
@@ -307,7 +307,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Modify the purchase price for a supplier
      *
@@ -363,7 +363,7 @@ 
         $fk_barcode_type = 0,
         $options = array()
     ) {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
         //global $mysoc;
 
@@ -667,7 +667,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Loads the price information of a provider
      *
@@ -677,7 +677,7 @@ 
      */
     public function fetch_product_fournisseur_price($rowid, $ignore_expression = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $sql = "SELECT pfp.rowid, pfp.price, pfp.quantity, pfp.unitprice, pfp.remise_percent, pfp.remise, pfp.tva_tx, pfp.default_vat_code, pfp.info_bits as fourn_tva_npr, pfp.fk_availability,";
@@ -762,7 +762,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    List all supplier prices of a product
      *
@@ -777,7 +777,7 @@ 
      */
     public function list_product_fournisseur_price($prodid, $sortfield = '', $sortorder = '', $limit = 0, $offset = 0, $socid = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $sql = "SELECT s.nom as supplier_name, s.rowid as fourn_id, p.ref as product_ref, p.tosell as status, p.tobuy as status_buy, ";
@@ -877,7 +877,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load properties for minimum price
      *
@@ -889,7 +889,7 @@ 
      */
     public function find_min_price_product_fournisseur($prodid, $qty = 0, $socid = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         if (empty($prodid)) {
@@ -1062,7 +1062,7 @@ 
         return $thirdparty->getNomUrl($withpicto, $option, $maxlen, $notooltip);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Display price of product
      *
@@ -1076,7 +1076,7 @@ 
      */
     public function display_price_product_fournisseur($showunitprice = 1, $showsuptitle = 1, $maxlen = 0, $notooltip = 0, $productFournList = array())
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $out = '';
@@ -1414,7 +1414,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the status
      *
@@ -1425,7 +1425,7 @@ 
      */
     public function LibStatut($status, $mode = 0, $type = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
             global $langs;
             //$langs->load("mymodule@mymodule");

Spacing +5 added lines, -5 removed lines

@@ -137,7 +137,7 @@ 
      */
     public $fourn_remise;
 
-    public $fourn_charges;  // when getDolGlobalString('PRODUCT_CHARGES') is set
+    public $fourn_charges; // when getDolGlobalString('PRODUCT_CHARGES') is set
 
     /**
      * @var int     product-supplier id
@@ -157,7 +157,7 @@ 
     public $fk_availability;
 
     public $fourn_unitprice;
-    public $fourn_unitprice_with_discount;  // not saved into database
+    public $fourn_unitprice_with_discount; // not saved into database
     public $fourn_tva_tx;
     public $fourn_tva_npr;
 
@@ -475,7 +475,7 @@ 
             }
             $sql = "UPDATE " . MAIN_DB_PREFIX . "product_fournisseur_price";
             $sql .= " SET fk_user = " . ((int) $user->id) . " ,";
-            $sql .= " datec = '" . $this->db->idate($now) . "' ,";  // Note: Even if this is an update, we update the creation date as the log of each change is tracked into product_fournisseur_log.
+            $sql .= " datec = '" . $this->db->idate($now) . "' ,"; // Note: Even if this is an update, we update the creation date as the log of each change is tracked into product_fournisseur_log.
             $sql .= " ref_fourn = '" . $this->db->escape($ref_fourn) . "',";
             $sql .= " desc_fourn = '" . $this->db->escape($desc_fourn) . "',";
             $sql .= " price = " . ((float) $buyprice) . ",";
@@ -825,10 +825,10 @@ 
                 $prodfourn->fourn_tva_tx = $record["tva_tx"];
                 $prodfourn->fourn_id = $record["fourn_id"];
                 $prodfourn->fourn_name = $record["supplier_name"];
-                $prodfourn->fk_availability         = $record["fk_availability"];
+                $prodfourn->fk_availability = $record["fk_availability"];
                 $prodfourn->delivery_time_days = $record["delivery_time_days"];
                 $prodfourn->id = $prodid;
-                $prodfourn->fourn_tva_npr                   = $record["info_bits"];
+                $prodfourn->fourn_tva_npr = $record["info_bits"];
                 $prodfourn->fk_supplier_price_expression = $record["fk_supplier_price_expression"];
                 $prodfourn->supplier_reputation = $record["supplier_reputation"];
                 $prodfourn->fourn_date_creation          = $this->db->jdate($record['datec']);

public/htdocs/fourn/class/api_supplier_invoices.class.php

Indentation +2 added lines, -2 removed lines

@@ -716,7 +716,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Clean sensible object datas
      *
@@ -725,7 +725,7 @@ 
      */
     protected function _cleanObjectDatas($object)
     {
-		// phpcs:enable
+        // phpcs:enable
         $object = parent::_cleanObjectDatas($object);
 
         unset($object->rowid);

public/htdocs/fourn/commande/card.php

Spacing +5 added lines, -5 removed lines

@@ -446,7 +446,7 @@ 
             $idprod = GETPOSTINT('idprod');
         }
 
-        $tva_tx = (GETPOST('tva_tx') ? GETPOST('tva_tx') : 0);      // Can be '1.2' or '1.2 (CODE)'
+        $tva_tx = (GETPOST('tva_tx') ? GETPOST('tva_tx') : 0); // Can be '1.2' or '1.2 (CODE)'
 
         $price_ht = price2num(GETPOST('price_ht'), 'MU', 2);
         $price_ht_devise = price2num(GETPOST('multicurrency_price_ht'), 'CU', 2);
@@ -1249,7 +1249,7 @@ 
             $object->mode_reglement_id = GETPOSTINT('mode_reglement_id');
             $object->fk_account        = GETPOSTINT('fk_account');
             $object->note_private = GETPOST('note_private', 'restricthtml');
-            $object->note_public    = GETPOST('note_public', 'restricthtml');
+            $object->note_public = GETPOST('note_public', 'restricthtml');
             $object->delivery_date = $datelivraison;
             $object->fk_incoterms = GETPOSTINT('incoterm_id');
             $object->location_incoterms = GETPOST('location_incoterms', 'alpha');
@@ -1685,7 +1685,7 @@ 
             $demand_reason_id = (!empty($objectsrc->demand_reason_id) ? $objectsrc->demand_reason_id : (!empty($soc->demand_reason_id) ? $soc->demand_reason_id : 0));
             //$remise_percent       = (!empty($objectsrc->remise_percent) ? $objectsrc->remise_percent : (!empty($soc->remise_supplier_percent) ? $soc->remise_supplier_percent : 0));
             //$remise_absolue       = (!empty($objectsrc->remise_absolue) ? $objectsrc->remise_absolue : (!empty($soc->remise_absolue) ? $soc->remise_absolue : 0));
-            $dateinvoice        = !getDolGlobalString('MAIN_AUTOFILL_DATE') ? -1 : '';
+            $dateinvoice = !getDolGlobalString('MAIN_AUTOFILL_DATE') ? -1 : '';
 
             $datedelivery = (!empty($objectsrc->delivery_date) ? $objectsrc->delivery_date : '');
 
@@ -1801,7 +1801,7 @@ 
         // Payment term
         print '<tr><td class="nowrap">' . $langs->trans('PaymentConditionsShort') . '</td><td>';
         print img_picto('', 'payment', 'class="pictofixedwidth"');
-        print $form->getSelectConditionsPaiements((GETPOSTISSET('cond_reglement_id') &&  GETPOST('cond_reglement_id') != 0) ? GETPOST('cond_reglement_id') : $cond_reglement_id, 'cond_reglement_id', -1, 1);
+        print $form->getSelectConditionsPaiements((GETPOSTISSET('cond_reglement_id') && GETPOST('cond_reglement_id') != 0) ? GETPOST('cond_reglement_id') : $cond_reglement_id, 'cond_reglement_id', -1, 1);
         print '</td></tr>';
 
         // Payment mode
@@ -2051,7 +2051,7 @@ 
                 'morecss' => 'minwidth300'
             )
         );
-        $formconfirm  = $form->formconfirm($_SERVER['PHP_SELF'] . "?id=$object->id", $langs->trans("DenyingThisOrder"), $langs->trans("ConfirmDenyingThisOrder", $object->ref), "confirm_refuse", $formquestion, 0, 1);
+        $formconfirm = $form->formconfirm($_SERVER['PHP_SELF'] . "?id=$object->id", $langs->trans("DenyingThisOrder"), $langs->trans("ConfirmDenyingThisOrder", $object->ref), "confirm_refuse", $formquestion, 0, 1);
     }
 
     // Confirmation of cancellation

public/htdocs/fourn/commande/dispatch.php

Spacing +3 added lines, -3 removed lines

@@ -102,8 +102,8 @@ 
     accessforbidden();
 }
 
-$usercancreate  = ($user->hasRight("fournisseur", "commande", "creer") || $user->hasRight("supplier_order", "creer"));
-$permissiontoadd    = $usercancreate; // Used by the include of actions_addupdatedelete.inc.php
+$usercancreate = ($user->hasRight("fournisseur", "commande", "creer") || $user->hasRight("supplier_order", "creer"));
+$permissiontoadd = $usercancreate; // Used by the include of actions_addupdatedelete.inc.php
 
 
 /*
@@ -629,7 +629,7 @@ 
     if (!empty($conf->global->STOCK_CALCULATE_ON_SUPPLIER_DISPATCH_ORDER)) {
         $disabled = 0;
     }*/
-    $disabled = 0;  // This is used to disable or not the bulk selection of target warehouse. No reason to have it disabled so forced to 0.
+    $disabled = 0; // This is used to disable or not the bulk selection of target warehouse. No reason to have it disabled so forced to 0.
 
     // Line of orders
     if ($object->statut <= CommandeFournisseur::STATUS_ACCEPTED || $object->statut >= CommandeFournisseur::STATUS_CANCELED) {