alxarafe / alixar

Dolibarr/Code/Fourn/Classes/FactureFournisseurRec.php

Indentation +4 added lines, -4 removed lines

@@ -699,7 +699,7 @@ 
         return $this->fetch_lines();
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Get lines of template invoices into this->lines
      *
@@ -707,7 +707,7 @@ 
      */
     public function fetch_lines()
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->lines = array();
 
         // Retrieve all extrafield for line
@@ -1512,7 +1512,7 @@ 
         return $this->LibStatut($this->frequency ? 1 : 0, $this->suspended, $mode, $alreadypaid, empty($this->type) ? 0 : $this->type);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return label of a status
      *
@@ -1525,7 +1525,7 @@ 
      */
     public function LibStatut($recur, $status, $mode = 0, $alreadypaid = -1, $type = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $langs->load('bills');
 

Spacing +9 added lines, -9 removed lines

@@ -355,13 +355,13 @@ 
             $sql .= ", '" . $this->db->idate($now) . "'";
             $sql .= ", " . ((int) $this->suspended);
             $sql .= ", '" . $this->db->escape($this->libelle) . "'";
-            $sql .= ", " . (!empty($facfourn_src->total_ttc) ? (float) $facfourn_src->total_ttc : '0');                              // amount
+            $sql .= ", " . (!empty($facfourn_src->total_ttc) ? (float) $facfourn_src->total_ttc : '0'); // amount
             $sql .= ", " . ((int) $user->id);
             $sql .= ", " . (!empty($this->fk_project) ? ((int) $this->fk_project) : 'NULL');
             $sql .= ", " . (!empty($facfourn_src->fk_account) ? ((int) $facfourn_src->fk_account) : 'NULL');
             $sql .= ", " . ($this->cond_reglement_id > 0 ? (int) $this->cond_reglement_id : 'NULL');
             $sql .= ", " . ($this->mode_reglement_id > 0 ? (int) $this->mode_reglement_id : 'NULL');
-            $sql .= ", " . ($facfourn_src->date_echeance > 0 ? "'" . $this->db->idate($facfourn_src->date_echeance) . "'" : 'NULL');      // date_lim_reglement
+            $sql .= ", " . ($facfourn_src->date_echeance > 0 ? "'" . $this->db->idate($facfourn_src->date_echeance) . "'" : 'NULL'); // date_lim_reglement
             $sql .= ", " . (!empty($this->note_private) ? "'" . $this->db->escape($this->note_private) . "'" : 'NULL');
             $sql .= ", " . (!empty($this->note_public) ? "'" . $this->db->escape($this->note_public) . "'" : 'NULL');
             $sql .= ", " . (!empty($this->model_pdf) ? "'" . $this->db->escape($this->model_pdf) . "'" : 'NULL');
@@ -516,7 +516,7 @@ 
         $error = 0;
 
         $sql = "UPDATE " . MAIN_DB_PREFIX . "facture_fourn_rec SET";
-        $sql .= " titre = '" . (!empty($this->title) ? $this->db->escape($this->title) : "") . "'," ;
+        $sql .= " titre = '" . (!empty($this->title) ? $this->db->escape($this->title) : "") . "',";
         $sql .= " subtype=" . (isset($this->subtype) ? $this->db->escape($this->subtype) : "null") . ",";
         $sql .= " ref_supplier = '" . (!empty($this->ref_supplier) ? $this->db->escape($this->ref_supplier) : "") . "',";
         $sql .= " entity = " . (!empty($this->entity) ? ((int) $this->entity) : 1) . ',';
@@ -634,7 +634,7 @@ 
                 $this->id                       = $obj->rowid;
                 $this->titre                    = $obj->title;
                 $this->title                    = $obj->title;
-                $this->subtype                        = $obj->subtype;
+                $this->subtype = $obj->subtype;
                 $this->ref                      = $obj->title;
                 $this->ref_supplier             = $obj->ref_supplier;
                 $this->entity                   = $obj->entity;
@@ -792,7 +792,7 @@ 
                 $line->product_type             = $objp->product_type;
                 $line->date_start               = $objp->date_start;
                 $line->date_end                 = $objp->date_end;
-                $line->info_bits                = $objp->info_bits  ;
+                $line->info_bits                = $objp->info_bits;
                 $line->special_code             = $objp->special_code;
                 $line->rang                     = $objp->rang;
                 $line->fk_unit                  = $objp->fk_unit;
@@ -875,7 +875,7 @@ 
             }
             // End call triggers
         }
-        if (! $error) {
+        if (!$error) {
             $this->db->commit();
             return 1;
         } else {
@@ -1019,7 +1019,7 @@ 
             $sql .= ', fk_user_author';
             $sql .= ', fk_multicurrency, multicurrency_code, multicurrency_subprice, multicurrency_total_ht, multicurrency_total_tva, multicurrency_total_ttc';
             $sql .= ') VALUES (';
-            $sql .= ' ' . (int) $facid;   // source supplier invoice id
+            $sql .= ' ' . (int) $facid; // source supplier invoice id
             $sql .= ', ' . (!empty($fk_product) ? "'" . $this->db->escape($fk_product) . "'" : 'null');
             $sql .= ', ' . (!empty($ref) ? "'" . $this->db->escape($ref) . "'" : 'null');
             $sql .= ', ' . (!empty($label) ? "'" . $this->db->escape($label) . "'" : 'null');
@@ -1360,7 +1360,7 @@ 
 
                     $new_fac_fourn->type = self::TYPE_STANDARD;
                     $new_fac_fourn->subtype = $facturerec->subtype;
-                    $new_fac_fourn->statut = self::STATUS_DRAFT;    // deprecated
+                    $new_fac_fourn->statut = self::STATUS_DRAFT; // deprecated
                     $new_fac_fourn->status = self::STATUS_DRAFT;
                     $new_fac_fourn->date = empty($facturerec->date_when) ? $now : $facturerec->date_when; // We could also use dol_now here but we prefer date_when so invoice has real date when we would like even if we generate later.
                     $new_fac_fourn->socid = $facturerec->socid;
@@ -1369,7 +1369,7 @@ 
                     $new_fac_fourn->model_pdf = $facturerec->model_pdf;
                     $new_fac_fourn->fk_project = $facturerec->fk_project;
                     $new_fac_fourn->label = $facturerec->label;
-                    $new_fac_fourn->libelle = $facturerec->label;   // deprecated
+                    $new_fac_fourn->libelle = $facturerec->label; // deprecated
 
                     $invoiceidgenerated = $new_fac_fourn->create($user);
                     $laststep = "Create invoiceidgenerated $invoiceidgenerated";

Dolibarr/Code/Fourn/Classes/CommandeFournisseur.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 +12 added lines, -12 removed lines

@@ -523,7 +523,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;
@@ -554,7 +554,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;
@@ -687,10 +687,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);
@@ -1626,10 +1626,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) {
@@ -2003,7 +2003,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
@@ -2146,7 +2146,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;
@@ -3353,7 +3353,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;

Dolibarr/Code/Fourn/Api/SupplierOrders.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);

Dolibarr/Code/Fourn/Api/SupplierInvoices.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);

Dolibarr/Code/ExpenseReport/Classes/ExpenseReportIk.php

Spacing +1 added lines, -1 removed lines

@@ -199,7 +199,7 @@
         $default_range = (int) $userauthor->default_range; // if not defined, then 0
         $ranges = $this->getRangesByCategory($fk_c_exp_tax_cat);
         // prevent out of range -1 indice
-        $indice = $default_range  - 1;
+        $indice = $default_range - 1;
         // subtract 1 because array start from 0
         if (empty($ranges) || $indice < 0 || !isset($ranges[$indice])) {
             return false;

Dolibarr/Code/ExpenseReport/Classes/ExpenseReportLine.php

Spacing +1 added lines, -1 removed lines

@@ -336,7 +336,7 @@
         } elseif ($mode == 'mon' || $mode == 'EX_MON') {
             $sql .= " AND DATE_FORMAT(d.date, '%Y-%m') = '" . dol_print_date($this->date, '%Y-%m') . "'"; // @todo DATE_FORMAT is forbidden
         } elseif ($mode == 'year' || $mode == 'EX_YEA') {
-            $sql .= " AND DATE_FORMAT(d.date, '%Y') = '" . dol_print_date($this->date, '%Y') . "'";     // @todo DATE_FORMAT is forbidden
+            $sql .= " AND DATE_FORMAT(d.date, '%Y') = '" . dol_print_date($this->date, '%Y') . "'"; // @todo DATE_FORMAT is forbidden
         }
 
         dol_syslog('ExpenseReportLine::getExpAmount');

Dolibarr/Code/ExpenseReport/Classes/ExpenseReport.php

Indentation +22 added lines, -22 removed lines

@@ -767,7 +767,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Classify the expense report as paid
      *
@@ -780,7 +780,7 @@ 
      */
     public function set_paid($id, $fuser, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         dol_syslog(get_class($this) . "::set_paid is deprecated, use setPaid instead", LOG_NOTICE);
         return $this->setPaid($id, $fuser, $notrigger);
     }
@@ -846,7 +846,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Returns the label of a status
      *
@@ -856,7 +856,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $labelStatus = $langs->transnoentitiesnoconv($this->labelStatus[$status]);
@@ -986,7 +986,7 @@ 
         return 1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * fetch_line_by_project
      *
@@ -996,7 +996,7 @@ 
      */
     public function fetch_line_by_project($projectid, $user)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $langs->load('trips');
@@ -1088,7 +1088,7 @@ 
         return 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * fetch_lines
      *
@@ -1096,7 +1096,7 @@ 
      */
     public function fetch_lines()
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->lines = array();
 
         $sql = ' SELECT de.rowid, de.comments, de.qty, de.value_unit, de.date, de.rang,';
@@ -1426,7 +1426,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * set_save_from_refuse
      *
@@ -1435,7 +1435,7 @@ 
      */
     public function set_save_from_refuse($fuser)
     {
-		// phpcs:enable
+        // phpcs:enable
         // Sélection de la date de début de la NDF
         $sql = 'SELECT date_debut';
         $sql .= ' FROM ' . MAIN_DB_PREFIX . $this->table_element;
@@ -1577,7 +1577,7 @@ 
         return 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * set_unpaid
      *
@@ -1589,7 +1589,7 @@ 
      */
     public function set_unpaid($fuser, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         dol_syslog(get_class($this) . "::set_unpaid is deprecated, use setUnpaid instead", LOG_NOTICE);
         return $this->setUnpaid($fuser, $notrigger);
     }
@@ -1645,7 +1645,7 @@ 
         return 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * set_cancel
      *
@@ -1656,7 +1656,7 @@ 
      */
     public function set_cancel($fuser, $detail, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $error = 0;
         $this->date_cancel = $this->db->idate(dol_now());
         if ($this->status != self::STATUS_CANCELED) {
@@ -1879,7 +1879,7 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update total of an expense report when you add a line.
      *
@@ -1889,7 +1889,7 @@ 
      */
     public function update_totaux_add($ligne_total_ht, $ligne_total_tva)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->total_ht += (float) $ligne_total_ht;
         $this->total_tva += (float) $ligne_total_tva;
         $this->total_ttc += $this->total_tva;
@@ -2375,7 +2375,7 @@ 
         return 1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * periode_existe
      *
@@ -2388,7 +2388,7 @@ 
     {
         global $conf;
 
-		// phpcs:enable
+        // phpcs:enable
         $sql = "SELECT rowid, date_debut, date_fin";
         $sql .= " FROM " . MAIN_DB_PREFIX . $this->table_element;
         $sql .= " WHERE entity = " . ((int) $conf->entity); // not shared, only for the current entity
@@ -2429,7 +2429,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return list of people with permission to validate expense reports.
      * Search for permission "approve expense report"
@@ -2438,7 +2438,7 @@ 
      */
     public function fetch_users_approver_expensereport()
     {
-		// phpcs:enable
+        // phpcs:enable
         $users_validator = array();
 
         $sql = "SELECT DISTINCT ur.fk_user";
@@ -2564,7 +2564,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Load indicators for dashboard (this->nbtodo and this->nbtodolate)
      *
@@ -2574,7 +2574,7 @@ 
      */
     public function load_board($user, $option = 'topay')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         if ($user->socid) {

Spacing +17 added lines, -17 removed lines

@@ -249,8 +249,8 @@ 
      */
     public $fk_user_approve;
 
-    public $localtax1;  // for backward compatibility (real field should be total_localtax1 defined into CommonObject)
-    public $localtax2;  // for backward compatibility (real field should be total_localtax2 defined into CommonObject)
+    public $localtax1; // for backward compatibility (real field should be total_localtax1 defined into CommonObject)
+    public $localtax2; // for backward compatibility (real field should be total_localtax2 defined into CommonObject)
 
     /**
      * Draft status
@@ -341,8 +341,8 @@ 
         $this->total_tva = 0;
         $this->total_localtax1 = 0;
         $this->total_localtax2 = 0;
-        $this->localtax1 = 0;   // For backward compatibility
-        $this->localtax2 = 0;   // For backward compatibility
+        $this->localtax1 = 0; // For backward compatibility
+        $this->localtax2 = 0; // For backward compatibility
         $this->modepaymentid = 0;
 
         // List of language codes for status
@@ -699,8 +699,8 @@ 
                 $this->total_ht     = $obj->total_ht;
                 $this->total_tva    = $obj->total_tva;
                 $this->total_ttc    = $obj->total_ttc;
-                $this->localtax1    = $obj->total_localtax1;        // For backward compatibility
-                $this->localtax2    = $obj->total_localtax2;        // For backward compatibility
+                $this->localtax1    = $obj->total_localtax1; // For backward compatibility
+                $this->localtax2    = $obj->total_localtax2; // For backward compatibility
                 $this->total_localtax1 = $obj->total_localtax1;
                 $this->total_localtax2 = $obj->total_localtax2;
 
@@ -1167,7 +1167,7 @@ 
 
                 $deplig->rule_warning_message = $objp->rule_warning_message;
 
-                $deplig->rang               = $objp->rang;
+                $deplig->rang = $objp->rang;
 
                 $this->lines[$i] = $deplig;
 
@@ -1963,8 +1963,8 @@ 
             $this->line = new ExpenseReportLine($this->db);
 
             // We don't know seller and buyer for expense reports
-            $seller = $mysoc;           // We use same than current company (expense report are often done in same country)
-            $seller->tva_assuj = 1;     // Most seller uses vat
+            $seller = $mysoc; // We use same than current company (expense report are often done in same country)
+            $seller->tva_assuj = 1; // Most seller uses vat
             $buyer = new Societe($this->db);
 
             $localtaxes_type = getLocalTaxesFromRate($vatrate, 0, $buyer, $seller);
@@ -2046,8 +2046,8 @@ 
 
         // We don't know seller and buyer for expense reports
         if (!is_object($seller)) {
-            $seller = $mysoc;           // We use same than current company (expense report are often done in same country)
-            $seller->tva_assuj = 1;     // Most seller uses vat
+            $seller = $mysoc; // We use same than current company (expense report are often done in same country)
+            $seller->tva_assuj = 1; // Most seller uses vat
         }
 
         $expensereportrule = new ExpenseReportRule($db);
@@ -2131,8 +2131,8 @@ 
 
         // We don't know seller and buyer for expense reports
         if (!is_object($seller)) {
-            $seller = $mysoc;           // We use same than current company (expense report are often done in same country)
-            $seller->tva_assuj = 1;     // Most seller uses vat
+            $seller = $mysoc; // We use same than current company (expense report are often done in same country)
+            $seller->tva_assuj = 1; // Most seller uses vat
         }
 
         $expenseik = new ExpenseReportIk($this->db);
@@ -2226,10 +2226,10 @@ 
             $type = 0; // TODO What if type is service ?
 
             // We don't know seller and buyer for expense reports
-            $seller = $mysoc;           // We use same than current company (expense report are often done in same country)
-            $seller->tva_assuj = 1;     // Most seller uses vat
-            $seller->localtax1_assuj = $mysoc->localtax1_assuj;     // We don't know, we reuse the state of company
-            $seller->localtax2_assuj = $mysoc->localtax1_assuj;     // We don't know, we reuse the state of company
+            $seller = $mysoc; // We use same than current company (expense report are often done in same country)
+            $seller->tva_assuj = 1; // Most seller uses vat
+            $seller->localtax1_assuj = $mysoc->localtax1_assuj; // We don't know, we reuse the state of company
+            $seller->localtax2_assuj = $mysoc->localtax1_assuj; // We don't know, we reuse the state of company
             $buyer = new Societe($this->db);
 
             $localtaxes_type = getLocalTaxesFromRate($vatrate, 0, $buyer, $seller);

Dolibarr/Code/ExpenseReport/Classes/PaymentExpenseReport.php

Indentation +8 added lines, -8 removed lines

@@ -297,7 +297,7 @@ 
         }
     }
 
-	// phpcs:disable Generic.CodeAnalysis.UnusedFunctionParameter
+    // phpcs:disable Generic.CodeAnalysis.UnusedFunctionParameter
     /**
      *  Update database
      *
@@ -307,7 +307,7 @@ 
      */
     public function update($user = null, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $error = 0;
 
         // Clean parameters
@@ -375,7 +375,7 @@ 
         }
     }
 
-	// phpcs:disable Generic.CodeAnalysis.UnusedFunctionParameter
+    // phpcs:disable Generic.CodeAnalysis.UnusedFunctionParameter
     /**
      *  Delete object in database
      *
@@ -385,7 +385,7 @@ 
      */
     public function delete($user, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $error = 0;
 
         $this->db->begin();
@@ -487,7 +487,7 @@ 
         return '';
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the label of a given status
      *
@@ -497,7 +497,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         //global $langs;
 
         return '';
@@ -635,7 +635,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update link between the expense report payment and the generated line in llx_bank
      *
@@ -644,7 +644,7 @@ 
      */
     public function update_fk_bank($id_bank)
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "UPDATE " . MAIN_DB_PREFIX . "payment_expensereport SET fk_bank = " . ((int) $id_bank) . " WHERE rowid = " . ((int) $this->id);
 
         dol_syslog(get_class($this) . "::update_fk_bank", LOG_DEBUG);

Spacing +1 added lines, -1 removed lines

@@ -211,7 +211,7 @@
             $sql .= " '" . $this->db->idate($this->datep) . "',";
             $sql .= " " . price2num($totalamount) . ",";
             $sql .= " " . ((int) $this->fk_typepayment) . ", '" . $this->db->escape($this->num_payment) . "', '" . $this->db->escape($this->note_public) . "', " . ((int) $user->id) . ",";
-            $sql .= " 0)";  // fk_bank is ID of transaction into ll_bank
+            $sql .= " 0)"; // fk_bank is ID of transaction into ll_bank
 
             dol_syslog(get_class($this) . "::create", LOG_DEBUG);
             $resql = $this->db->query($sql);

Dolibarr/Code/ExpenseReport/Api/ExpenseReports.php

Indentation +2 added lines, -2 removed lines

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