alxarafe / alixar

Dolibarr/Code/Compta/Classes/Cchargesociales.php

Indentation +2 added lines, -2 removed lines

@@ -458,7 +458,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }*/
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the label of a given status
      *
@@ -468,7 +468,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         if ($mode == 0) {

Dolibarr/Code/Compta/Classes/Facture.php

Spacing +12 added lines, -12 removed lines

@@ -532,7 +532,7 @@ 
             if (!empty($_facrec->frequency)) {  // Invoice are created on same thirdparty than template when there is a recurrence, but not necessarily when there is no recurrence.
                 $this->socid = $_facrec->socid;
             }
-            $this->entity            = $_facrec->entity; // Invoice created in same entity than template
+            $this->entity = $_facrec->entity; // Invoice created in same entity than template
 
             // Fields coming from GUI.
             // @TODO Value of template should be used as default value on the form on the GUI, and we should here always use the value from GUI
@@ -815,7 +815,7 @@ 
 
                         // Complete vat rate with code
                         $vatrate = $newinvoiceline->tva_tx;
-                        if ($newinvoiceline->vat_src_code && ! preg_match('/\(.*\)/', (string) $vatrate)) {
+                        if ($newinvoiceline->vat_src_code && !preg_match('/\(.*\)/', (string) $vatrate)) {
                             $vatrate .= ' (' . $newinvoiceline->vat_src_code . ')';
                         }
 
@@ -1047,7 +1047,7 @@ 
                         $buyprice,
                         $_facrec->lines[$i]->label,
                         empty($_facrec->lines[$i]->array_options) ? null : $_facrec->lines[$i]->array_options,
-                        100,    // situation percent is undefined on recurring invoice lines
+                        100, // situation percent is undefined on recurring invoice lines
                         '',
                         $_facrec->lines[$i]->fk_unit,
                         $_facrec->lines[$i]->multicurrency_subprice,
@@ -1157,7 +1157,7 @@ 
 
         $facture->origin            = $this->origin;
         $facture->origin_id         = $this->origin_id;
-        $facture->fk_account         = $this->fk_account;
+        $facture->fk_account = $this->fk_account;
 
         $facture->lines = $this->lines; // Array of lines of invoice
         $facture->situation_counter = $this->situation_counter;
@@ -1631,7 +1631,7 @@ 
     {
         global $conf, $langs, $hookmanager, $action;
 
-        if (! in_array($origin->element, array('propal', 'commande'))) {
+        if (!in_array($origin->element, array('propal', 'commande'))) {
             $origin->error = 'ErrorCanOnlyAutomaticallyGenerateADepositFromProposalOrOrder';
             return null;
         }
@@ -2192,7 +2192,7 @@ 
                 $this->fk_project = $obj->fk_project;
                 $this->project = null; // Clear if another value was already set by fetch_projet
 
-                $this->statut = $obj->status;   // deprecated
+                $this->statut = $obj->status; // deprecated
                 $this->status = $obj->status;
 
                 $this->date_lim_reglement = $this->db->jdate($obj->dlr);
@@ -3202,7 +3202,7 @@ 
             return -1;
         }
         if (
-            (preg_match('/^[\(]?PROV/i', $this->ref) || empty($this->ref)) &&   // empty should not happened, but when it occurs, the test save life
+            (preg_match('/^[\(]?PROV/i', $this->ref) || empty($this->ref)) && // empty should not happened, but when it occurs, the test save life
             getDolGlobalString('FAC_FORCE_DATE_VALIDATION')                     // If option enabled, we force invoice date
         ) {
             $this->date = dol_now();
@@ -5695,14 +5695,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('facture', 0) . ")";   // One batch process only one company (no sharing)
+        $sql .= " AND f.entity IN (" . getEntity('facture', 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) . "'";
         }
@@ -5920,7 +5920,7 @@ 
                                 $actioncomm->create($user);
                             }
 
-                            $this->db->commit();    // We always commit
+                            $this->db->commit(); // We always commit
                         }
 
                         if ($errormesg) {
@@ -5962,7 +5962,7 @@ 
         // get date of last validated invoices of same type
         $sql  = "SELECT datef";
         $sql .= " FROM " . MAIN_DB_PREFIX . "facture";
-        $sql .= " WHERE type = " . (int) $this->type ;
+        $sql .= " WHERE type = " . (int) $this->type;
         $sql .= " AND date_valid IS NOT NULL";
         $sql .= " AND entity IN (" . getEntity('invoice') . ")";
         $sql .= " ORDER BY datef DESC LIMIT 1";

Indentation +31 added lines, -31 removed lines

@@ -2268,7 +2268,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load all detailed lines into this->lines
      *
@@ -2279,7 +2279,7 @@ 
      */
     public function fetch_lines($only_product = 0, $loadalsotranslation = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->lines = array();
 
         $sql = 'SELECT l.rowid, l.fk_facture, l.fk_product, l.fk_parent_line, l.label as custom_label, l.description, l.product_type, l.price, l.qty, l.vat_src_code, l.tva_tx,';
@@ -2568,7 +2568,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)
      *
@@ -2577,7 +2577,7 @@ 
      */
     public function insert_discount($idremise)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         include_once DOL_DOCUMENT_ROOT . '/core/lib/price.lib.php';
@@ -2665,7 +2665,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set customer ref
      *
@@ -2675,7 +2675,7 @@ 
      */
     public function set_ref_client($ref_client, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user;
 
         $error = 0;
@@ -2915,7 +2915,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Tag the invoice as paid completely (if close_code is filled) => this->fk_statut=2, this->paye=1
      *  or partially (if close_code filled) + appel trigger BILL_PAYED => this->fk_statut=2, this->paye stay 0
@@ -2929,7 +2929,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);
     }
@@ -2996,7 +2996,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Tags the invoice as incompletely paid and call the trigger BILL_UNPAYED
      *  This method is used when a direct debit (fr:prelevement) is refused
@@ -3009,7 +3009,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);
     }
@@ -3059,7 +3059,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Tag invoice as canceled, with no payment on it (example for replacement invoice or payment never received) + call trigger BILL_CANCEL
      *  Warning, if option to decrease stock on invoice was set, this function does not change stock (it might be a cancel because
@@ -3074,7 +3074,7 @@ 
      */
     public function set_canceled($user, $close_code = '', $close_note = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         dol_syslog(get_class($this) . "::set_canceled is deprecated, use setCanceled instead", LOG_NOTICE);
         return $this->setCanceled($user, $close_code, $close_note);
     }
@@ -4303,7 +4303,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Update invoice line with percentage
      *
@@ -4314,7 +4314,7 @@ 
      */
     public function update_percent($line, $percent, $update_price = true)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $mysoc, $user;
 
         // Progress should never be changed for discount lines
@@ -4411,7 +4411,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set percent discount
      *
@@ -4424,7 +4424,7 @@ 
      */
     public function set_remise($user, $remise, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         dol_syslog(get_class($this) . "::set_remise is deprecated, use setDiscount instead", LOG_NOTICE);
         // @phan-suppress-next-line PhanDeprecatedFunction
         return $this->setDiscount($user, $remise, $notrigger);
@@ -4493,7 +4493,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set absolute discount
      *
@@ -4705,7 +4705,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of invoices (eventually filtered on a user) into an array
      *
@@ -4721,7 +4721,7 @@ 
      */
     public function liste_array($shortlist = 0, $draft = 0, $excluser = null, $socid = 0, $limit = 0, $offset = 0, $sortfield = 'f.datef,f.rowid', $sortorder = 'DESC')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user;
 
         $ga = array();
@@ -4785,7 +4785,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of invoices qualified to be replaced by another invoice.
      *  Invoices matching the following rules are returned:
@@ -4796,7 +4796,7 @@ 
      */
     public function list_replacable_invoices($socid = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $return = array();
@@ -4839,7 +4839,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of invoices qualified to be corrected by a credit note.
      *  Invoices matching the following rules are returned:
@@ -4850,7 +4850,7 @@ 
      */
     public function list_qualified_avoir_invoices($socid = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $return = array();
@@ -4916,7 +4916,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load indicators for dashboard (this->nbtodo and this->nbtodolate)
      *
@@ -4925,7 +4925,7 @@ 
      */
     public function load_board($user)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $clause = " WHERE";
@@ -5278,7 +5278,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Checks if the invoice is the first of a cycle
      *
@@ -5286,11 +5286,11 @@ 
      */
     public function is_first()
     {
-		// phpcs:enable
+        // phpcs:enable
         return ($this->situation_counter == 1);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Returns an array containing the previous situations as Facture objects
      *
@@ -5298,7 +5298,7 @@ 
      */
     public function get_prev_sits()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $sql = 'SELECT rowid FROM ' . MAIN_DB_PREFIX . 'facture';
@@ -5367,7 +5367,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Checks if the invoice is the last in its cycle
      *
@@ -5375,7 +5375,7 @@ 
      */
     public function is_last_in_cycle()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         if (!empty($this->situation_cycle_ref)) {

Dolibarr/Code/Compta/Classes/Paiement.php

Spacing +2 added lines, -2 removed lines

@@ -738,8 +738,8 @@
 
             // if dolibarr currency != bank currency then we received an amount in customer currency (currently I don't manage the case : my currency is USD, the customer currency is EUR and he paid me in GBP. Seems no sense for me)
             if (isModEnabled('multicurrency') && $conf->currency != $acc->currency_code) {
-                $totalamount = $this->multicurrency_amount;     // We will insert into llx_bank.amount in foreign currency
-                $totalamount_main_currency = $this->amount;     // We will also save the amount in main currency into column llx_bank.amount_main_currency
+                $totalamount = $this->multicurrency_amount; // We will insert into llx_bank.amount in foreign currency
+                $totalamount_main_currency = $this->amount; // We will also save the amount in main currency into column llx_bank.amount_main_currency
             }
 
             if ($mode == 'payment_supplier') {

Indentation +10 added lines, -10 removed lines

@@ -886,7 +886,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Mise a jour du lien entre le paiement et la ligne generee dans llx_bank
      *
@@ -895,7 +895,7 @@ 
      */
     public function update_fk_bank($id_bank)
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = 'UPDATE ' . MAIN_DB_PREFIX . $this->table_element . ' set fk_bank = ' . ((int) $id_bank);
         $sql .= " WHERE rowid = " . ((int) $this->id);
 
@@ -910,7 +910,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Updates the payment date
      *
@@ -919,7 +919,7 @@ 
      */
     public function update_date($date)
     {
-		// phpcs:enable
+        // phpcs:enable
         $error = 0;
 
         if (!empty($date) && $this->statut != 1) {
@@ -967,7 +967,7 @@ 
         return -1; //no date given or already validated
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Updates the payment number
      *
@@ -976,7 +976,7 @@ 
      */
     public function update_num($num_payment)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (!empty($num_payment) && $this->statut != 1) {
             $sql = "UPDATE " . MAIN_DB_PREFIX . $this->table_element;
             $sql .= " SET num_paiement = '" . $this->db->escape($num_payment) . "'";
@@ -1373,7 +1373,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
      *
@@ -1383,7 +1383,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs; // TODO Renvoyer le libelle anglais et faire traduction a affichage
 
         $langs->load('compta');
@@ -1425,7 +1425,7 @@ 
         return '';
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load the third party of object, from id into this->thirdparty.
      *  For payments, take the thirdparty linked to the first invoice found. This is enough because payments are done on invoices of the same thirdparty.
@@ -1435,7 +1435,7 @@ 
      */
     public function fetch_thirdparty($force_thirdparty_id = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
 
         if (empty($force_thirdparty_id)) {
             $billsarray = $this->getBillsArray(); // From payment, the fk_soc isn't available, we should load the first supplier invoice to get him

Dolibarr/Code/Comm/Classes/CActionComm.php

Indentation +2 added lines, -2 removed lines

@@ -145,7 +145,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of event types: array(id=>label) or array(code=>label)
      *
@@ -159,7 +159,7 @@ 
      */
     public function liste_array($active = '', $idorcode = 'id', $excludetype = '', $onlyautoornot = 0, $morefilter = '', $shortlabel = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf, $user;
         $langs->load("commercial");
 

Spacing +3 added lines, -3 removed lines

@@ -263,7 +263,7 @@ 
                             }
                         }
                         // For the case type is not 'system...' neither 'module', we just check module is on
-                        if (! in_array($obj->type, array('system', 'systemauto', 'module', 'moduleauto'))) {
+                        if (!in_array($obj->type, array('system', 'systemauto', 'module', 'moduleauto'))) {
                             $tmpmodule = $obj->module;
                             //var_dump($tmpmodule);
                             if ($tmpmodule && isset($conf->$tmpmodule) && isModEnabled($tmpmodule)) {
@@ -351,10 +351,10 @@ 
                 $idorcode = 'all';
             }
             $TType = $TSystem[$idorcode];
-            if (! empty($TSystemAuto[$idorcode])) {
+            if (!empty($TSystemAuto[$idorcode])) {
                 $TType = array_merge($TType, $TSystemAuto[$idorcode]);
             }
-            if (! empty($TModule[$idorcode])) {
+            if (!empty($TModule[$idorcode])) {
                 $TType = array_merge($TType, $TModule[$idorcode]);
             }
             $this->liste_array = $TType;

Dolibarr/Code/Comm/Classes/AdvanceTargetingMailing.php

Indentation +8 added lines, -8 removed lines

@@ -265,7 +265,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load object in memory from the database
      *
@@ -274,7 +274,7 @@ 
      */
     public function fetch_by_mailing($id = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $sql = "SELECT";
         $sql .= " t.rowid,";
@@ -327,7 +327,7 @@ 
 
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load object in memory from the database
      *
@@ -337,7 +337,7 @@ 
      */
     public function fetch_by_element($id = 0, $type_element = 'mailing')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $sql = "SELECT";
         $sql .= " t.rowid,";
@@ -519,7 +519,7 @@ 
 
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Load object in memory from database
      *
@@ -528,7 +528,7 @@ 
      */
     public function query_thirdparty($arrayquery)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf, $extrafields;
 
         $sql = "SELECT";
@@ -676,7 +676,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Load object in memory from database
      *
@@ -686,7 +686,7 @@ 
      */
     public function query_contact($arrayquery, $withThirdpartyFilter = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf;
 
         $sql = "SELECT";

Dolibarr/Code/Comm/Classes/ActionCommReminder.php

Indentation +2 added lines, -2 removed lines

@@ -226,7 +226,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the status
      *
@@ -236,7 +236,7 @@ 
      */
     public static function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $labelStatus = $langs->transnoentitiesnoconv('ToDo');

Dolibarr/Code/Comm/Classes/ICal.php

Indentation +30 added lines, -30 removed lines

@@ -85,7 +85,7 @@ 
     {
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Read text file, icalender text file
      *
@@ -94,7 +94,7 @@ 
      */
     public function read_file($file)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->file = $file;
         $file_text = '';
 
@@ -111,7 +111,7 @@ 
         return $file_text; // return all text
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Returns the number of calendar events
      *
@@ -119,11 +119,11 @@ 
      */
     public function get_event_count()
     {
-		// phpcs:enable
+        // phpcs:enable
         return $this->event_count;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Returns the number of to do
      *
@@ -131,7 +131,7 @@ 
      */
     public function get_todo_count()
     {
-		// phpcs:enable
+        // phpcs:enable
         return $this->todo_count;
     }
 
@@ -261,7 +261,7 @@ 
         return $this->cal;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Add to $this->ical array one value and key.
      *
@@ -272,7 +272,7 @@ 
      */
     public function add_to_array($type, $key, $value)
     {
-		// phpcs:enable
+        // phpcs:enable
 
         //print 'type='.$type.' key='.$key.' value='.$value.'<br>'."\n";
 
@@ -324,7 +324,7 @@ 
         $this->last_key = $key;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Parse text "XXXX:value text some with : " and return array($key = "XXXX", $value="value");
      *
@@ -333,7 +333,7 @@ 
      */
     public function retun_key_value($text)
     {
-		// phpcs:enable
+        // phpcs:enable
         /*
         preg_match("/([^:]+)[:]([\w\W]+)/", $text, $matches);
 
@@ -349,7 +349,7 @@ 
         return explode(':', $text, 2);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Parse RRULE  return array
      *
@@ -358,7 +358,7 @@ 
      */
     public function ical_rrule($value)
     {
-		// phpcs:enable
+        // phpcs:enable
         $result = array();
         $rrule = explode(';', $value);
         foreach ($rrule as $line) {
@@ -368,7 +368,7 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return Unix time from ical date time format (YYYYMMDD[T]HHMMSS[Z] or YYYYMMDD[T]HHMMSS)
      *
@@ -377,7 +377,7 @@ 
      */
     public function ical_date_to_unix($ical_date)
     {
-		// phpcs:enable
+        // phpcs:enable
         $ical_date = str_replace('T', '', $ical_date);
         $ical_date = str_replace('Z', '', $ical_date);
 
@@ -393,7 +393,7 @@ 
         return $ntime; // ntime is a GTM time
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return unix date from iCal date format
      *
@@ -403,7 +403,7 @@ 
      */
     public function ical_dt_date($key, $value)
     {
-		// phpcs:enable
+        // phpcs:enable
         $return_value = array();
 
         // Analyse TZID
@@ -424,7 +424,7 @@ 
         return array($key, $return_value);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return sorted eventlist as array or false if calendar is empty
      *
@@ -432,7 +432,7 @@ 
      */
     public function get_sort_event_list()
     {
-		// phpcs:enable
+        // phpcs:enable
         $temp = $this->get_event_list();
         if (!empty($temp)) {
             usort($temp, array(&$this, "ical_dtstart_compare"));
@@ -442,7 +442,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Compare two unix timestamp
      *
@@ -452,11 +452,11 @@ 
      */
     public function ical_dtstart_compare($a, $b)
     {
-		// phpcs:enable
+        // phpcs:enable
         return strnatcasecmp($a['DTSTART']['unixtime'], $b['DTSTART']['unixtime']);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return eventlist array (not sorted eventlist array)
      *
@@ -464,11 +464,11 @@ 
      */
     public function get_event_list()
     {
-		// phpcs:enable
+        // phpcs:enable
         return (empty($this->cal['VEVENT']) ? array() : $this->cal['VEVENT']);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return freebusy array (not sort eventlist array)
      *
@@ -476,11 +476,11 @@ 
      */
     public function get_freebusy_list()
     {
-		// phpcs:enable
+        // phpcs:enable
         return (empty($this->cal['VFREEBUSY']) ? array() : $this->cal['VFREEBUSY']);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return to do array (not sorted todo array)
      *
@@ -488,11 +488,11 @@ 
      */
     public function get_todo_list()
     {
-		// phpcs:enable
+        // phpcs:enable
         return $this->cal['VTODO'];
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return base calendar data
      *
@@ -500,11 +500,11 @@ 
      */
     public function get_calender_data()
     {
-		// phpcs:enable
+        // phpcs:enable
         return $this->cal['VCALENDAR'];
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return array with all data
      *
@@ -512,7 +512,7 @@ 
      */
     public function get_all_data()
     {
-		// phpcs:enable
+        // phpcs:enable
         return $this->cal;
     }
 }

Dolibarr/Code/Comm/Classes/Propal.php

Indentation +14 added lines, -14 removed lines

@@ -194,9 +194,9 @@ 
     public $fin_validite;    // code
         public $user_author_id;         // label
         /**
-     * @deprecated
-     * @see $total_ht
-     */
+         * @deprecated
+         * @see $total_ht
+         */
     public $price;     // label doc
     /**
      * @deprecated
@@ -204,9 +204,9 @@ 
      */
     public $tva;    // code
         /**
-     * @deprecated
-     * @see $total_ttc
-     */
+         * @deprecated
+         * @see $total_ttc
+         */
     public $total;         // label
 public $cond_reglement_code;
 public $cond_reglement;
@@ -226,14 +226,14 @@ 
      */
     public $availability_id;       // id
         /**
-     * @var int availability ID
-     * @deprecated
-     * @see $availability_id
-     */
+         * @var int availability ID
+         * @deprecated
+         * @see $availability_id
+         */
     public $fk_availability;     // code
         /**
-     * @var string availability code
-     */
+         * @var string availability code
+         */
     public $availability_code;          // label
     /**
      * @var string availability label
@@ -338,8 +338,8 @@ 
         'import_key' => array('type' => 'varchar(14)', 'label' => 'ImportId', 'enabled' => 1, 'visible' => -2, 'position' => 900),
     );
         /**
-     * {@inheritdoc}
-     */
+         * {@inheritdoc}
+         */
     protected $table_ref_field = 'ref'; // Todo rename into STATUS_CLOSE ?
 
     /**

Spacing +73 added lines, -73 removed lines

@@ -191,23 +191,23 @@ 
      * @var integer|string $delivery_date ;
      */
     public $delivery_date;
-    public $fin_validite;    // code
-        public $user_author_id;         // label
+    public $fin_validite; // code
+        public $user_author_id; // label
         /**
      * @deprecated
      * @see $total_ht
      */
-    public $price;     // label doc
+    public $price; // label doc
     /**
      * @deprecated
      * @see $total_tva
      */
-    public $tva;    // code
+    public $tva; // code
         /**
      * @deprecated
      * @see $total_ttc
      */
-    public $total;         // label
+    public $total; // label
 public $cond_reglement_code;
 public $cond_reglement;
 public $cond_reglement_doc;
@@ -224,17 +224,17 @@ 
     /**
      * @var int availability ID
      */
-    public $availability_id;       // id
+    public $availability_id; // id
         /**
      * @var int availability ID
      * @deprecated
      * @see $availability_id
      */
-    public $fk_availability;     // code
+    public $fk_availability; // code
         /**
      * @var string availability code
      */
-    public $availability_code;          // label
+    public $availability_code; // label
     /**
      * @var string availability label
      */
@@ -513,14 +513,14 @@ 
 
         if (!empty($ref)) {
             if (!empty($forceentity)) {
-                $sql .= " WHERE p.entity = " . (int)$forceentity; // Check only the current entity because we may have the same reference in several entities
+                $sql .= " WHERE p.entity = " . (int) $forceentity; // Check only the current entity because we may have the same reference in several entities
             } else {
                 $sql .= " WHERE p.entity IN (" . getEntity('propal') . ")";
             }
             $sql .= " AND p.ref='" . $this->db->escape($ref) . "'";
         } else {
             // Don't use entity if you use rowid
-            $sql .= " WHERE p.rowid = " . ((int)$rowid);
+            $sql .= " WHERE p.rowid = " . ((int) $rowid);
         }
 
         dol_syslog(get_class($this) . "::fetch", LOG_DEBUG);
@@ -537,7 +537,7 @@ 
                 $this->ref_customer = $obj->ref_client;
                 $this->ref_ext = $obj->ref_ext;
 
-                $this->total = $obj->total_ttc;          // TODO deprecated
+                $this->total = $obj->total_ttc; // TODO deprecated
                 $this->total_ttc = $obj->total_ttc;
                 $this->total_ht = $obj->total_ht;
                 $this->total_tva = $obj->total_tva;
@@ -556,7 +556,7 @@ 
                 $this->note_private = $obj->note_private;
                 $this->note_public = $obj->note_public;
 
-                $this->status = (int)$obj->fk_statut;
+                $this->status = (int) $obj->fk_statut;
                 $this->statut = $this->status; // deprecated
 
                 $this->datec = $this->db->jdate($obj->datec); // TODO deprecated
@@ -589,7 +589,7 @@ 
                 $this->cond_reglement_doc = $obj->cond_reglement_libelle_doc;
                 $this->deposit_percent = $obj->deposit_percent;
 
-                $this->extraparams = !empty($obj->extraparams) ? (array)json_decode($obj->extraparams, true) : array();
+                $this->extraparams = !empty($obj->extraparams) ? (array) json_decode($obj->extraparams, true) : array();
 
                 $this->user_author_id = $obj->fk_user_author;
                 $this->user_validation_id = $obj->fk_user_valid;
@@ -655,7 +655,7 @@ 
         $sql .= ' d.fk_multicurrency, d.multicurrency_code, d.multicurrency_subprice, d.multicurrency_total_ht, d.multicurrency_total_tva, d.multicurrency_total_ttc';
         $sql .= ' FROM ' . MAIN_DB_PREFIX . 'propaldet as d';
         $sql .= ' LEFT JOIN ' . MAIN_DB_PREFIX . 'product as p ON (d.fk_product = p.rowid)';
-        $sql .= ' WHERE d.fk_propal = ' . ((int)$this->id);
+        $sql .= ' WHERE d.fk_propal = ' . ((int) $this->id);
         if ($only_product) {
             $sql .= ' AND p.fk_product_type = 0';
         }
@@ -875,10 +875,10 @@ 
 
         // Clean parameters
         $remise_percent = price2num($remise_percent);
-        $qty = (float)price2num($qty);
+        $qty = (float) price2num($qty);
         $pu = price2num($pu);
         $pu_ht_devise = price2num($pu_ht_devise);
-        if (!preg_match('/\((.*)\)/', (string)$txtva)) {
+        if (!preg_match('/\((.*)\)/', (string) $txtva)) {
             $txtva = price2num($txtva); // $txtva can have format '5.0(XXX)' or '5'
         }
         $txlocaltax1 = price2num($txlocaltax1);
@@ -939,9 +939,9 @@ 
             // Anciens indicateurs: $price, $remise (a ne plus utiliser)
             $price = $pu;
             $remise = 0;
-            if ((float)$remise_percent > 0) {
-                $remise = round(((float)$pu * (float)$remise_percent / 100), 2);
-                $price = (float)$pu - $remise;
+            if ((float) $remise_percent > 0) {
+                $remise = round(((float) $pu * (float) $remise_percent / 100), 2);
+                $price = (float) $pu - $remise;
             }
 
             //Fetch current line from the database and then clone the object and set it in $oldline property
@@ -1099,7 +1099,7 @@ 
         $sql .= " note_public=" . (isset($this->note_public) ? "'" . $this->db->escape($this->note_public) . "'" : "null") . ",";
         $sql .= " model_pdf=" . (isset($this->model_pdf) ? "'" . $this->db->escape($this->model_pdf) . "'" : "null") . ",";
         $sql .= " import_key=" . (isset($this->import_key) ? "'" . $this->db->escape($this->import_key) . "'" : "null");
-        $sql .= " WHERE rowid=" . ((int)$this->id);
+        $sql .= " WHERE rowid=" . ((int) $this->id);
 
         $this->db->begin();
 
@@ -1218,8 +1218,8 @@ 
         // Delete extrafields of lines and lines
         if (!$error && !empty($this->table_element_line)) {
             $tabletodelete = $this->table_element_line;
-            $sqlef = "DELETE FROM " . MAIN_DB_PREFIX . $tabletodelete . "_extrafields WHERE fk_object IN (SELECT rowid FROM " . MAIN_DB_PREFIX . $tabletodelete . " WHERE " . $this->fk_element . " = " . ((int)$this->id) . ")";
-            $sql = "DELETE FROM " . MAIN_DB_PREFIX . $tabletodelete . " WHERE " . $this->fk_element . " = " . ((int)$this->id);
+            $sqlef = "DELETE FROM " . MAIN_DB_PREFIX . $tabletodelete . "_extrafields WHERE fk_object IN (SELECT rowid FROM " . MAIN_DB_PREFIX . $tabletodelete . " WHERE " . $this->fk_element . " = " . ((int) $this->id) . ")";
+            $sql = "DELETE FROM " . MAIN_DB_PREFIX . $tabletodelete . " WHERE " . $this->fk_element . " = " . ((int) $this->id);
             if (!$this->db->query($sqlef) || !$this->db->query($sql)) {
                 $error++;
                 $this->error = $this->db->lasterror();
@@ -1255,7 +1255,7 @@ 
 
         // Delete main record
         if (!$error) {
-            $sql = "DELETE FROM " . MAIN_DB_PREFIX . $this->table_element . " WHERE rowid = " . ((int)$this->id);
+            $sql = "DELETE FROM " . MAIN_DB_PREFIX . $this->table_element . " WHERE rowid = " . ((int) $this->id);
             $res = $this->db->query($sql);
             if (!$res) {
                 $error++;
@@ -1291,7 +1291,7 @@ 
                     }
                 }
                 if (file_exists($dir)) {
-                    $res = @dol_delete_dir_recursive($dir);     // delete files physically + into ecm tables
+                    $res = @dol_delete_dir_recursive($dir); // delete files physically + into ecm tables
                     if (!$res) {
                         $this->error = 'ErrorFailToDeleteDir';
                         $this->errors[] = $this->error;
@@ -1612,15 +1612,15 @@ 
         $sql .= ", '" . $this->db->idate($this->date) . "'";
         $sql .= ", '" . $this->db->idate($now) . "'";
         $sql .= ", '(PROV)'";
-        $sql .= ", " . ($user->id > 0 ? ((int)$user->id) : "NULL");
+        $sql .= ", " . ($user->id > 0 ? ((int) $user->id) : "NULL");
         $sql .= ", '" . $this->db->escape($this->note_private) . "'";
         $sql .= ", '" . $this->db->escape($this->note_public) . "'";
         $sql .= ", '" . $this->db->escape($this->model_pdf) . "'";
         $sql .= ", " . ($this->fin_validite != '' ? "'" . $this->db->idate($this->fin_validite) . "'" : "NULL");
-        $sql .= ", " . ($this->cond_reglement_id > 0 ? ((int)$this->cond_reglement_id) : 'NULL');
+        $sql .= ", " . ($this->cond_reglement_id > 0 ? ((int) $this->cond_reglement_id) : 'NULL');
         $sql .= ", " . (!empty($this->deposit_percent) ? "'" . $this->db->escape($this->deposit_percent) . "'" : 'NULL');
-        $sql .= ", " . ($this->mode_reglement_id > 0 ? ((int)$this->mode_reglement_id) : 'NULL');
-        $sql .= ", " . ($this->fk_account > 0 ? ((int)$this->fk_account) : 'NULL');
+        $sql .= ", " . ($this->mode_reglement_id > 0 ? ((int) $this->mode_reglement_id) : 'NULL');
+        $sql .= ", " . ($this->fk_account > 0 ? ((int) $this->fk_account) : 'NULL');
         $sql .= ", '" . $this->db->escape($this->ref_client) . "'";
         $sql .= ", '" . $this->db->escape($this->ref_ext) . "'";
         $sql .= ", " . (empty($delivery_date) ? "NULL" : "'" . $this->db->idate($delivery_date) . "'");
@@ -1629,12 +1629,12 @@ 
         $sql .= ", " . $this->availability_id;
         $sql .= ", " . $this->demand_reason_id;
         $sql .= ", " . ($this->fk_project ? $this->fk_project : "null");
-        $sql .= ", " . (int)$this->fk_incoterms;
+        $sql .= ", " . (int) $this->fk_incoterms;
         $sql .= ", '" . $this->db->escape($this->location_incoterms) . "'";
         $sql .= ", " . setEntity($this);
-        $sql .= ", " . (int)$this->fk_multicurrency;
+        $sql .= ", " . (int) $this->fk_multicurrency;
         $sql .= ", '" . $this->db->escape($this->multicurrency_code) . "'";
-        $sql .= ", " . (float)$this->multicurrency_tx;
+        $sql .= ", " . (float) $this->multicurrency_tx;
         $sql .= ")";
 
         dol_syslog(get_class($this) . "::create", LOG_DEBUG);
@@ -1644,7 +1644,7 @@ 
 
             if ($this->id) {
                 $this->ref = '(PROV' . $this->id . ')';
-                $sql = 'UPDATE ' . MAIN_DB_PREFIX . "propal SET ref='" . $this->db->escape($this->ref) . "' WHERE rowid=" . ((int)$this->id);
+                $sql = 'UPDATE ' . MAIN_DB_PREFIX . "propal SET ref='" . $this->db->escape($this->ref) . "' WHERE rowid=" . ((int) $this->id);
 
                 dol_syslog(get_class($this) . "::create", LOG_DEBUG);
                 $resql = $this->db->query($sql);
@@ -1689,7 +1689,7 @@ 
                     for ($i = 0; $i < $num; $i++) {
                         if (!is_object($this->lines[$i])) { // If this->lines is not array of objects, coming from REST API
                             // Convert into object this->lines[$i].
-                            $line = (object)$this->lines[$i];
+                            $line = (object) $this->lines[$i];
                         } else {
                             $line = $this->lines[$i];
                         }
@@ -1882,11 +1882,11 @@ 
             }
 
             $remise_percent = price2num($remise_percent);
-            $qty = (float)price2num($qty);
+            $qty = (float) price2num($qty);
             $pu_ht = price2num($pu_ht);
             $pu_ht_devise = price2num($pu_ht_devise);
             $pu_ttc = price2num($pu_ttc);
-            if (!preg_match('/\((.*)\)/', (string)$txtva)) {
+            if (!preg_match('/\((.*)\)/', (string) $txtva)) {
                 $txtva = price2num($txtva); // $txtva can have format '5,1' or '5.1' or '5.1(XXX)', we must clean only if '5,1'
             }
             $txlocaltax1 = price2num($txlocaltax1);
@@ -1968,9 +1968,9 @@ 
             // Anciens indicateurs: $price, $remise (a ne plus utiliser)
             $price = $pu;
             $remise = 0;
-            if ((float)$remise_percent > 0) {
-                $remise = round(((float)$pu * (float)$remise_percent / 100), 2);
-                $price = (float)$pu - $remise;
+            if ((float) $remise_percent > 0) {
+                $remise = round(((float) $pu * (float) $remise_percent / 100), 2);
+                $price = (float) $pu - $remise;
             }
 
             // Insert line
@@ -2121,8 +2121,8 @@ 
 
         $sql = "UPDATE " . MAIN_DB_PREFIX . "propal";
         $sql .= " SET ref = '" . $this->db->escape($num) . "',";
-        $sql .= " fk_statut = " . self::STATUS_VALIDATED . ", date_valid='" . $this->db->idate($now) . "', fk_user_valid=" . ((int)$user->id);
-        $sql .= " WHERE rowid = " . ((int)$this->id) . " AND fk_statut = " . self::STATUS_DRAFT;
+        $sql .= " fk_statut = " . self::STATUS_VALIDATED . ", date_valid='" . $this->db->idate($now) . "', fk_user_valid=" . ((int) $user->id);
+        $sql .= " WHERE rowid = " . ((int) $this->id) . " AND fk_statut = " . self::STATUS_DRAFT;
 
         dol_syslog(get_class($this) . "::valid", LOG_DEBUG);
         $resql = $this->db->query($sql);
@@ -2148,7 +2148,7 @@ 
             if (preg_match('/^[\(]?PROV/i', $this->ref)) {
                 // Now we rename also files into index
                 $sql = 'UPDATE ' . MAIN_DB_PREFIX . "ecm_files set filename = CONCAT('" . $this->db->escape($this->newref) . "', SUBSTR(filename, " . (strlen($this->ref) + 1) . ")), filepath = 'propale/" . $this->db->escape($this->newref) . "'";
-                $sql .= " WHERE filename LIKE '" . $this->db->escape($this->ref) . "%' AND filepath = 'propale/" . $this->db->escape($this->ref) . "' and entity = " . ((int)$conf->entity);
+                $sql .= " WHERE filename LIKE '" . $this->db->escape($this->ref) . "%' AND filepath = 'propale/" . $this->db->escape($this->ref) . "' and entity = " . ((int) $conf->entity);
                 $resql = $this->db->query($sql);
                 if (!$resql) {
                     $error++;
@@ -2219,12 +2219,12 @@ 
             $file = $classname . ".php";
 
             // Include file with class
-            $dirmodels = array_merge(array('/'), (array)$conf->modules_parts['models']);
+            $dirmodels = array_merge(array('/'), (array) $conf->modules_parts['models']);
             foreach ($dirmodels as $reldir) {
                 $dir = dol_buildpath($reldir . "core/modules/propale/");
 
                 // Load file with numbering class (if found)
-                $mybool = ((bool)@include_once $dir . $file) || $mybool;
+                $mybool = ((bool) @include_once $dir . $file) || $mybool;
             }
 
             if (!$mybool) {
@@ -2275,7 +2275,7 @@ 
             $this->db->begin();
 
             $sql = "UPDATE " . MAIN_DB_PREFIX . "propal SET datep = '" . $this->db->idate($date) . "'";
-            $sql .= " WHERE rowid = " . ((int)$this->id);
+            $sql .= " WHERE rowid = " . ((int) $this->id);
 
             dol_syslog(__METHOD__, LOG_DEBUG);
             $resql = $this->db->query($sql);
@@ -2334,7 +2334,7 @@ 
             $this->db->begin();
 
             $sql = "UPDATE " . MAIN_DB_PREFIX . "propal SET fin_validite = " . ($date_end_validity != '' ? "'" . $this->db->idate($date_end_validity) . "'" : 'null');
-            $sql .= " WHERE rowid = " . ((int)$this->id);
+            $sql .= " WHERE rowid = " . ((int) $this->id);
 
             dol_syslog(__METHOD__, LOG_DEBUG);
 
@@ -2409,7 +2409,7 @@ 
 
             $sql = "UPDATE " . MAIN_DB_PREFIX . "propal ";
             $sql .= " SET date_livraison = " . ($delivery_date != '' ? "'" . $this->db->idate($delivery_date) . "'" : 'null');
-            $sql .= " WHERE rowid = " . ((int)$this->id);
+            $sql .= " WHERE rowid = " . ((int) $this->id);
 
             dol_syslog(__METHOD__, LOG_DEBUG);
             $resql = $this->db->query($sql);
@@ -2465,8 +2465,8 @@ 
             $this->db->begin();
 
             $sql = "UPDATE " . MAIN_DB_PREFIX . "propal";
-            $sql .= " SET fk_availability = " . ((int)$id);
-            $sql .= " WHERE rowid = " . ((int)$this->id);
+            $sql .= " SET fk_availability = " . ((int) $id);
+            $sql .= " WHERE rowid = " . ((int) $this->id);
 
             dol_syslog(__METHOD__ . ' availability(' . $id . ')', LOG_DEBUG);
             $resql = $this->db->query($sql);
@@ -2527,8 +2527,8 @@ 
             $this->db->begin();
 
             $sql = "UPDATE " . MAIN_DB_PREFIX . "propal ";
-            $sql .= " SET fk_input_reason = " . ((int)$id);
-            $sql .= " WHERE rowid = " . ((int)$this->id);
+            $sql .= " SET fk_input_reason = " . ((int) $id);
+            $sql .= " WHERE rowid = " . ((int) $this->id);
 
             dol_syslog(__METHOD__, LOG_DEBUG);
             $resql = $this->db->query($sql);
@@ -2591,7 +2591,7 @@ 
             $this->db->begin();
 
             $sql = "UPDATE " . MAIN_DB_PREFIX . "propal SET ref_client = " . (empty($ref_client) ? 'NULL' : "'" . $this->db->escape($ref_client) . "'");
-            $sql .= " WHERE rowid = " . ((int)$this->id);
+            $sql .= " WHERE rowid = " . ((int) $this->id);
 
             dol_syslog(__METHOD__ . ' $this->id=' . $this->id . ', ref_client=' . $ref_client, LOG_DEBUG);
             $resql = $this->db->query($sql);
@@ -2646,12 +2646,12 @@ 
         $error = 0;
 
         $sql = "UPDATE " . MAIN_DB_PREFIX . "propal";
-        $sql .= " SET fk_statut = " . ((int)$status) . ",";
+        $sql .= " SET fk_statut = " . ((int) $status) . ",";
         if (!empty($note)) {
             $sql .= " note_private = '" . $this->db->escape($note) . "',";
         }
         $sql .= " date_cloture=NULL, fk_user_cloture=NULL";
-        $sql .= " WHERE rowid = " . ((int)$this->id);
+        $sql .= " WHERE rowid = " . ((int) $this->id);
 
         $this->db->begin();
 
@@ -2729,17 +2729,17 @@ 
         }
 
         $sql = "UPDATE " . MAIN_DB_PREFIX . "propal";
-        $sql .= " SET fk_statut = " . ((int)$status) . ", note_private = '" . $this->db->escape($newprivatenote) . "', date_signature='" . $this->db->idate($date_signature) . "', fk_user_signature=" . $fk_user_signature;
-        $sql .= " WHERE rowid = " . ((int)$this->id);
+        $sql .= " SET fk_statut = " . ((int) $status) . ", note_private = '" . $this->db->escape($newprivatenote) . "', date_signature='" . $this->db->idate($date_signature) . "', fk_user_signature=" . $fk_user_signature;
+        $sql .= " WHERE rowid = " . ((int) $this->id);
 
         $resql = $this->db->query($sql);
         if ($resql) {
             // Status self::STATUS_REFUSED by default
             $modelpdf = getDolGlobalString('PROPALE_ADDON_PDF_ODT_CLOSED', $this->model_pdf);
-            $trigger_name = 'PROPAL_CLOSE_REFUSED';     // used later in call_trigger()
+            $trigger_name = 'PROPAL_CLOSE_REFUSED'; // used later in call_trigger()
 
             if ($status == self::STATUS_SIGNED) {   // Status self::STATUS_SIGNED
-                $trigger_name = 'PROPAL_CLOSE_SIGNED';  // used later in call_trigger()
+                $trigger_name = 'PROPAL_CLOSE_SIGNED'; // used later in call_trigger()
                 $modelpdf = getDolGlobalString('PROPALE_ADDON_PDF_ODT_TOBILL') ? $conf->global->PROPALE_ADDON_PDF_ODT_TOBILL : $this->model_pdf;
 
                 // The connected company is classified as a client
@@ -2820,7 +2820,7 @@ 
         $sql .= " c.datec, c.date_valid as datev, c.date_signature, c.date_cloture,";
         $sql .= " c.fk_user_author, c.fk_user_valid, c.fk_user_signature, c.fk_user_cloture";
         $sql .= " FROM " . MAIN_DB_PREFIX . "propal as c";
-        $sql .= " WHERE c.rowid = " . ((int)$id);
+        $sql .= " WHERE c.rowid = " . ((int) $id);
 
         $result = $this->db->query($sql);
 
@@ -2911,8 +2911,8 @@ 
         $newprivatenote = dol_concatdesc($this->note_private, $note);
 
         $sql = 'UPDATE ' . MAIN_DB_PREFIX . 'propal SET fk_statut = ' . self::STATUS_BILLED . ", ";
-        $sql .= " note_private = '" . $this->db->escape($newprivatenote) . "', date_cloture='" . $this->db->idate($now) . "', fk_user_cloture=" . ((int)$user->id);
-        $sql .= ' WHERE rowid = ' . ((int)$this->id) . ' AND fk_statut = ' . ((int)self::STATUS_SIGNED);
+        $sql .= " note_private = '" . $this->db->escape($newprivatenote) . "', date_cloture='" . $this->db->idate($now) . "', fk_user_cloture=" . ((int) $user->id);
+        $sql .= ' WHERE rowid = ' . ((int) $this->id) . ' AND fk_statut = ' . ((int) self::STATUS_SIGNED);
 
         dol_syslog(__METHOD__, LOG_DEBUG);
         $resql = $this->db->query($sql);
@@ -2986,8 +2986,8 @@ 
 
         $sql = "UPDATE " . MAIN_DB_PREFIX . "propal";
         $sql .= " SET fk_statut = " . self::STATUS_CANCELED . ",";
-        $sql .= " fk_user_modif = " . ((int)$user->id);
-        $sql .= " WHERE rowid = " . ((int)$this->id);
+        $sql .= " fk_user_modif = " . ((int) $user->id);
+        $sql .= " WHERE rowid = " . ((int) $this->id);
 
         dol_syslog(get_class($this) . "::cancel", LOG_DEBUG);
         if ($this->db->query($sql)) {
@@ -3045,7 +3045,7 @@ 
         $sql = "UPDATE " . MAIN_DB_PREFIX . "propal";
         $sql .= " SET fk_statut = " . self::STATUS_DRAFT;
         $sql .= ",  online_sign_ip = NULL , online_sign_name = NULL";
-        $sql .= " WHERE rowid = " . ((int)$this->id);
+        $sql .= " WHERE rowid = " . ((int) $this->id);
 
         $resql = $this->db->query($sql);
         if (!$resql) {
@@ -3120,18 +3120,18 @@ 
             if ($search_sale == -2) {
                 $sql .= " AND NOT EXISTS (SELECT sc.fk_soc FROM " . MAIN_DB_PREFIX . "societe_commerciaux as sc WHERE sc.fk_soc = p.fk_soc)";
             } elseif ($search_sale > 0) {
-                $sql .= " AND EXISTS (SELECT sc.fk_soc FROM " . MAIN_DB_PREFIX . "societe_commerciaux as sc WHERE sc.fk_soc = p.fk_soc AND sc.fk_user = " . ((int)$search_sale) . ")";
+                $sql .= " AND EXISTS (SELECT sc.fk_soc FROM " . MAIN_DB_PREFIX . "societe_commerciaux as sc WHERE sc.fk_soc = p.fk_soc AND sc.fk_user = " . ((int) $search_sale) . ")";
             }
         }
         // Search on socid
         if ($socid) {
-            $sql .= " AND p.fk_soc = " . ((int)$socid);
+            $sql .= " AND p.fk_soc = " . ((int) $socid);
         }
         if ($draft) {
-            $sql .= " AND p.fk_statut = " . ((int)self::STATUS_DRAFT);
+            $sql .= " AND p.fk_statut = " . ((int) self::STATUS_DRAFT);
         }
         if ($notcurrentuser > 0) {
-            $sql .= " AND p.fk_user_author <> " . ((int)$user->id);
+            $sql .= " AND p.fk_user_author <> " . ((int) $user->id);
         }
         $sql .= $this->db->order($sortfield, $sortorder);
         $sql .= $this->db->plimit($limit, $offset);
@@ -3269,8 +3269,8 @@ 
             $this->db->begin();
 
             $sql = 'UPDATE ' . MAIN_DB_PREFIX . 'propal';
-            $sql .= ' SET fk_availability = ' . ((int)$availability_id);
-            $sql .= ' WHERE rowid=' . ((int)$this->id);
+            $sql .= ' SET fk_availability = ' . ((int) $availability_id);
+            $sql .= ' WHERE rowid=' . ((int) $this->id);
 
             dol_syslog(__METHOD__ . ' availability(' . $availability_id . ')', LOG_DEBUG);
             $resql = $this->db->query($sql);
@@ -3332,8 +3332,8 @@ 
             $this->db->begin();
 
             $sql = 'UPDATE ' . MAIN_DB_PREFIX . 'propal';
-            $sql .= ' SET fk_input_reason = ' . ((int)$demand_reason_id);
-            $sql .= ' WHERE rowid=' . ((int)$this->id);
+            $sql .= ' SET fk_input_reason = ' . ((int) $demand_reason_id);
+            $sql .= ' WHERE rowid=' . ((int) $this->id);
 
             dol_syslog(__METHOD__ . ' demand_reason(' . $demand_reason_id . ')', LOG_DEBUG);
             $resql = $this->db->query($sql);
@@ -3409,7 +3409,7 @@ 
             if ($search_sale == -2) {
                 $sql .= " AND NOT EXISTS (SELECT sc.fk_soc FROM " . MAIN_DB_PREFIX . "societe_commerciaux as sc WHERE sc.fk_soc = p.fk_soc)";
             } elseif ($search_sale > 0) {
-                $sql .= " AND EXISTS (SELECT sc.fk_soc FROM " . MAIN_DB_PREFIX . "societe_commerciaux as sc WHERE sc.fk_soc = p.fk_soc AND sc.fk_user = " . ((int)$search_sale) . ")";
+                $sql .= " AND EXISTS (SELECT sc.fk_soc FROM " . MAIN_DB_PREFIX . "societe_commerciaux as sc WHERE sc.fk_soc = p.fk_soc AND sc.fk_user = " . ((int) $search_sale) . ")";
             }
         }
 
@@ -3585,7 +3585,7 @@ 
             if ($search_sale == -2) {
                 $sql .= " AND NOT EXISTS (SELECT sc.fk_soc FROM " . MAIN_DB_PREFIX . "societe_commerciaux as sc WHERE sc.fk_soc = p.fk_soc)";
             } elseif ($search_sale > 0) {
-                $sql .= " AND EXISTS (SELECT sc.fk_soc FROM " . MAIN_DB_PREFIX . "societe_commerciaux as sc WHERE sc.fk_soc = p.fk_soc AND sc.fk_user = " . ((int)$search_sale) . ")";
+                $sql .= " AND EXISTS (SELECT sc.fk_soc FROM " . MAIN_DB_PREFIX . "societe_commerciaux as sc WHERE sc.fk_soc = p.fk_soc AND sc.fk_user = " . ((int) $search_sale) . ")";
             }
         }
 

Dolibarr/Code/Comm/Classes/PropaleLigne.php

Indentation +2 added lines, -2 removed lines

@@ -665,7 +665,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update DB line fields total_xxx
      *  Used by migration
@@ -674,7 +674,7 @@ 
      */
     public function update_total()
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->db->begin();
 
         // Mise a jour ligne en base