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

public/htdocs/modulebuilder/template/core/modules/mymodule/doc/pdf_standard_myobject.modules.php

Indentation +9 added lines, -9 removed lines

@@ -150,7 +150,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Function to build pdf onto disk
      *
@@ -164,7 +164,7 @@ 
      */
     public function write_file($object, $outputlangs, $srctemplatepath = '', $hidedetails = 0, $hidedesc = 0, $hideref = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user, $langs, $conf, $mysoc, $db, $hookmanager, $nblines;
 
         dol_syslog("write_file outputlangs->defaultlang=" . (is_object($outputlangs) ? $outputlangs->defaultlang : 'null'));
@@ -280,7 +280,7 @@ 
             if (file_exists($dir)) {
                 // Add pdfgeneration hook
                 if (!is_object($hookmanager)) {
-					$hookmanager = new HookManager($this->db);
+                    $hookmanager = new HookManager($this->db);
                 }
                 $hookmanager->initHooks(array('pdfgeneration'));
                 $parameters = array('file' => $file, 'object' => $object, 'outputlangs' => $outputlangs);
@@ -824,7 +824,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of active generation modules
      *
@@ -834,11 +834,11 @@ 
      */
     public static function liste_modeles($db, $maxfilenamelength = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         return parent::liste_modeles($db, $maxfilenamelength); // TODO: Change the autogenerated stub
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *   Show table for lines
      *
@@ -899,7 +899,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *  Show top header of page.
      *
@@ -912,7 +912,7 @@ 
      */
     protected function _pagehead(&$pdf, $object, $showaddress, $outputlangs, $outputlangsbis = null)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $ltrdirection = 'L';
@@ -1163,7 +1163,7 @@ 
         return $top_shift;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *      Show footer of page. Need this->emetteur object
      *

Dolibarr/Code/User/Classes/User.php

Spacing +12 added lines, -12 removed lines

@@ -608,7 +608,7 @@ 
                 $this->note_public = $obj->note_public;
                 $this->note_private = $obj->note_private;
 
-                $this->statut       = $obj->status;         // deprecated
+                $this->statut       = $obj->status; // deprecated
                 $this->status       = $obj->status;
 
                 $this->photo        = $obj->photo;
@@ -735,7 +735,7 @@ 
         if (getDolGlobalString('MAIN_ENABLE_DEFAULT_VALUES')) {
             // Load user->default_values for user. TODO Save this in memcached ?
             $defaultValues = new DefaultValues($this->db);
-            $result = $defaultValues->fetchAll('', '', 0, 0, '(t.user_id:in:0,' . $this->id . ') AND (entity:in:' . (isset($this->entity) ? $this->entity : $conf->entity) . ',' . $conf->entity . ')');    // User 0 (all) + me (if defined)
+            $result = $defaultValues->fetchAll('', '', 0, 0, '(t.user_id:in:0,' . $this->id . ') AND (entity:in:' . (isset($this->entity) ? $this->entity : $conf->entity) . ',' . $conf->entity . ')'); // User 0 (all) + me (if defined)
             //$result = $defaultValues->fetchAll('', '', 0, 0, array('t.user_id'=>array(0, $this->id), 'entity'=>array((isset($this->entity) ? $this->entity : $conf->entity), $conf->entity)));    // User 0 (all) + me (if defined)
 
             if (!is_array($result) && $result < 0) {
@@ -849,16 +849,16 @@ 
         // Special case for external user
         if (!empty($this->socid)) {
             if ($module == 'societe' && ($permlevel1 == 'creer' || $permlevel1 == 'write')) {
-                return 0;   // An external user never has the permission ->societe->write to see all thirdparties (always restricted to himself)
+                return 0; // An external user never has the permission ->societe->write to see all thirdparties (always restricted to himself)
             }
             if ($module == 'societe' && $permlevel1 == 'client' && $permlevel2 == 'voir') {
-                return 0;   // An external user never has the permission ->societe->client->voir to see all thirdparties (always restricted to himself)
+                return 0; // An external user never has the permission ->societe->client->voir to see all thirdparties (always restricted to himself)
             }
             if ($module == 'societe' && $permlevel1 == 'export') {
-                return 0;   // An external user never has the permission ->societe->export to see all thirdparties (always restricted to himself)
+                return 0; // An external user never has the permission ->societe->export to see all thirdparties (always restricted to himself)
             }
             if ($module == 'societe' && ($permlevel1 == 'supprimer' || $permlevel1 == 'delete')) {
-                return 0;   // An external user never has the permission ->societe->delete to see all thirdparties (always restricted to himself)
+                return 0; // An external user never has the permission ->societe->delete to see all thirdparties (always restricted to himself)
             }
         }
 
@@ -1006,7 +1006,7 @@ 
             $sql .= " FROM " . $this->db->prefix() . "rights_def";
             $sql .= " WHERE entity = " . ((int) $entity);
             if (!empty($whereforadd) && $whereforadd != 'allmodules') {
-                $sql .= " AND (" . $whereforadd . ")";  // Note: parenthesis are important because whereforadd can contains OR. Also note that $whereforadd is already sanitized
+                $sql .= " AND (" . $whereforadd . ")"; // Note: parenthesis are important because whereforadd can contains OR. Also note that $whereforadd is already sanitized
             }
 
             $sqldelete = "DELETE FROM " . $this->db->prefix() . "user_rights";
@@ -1142,7 +1142,7 @@ 
             $sql .= " FROM " . $this->db->prefix() . "rights_def";
             $sql .= " WHERE entity IN (" . $this->db->sanitize($entity, 0, 0, 0, 0) . ")";
             if (!empty($wherefordel) && $wherefordel != 'allmodules') {
-                $sql .= " AND (" . $wherefordel . ")";  // Note: parenthesis are important because wherefordel can contains OR. Also note that $wherefordel is already sanitized
+                $sql .= " AND (" . $wherefordel . ")"; // Note: parenthesis are important because wherefordel can contains OR. Also note that $wherefordel is already sanitized
             }
 
             // avoid admin to remove his own important rights
@@ -1316,12 +1316,12 @@ 
                     $sql .= " AND r.entity = " . ((int) $conf->entity);
                 }
             } else {
-                $sql .= " AND gr.entity = " . ((int) $conf->entity);  // Only groups created in current entity
+                $sql .= " AND gr.entity = " . ((int) $conf->entity); // Only groups created in current entity
                 // The entity on the table gu=usergroup_user should be useless and should never be used because it is already into gr and r.
                 // but when using MULTICOMPANY_TRANSVERSE_MODE, we may have inserted record that make rubbish result here due to the duplicate record of
                 // other entities, so we are forced to add a filter on gu here
                 $sql .= " AND gu.entity IN (0," . $conf->entity . ")";
-                $sql .= " AND r.entity = " . ((int) $conf->entity);   // Only permission of modules enabled in current entity
+                $sql .= " AND r.entity = " . ((int) $conf->entity); // Only permission of modules enabled in current entity
             }
             // End of strange business rule
             $sql .= " AND gr.fk_usergroup = gu.fk_usergroup";
@@ -2794,7 +2794,7 @@ 
         $sql .= " WHERE fk_user  = " . ((int) $this->id);
         $sql .= " AND fk_usergroup = " . ((int) $group);
         if (empty($entity)) {
-            $sql .= " AND entity IN (0, 1)";    // group may be in entity 0 (so $entity=0) and link with user into entity 1.
+            $sql .= " AND entity IN (0, 1)"; // group may be in entity 0 (so $entity=0) and link with user into entity 1.
         } else {
             $sql .= " AND entity = " . ((int) $entity);
         }
@@ -3514,7 +3514,7 @@ 
         $this->iplastlogin = '127.0.0.1';
         $this->datepreviouslogin = $now;
         $this->ippreviouslogin = '127.0.0.1';
-        $this->statut = 1;      // deprecated
+        $this->statut = 1; // deprecated
         $this->status = 1;
 
         $this->entity = 1;

Indentation +36 added lines, -36 removed lines

@@ -1764,7 +1764,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Create a user from a contact object. User will be internal but if contact is linked to a third party, user will be external
      *
@@ -1775,7 +1775,7 @@ 
      */
     public function create_from_contact($contact, $login = '', $password = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user, $langs;
 
         $error = 0;
@@ -1850,7 +1850,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Create a user into database from a member object.
      *  If $member->fk_soc is set, it will be an external user.
@@ -1861,7 +1861,7 @@ 
      */
     public function create_from_member($member, $login = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user;
 
         // Set properties on new user
@@ -1939,7 +1939,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Assign rights by default
      *
@@ -1947,7 +1947,7 @@ 
      */
     public function set_default_rights()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $rd = array();
@@ -2336,7 +2336,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update the user's last login date in the database.
      *  Function called when a new connection is made by the user
@@ -2345,7 +2345,7 @@ 
      */
     public function update_last_login_date()
     {
-		// phpcs:enable
+        // phpcs:enable
         $now = dol_now();
 
         $userremoteip = getUserRemoteIP();
@@ -2516,7 +2516,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Send a new password (or instructions to reset it) by email
      *
@@ -2527,7 +2527,7 @@ 
      */
     public function send_password($user, $password = '', $changelater = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $mysoc;
         global $dolibarr_main_url_root;
 
@@ -2644,7 +2644,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Read clicktodial information for user
      *
@@ -2652,7 +2652,7 @@ 
      */
     public function fetch_clicktodial()
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "SELECT url, login, pass, poste ";
         $sql .= " FROM " . $this->db->prefix() . "user_clicktodial as u";
         $sql .= " WHERE u.fk_user = " . ((int) $this->id);
@@ -2678,7 +2678,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update clicktodial info
      *
@@ -2686,7 +2686,7 @@ 
      */
     public function update_clicktodial()
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->db->begin();
 
         $sql = "DELETE FROM " . $this->db->prefix() . "user_clicktodial";
@@ -2716,7 +2716,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Add user into a group
      *
@@ -2727,7 +2727,7 @@ 
      */
     public function SetInGroup($group, $entity, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $user;
 
         $error = 0;
@@ -2772,7 +2772,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Remove a user from a group
      *
@@ -2783,7 +2783,7 @@ 
      */
     public function RemoveFromGroup($group, $entity, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $user;
 
         $error = 0;
@@ -3176,7 +3176,7 @@ 
         return $this->LibStatut(isset($this->statut) ? (int) $this->statut : (int) $this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the label of a status of user (active, inactive)
      *
@@ -3186,7 +3186,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
@@ -3278,8 +3278,8 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Retourne chaine DN complete dans l'annuaire LDAP pour l'objet
      *
@@ -3291,7 +3291,7 @@ 
      */
     public function _load_ldap_dn($info, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
         $dn = '';
         if ($mode == 0) {
@@ -3304,8 +3304,8 @@ 
         return $dn;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Initialize the info array (array of LDAP values) that will be used to call LDAP functions
      *
@@ -3313,7 +3313,7 @@ 
      */
     public function _load_ldap_info()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $info = array();
@@ -3619,7 +3619,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update user using data from the LDAP
      *
@@ -3628,7 +3628,7 @@ 
      */
     public function update_ldap2dolibarr(&$ldapuser)
     {
-		// phpcs:enable
+        // phpcs:enable
         // TODO: Voir pourquoi le update met à jour avec toutes les valeurs vide (global $user écrase ?)
         global $user, $conf;
 
@@ -3673,7 +3673,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return and array with all instantiated first level children users of current user
      *
@@ -3682,7 +3682,7 @@ 
      */
     public function get_children()
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "SELECT rowid FROM " . $this->db->prefix() . "user";
         $sql .= " WHERE fk_user = " . ((int) $this->id);
 
@@ -3733,7 +3733,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Build the hierarchy/tree of users into an array.
      *  Set and return this->users that is an array sorted according to tree with arrays of:
@@ -3749,7 +3749,7 @@ 
      */
     public function get_full_tree($deleteafterid = 0, $filter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
         global $hookmanager;
 
@@ -3878,7 +3878,7 @@ 
         return $childids;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  For user id_user and its children available in this->users, define property fullpath and fullname.
      *  Function called by get_full_tree().
@@ -3889,7 +3889,7 @@ 
      */
     public function build_path_from_id_user($id_user, $protection = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         //dol_syslog(get_only_class($this)."::build_path_from_id_user id_user=".$id_user." protection=".$protection, LOG_DEBUG);
 
         if (!empty($this->users[$id_user]['fullpath'])) {
@@ -4009,7 +4009,7 @@ 
         return $this->commonGenerateDocument($modelpath, $modele, $outputlangs, $hidedetails, $hidedesc, $hideref, $moreparams);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return property of user from its id
      *
@@ -4019,7 +4019,7 @@ 
      */
     public function user_get_property($rowid, $mode)
     {
-		// phpcs:enable
+        // phpcs:enable
         $user_property = '';
 
         if (empty($rowid)) {

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