alxarafe / alixar

Dolibarr/Code/SupplierProposal/Api/SupplierProposals.php

Indentation +2 added lines, -2 removed lines

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

public/htdocs/index.php

Spacing +11 added lines, -11 removed lines

@@ -228,7 +228,7 @@ 
     }
 
     // Number of supplier proposals open (expired)
-    if (isModEnabled('supplier_proposal')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_SUPPLIER') && $user->hasRight('supplier_proposal', 'lire')) {
+    if (isModEnabled('supplier_proposal') && !getDolGlobalString('MAIN_DISABLE_BLOCK_SUPPLIER') && $user->hasRight('supplier_proposal', 'lire')) {
         $langs->load("supplier_proposal");
         $board = new SupplierProposal($db);
         $dashboardlines[$board->element . '_opened'] = $board->load_board($user, "opened");
@@ -237,7 +237,7 @@ 
     }
 
     // Number of sales orders
-    if (isModEnabled('order')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_CUSTOMER') && $user->hasRight('commande', 'lire')) {
+    if (isModEnabled('order') && !getDolGlobalString('MAIN_DISABLE_BLOCK_CUSTOMER') && $user->hasRight('commande', 'lire')) {
         $board = new Commande($db);
         // Number of customer orders to be shipped (validated and in progress)
         $dashboardlines[$board->element . '_toship'] = $board->load_board($user, 'toship');
@@ -250,14 +250,14 @@ 
     }
 
     // Number of suppliers orders
-    if (isModEnabled('supplier_order')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_SUPPLIER') && $user->hasRight('fournisseur', 'commande', 'lire')) {
+    if (isModEnabled('supplier_order') && !getDolGlobalString('MAIN_DISABLE_BLOCK_SUPPLIER') && $user->hasRight('fournisseur', 'commande', 'lire')) {
         $board = new CommandeFournisseur($db);
         $dashboardlines[$board->element . '_opened'] = $board->load_board($user, "opened");
         $dashboardlines[$board->element . '_awaiting'] = $board->load_board($user, 'awaiting');
     }
 
     // Number of contract / services enabled (delayed)
-    if (isModEnabled('contract')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_CONTRACT') && $user->hasRight('contrat', 'lire')) {
+    if (isModEnabled('contract') && !getDolGlobalString('MAIN_DISABLE_BLOCK_CONTRACT') && $user->hasRight('contrat', 'lire')) {
         $board = new Contrat($db);
         $dashboardlines[$board->element . '_inactive'] = $board->load_board($user, "inactive");
         // Number of active services (expired)
@@ -265,7 +265,7 @@ 
     }
 
     // Number of tickets open
-    if (isModEnabled('ticket')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_TICKET') && $user->hasRight('ticket', 'read')) {
+    if (isModEnabled('ticket') && !getDolGlobalString('MAIN_DISABLE_BLOCK_TICKET') && $user->hasRight('ticket', 'read')) {
         $board = new Ticket($db);
         $dashboardlines[$board->element . '_opened'] = $board->load_board($user, "opened");
         // Number of active services (expired)
@@ -285,7 +285,7 @@ 
     }
 
     // Number of transactions to conciliate
-    if (isModEnabled('bank')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_BANK') && $user->hasRight('banque', 'lire') && !$user->socid) {
+    if (isModEnabled('bank') && !getDolGlobalString('MAIN_DISABLE_BLOCK_BANK') && $user->hasRight('banque', 'lire') && !$user->socid) {
         $board = new Account($db);
         $nb = $board->countAccountToReconcile(); // Get nb of account to reconciliate
         if ($nb > 0) {
@@ -295,7 +295,7 @@ 
 
 
     // Number of cheque to send
-    if (isModEnabled('bank')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_BANK') && $user->hasRight('banque', 'lire') && !$user->socid) {
+    if (isModEnabled('bank') && !getDolGlobalString('MAIN_DISABLE_BLOCK_BANK') && $user->hasRight('banque', 'lire') && !$user->socid) {
         if (!getDolGlobalString('BANK_DISABLE_CHECK_DEPOSIT')) {
             $board = new RemiseCheque($db);
             $dashboardlines[$board->element] = $board->load_board($user);
@@ -311,26 +311,26 @@ 
     }
 
     // Number of foundation members
-    if (isModEnabled('member')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_ADHERENT') && $user->hasRight('adherent', 'lire') && !$user->socid) {
+    if (isModEnabled('member') && !getDolGlobalString('MAIN_DISABLE_BLOCK_ADHERENT') && $user->hasRight('adherent', 'lire') && !$user->socid) {
         $board = new Adherent($db);
         $dashboardlines[$board->element . '_shift'] = $board->load_board($user, 'shift');
         $dashboardlines[$board->element . '_expired'] = $board->load_board($user, 'expired');
     }
 
     // Number of expense reports to approve
-    if (isModEnabled('expensereport')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_EXPENSEREPORT') && $user->hasRight('expensereport', 'approve')) {
+    if (isModEnabled('expensereport') && !getDolGlobalString('MAIN_DISABLE_BLOCK_EXPENSEREPORT') && $user->hasRight('expensereport', 'approve')) {
         $board = new ExpenseReport($db);
         $dashboardlines[$board->element . '_toapprove'] = $board->load_board($user, 'toapprove');
     }
 
     // Number of expense reports to pay
-    if (isModEnabled('expensereport')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_EXPENSEREPORT') && $user->hasRight('expensereport', 'to_paid')) {
+    if (isModEnabled('expensereport') && !getDolGlobalString('MAIN_DISABLE_BLOCK_EXPENSEREPORT') && $user->hasRight('expensereport', 'to_paid')) {
         $board = new ExpenseReport($db);
         $dashboardlines[$board->element . '_topay'] = $board->load_board($user, 'topay');
     }
 
     // Number of holidays to approve
-    if (isModEnabled('holiday')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_HOLIDAY') && $user->hasRight('holiday', 'approve')) {
+    if (isModEnabled('holiday') && !getDolGlobalString('MAIN_DISABLE_BLOCK_HOLIDAY') && $user->hasRight('holiday', 'approve')) {
         $board = new Holiday($db);
         $dashboardlines[$board->element] = $board->load_board($user);
     }

Dolibarr/Code/Holiday/Classes/Holiday.php

Indentation +6 added lines, -6 removed lines

@@ -1488,7 +1488,7 @@ 
         return $this->LibStatut($this->status, $mode, $this->date_debut);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Returns the label of a status
      *
@@ -1499,7 +1499,7 @@ 
      */
     public function LibStatut($status, $mode = 0, $startdate = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
@@ -2041,7 +2041,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return list of people with permission to validate leave requests.
      * Search for permission "approve leave requests"
@@ -2050,7 +2050,7 @@ 
      */
     public function fetch_users_approver_holiday()
     {
-		// phpcs:enable
+        // phpcs:enable
         $users_validator = array();
 
         $sql = "SELECT DISTINCT ur.fk_user";
@@ -2431,7 +2431,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Load indicators for dashboard (this->nbtodo and this->nbtodolate)
      *
@@ -2440,7 +2440,7 @@ 
      */
     public function load_board($user)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         if ($user->socid) {

Spacing +1 added lines, -1 removed lines

@@ -451,7 +451,7 @@
                 $this->date_fin_gmt = $this->db->jdate($obj->date_fin, 1);
                 $this->halfday = $obj->halfday;
                 $this->status = $obj->status;
-                $this->statut = $obj->status;   // deprecated
+                $this->statut = $obj->status; // deprecated
                 $this->fk_validator = $obj->fk_validator;
                 $this->date_valid = $this->db->jdate($obj->date_valid);
                 $this->fk_user_valid = $obj->fk_user_valid;

Dolibarr/Code/Compta/Classes/RemiseCheque.php

Spacing +23 added lines, -23 removed lines

@@ -59,7 +59,7 @@ 
     //! Numero d'erreur Plage 1024-1279
     public $errno;
 
-    public $type = 'CHQ';       // 'CHQ', 'TRA', ...
+    public $type = 'CHQ'; // 'CHQ', 'TRA', ...
 
     public $amount;
     public $date_bordereau;
@@ -105,7 +105,7 @@ 
         $sql .= " LEFT JOIN " . MAIN_DB_PREFIX . "bank_account as ba ON bc.fk_bank_account = ba.rowid";
         $sql .= " WHERE bc.entity = " . $conf->entity;
         if ($id) {
-            $sql .= " AND bc.rowid = " . ((int)$id);
+            $sql .= " AND bc.rowid = " . ((int) $id);
         }
         if ($ref) {
             $sql .= " AND bc.ref = '" . $this->db->escape($ref) . "'";
@@ -183,12 +183,12 @@ 
         $sql .= ") VALUES (";
         $sql .= "'" . $this->db->idate($now) . "'";
         $sql .= ", '" . $this->db->idate($now) . "'";
-        $sql .= ", " . ((int)$user->id);
-        $sql .= ", " . ((int)$account_id);
+        $sql .= ", " . ((int) $user->id);
+        $sql .= ", " . ((int) $account_id);
         $sql .= ", 0";
         $sql .= ", 0";
         $sql .= ", 0";
-        $sql .= ", " . ((int)$conf->entity);
+        $sql .= ", " . ((int) $conf->entity);
         $sql .= ", 0";
         $sql .= ", ''";
         $sql .= ", '" . $this->db->escape($this->type) . "'";
@@ -205,7 +205,7 @@ 
             if ($this->id > 0 && $this->errno == 0) {
                 $sql = "UPDATE " . MAIN_DB_PREFIX . "bordereau_cheque";
                 $sql .= " SET ref = '(PROV" . $this->id . ")'";
-                $sql .= " WHERE rowid=" . ((int)$this->id);
+                $sql .= " WHERE rowid=" . ((int) $this->id);
 
                 $resql = $this->db->query($sql);
                 if (!$resql) {
@@ -222,7 +222,7 @@ 
                 $sql .= " WHERE b.fk_type = '" . $this->db->escape($this->type) . "'";
                 $sql .= " AND b.amount > 0";
                 $sql .= " AND b.fk_bordereau = 0";
-                $sql .= " AND b.fk_account = " . ((int)$account_id);
+                $sql .= " AND b.fk_account = " . ((int) $account_id);
                 if ($limit) {
                     $sql .= $this->db->plimit($limit);
                 }
@@ -251,8 +251,8 @@ 
 
                     if ($checkremise) {
                         $sql = "UPDATE " . MAIN_DB_PREFIX . "bank";
-                        $sql .= " SET fk_bordereau = " . ((int)$this->id);
-                        $sql .= " WHERE rowid = " . ((int)$lineid);
+                        $sql .= " SET fk_bordereau = " . ((int) $this->id);
+                        $sql .= " WHERE rowid = " . ((int) $lineid);
 
                         $resql = $this->db->query($sql);
                         if (!$resql) {
@@ -306,7 +306,7 @@ 
         $this->db->begin();
 
         $sql = "DELETE FROM " . MAIN_DB_PREFIX . "bordereau_cheque";
-        $sql .= " WHERE rowid = " . ((int)$this->id);
+        $sql .= " WHERE rowid = " . ((int) $this->id);
         $sql .= " AND entity = " . $conf->entity;
 
         $resql = $this->db->query($sql);
@@ -321,7 +321,7 @@ 
             if ($this->errno === 0) {
                 $sql = "UPDATE " . MAIN_DB_PREFIX . "bank";
                 $sql .= " SET fk_bordereau = 0";
-                $sql .= " WHERE fk_bordereau = " . ((int)$this->id);
+                $sql .= " WHERE fk_bordereau = " . ((int) $this->id);
 
                 $resql = $this->db->query($sql);
                 if (!$resql) {
@@ -360,7 +360,7 @@ 
         if ($this->errno == 0 && $numref) {
             $sql = "UPDATE " . MAIN_DB_PREFIX . "bordereau_cheque";
             $sql .= " SET statut = 1, ref = '" . $this->db->escape($numref) . "'";
-            $sql .= " WHERE rowid = " . ((int)$this->id);
+            $sql .= " WHERE rowid = " . ((int) $this->id);
             $sql .= " AND entity = " . $conf->entity;
             $sql .= " AND statut = 0";
 
@@ -421,7 +421,7 @@ 
             $classname = getDolGlobalString('CHEQUERECEIPTS_ADDON');
 
             // 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/cheque/");
@@ -603,7 +603,7 @@ 
             $sql .= ", " . MAIN_DB_PREFIX . "bordereau_cheque as bc";
             $sql .= " WHERE b.fk_account = ba.rowid";
             $sql .= " AND b.fk_bordereau = bc.rowid";
-            $sql .= " AND bc.rowid = " . ((int)$this->id);
+            $sql .= " AND bc.rowid = " . ((int) $this->id);
             $sql .= " AND bc.entity = " . $conf->entity;
             $sql .= " ORDER BY b.dateo ASC, b.rowid ASC";
 
@@ -666,7 +666,7 @@ 
         $nb = 0;
         $sql = "SELECT amount ";
         $sql .= " FROM " . MAIN_DB_PREFIX . "bank";
-        $sql .= " WHERE fk_bordereau = " . ((int)$this->id);
+        $sql .= " WHERE fk_bordereau = " . ((int) $this->id);
 
         $resql = $this->db->query($sql);
         if ($resql) {
@@ -679,9 +679,9 @@ 
 
             $sql = "UPDATE " . MAIN_DB_PREFIX . "bordereau_cheque";
             $sql .= " SET amount = " . price2num($total);
-            $sql .= ", nbcheque = " . ((int)$nb);
-            $sql .= " WHERE rowid = " . ((int)$this->id);
-            $sql .= " AND entity = " . ((int)$conf->entity);
+            $sql .= ", nbcheque = " . ((int) $nb);
+            $sql .= " WHERE rowid = " . ((int) $this->id);
+            $sql .= " AND entity = " . ((int) $conf->entity);
 
             $resql = $this->db->query($sql);
             if (!$resql) {
@@ -716,8 +716,8 @@ 
         if ($this->id > 0) {
             $sql = "UPDATE " . MAIN_DB_PREFIX . "bank";
             $sql .= " SET fk_bordereau = 0";
-            $sql .= " WHERE rowid = " . ((int)$account_id);
-            $sql .= " AND fk_bordereau = " . ((int)$this->id);
+            $sql .= " WHERE rowid = " . ((int) $account_id);
+            $sql .= " AND fk_bordereau = " . ((int) $this->id);
 
             $resql = $this->db->query($sql);
             if ($resql) {
@@ -765,7 +765,7 @@ 
         // Get invoices list to reopen them
         $sql = 'SELECT pf.fk_facture, pf.amount';
         $sql .= ' FROM ' . MAIN_DB_PREFIX . 'paiement_facture as pf';
-        $sql .= ' WHERE pf.fk_paiement = ' . ((int)$payment->id);
+        $sql .= ' WHERE pf.fk_paiement = ' . ((int) $payment->id);
 
         $resql = $this->db->query($sql);
         if ($resql) {
@@ -830,7 +830,7 @@ 
         if ($user->hasRight('banque', 'cheque')) {
             $sql = "UPDATE " . MAIN_DB_PREFIX . "bordereau_cheque";
             $sql .= " SET date_bordereau = " . ($date ? "'" . $this->db->idate($date) . "'" : 'null');
-            $sql .= " WHERE rowid = " . ((int)$this->id);
+            $sql .= " WHERE rowid = " . ((int) $this->id);
 
             dol_syslog("RemiseCheque::set_date", LOG_DEBUG);
             $resql = $this->db->query($sql);
@@ -861,7 +861,7 @@ 
         if ($user->hasRight('banque', 'cheque')) {
             $sql = "UPDATE " . MAIN_DB_PREFIX . "bordereau_cheque";
             $sql .= " SET ref = '" . $this->db->escape($ref) . "'";
-            $sql .= " WHERE rowid = " . ((int)$this->id);
+            $sql .= " WHERE rowid = " . ((int) $this->id);
 
             dol_syslog("RemiseCheque::set_number", LOG_DEBUG);
             $resql = $this->db->query($sql);

Dolibarr/Code/Compta/Classes/FactureLigneRec.php

Spacing +9 added lines, -9 removed lines

@@ -117,7 +117,7 @@ 
         }
 
         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) {
@@ -157,7 +157,7 @@ 
         $sql .= ' p.ref as product_ref, p.fk_product_type as fk_product_type, p.label as product_label, p.description as product_desc';
         $sql .= ' FROM ' . MAIN_DB_PREFIX . 'facturedet_rec as l';
         $sql .= ' LEFT JOIN ' . MAIN_DB_PREFIX . 'product as p ON l.fk_product = p.rowid';
-        $sql .= ' WHERE l.rowid = ' . ((int)$rowid);
+        $sql .= ' WHERE l.rowid = ' . ((int) $rowid);
         $sql .= ' ORDER BY l.rang';
 
         dol_syslog('FactureRec::fetch', LOG_DEBUG);
@@ -245,7 +245,7 @@ 
         include_once DOL_DOCUMENT_ROOT . '/core/lib/price.lib.php';
 
         $sql = "UPDATE " . MAIN_DB_PREFIX . "facturedet_rec SET";
-        $sql .= " fk_facture = " . ((int)$this->fk_facture);
+        $sql .= " fk_facture = " . ((int) $this->fk_facture);
         $sql .= ", fk_parent_line=" . ($this->fk_parent_line > 0 ? $this->fk_parent_line : "null");
         $sql .= ", label=" . (!empty($this->label) ? "'" . $this->db->escape($this->label) . "'" : "null");
         $sql .= ", description='" . $this->db->escape($this->desc) . "'";
@@ -258,12 +258,12 @@ 
         $sql .= ", localtax2_tx=" . price2num($this->localtax2_tx);
         $sql .= ", localtax2_type='" . $this->db->escape($this->localtax2_type) . "'";
         $sql .= ", fk_product=" . ($this->fk_product > 0 ? $this->fk_product : "null");
-        $sql .= ", product_type=" . ((int)$this->product_type);
+        $sql .= ", product_type=" . ((int) $this->product_type);
         $sql .= ", remise_percent=" . price2num($this->remise_percent);
         $sql .= ", subprice=" . price2num($this->subprice);
         $sql .= ", info_bits=" . price2num($this->info_bits);
-        $sql .= ", date_start_fill=" . (int)$this->date_start_fill;
-        $sql .= ", date_end_fill=" . (int)$this->date_end_fill;
+        $sql .= ", date_start_fill=" . (int) $this->date_start_fill;
+        $sql .= ", date_end_fill=" . (int) $this->date_end_fill;
         if (empty($this->skip_update_total)) {
             $sql .= ", total_ht=" . price2num($this->total_ht);
             $sql .= ", total_tva=" . price2num($this->total_tva);
@@ -271,11 +271,11 @@ 
             $sql .= ", total_localtax2=" . price2num($this->total_localtax2);
             $sql .= ", total_ttc=" . price2num($this->total_ttc);
         }
-        $sql .= ", rang=" . ((int)$this->rang);
-        $sql .= ", special_code=" . ((int)$this->special_code);
+        $sql .= ", rang=" . ((int) $this->rang);
+        $sql .= ", special_code=" . ((int) $this->special_code);
         $sql .= ", fk_unit=" . ($this->fk_unit ? "'" . $this->db->escape($this->fk_unit) . "'" : "null");
         $sql .= ", fk_contract_line=" . ($this->fk_contract_line ? $this->fk_contract_line : "null");
-        $sql .= " WHERE rowid = " . ((int)$this->id);
+        $sql .= " WHERE rowid = " . ((int) $this->id);
 
         $this->db->begin();
 

Dolibarr/Code/EventOrganizaction/Classes/ConferenceOrBoothAttendee.php

Indentation +4 added lines, -4 removed lines

@@ -636,7 +636,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Load the project with id $this->fk_project into this->project
      *
@@ -644,7 +644,7 @@ 
      */
     public function fetch_projet()
     {
-		// phpcs:enable
+        // phpcs:enable
 
         if (empty($this->fk_project) && !empty($this->fk_projet)) {
             $this->fk_project = $this->fk_projet; // For backward compatibility
@@ -863,7 +863,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the status
      *
@@ -873,7 +873,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {

Dolibarr/Code/EmailCollector/Classes/EmailCollector.php

Spacing +30 added lines, -30 removed lines

@@ -354,7 +354,7 @@ 
 
         $sql = 'SELECT rowid, type, rulevalue, status';
         $sql .= ' FROM ' . MAIN_DB_PREFIX . 'emailcollector_emailcollectorfilter';
-        $sql .= ' WHERE fk_emailcollector = ' . ((int)$this->id);
+        $sql .= ' WHERE fk_emailcollector = ' . ((int) $this->id);
         //$sql.= ' ORDER BY position';
 
         $resql = $this->db->query($sql);
@@ -386,7 +386,7 @@ 
 
         $sql = 'SELECT rowid, type, actionparam, status';
         $sql .= ' FROM ' . MAIN_DB_PREFIX . 'emailcollector_emailcollectoraction';
-        $sql .= ' WHERE fk_emailcollector = ' . ((int)$this->id);
+        $sql .= ' WHERE fk_emailcollector = ' . ((int) $this->id);
         $sql .= ' ORDER BY position';
 
         $resql = $this->db->query($sql);
@@ -617,7 +617,7 @@ 
         $sql = 'SELECT rowid, date_creation as datec, tms as datem,';
         $sql .= ' fk_user_creat, fk_user_modif';
         $sql .= ' FROM ' . MAIN_DB_PREFIX . $this->table_element . ' as t';
-        $sql .= ' WHERE t.rowid = ' . ((int)$id);
+        $sql .= ' WHERE t.rowid = ' . ((int) $id);
         $result = $this->db->query($sql);
         if ($result) {
             if ($this->db->num_rows($result)) {
@@ -1125,7 +1125,7 @@ 
                 }
 
                 if ($rule['type'] == 'from') {
-                    $tmprulevaluearray = explode('*', $rule['rulevalue']);  // Search on abc*def means searching on 'abc' and on 'def'
+                    $tmprulevaluearray = explode('*', $rule['rulevalue']); // Search on abc*def means searching on 'abc' and on 'def'
                     if (count($tmprulevaluearray) >= 2) {
                         foreach ($tmprulevaluearray as $tmprulevalue) {
                             $search .= ($search ? ' ' : '') . $not . 'FROM "' . str_replace('"', '', $tmprulevalue) . '"';
@@ -1135,7 +1135,7 @@ 
                     }
                 }
                 if ($rule['type'] == 'to') {
-                    $tmprulevaluearray = explode('*', $rule['rulevalue']);  // Search on abc*def means searching on 'abc' and on 'def'
+                    $tmprulevaluearray = explode('*', $rule['rulevalue']); // Search on abc*def means searching on 'abc' and on 'def'
                     if (count($tmprulevaluearray) >= 2) {
                         foreach ($tmprulevaluearray as $tmprulevalue) {
                             $search .= ($search ? ' ' : '') . $not . 'TO "' . str_replace('"', '', $tmprulevalue) . '"';
@@ -1322,7 +1322,7 @@ 
             }
         }
 
-        $arrayofemailtodelete = array();    // Track email to delete to make the deletion at end.
+        $arrayofemailtodelete = array(); // Track email to delete to make the deletion at end.
 
         // Loop on each email found
         if (!$error && !empty($arrayofemail) && count($arrayofemail) > 0) {
@@ -1392,7 +1392,7 @@ 
                     // $operationslog .= " - ".dol_escape_htmltag((string) $imapemail);
                     $msgid = str_replace(array('<', '>'), '', $overview['message_id']);
                 } else {
-                    $operationslog .= " - " . dol_escape_htmltag((string)$imapemail);
+                    $operationslog .= " - " . dol_escape_htmltag((string) $imapemail);
                     $msgid = str_replace(array('<', '>'), '', $overview[0]->message_id);
                 }
                 $operationslog .= " - MsgId: " . $msgid . " - References: " . dol_escape_htmltag($headers['References'] ?? '') . " - Subject: " . dol_escape_htmltag($headers['Subject']);
@@ -1654,15 +1654,15 @@ 
                     $tmpdate = $overview['date']->toDate();
                     $tmptimezone = $tmpdate->getTimezone()->getName();
 
-                    $dateemail = dol_stringtotime((string)$overview['date'], 'gmt');    // if $overview['timezone'] is "+00:00"
+                    $dateemail = dol_stringtotime((string) $overview['date'], 'gmt'); // if $overview['timezone'] is "+00:00"
                     if (preg_match('/^([+\-])(\d\d):(\d\d)/', $tmptimezone, $reg)) {
                         if ($reg[1] == '+' && ($reg[2] != '00' || $reg[3] != '00')) {
-                            $dateemail -= (3600 * (int)$reg[2]);
-                            $dateemail -= (60 * (int)$reg[3]);
+                            $dateemail -= (3600 * (int) $reg[2]);
+                            $dateemail -= (60 * (int) $reg[3]);
                         }
                         if ($reg[1] == '-' && ($reg[2] != '00' || $reg[3] != '00')) {
-                            $dateemail += (3600 * (int)$reg[2]);
-                            $dateemail += (60 * (int)$reg[3]);
+                            $dateemail += (3600 * (int) $reg[2]);
+                            $dateemail += (60 * (int) $reg[3]);
                         }
                     }
                     $subject = $overview['subject'];
@@ -1674,7 +1674,7 @@ 
                     $to = $overview[0]->to;
                     $sendtocc = !empty($overview[0]->cc) ? $overview[0]->cc : '';
                     $sendtobcc = !empty($overview[0]->bcc) ? $overview[0]->bcc : '';
-                    $dateemail = dol_stringtotime((string)$overview[0]->udate, 'gmt');
+                    $dateemail = dol_stringtotime((string) $overview[0]->udate, 'gmt');
                     $subject = $overview[0]->subject;
                     //var_dump($msgid);exit;
                 }
@@ -1739,15 +1739,15 @@ 
                 foreach ($arrayofreferences as $reference) {
                     //print "Process mail ".$iforemailloop." email_msgid ".$msgid.", date ".dol_print_date($dateemail, 'dayhour', 'gmt').", subject ".$subject.", reference ".dol_escape_htmltag($reference)."<br>\n";
                     if (!empty($trackidfoundintorecipienttype)) {
-                        $resultsearchtrackid = -1;      // trackid found
+                        $resultsearchtrackid = -1; // trackid found
                         $reg[1] = $trackidfoundintorecipienttype;
                         $reg[2] = $trackidfoundintorecipientid;
                     } elseif (!empty($trackidfoundintomsgidtype)) {
-                        $resultsearchtrackid = -1;      // trackid found
+                        $resultsearchtrackid = -1; // trackid found
                         $reg[1] = $trackidfoundintomsgidtype;
                         $reg[2] = $trackidfoundintomsgidid;
                     } else {
-                        $resultsearchtrackid = preg_match('/dolibarr-([a-z]+)([0-9]+)@' . preg_quote($host, '/') . '/', $reference, $reg);  // trackid found or not
+                        $resultsearchtrackid = preg_match('/dolibarr-([a-z]+)([0-9]+)@' . preg_quote($host, '/') . '/', $reference, $reg); // trackid found or not
                         if (empty($resultsearchtrackid) && getDolGlobalString('EMAIL_ALTERNATIVE_HOST_SIGNATURE')) {
                             $resultsearchtrackid = preg_match('/dolibarr-([a-z]+)([0-9]+)@' . preg_quote(getDolGlobalString('EMAIL_ALTERNATIVE_HOST_SIGNATURE'), '/') . '/', $reference, $reg); // trackid found
                         }
@@ -2045,7 +2045,7 @@ 
 
                         // Make Operation
                         dol_syslog("Execute action " . $operation['type'] . " actionparam=" . $operation['actionparam'] . ' thirdpartystatic->id=' . $thirdpartystatic->id . ' contactstatic->id=' . $contactstatic->id . ' projectstatic->id=' . $projectstatic->id);
-                        dol_syslog("Execute action fk_element_id=" . $fk_element_id . " fk_element_type=" . $fk_element_type);    // If a Dolibarr tracker id is found, we should now the id of object
+                        dol_syslog("Execute action fk_element_id=" . $fk_element_id . " fk_element_type=" . $fk_element_type); // If a Dolibarr tracker id is found, we should now the id of object
 
                         // Try to guess if this is an email in or out.
                         $actioncode = 'EMAIL_IN';
@@ -2098,7 +2098,7 @@ 
                                     $operationslog .= '<br>Ticket not found using trackid=' . $trackid . ' or msgid=' . $msgid;
                                     $ticketalreadyexists = 0;
                                 } else {
-                                    $operationslog .= '<br>Ticket already found using trackid=' . $trackid . ' or msgid=' . $msgid;   // We change the operation type to do
+                                    $operationslog .= '<br>Ticket already found using trackid=' . $trackid . ' or msgid=' . $msgid; // We change the operation type to do
                                     $ticketalreadyexists = 1;
                                     $operation['type'] = 'recordevent';
                                 }
@@ -2472,8 +2472,8 @@ 
                                 $actioncomm->label = $langs->trans("ActionAC_" . $actioncode) . ' - ' . $langs->trans("MailFrom") . ' ' . $from;
                                 $actioncomm->note_private = $descriptionfull;
                                 $actioncomm->fk_project = $projectstatic->id;
-                                $actioncomm->datep = $dateemail;  // date of email
-                                $actioncomm->datef = $dateemail;  // date of email
+                                $actioncomm->datep = $dateemail; // date of email
+                                $actioncomm->datef = $dateemail; // date of email
                                 $actioncomm->percentage = -1; // Not applicable
                                 $actioncomm->socid = $thirdpartystatic->id;
                                 $actioncomm->contact_id = $contactstatic->id;
@@ -2671,7 +2671,7 @@ 
                                 }
                                 $hookmanager->initHooks(array('emailcolector'));
                                 $parameters = array('arrayobject' => $arrayobject);
-                                $reshook = $hookmanager->executeHooks('addmoduletoeamailcollectorjoinpiece', $parameters);    // Note that $action and $object may have been modified by some hooks
+                                $reshook = $hookmanager->executeHooks('addmoduletoeamailcollectorjoinpiece', $parameters); // Note that $action and $object may have been modified by some hooks
                                 if ($reshook > 0) {
                                     $arrayobject = $hookmanager->resArray;
                                 }
@@ -2760,7 +2760,7 @@ 
                                 $percent_opp_status = dol_getIdFromCode($this->db, 'PROSP', 'c_lead_status', 'code', 'percent');
 
                                 $projecttocreate->title = $subject;
-                                $projecttocreate->date_start = $date;   // date of email
+                                $projecttocreate->date_start = $date; // date of email
                                 $projecttocreate->date_end = 0;
                                 $projecttocreate->opp_status = $id_opp_status;
                                 $projecttocreate->opp_percent = $percent_opp_status;
@@ -2785,7 +2785,7 @@ 
                                     $file = '';
                                     $classname = '';
                                     $reldir = '';
-                                    $dirmodels = array_merge(array('/'), (array)$conf->modules_parts['models']);
+                                    $dirmodels = array_merge(array('/'), (array) $conf->modules_parts['models']);
                                     foreach ($dirmodels as $reldir) {
                                         $file = dol_buildpath($reldir . "core/modules/project/" . $modele . '.php', 0);
                                         if (file_exists($file)) {
@@ -2840,7 +2840,7 @@ 
                                                 if (getDolGlobalString('MAIN_IMAP_USE_PHPIMAP')) {
                                                     foreach ($attachments as $attachment) {
                                                         // $attachment->save($destdir.'/');
-                                                        $typeattachment = (string)$attachment->getDisposition();
+                                                        $typeattachment = (string) $attachment->getDisposition();
                                                         $filename = $attachment->getFilename();
                                                         $content = $attachment->getContent();
                                                         $this->saveAttachment($destdir, $filename, $content);
@@ -2925,7 +2925,7 @@ 
                                     $file = '';
                                     $classname = '';
                                     $reldir = '';
-                                    $dirmodels = array_merge(array('/'), (array)$conf->modules_parts['models']);
+                                    $dirmodels = array_merge(array('/'), (array) $conf->modules_parts['models']);
                                     foreach ($dirmodels as $reldir) {
                                         $file = dol_buildpath($reldir . "core/modules/ticket/" . $modele . '.php', 0);
                                         if (file_exists($file)) {
@@ -2975,7 +2975,7 @@ 
                                                 if (getDolGlobalString('MAIN_IMAP_USE_PHPIMAP')) {
                                                     foreach ($attachments as $attachment) {
                                                         // $attachment->save($destdir.'/');
-                                                        $typeattachment = (string)$attachment->getDisposition();
+                                                        $typeattachment = (string) $attachment->getDisposition();
                                                         $filename = $attachment->getFilename();
                                                         $content = $attachment->getContent();
                                                         $this->saveAttachment($destdir, $filename, $content);
@@ -3022,7 +3022,7 @@ 
                                 $candidaturetocreate->note_private = $descriptionfull;
                                 $candidaturetocreate->entity = $conf->entity;
                                 $candidaturetocreate->email_msgid = $msgid;
-                                $candidaturetocreate->email_date = $date;       // date of email
+                                $candidaturetocreate->email_date = $date; // date of email
                                 $candidaturetocreate->status = $candidaturetocreate::STATUS_DRAFT;
                                 //$candidaturetocreate->fk_contact = $contactstatic->id;
 
@@ -3324,8 +3324,8 @@ 
     {
         if (function_exists('mb_convert_encoding')) {
             // change spaces by entropy because mb_convert fail with spaces
-            $str = preg_replace("/ /", "xxxSPACExxx", $str);        // the replacement string must be valid in utf7 so _ can't be used
-            $str = preg_replace("/\[Gmail\]/", "xxxGMAILxxx", $str);    // the replacement string must be valid in utf7 so _ can't be used
+            $str = preg_replace("/ /", "xxxSPACExxx", $str); // the replacement string must be valid in utf7 so _ can't be used
+            $str = preg_replace("/\[Gmail\]/", "xxxGMAILxxx", $str); // the replacement string must be valid in utf7 so _ can't be used
             // if mb_convert work
             if ($str = mb_convert_encoding($str, "UTF-7")) {
                 // change characters
@@ -3851,7 +3851,7 @@ 
         if (is_object($imapemail)) {
             return $imapemail->getAttributes()["uid"];
         } else {
-            return (string)$imapemail;
+            return (string) $imapemail;
         }
     }
 }

public/htdocs/core/triggers/interface_20_modWorkflow_WorkflowManager.class.php

Spacing +8 added lines, -8 removed lines

@@ -103,7 +103,7 @@ 
 
                 $object->clearObjectLinkedCache();
 
-                return (int)$ret;
+                return (int) $ret;
             }
         }
 
@@ -215,7 +215,7 @@ 
                         foreach ($object->linkedObjects['shipping'] as $element) {
                             $ret = $element->setClosed();
                             if ($ret < 0) {
-                                return (int)$ret;
+                                return (int) $ret;
                             }
                         }
                     }
@@ -236,7 +236,7 @@ 
                         foreach ($object->linkedObjects['shipping'] as $element) {
                             $ret = $element->setBilled();
                             if ($ret < 0) {
-                                return (int)$ret;
+                                return (int) $ret;
                             }
                         }
                     }
@@ -254,7 +254,7 @@ 
                         $areAllInvoicesValidated = true;
                         foreach ($orderLinked->linkedObjects['facture'] as $key => $invoice) {
                             if ($invoice->statut == Facture::STATUS_VALIDATED || $object->id == $invoice->id) {
-                                $totalHTInvoices += (float)$invoice->total_ht;
+                                $totalHTInvoices += (float) $invoice->total_ht;
                             } else {
                                 $areAllInvoicesValidated = false;
                                 break;
@@ -262,7 +262,7 @@ 
                         }
                         if ($areAllInvoicesValidated) {
                             $isSameTotal = (price2num($totalHTInvoices, 'MT') == price2num($orderLinked->total_ht, 'MT'));
-                            dol_syslog("Amount of linked invoices = " . $totalHTInvoices . ", of order = " . $orderLinked->total_ht . ", isSameTotal = " . (string)$isSameTotal, LOG_DEBUG);
+                            dol_syslog("Amount of linked invoices = " . $totalHTInvoices . ", of order = " . $orderLinked->total_ht . ", isSameTotal = " . (string) $isSameTotal, LOG_DEBUG);
                             if ($isSameTotal) {
                                 $ret = $orderLinked->classifyBilled($user);
                                 if ($ret < 0) {
@@ -592,9 +592,9 @@ 
             // Automatically create intervention
             if (isModEnabled('intervention') && isModEnabled('ticket') && !empty($conf->workflow->enabled) && getDolGlobalString('WORKFLOW_TICKET_CREATE_INTERVENTION')) {
                 $fichinter = new Fichinter($this->db);
-                $fichinter->socid = (int)$object->fk_soc;
-                $fichinter->fk_project = (int)$object->fk_project;
-                $fichinter->fk_contrat = (int)$object->fk_contract;
+                $fichinter->socid = (int) $object->fk_soc;
+                $fichinter->fk_project = (int) $object->fk_project;
+                $fichinter->fk_contrat = (int) $object->fk_contract;
                 $fichinter->author = $user->id;
                 $fichinter->model_pdf = (getDolGlobalString('FICHEINTER_ADDON_PDF')) ? $conf->global->FICHEINTER_ADDON_PDF : 'soleil';
                 $fichinter->origin = $object->element;

public/htdocs/core/triggers/interface_50_modAgenda_ActionsAuto.class.php

Spacing +2 added lines, -2 removed lines

@@ -1510,8 +1510,8 @@
         $actioncomm = new ActionComm($this->db);
         $actioncomm->type_code = $object->actiontypecode; // Type of event ('AC_OTH', 'AC_OTH_AUTO', 'AC_XXX'...)
         $actioncomm->code = 'AC_' . $action;
-        $actioncomm->label = $object->actionmsg2;     // Label of event
-        $actioncomm->note_private = $object->actionmsg;     // Description
+        $actioncomm->label = $object->actionmsg2; // Label of event
+        $actioncomm->note_private = $object->actionmsg; // Description
         $actioncomm->fk_project = $projectid;
         $actioncomm->datep = $now;
         $actioncomm->datef = $now;