alxarafe / alixar

htdocs/fourn/class/paiementfourn.class.php

Indentation +4 added lines, -4 removed lines

@@ -594,7 +594,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
      *
@@ -604,7 +604,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $langs->load('compta');
@@ -891,7 +891,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load the third party of object, from id into this->thirdparty
      *
@@ -900,7 +900,7 @@ 
      */
     public function fetch_thirdparty($force_thirdparty_id = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         require_once 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;

htdocs/fourn/class/fournisseur.facture-rec.class.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 +16 added lines, -16 removed lines

@@ -337,13 +337,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');
@@ -498,7 +498,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 .= " ref_supplier = '" . (!empty($this->ref_supplier) ? $this->db->escape($this->ref_supplier) : "") . "',";
         $sql .= " entity = " . (!empty($this->entity) ? ((int) $this->entity) : 1) . ',';
         if (!empty($this->socid) && $this->socid > 0) {
@@ -772,7 +772,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;
@@ -855,7 +855,7 @@ 
             }
             // End call triggers
         }
-        if (! $error) {
+        if (!$error) {
             $this->db->commit();
             return 1;
         } else {
@@ -999,7 +999,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');
@@ -1339,7 +1339,7 @@ 
                     $new_fac_fourn->fk_fac_rec_source = $facturerec->id; // We will create $facture from this recurring invoice
 
                     $new_fac_fourn->type = self::TYPE_STANDARD;
-                    $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;
@@ -1348,7 +1348,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";
@@ -1576,7 +1576,7 @@ 
                     $statusType = 'status6';
                     $labelStatus = $langs->transnoentitiesnoconv('Disabled');
                 } else {
-                    $statusType = 'status0';  // @phan-suppress-current-line PhanPluginRedundantAssignment
+                    $statusType = 'status0'; // @phan-suppress-current-line PhanPluginRedundantAssignment
                     $labelStatus = $langs->transnoentitiesnoconv('Draft');
                 }
             }
@@ -1595,7 +1595,7 @@ 
                     $statusType = 'status6';
                     $labelStatus = $langs->transnoentitiesnoconv('Disabled');
                 } else {
-                    $statusType = 'status0';  // @phan-suppress-current-line PhanPluginRedundantAssignment
+                    $statusType = 'status0'; // @phan-suppress-current-line PhanPluginRedundantAssignment
                     $labelStatus = $langs->transnoentitiesnoconv('Draft');
                 }
             }
@@ -1614,7 +1614,7 @@ 
                     $statusType = 'status6';
                     $labelStatus = $langs->transnoentitiesnoconv('Disabled');
                 } else {
-                    $statusType = 'status0';  // @phan-suppress-current-line PhanPluginRedundantAssignment
+                    $statusType = 'status0'; // @phan-suppress-current-line PhanPluginRedundantAssignment
                     $labelStatus = $langs->transnoentitiesnoconv('Draft');
                 }
             }
@@ -1636,7 +1636,7 @@ 
                     $statusType = 'status6';
                     $labelStatus = $langs->transnoentitiesnoconv('Disabled');
                 } else {
-                    $statusType = 'status0';  // @phan-suppress-current-line PhanPluginRedundantAssignment
+                    $statusType = 'status0'; // @phan-suppress-current-line PhanPluginRedundantAssignment
                     $labelStatus = $langs->transnoentitiesnoconv('Draft');
                 }
             }
@@ -2047,8 +2047,8 @@ 
         $error = 0;
         $this->db->begin();
 
-        if (! $error) {
-            if (! $notrigger) {
+        if (!$error) {
+            if (!$notrigger) {
                 // Call triggers
                 $result = $this->call_trigger('LINESUPPLIERBILLREC_DELETE', $user);
                 if ($result < 0) {
@@ -2058,14 +2058,14 @@ 
             }
         }
 
-        if (! $error) {
+        if (!$error) {
             $result = $this->deleteExtraFields();
             if ($result < 0) {
                 $error++;
             }
         }
 
-        if (! $error) {
+        if (!$error) {
             $sql = 'DELETE FROM ' . MAIN_DB_PREFIX . $this->table_element . ' WHERE rowid=' . (int) $this->id;
 
             $res = $this->db->query($sql);

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;
 

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

Indentation +2 added lines, -2 removed lines

@@ -568,7 +568,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return label of a status
      *
@@ -578,7 +578,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $langs->load('orders');
 

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 +13 added lines, -13 removed lines

@@ -217,7 +217,7 @@ 
     /**
      * @var int ID
      */
-    public $fk_account;     // default bank account
+    public $fk_account; // default bank account
 
     /**
      * @var int Transport mode id
@@ -444,7 +444,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'))) {
@@ -458,7 +458,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;
@@ -710,7 +710,7 @@ 
             /*
              * Insert lines of template invoices
              */
-            if (! $error && $this->fac_rec > 0) {
+            if (!$error && $this->fac_rec > 0) {
                 foreach ($_facrec->lines as $i => $val) {
                     if ($_facrec->lines[$i]->fk_product) {
                         $prod = new Product($this->db);
@@ -737,7 +737,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 DOL_DOCUMENT_ROOT . '/fourn/class/fournisseur.product.class.php';
                         $producttmp = new ProductFournisseur($this->db);
                         $producttmp->fetch($_facrec->lines[$i]->fk_product);
@@ -986,7 +986,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
@@ -2261,7 +2261,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
@@ -2712,7 +2712,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++;
@@ -3144,7 +3144,7 @@ 
         // 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
@@ -3444,14 +3444,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) . "'";
         }
@@ -3671,7 +3671,7 @@ 
                                 $actioncomm->create($user);
                             }
 
-                            $this->db->commit();    // We always commit
+                            $this->db->commit(); // We always commit
                         }
 
                         if ($errormesg) {

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

htdocs/contrat/class/api_contracts.class.php

Indentation +4 added lines, -4 removed lines

@@ -18,9 +18,9 @@ 
  * along with this program. If not, see <https://www.gnu.org/licenses/>.
  */
 
- use Luracast\Restler\RestException;
+    use Luracast\Restler\RestException;
 
- require_once DOL_DOCUMENT_ROOT . '/contrat/class/contrat.class.php';
+    require_once DOL_DOCUMENT_ROOT . '/contrat/class/contrat.class.php';
 
 /**
  * API class for contracts
@@ -646,7 +646,7 @@ 
 
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Clean sensible object datas
      *
@@ -655,7 +655,7 @@ 
      */
     protected function _cleanObjectDatas($object)
     {
-		// phpcs:enable
+        // phpcs:enable
         $object = parent::_cleanObjectDatas($object);
 
         unset($object->address);

htdocs/contrat/class/contrat.class.php

Indentation +22 added lines, -22 removed lines

@@ -363,7 +363,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Activate a contract line
      *
@@ -376,7 +376,7 @@ 
      */
     public function active_line($user, $line_id, $date_start, $date_end = '', $comment = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         $result = $this->lines[$this->lines_id_index_mapper[$line_id]]->active_line($user, $date_start, $date_end, $comment);
         if ($result < 0) {
             $this->error = $this->lines[$this->lines_id_index_mapper[$line_id]]->error;
@@ -386,7 +386,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Close a contract line
      *
@@ -398,7 +398,7 @@ 
      */
     public function close_line($user, $line_id, $date_end, $comment = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         $result = $this->lines[$this->lines_id_index_mapper[$line_id]]->close_line($user, $date_end, $comment);
         if ($result < 0) {
             $this->error = $this->lines[$this->lines_id_index_mapper[$line_id]]->error;
@@ -820,7 +820,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load lines array into this->lines.
      *  This set also nbofserviceswait, nbofservicesopened, nbofservicesexpired and nbofservicesclosed
@@ -832,7 +832,7 @@ 
      */
     public function fetch_lines($only_services = 0, $loadalsotranslation = 0, $noextrafields = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->nbofservices = 0;
         $this->nbofserviceswait = 0;
         $this->nbofservicesopened = 0;
@@ -1963,7 +1963,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update statut of contract according to services
      *
@@ -1973,7 +1973,7 @@ 
      */
     public function update_statut($user)
     {
-		// phpcs:enable
+        // phpcs:enable
         dol_syslog(__METHOD__ . " is deprecated", LOG_WARNING);
 
         // If draft, we keep it (should not happen)
@@ -2005,7 +2005,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 contrat status
      *
@@ -2015,7 +2015,7 @@ 
      */
     public function LibStatut($status, $mode)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
@@ -2225,7 +2225,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of line rowid
      *
@@ -2234,7 +2234,7 @@ 
      */
     public function array_detail($status = -1)
     {
-		// phpcs:enable
+        // phpcs:enable
         $tab = array();
 
         $sql = "SELECT cd.rowid";
@@ -2306,7 +2306,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Load indicators for dashboard (this->nbtodo and this->nbtodolate)
      *
@@ -2316,7 +2316,7 @@ 
      */
     public function load_board($user, $mode)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $this->from = " FROM " . MAIN_DB_PREFIX . "contrat as c";
@@ -3199,7 +3199,7 @@ 
         return $this->LibStatut($this->statut, $mode, ((!empty($this->date_end)) ? ($this->date_end < dol_now() ? 1 : 0) : -1));
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return label of a contract line status
      *
@@ -3211,7 +3211,7 @@ 
      */
     public static function LibStatut($status, $mode, $expired = -1, $moreatt = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $langs->load("contracts");
 
@@ -3646,7 +3646,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update in database the fields total_xxx of lines
      *  Used by migration process
@@ -3655,7 +3655,7 @@ 
      */
     public function update_total()
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->db->begin();
 
         // Mise a jour ligne en base
@@ -3773,7 +3773,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Activate a contract line
      *
@@ -3785,7 +3785,7 @@ 
      */
     public function active_line($user, $date, $date_end = '', $comment = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         $error = 0;
 
         $this->db->begin();
@@ -3831,7 +3831,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Close a contract line
      *
@@ -3843,7 +3843,7 @@ 
      */
     public function close_line($user, $date_end_real, $comment = '', $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->date_cloture = $date_end_real;
         $this->date_end_real = $date_end_real;
         $this->user_closing_id = $user->id;

Spacing +17 added lines, -17 removed lines

@@ -286,12 +286,12 @@ 
     /*
      * No signature
      */
-    const STATUS_NO_SIGNATURE    = 0;
+    const STATUS_NO_SIGNATURE = 0;
 
     /*
      * Signed by sender
      */
-    const STATUS_SIGNED_SENDER   = 1;
+    const STATUS_SIGNED_SENDER = 1;
 
     /*
      * Signed by receiver
@@ -301,7 +301,7 @@ 
     /*
      * Signed by all
      */
-    const STATUS_SIGNED_ALL      = 9; // To handle future kind of signature (ex: tripartite contract)
+    const STATUS_SIGNED_ALL = 9; // To handle future kind of signature (ex: tripartite contract)
 
 
     /**
@@ -437,7 +437,7 @@ 
             if ($contratline->statut != ContratLigne::STATUS_OPEN) {
                 $contratline->context = $this->context;
 
-                $result = $contratline->active_line($user, $date_start, !empty($date_end) ? $date_end : -1, $comment);  // This call trigger LINECONTRACT_ACTIVATE
+                $result = $contratline->active_line($user, $date_start, !empty($date_end) ? $date_end : -1, $comment); // This call trigger LINECONTRACT_ACTIVATE
                 if ($result < 0) {
                     $error++;
                     $this->error = $contratline->error;
@@ -487,7 +487,7 @@ 
             // Close lines not already closed
             if ($contratline->statut != ContratLigne::STATUS_CLOSED) {
                 $contratline->date_end_real = $now;
-                $contratline->date_cloture = $now;  // For backward compatibility
+                $contratline->date_cloture = $now; // For backward compatibility
                 $contratline->user_closing_id = $user->id;
                 $contratline->statut = ContratLigne::STATUS_CLOSED;
                 $result = $contratline->close_line($user, $now, $comment, $notrigger);
@@ -942,7 +942,7 @@ 
                 //$line->date_fin_prevue   = $this->db->jdate($objp->date_fin_validite);
                 //$line->date_fin_reel     = $this->db->jdate($objp->date_cloture);
 
-                $line->rang     = $objp->rang;
+                $line->rang = $objp->rang;
 
                 // Retrieve all extrafields for contract line
                 // fetch optionals attributes and labels
@@ -1454,7 +1454,7 @@ 
         }
 
         if (!$error) {
-            $result = $this->insertExtraFields();   // This delete and reinsert extrafields
+            $result = $this->insertExtraFields(); // This delete and reinsert extrafields
             if ($result < 0) {
                 $error++;
             }
@@ -2558,7 +2558,7 @@ 
         global $user;
 
         $ticket = new Ticket($this->db);
-        $nbTicket =  $ticket->fetchAll($user, 'ASC', 't.datec', '', 0, '', array('t.fk_contract' => $this->id));
+        $nbTicket = $ticket->fetchAll($user, 'ASC', 't.datec', '', 0, '', array('t.fk_contract' => $this->id));
 
         return ($nbTicket < 0 ? $nbTicket : $ticket->lines);
     }
@@ -2580,7 +2580,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;
@@ -2806,7 +2806,7 @@ 
 
                     // Load contract
                     $object = new Contrat($this->db);
-                    $object->fetch($obj->rowid);        // fetch also lines
+                    $object->fetch($obj->rowid); // fetch also lines
                     //$object->fetch_thirdparty();
 
                     if ($object->id <= 0) {
@@ -2853,7 +2853,7 @@ 
                     if ($expirationdate && $expirationdate < $enddatetoscan) {
                         dol_syslog("Define the newdate of end of services from expirationdate=" . $expirationdate);
                         $newdate = $expirationdate;
-                        $protecti = 0;  //$protecti is to avoid infinite loop
+                        $protecti = 0; //$protecti is to avoid infinite loop
                         while ($newdate < $enddatetoscan && $protecti < 1000) {
                             $newdate = dol_time_plus_duree($newdate, $duration_value, $duration_unit);
                             $protecti++;
@@ -2881,20 +2881,20 @@ 
 
                                 // Create an event
                                 $actioncomm = new ActionComm($this->db);
-                                $actioncomm->type_code    = 'AC_OTH_AUTO';      // Type of event ('AC_OTH', 'AC_OTH_AUTO', 'AC_XXX'...)
+                                $actioncomm->type_code    = 'AC_OTH_AUTO'; // Type of event ('AC_OTH', 'AC_OTH_AUTO', 'AC_XXX'...)
                                 $actioncomm->code         = 'AC_' . $actioncode;
                                 $actioncomm->label        = $label;
                                 $actioncomm->datep        = $now;
                                 $actioncomm->datef        = $now;
-                                $actioncomm->percentage   = -1;   // Not applicable
+                                $actioncomm->percentage   = -1; // Not applicable
                                 $actioncomm->socid        = $object->socid;
-                                $actioncomm->authorid     = $user->id;   // User saving action
-                                $actioncomm->userownerid  = $user->id;  // Owner of action
+                                $actioncomm->authorid     = $user->id; // User saving action
+                                $actioncomm->userownerid  = $user->id; // Owner of action
                                 $actioncomm->fk_element   = $object->id;
                                 $actioncomm->elementtype  = 'contract';
                                 $actioncomm->note_private = $comment;
 
-                                $ret = $actioncomm->create($user);       // User creating action
+                                $ret = $actioncomm->create($user); // User creating action
                             } else {
                                 $contracterror[$object->id] = $object->ref;
 
@@ -2903,7 +2903,7 @@ 
                                 $this->error = $this->db->lasterror();
                             }
 
-                            if (! $errorforlocaltransaction) {
+                            if (!$errorforlocaltransaction) {
                                 $this->db->commit();
                             } else {
                                 $this->db->rollback();

htdocs/multicurrency/class/api_multicurrencies.class.php

Indentation +4 added lines, -4 removed lines

@@ -339,7 +339,7 @@ 
         return $this->_cleanObjectDatas($multicurrency);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Clean sensible object datas
      *
@@ -348,7 +348,7 @@ 
      */
     protected function _cleanObjectDatas($object)
     {
-		// phpcs:enable
+        // phpcs:enable
         $object = parent::_cleanObjectDatas($object);
 
         // Clear all fields out of interest
@@ -365,7 +365,7 @@ 
         return $object;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Clean sensible MultiCurrencyRate object datas
      *
@@ -374,7 +374,7 @@ 
      */
     protected function _cleanObjectDatasRate($object)
     {
-		// phpcs:enable
+        // phpcs:enable
         $object = parent::_cleanObjectDatas($object);
 
         // Clear all fields out of interest