alxarafe / alixar

htdocs/admin/modules.php

Spacing +11 added lines, -11 removed lines

@@ -70,9 +70,9 @@ 
 // For dolistore search
 $options = [];
 $options['per_page'] = 20;
-$options['categorie'] = ((int)(GETPOSTINT('categorie') ? GETPOSTINT('categorie') : 0));
-$options['start'] = ((int)(GETPOSTINT('start') ? GETPOSTINT('start') : 0));
-$options['end'] = ((int)(GETPOSTINT('end') ? GETPOSTINT('end') : 0));
+$options['categorie'] = ((int) (GETPOSTINT('categorie') ? GETPOSTINT('categorie') : 0));
+$options['start'] = ((int) (GETPOSTINT('start') ? GETPOSTINT('start') : 0));
+$options['end'] = ((int) (GETPOSTINT('end') ? GETPOSTINT('end') : 0));
 $options['search'] = GETPOST('search_keyword', 'alpha');
 $dolistore = new Dolistore(false);
 
@@ -311,13 +311,13 @@ 
     exit;
 } elseif (getDolGlobalInt("MAIN_FEATURES_LEVEL") > 1 && $action == 'reload' && $user->admin && GETPOST('confirm') == 'yes') {
     $result = unActivateModule($value, 0);
-    dolibarr_set_const($db, "MAIN_IHM_PARAMS_REV", (int)$conf->global->MAIN_IHM_PARAMS_REV + 1, 'chaine', 0, '', $conf->entity);
+    dolibarr_set_const($db, "MAIN_IHM_PARAMS_REV", (int) $conf->global->MAIN_IHM_PARAMS_REV + 1, 'chaine', 0, '', $conf->entity);
     if ($result) {
         setEventMessages($result, null, 'errors');
         header("Location: " . $_SERVER['PHP_SELF'] . "?mode=" . $mode . $param . ($page_y ? '&page_y=' . $page_y : ''));
     }
     $resarray = activateModule($value, 0, 1);
-    dolibarr_set_const($db, "MAIN_IHM_PARAMS_REV", (int)$conf->global->MAIN_IHM_PARAMS_REV + 1, 'chaine', 0, '', $conf->entity);
+    dolibarr_set_const($db, "MAIN_IHM_PARAMS_REV", (int) $conf->global->MAIN_IHM_PARAMS_REV + 1, 'chaine', 0, '', $conf->entity);
     if (!empty($resarray['errors'])) {
         setEventMessages('', $resarray['errors'], 'errors');
     } else {
@@ -462,7 +462,7 @@ 
             } else {
                 $familykey = $objMod->family;
             }
-            '@phan-var-force string $familykey';  // if not, phan considers $familykey may be null
+            '@phan-var-force string $familykey'; // if not, phan considers $familykey may be null
 
             $moduleposition = ($objMod->module_position ? $objMod->module_position : '50');
             if ($objMod->isCoreOrExternalModule() == 'external' && $moduleposition < 100000) {
@@ -483,7 +483,7 @@ 
             if ($external && !in_array($familykey, $listOfOfficialModuleGroups)) {
                 // If module is extern and into a custom group (not into an official predefined one), it must appear at end (custom groups should not be before official groups).
                 if (is_numeric($familyposition)) {
-                    $familyposition = sprintf("%03d", (int)$familyposition + 100);
+                    $familyposition = sprintf("%03d", (int) $familyposition + 100);
                 }
             }
 
@@ -940,10 +940,10 @@ 
                             }
                         }
                     }
-                } elseif (preg_match('/^([^@]+)@([^@]+)$/i', (string)$objMod->config_page_url, $regs)) {
+                } elseif (preg_match('/^([^@]+)@([^@]+)$/i', (string) $objMod->config_page_url, $regs)) {
                     $codetoconfig .= '<a class="valignmiddle" href="' . dol_buildpath('/' . $regs[2] . '/admin/' . $regs[1], 1) . '?save_lastsearch_values=1&backtopage=' . urlencode($backtourl) . '" title="' . $langs->trans("Setup") . '">' . img_picto($langs->trans("Setup"), "setup", 'style="padding-right: 6px"', false, 0, 0, '', 'fa-15') . '</a>';
                 } else {
-                    $codetoconfig .= '<a class="valignmiddle" href="' . ((string)$objMod->config_page_url) . '?save_lastsearch_values=1&backtopage=' . urlencode($backtourl) . '" title="' . $langs->trans("Setup") . '">' . img_picto($langs->trans("Setup"), "setup", 'style="padding-right: 6px"', false, 0, 0, '', 'fa-15') . '</a>';
+                    $codetoconfig .= '<a class="valignmiddle" href="' . ((string) $objMod->config_page_url) . '?save_lastsearch_values=1&backtopage=' . urlencode($backtourl) . '" title="' . $langs->trans("Setup") . '">' . img_picto($langs->trans("Setup"), "setup", 'style="padding-right: 6px"', false, 0, 0, '', 'fa-15') . '</a>';
                 }
             } else {
                 $codetoconfig .= img_picto($langs->trans("NothingToSetup"), "setup", 'class="opacitytransp" style="padding-right: 6px"', false, 0, 0, '', 'fa-15');
@@ -1038,7 +1038,7 @@ 
             // Help
             print '<td class="center nowrap" style="width: 82px;">';
             //print $form->textwithpicto('', $text, 1, $imginfo, 'minheight20', 0, 2, 1);
-            print '<a href="javascript:document_preview(\'' . DOL_URL_ROOT . '/admin/modulehelp.php?id=' . ((int)$objMod->numero) . '\',\'text/html\',\'' . dol_escape_js($langs->trans("Module")) . '\')">' . img_picto(($objMod->isCoreOrExternalModule() == 'external' ? $langs->trans("ExternalModule") . ' - ' : '') . $langs->trans("ClickToShowDescription"), $imginfo) . '</a>';
+            print '<a href="javascript:document_preview(\'' . DOL_URL_ROOT . '/admin/modulehelp.php?id=' . ((int) $objMod->numero) . '\',\'text/html\',\'' . dol_escape_js($langs->trans("Module")) . '\')">' . img_picto(($objMod->isCoreOrExternalModule() == 'external' ? $langs->trans("ExternalModule") . ' - ' : '') . $langs->trans("ClickToShowDescription"), $imginfo) . '</a>';
             print '</td>';
 
             // Version
@@ -1168,7 +1168,7 @@ 
         <div id="category-tree-left">
             <ul class="tree">
                 <?php
-                echo $dolistore->get_categories();  // Do not use dol_escape_htmltag here, it is already a structured content?>
+                echo $dolistore->get_categories(); // Do not use dol_escape_htmltag here, it is already a structured content?>
             </ul>
         </div>
         <div id="listing-content">

htdocs/core/modules/DolibarrModules.class.php

Spacing +18 added lines, -18 removed lines

@@ -455,7 +455,7 @@ 
         if ($pathoffile) {     // Mostly for external modules
             $content = file_get_contents($pathoffile);
 
-            if ((float)DOL_VERSION >= 6.0) {
+            if ((float) DOL_VERSION >= 6.0) {
                 @include_once BASE_PATH . '/../Dolibarr/Lib/ParseMd.php';
 
                 $content = dolMd2Html(
@@ -557,7 +557,7 @@ 
         if ($filefound) {     // Mostly for external modules
             $content = file_get_contents($pathoffile);
 
-            if ((float)DOL_VERSION >= 6.0) {
+            if ((float) DOL_VERSION >= 6.0) {
                 @include_once BASE_PATH . '/../Dolibarr/Lib/ParseMd.php';
 
                 $content = dolMd2Html($content, 'parsedown', ['doc/' => dol_buildpath(strtolower($this->name) . '/doc/', 1)]);
@@ -676,7 +676,7 @@ 
 
         $sql = "SELECT tms FROM " . $dbPrefix . "const";
         $sql .= " WHERE " . $this->db->decrypt('name') . " = '" . $this->db->escape($this->const_name) . "'";
-        $sql .= " AND entity IN (0, " . ((int)$conf->entity) . ")";
+        $sql .= " AND entity IN (0, " . ((int) $conf->entity) . ")";
 
         dol_syslog(get_class($this) . "::getLastActiveDate", LOG_DEBUG);
         $resql = $this->db->query($sql);
@@ -889,7 +889,7 @@ 
         $sql = "INSERT INTO " . $dbPrefix . "const (name, value, visible, entity, note) VALUES";
         $sql .= " (" . $this->db->encrypt($this->const_name);
         $sql .= ", " . $this->db->encrypt('1');
-        $sql .= ", 0, " . ((int)$entity);
+        $sql .= ", 0, " . ((int) $entity);
         $sql .= ", '" . $this->db->escape($note) . "')";
 
         dol_syslog(get_class($this) . "::_active insert activation constant", LOG_DEBUG);
@@ -950,7 +950,7 @@ 
                     $sql .= ", " . $this->db->encrypt($newvalue);
                     $sql .= ", null";
                     $sql .= ", '0'";
-                    $sql .= ", " . ((int)$entity);
+                    $sql .= ", " . ((int) $entity);
                     $sql .= ")";
 
                     $resql = $this->db->query($sql);
@@ -1064,7 +1064,7 @@ 
                     $sql .= ", " . $this->db->encrypt($newvalue, 1);
                     $sql .= ", null";
                     $sql .= ", '0'";
-                    $sql .= ", " . ((int)$entity);
+                    $sql .= ", " . ((int) $entity);
                     $sql .= ")";
 
                     dol_syslog(get_class($this) . "::insert_module_parts for key=" . $this->const_name . "_" . strtoupper($key), LOG_DEBUG);
@@ -1122,7 +1122,7 @@ 
             $sql = "SELECT count(*) as nb";
             $sql .= " FROM " . $dbPrefix . "const";
             $sql .= " WHERE " . $this->db->decrypt('name') . " = '" . $this->db->escape($name) . "'";
-            $sql .= " AND entity = " . ((int)$entity);
+            $sql .= " AND entity = " . ((int) $entity);
 
             $result = $this->db->query($sql);
             if ($result) {
@@ -1225,7 +1225,7 @@ 
                                 }
 
                                 $sql = "INSERT INTO " . $dbPrefix . "boxes (box_id, position, box_order, fk_user, entity)";
-                                $sql .= " VALUES (" . ((int)$lastid) . ", " . ((int)$key2) . ", '0', 0, " . ((int)$conf->entity) . ")";
+                                $sql .= " VALUES (" . ((int) $lastid) . ", " . ((int) $key2) . ", '0', 0, " . ((int) $conf->entity) . ")";
 
                                 dol_syslog(get_class($this) . "::insert_boxes onto page " . $key2 . "=" . $val2, LOG_DEBUG);
                                 $resql = $this->db->query($sql);
@@ -1311,7 +1311,7 @@ 
                 if ($parameters) {
                     $sql .= " AND params = '".$this->db->escape($parameters)."'";
                 }*/
-                $sql .= " AND entity = " . ((int)$entity); // Must be exact entity
+                $sql .= " AND entity = " . ((int) $entity); // Must be exact entity
 
                 $now = dol_now();
 
@@ -1359,7 +1359,7 @@ 
                                 $sql .= "'" . $this->db->escape($priority) . "', ";
                             }
                             if (is_int($status)) {
-                                $sql .= ((int)$status) . ", ";
+                                $sql .= ((int) $status) . ", ";
                             }
                             $sql .= $entity . ",";
                             $sql .= "'" . $this->db->escape($test) . "'";
@@ -1425,7 +1425,7 @@ 
             if ($obj !== null && !empty($obj->value) && !empty($this->rights)) {
                 // If the module is active
                 foreach ($this->rights as $key => $value) {
-                    $r_id = $this->rights[$key][0];   // permission id in llx_rights_def (not unique because primary key is couple id-entity)
+                    $r_id = $this->rights[$key][0]; // permission id in llx_rights_def (not unique because primary key is couple id-entity)
                     $r_desc = $this->rights[$key][1];
                     $r_type = isset($this->rights[$key][2]) ? $this->rights[$key][2] : '';
                     $r_def = empty($this->rights[$key][3]) ? 0 : $this->rights[$key][3];
@@ -1439,7 +1439,7 @@ 
 
                     // Search if perm already present
                     $sql = "SELECT count(*) as nb FROM " . $dbPrefix . "rights_def";
-                    $sql .= " WHERE id = " . ((int)$r_id) . " AND entity = " . ((int)$entity);
+                    $sql .= " WHERE id = " . ((int) $r_id) . " AND entity = " . ((int) $entity);
 
                     $resqlselect = $this->db->query($sql);
                     if ($resqlselect) {
@@ -1726,7 +1726,7 @@ 
 
             if ($row[0] == 0) {
                 $sql = "INSERT INTO " . $dbPrefix . "const (name, type, value, note, visible, entity)";
-                $sql .= " VALUES (" . $this->db->encrypt($name) . ", 'chaine', " . $this->db->encrypt($dir) . ", '" . $this->db->escape("Directory for module " . $this->name) . "', '0', " . ((int)$conf->entity) . ")";
+                $sql .= " VALUES (" . $this->db->encrypt($name) . ", 'chaine', " . $this->db->encrypt($dir) . ", '" . $this->db->escape("Directory for module " . $this->name) . "', '0', " . ((int) $conf->entity) . ")";
 
                 dol_syslog(get_class($this) . "::insert_dirs", LOG_DEBUG);
                 $this->db->query($sql);
@@ -1819,7 +1819,7 @@ 
         }
 
         // Run complementary sql requests
-        $num = count((array)$array_sql);
+        $num = count((array) $array_sql);
         for ($i = 0; $i < $num; $i++) {
             if (!$err) {
                 dol_syslog(get_class($this) . "::_remove", LOG_DEBUG);
@@ -1929,7 +1929,7 @@ 
 
                 $sql = "DELETE FROM " . $dbPrefix . "const";
                 $sql .= " WHERE " . $this->db->decrypt('name') . " LIKE '" . $this->db->escape($this->const_name) . "_" . strtoupper($key) . "'";
-                $sql .= " AND entity = " . ((int)$entity);
+                $sql .= " AND entity = " . ((int) $entity);
 
                 if (!$this->db->query($sql)) {
                     $this->error = $this->db->lasterror();
@@ -2042,7 +2042,7 @@ 
 
                 $sql = "DELETE FROM " . $dbPrefix . "boxes_def";
                 $sql .= " WHERE file = '" . $this->db->escape($file) . "'";
-                $sql .= " AND entity = " . $conf->entity;     // Do not use getEntity here, we want to delete only in current company
+                $sql .= " AND entity = " . $conf->entity; // Do not use getEntity here, we want to delete only in current company
 
                 dol_syslog(get_class($this) . "::delete_boxes", LOG_DEBUG);
                 $resql = $this->db->query($sql);
@@ -2255,7 +2255,7 @@ 
 
         $return .= '<div class="valignmiddle inline-block info-box-more">';
         //if ($versiontrans) print img_warning($langs->trans("Version").' '.$this->getVersion(1)).' ';
-        $return .= '<a class="valignmiddle inline-block" href="javascript:document_preview(\'' . DOL_URL_ROOT . '/admin/modulehelp.php?id=' . ((int)$this->numero) . '\',\'text/html\',\'' . dol_escape_js($langs->trans("Module")) . '\')">' . img_picto(($this->isCoreOrExternalModule() == 'external' ? $langs->trans("ExternalModule") . ' - ' : '') . $langs->trans("ClickToShowDescription"), $imginfo) . '</a>';
+        $return .= '<a class="valignmiddle inline-block" href="javascript:document_preview(\'' . DOL_URL_ROOT . '/admin/modulehelp.php?id=' . ((int) $this->numero) . '\',\'text/html\',\'' . dol_escape_js($langs->trans("Module")) . '\')">' . img_picto(($this->isCoreOrExternalModule() == 'external' ? $langs->trans("ExternalModule") . ' - ' : '') . $langs->trans("ClickToShowDescription"), $imginfo) . '</a>';
         $return .= '</div><br>';
 
         $return .= '<div class="valignmiddle inline-block info-box-actions">';
@@ -2416,7 +2416,7 @@ 
     {
         require_once BASE_PATH . '/../Dolibarr/Lib/GetUrl.php';
         if (!empty($this->url_last_version)) {
-            $lastVersion = getURLContent($this->url_last_version, 'GET', '', 1, [], ['http', 'https'], 0);    // Accept http or https links on external remote server only
+            $lastVersion = getURLContent($this->url_last_version, 'GET', '', 1, [], ['http', 'https'], 0); // Accept http or https links on external remote server only
             if (isset($lastVersion['content']) && strlen($lastVersion['content']) < 30) {
                 // Security warning :  be careful with remote data content, the module editor could be hacked (or evil) so limit to a-z A-Z 0-9 _ . -
                 $this->lastVersion = preg_replace("/[^a-zA-Z0-9_\.\-]+/", "", $lastVersion['content']);

htdocs/core/modules/modAdherent.class.php

Spacing +2 added lines, -2 removed lines

@@ -456,8 +456,8 @@
         }*/
 
         $sql = array(
-            "DELETE FROM " . MAIN_DB_PREFIX . "document_model WHERE nom = '" . $this->db->escape($this->const[0][2]) . "' AND type='member' AND entity = " . ((int)$conf->entity),
-            "INSERT INTO " . MAIN_DB_PREFIX . "document_model (nom, type, entity) VALUES('" . $this->db->escape($this->const[0][2]) . "','member'," . ((int)$conf->entity) . ")"
+            "DELETE FROM " . MAIN_DB_PREFIX . "document_model WHERE nom = '" . $this->db->escape($this->const[0][2]) . "' AND type='member' AND entity = " . ((int) $conf->entity),
+            "INSERT INTO " . MAIN_DB_PREFIX . "document_model (nom, type, entity) VALUES('" . $this->db->escape($this->const[0][2]) . "','member'," . ((int) $conf->entity) . ")"
         );
 
         return $this->_init($sql, $options);

Dolibarr/Modules/User/Model/User.php

Spacing +110 added lines, -110 removed lines

@@ -439,8 +439,8 @@ 
             // If we ask to add a given permission, we first load properties of this permission (module, perms and subperms).
             $sql = "SELECT module, perms, subperms";
             $sql .= " FROM " . $dbPrefix . "rights_def";
-            $sql .= " WHERE id = " . ((int)$rid);
-            $sql .= " AND entity = " . ((int)$entity);
+            $sql .= " WHERE id = " . ((int) $rid);
+            $sql .= " AND entity = " . ((int) $entity);
 
             $result = $this->db->query($sql);
             if ($result) {
@@ -457,7 +457,7 @@ 
             }
 
             // Define the where for the permission to add
-            $whereforadd = "id=" . ((int)$rid);
+            $whereforadd = "id=" . ((int) $rid);
             // Add also inherited permissions
             if (!empty($subperms)) {
                 $whereforadd .= " OR (module='" . $this->db->escape($module) . "' AND perms='" . $this->db->escape($perms) . "' AND (subperms='lire' OR subperms='read'))";
@@ -486,15 +486,15 @@ 
             //print "$module-$perms-$subperms";
             $sql = "SELECT id";
             $sql .= " FROM " . $dbPrefix . "rights_def";
-            $sql .= " WHERE entity = " . ((int)$entity);
+            $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 " . $dbPrefix . "user_rights";
-            $sqldelete .= " WHERE fk_user = " . ((int)$this->id) . " AND fk_id IN (";
+            $sqldelete .= " WHERE fk_user = " . ((int) $this->id) . " AND fk_id IN (";
             $sqldelete .= $sql;
-            $sqldelete .= ") AND entity = " . ((int)$entity);
+            $sqldelete .= ") AND entity = " . ((int) $entity);
             if (!$this->db->query($sqldelete)) {
                 $error++;
             }
@@ -510,7 +510,7 @@ 
                         if ($obj) {
                             $nid = $obj->id;
 
-                            $sql = "INSERT INTO " . $dbPrefix . "user_rights (entity, fk_user, fk_id) VALUES (" . ((int)$entity) . ", " . ((int)$this->id) . ", " . ((int)$nid) . ")";
+                            $sql = "INSERT INTO " . $dbPrefix . "user_rights (entity, fk_user, fk_id) VALUES (" . ((int) $entity) . ", " . ((int) $this->id) . ", " . ((int) $nid) . ")";
                             if (!$this->db->query($sql)) {
                                 $error++;
                             }
@@ -576,7 +576,7 @@ 
             $sql = "SELECT module, perms, subperms";
             $sql .= " FROM " . $dbPrefix . "rights_def";
             $sql .= " WHERE id = '" . $this->db->escape($rid) . "'";
-            $sql .= " AND entity = " . ((int)$entity);
+            $sql .= " AND entity = " . ((int) $entity);
 
             $result = $this->db->query($sql);
             if ($result) {
@@ -593,7 +593,7 @@ 
             }
 
             // Where clause for the list of permissions to delete
-            $wherefordel = "id=" . ((int)$rid);
+            $wherefordel = "id=" . ((int) $rid);
             // Suppression des droits induits
             if ($subperms == 'lire' || $subperms == 'read') {
                 $wherefordel .= " OR (module='" . $this->db->escape($module) . "' AND perms='" . $this->db->escape($perms) . "' AND subperms IS NOT NULL)";
@@ -622,9 +622,9 @@ 
             //print "$module-$perms-$subperms";
             $sql = "SELECT id";
             $sql .= " FROM " . $dbPrefix . "rights_def";
-            $sql .= " WHERE entity = " . ((int)$entity);
+            $sql .= " WHERE entity = " . ((int) $entity);
             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 can remove his own important rights
@@ -636,10 +636,10 @@ 
             }
 
             $sqldelete = "DELETE FROM " . $dbPrefix . "user_rights";
-            $sqldelete .= " WHERE fk_user = " . ((int)$this->id) . " AND fk_id IN (";
+            $sqldelete .= " WHERE fk_user = " . ((int) $this->id) . " AND fk_id IN (";
             $sqldelete .= $sql;
             $sqldelete .= ")";
-            $sqldelete .= " AND entity = " . ((int)$entity);
+            $sqldelete .= " AND entity = " . ((int) $entity);
 
             $resql = $this->db->query($sqldelete);
             if (!$resql) {
@@ -696,7 +696,7 @@ 
     public function getrights($moduletag = '', $forcereload = 0)
     {
         global $conf;
-        $dbPrefix=$conf->db->prefix;
+        $dbPrefix = $conf->db->prefix;
 
         $alreadyloaded = false;
 
@@ -736,9 +736,9 @@ 
         } else {
             // On table r=rights_def, the unique key is (id, entity) because id is hard coded into module descriptor and insert during module activation.
             // So we must include the filter on entity on both table r. and ur.
-            $sql .= " AND r.entity = " . ((int)$conf->entity) . " AND ur.entity = " . ((int)$conf->entity);
+            $sql .= " AND r.entity = " . ((int) $conf->entity) . " AND ur.entity = " . ((int) $conf->entity);
         }
-        $sql .= " AND ur.fk_user= " . ((int)$this->id);
+        $sql .= " AND ur.fk_user= " . ((int) $this->id);
         $sql .= " AND r.perms IS NOT NULL";
         if (!getDolGlobalString('MAIN_USE_ADVANCED_PERMS')) {
             $sql .= " AND r.perms NOT LIKE '%_advance'"; // Hide advanced perms if option is not enabled
@@ -797,19 +797,19 @@ 
             if (isModEnabled('multicompany') && getDolGlobalString('MULTICOMPANY_TRANSVERSE_MODE')) {
                 $sql .= " AND gu.entity IN (0," . $conf->entity . ")";
             } else {
-                $sql .= " AND r.entity = " . ((int)$conf->entity);
+                $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 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 insert record that make rubbish result due to duplicate record of
             // other entities, so we are forced to add a filter 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";
-        $sql .= " AND gu.fk_user = " . ((int)$this->id);
+        $sql .= " AND gu.fk_user = " . ((int) $this->id);
         $sql .= " AND r.perms IS NOT NULL";
         if ($moduletag) {
             $sql .= " AND r.module = '" . $this->db->escape($moduletag) . "'";
@@ -959,8 +959,8 @@ 
 
         // Save in database
         $sql = "UPDATE " . $dbPrefix . "user";
-        $sql .= " SET statut = " . ((int)$status);
-        $sql .= " WHERE rowid = " . ((int)$this->id);
+        $sql .= " SET statut = " . ((int) $status);
+        $sql .= " WHERE rowid = " . ((int) $this->id);
         $result = $this->db->query($sql);
 
         dol_syslog(get_class($this) . "::setstatus", LOG_DEBUG);
@@ -1025,7 +1025,7 @@ 
         dol_syslog(get_class($this) . "::delete", LOG_DEBUG);
 
         // Remove rights
-        $sql = "DELETE FROM " . $dbPrefix . "user_rights WHERE fk_user = " . ((int)$this->id);
+        $sql = "DELETE FROM " . $dbPrefix . "user_rights WHERE fk_user = " . ((int) $this->id);
 
         if (!$error && !$this->db->query($sql)) {
             $error++;
@@ -1033,14 +1033,14 @@ 
         }
 
         // Remove group
-        $sql = "DELETE FROM " . $dbPrefix . "usergroup_user WHERE fk_user  = " . ((int)$this->id);
+        $sql = "DELETE FROM " . $dbPrefix . "usergroup_user WHERE fk_user  = " . ((int) $this->id);
         if (!$error && !$this->db->query($sql)) {
             $error++;
             $this->error = $this->db->lasterror();
         }
 
         // Remove params
-        $sql = "DELETE FROM " . $dbPrefix . "user_param WHERE fk_user  = " . ((int)$this->id);
+        $sql = "DELETE FROM " . $dbPrefix . "user_param WHERE fk_user  = " . ((int) $this->id);
         if (!$error && !$this->db->query($sql)) {
             $error++;
             $this->error = $this->db->lasterror();
@@ -1048,7 +1048,7 @@ 
 
         // If contact, remove link
         if ($this->contact_id > 0) {
-            $sql = "UPDATE " . $dbPrefix . "socpeople SET fk_user_creat = null WHERE rowid = " . ((int)$this->contact_id);
+            $sql = "UPDATE " . $dbPrefix . "socpeople SET fk_user_creat = null WHERE rowid = " . ((int) $this->contact_id);
             if (!$error && !$this->db->query($sql)) {
                 $error++;
                 $this->error = $this->db->lasterror();
@@ -1066,7 +1066,7 @@ 
 
         // Remove user
         if (!$error) {
-            $sql = "DELETE FROM " . $dbPrefix . "user WHERE rowid = " . ((int)$this->id);
+            $sql = "DELETE FROM " . $dbPrefix . "user WHERE rowid = " . ((int) $this->id);
             dol_syslog(get_class($this) . "::delete", LOG_DEBUG);
             if (!$this->db->query($sql)) {
                 $error++;
@@ -1161,7 +1161,7 @@ 
 
         if ($entity < 0) {
             if ((!isModEnabled('multicompany') || !getDolGlobalString('MULTICOMPANY_TRANSVERSE_MODE')) && (!empty($user->entity))) {
-                $sql .= " WHERE u.entity IN (0, " . ((int)$conf->entity) . ")";
+                $sql .= " WHERE u.entity IN (0, " . ((int) $conf->entity) . ")";
             } else {
                 $sql .= " WHERE u.entity IS NOT NULL"; // multicompany is on in transverse mode or user making fetch is on entity 0, so user is allowed to fetch anywhere into database
             }
@@ -1170,7 +1170,7 @@ 
             if (isModEnabled('multicompany') && getDolGlobalString('MULTICOMPANY_TRANSVERSE_MODE')) {
                 $sql .= " WHERE u.entity IS NOT NULL"; // multicompany is on in transverse mode or user making fetch is on entity 0, so user is allowed to fetch anywhere into database
             } else {
-                $sql .= " WHERE u.entity IN (0, " . ((int)(($entity != '' && $entity >= 0) ? $entity : $conf->entity)) . ")"; // search in entity provided in parameter
+                $sql .= " WHERE u.entity IN (0, " . ((int) (($entity != '' && $entity >= 0) ? $entity : $conf->entity)) . ")"; // search in entity provided in parameter
             }
         }
 
@@ -1186,9 +1186,9 @@ 
             }
             $sql .= ")";
         } elseif ($fk_socpeople > 0) {
-            $sql .= " AND u.fk_socpeople = " . ((int)$fk_socpeople);
+            $sql .= " AND u.fk_socpeople = " . ((int) $fk_socpeople);
         } else {
-            $sql .= " AND u.rowid = " . ((int)$id);
+            $sql .= " AND u.rowid = " . ((int) $id);
         }
         $sql .= " ORDER BY u.entity ASC"; // Avoid random result when there is 2 login in 2 different entities
 
@@ -1252,15 +1252,15 @@ 
                 $this->email = $obj->email;
                 $this->email_oauth2 = $obj->email_oauth2;
                 $this->personal_email = $obj->personal_email;
-                $this->socialnetworks = ($obj->socialnetworks ? (array)json_decode($obj->socialnetworks, true) : []);
+                $this->socialnetworks = ($obj->socialnetworks ? (array) json_decode($obj->socialnetworks, true) : []);
                 $this->job = $obj->job;
                 $this->signature = $obj->signature;
                 $this->admin = $obj->admin;
                 $this->note_public = $obj->note_public;
                 $this->note_private = $obj->note_private;
-                $this->note = $obj->note_private;   // deprecated
+                $this->note = $obj->note_private; // deprecated
 
-                $this->statut = $obj->status;         // deprecated
+                $this->statut = $obj->status; // deprecated
                 $this->status = $obj->status;
 
                 $this->photo = $obj->photo;
@@ -1347,8 +1347,8 @@ 
 
         // Load user->conf for user
         $sql = "SELECT param, value FROM " . $dbPrefix . "user_param";
-        $sql .= " WHERE fk_user = " . ((int)$this->id);
-        $sql .= " AND entity = " . ((int)$conf->entity);
+        $sql .= " WHERE fk_user = " . ((int) $this->id);
+        $sql .= " AND entity = " . ((int) $conf->entity);
         //dol_syslog(get_class($this).'::fetch load personalized conf', LOG_DEBUG);
         $resql = $this->db->query($sql);
         if ($resql) {
@@ -1385,7 +1385,7 @@ 
             // 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) {
@@ -1546,12 +1546,12 @@ 
         $result = $this->create($user, 1);
         if ($result > 0) {
             $sql = "UPDATE " . $dbPrefix . "user";
-            $sql .= " SET fk_socpeople=" . ((int)$contact->id);
+            $sql .= " SET fk_socpeople=" . ((int) $contact->id);
             $sql .= ", civility='" . $this->db->escape($contact->civility_code) . "'";
             if ($contact->socid > 0) {
-                $sql .= ", fk_soc=" . ((int)$contact->socid);
+                $sql .= ", fk_soc=" . ((int) $contact->socid);
             }
-            $sql .= " WHERE rowid=" . ((int)$this->id);
+            $sql .= " WHERE rowid=" . ((int) $this->id);
 
             $resql = $this->db->query($sql);
 
@@ -1606,8 +1606,8 @@ 
         // Clean parameters
         $this->setUpperOrLowerCase();
 
-        $this->civility_code = trim((string)$this->civility_code);
-        $this->login = trim((string)$this->login);
+        $this->civility_code = trim((string) $this->civility_code);
+        $this->login = trim((string) $this->login);
         if (!isset($this->entity)) {
             $this->entity = $conf->entity; // If not defined, we use default value
         }
@@ -1639,7 +1639,7 @@ 
 
         // Check if login already exists in same entity or into entity 0.
         if ($this->login) {
-            $sqltochecklogin = "SELECT COUNT(*) as nb FROM " . $dbPrefix . "user WHERE entity IN (" . $this->db->sanitize((int)$this->entity) . ", 0) AND login = '" . $this->db->escape($this->login) . "'";
+            $sqltochecklogin = "SELECT COUNT(*) as nb FROM " . $dbPrefix . "user WHERE entity IN (" . $this->db->sanitize((int) $this->entity) . ", 0) AND login = '" . $this->db->escape($this->login) . "'";
             $resqltochecklogin = $this->db->query($sqltochecklogin);
             if ($resqltochecklogin) {
                 $objtochecklogin = $this->db->fetch_object($resqltochecklogin);
@@ -1654,7 +1654,7 @@ 
             }
         }
         if (!empty($this->email)) {
-            $sqltochecklogin = "SELECT COUNT(*) as nb FROM " . $dbPrefix . "user WHERE entity IN (" . $this->db->sanitize((int)$this->entity) . ", 0) AND email = '" . $this->db->escape($this->email) . "'";
+            $sqltochecklogin = "SELECT COUNT(*) as nb FROM " . $dbPrefix . "user WHERE entity IN (" . $this->db->sanitize((int) $this->entity) . ", 0) AND email = '" . $this->db->escape($this->email) . "'";
             $resqltochecklogin = $this->db->query($sqltochecklogin);
             if ($resqltochecklogin) {
                 $objtochecklogin = $this->db->fetch_object($resqltochecklogin);
@@ -1671,7 +1671,7 @@ 
 
         // Insert into database
         $sql = "INSERT INTO " . $dbPrefix . "user (datec, login, ldap_sid, entity)";
-        $sql .= " VALUES('" . $this->db->idate($this->datec) . "', '" . $this->db->escape($this->login) . "', '" . $this->db->escape($this->ldap_sid) . "', " . ((int)$this->entity) . ")";
+        $sql .= " VALUES('" . $this->db->idate($this->datec) . "', '" . $this->db->escape($this->login) . "', '" . $this->db->escape($this->ldap_sid) . "', " . ((int) $this->entity) . ")";
         $result = $this->db->query($sql);
 
         dol_syslog(get_class($this) . "::create", LOG_DEBUG);
@@ -1747,7 +1747,7 @@ 
         $num = 0;
         $sql = "SELECT id FROM " . $dbPrefix . "rights_def";
         $sql .= " WHERE bydefault = 1";
-        $sql .= " AND entity = " . ((int)$conf->entity);
+        $sql .= " AND entity = " . ((int) $conf->entity);
 
         $resql = $this->db->query($sql);
         if ($resql) {
@@ -1802,47 +1802,47 @@ 
         dol_syslog(get_class($this) . "::update notrigger=" . $notrigger . ", nosyncmember=" . $nosyncmember . ", nosyncmemberpass=" . $nosyncmemberpass);
 
         // Clean parameters
-        $this->civility_code = trim((string)$this->civility_code);
-        $this->lastname = trim((string)$this->lastname);
-        $this->firstname = trim((string)$this->firstname);
-        $this->ref_employee = trim((string)$this->ref_employee);
-        $this->national_registration_number = trim((string)$this->national_registration_number);
+        $this->civility_code = trim((string) $this->civility_code);
+        $this->lastname = trim((string) $this->lastname);
+        $this->firstname = trim((string) $this->firstname);
+        $this->ref_employee = trim((string) $this->ref_employee);
+        $this->national_registration_number = trim((string) $this->national_registration_number);
         $this->employee = ($this->employee > 0 ? $this->employee : 0);
-        $this->login = trim((string)$this->login);
-        $this->gender = trim((string)$this->gender);
+        $this->login = trim((string) $this->login);
+        $this->gender = trim((string) $this->gender);
 
-        $this->pass = trim((string)$this->pass);
-        $this->api_key = trim((string)$this->api_key);
+        $this->pass = trim((string) $this->pass);
+        $this->api_key = trim((string) $this->api_key);
         $this->datestartvalidity = empty($this->datestartvalidity) ? '' : $this->datestartvalidity;
         $this->dateendvalidity = empty($this->dateendvalidity) ? '' : $this->dateendvalidity;
 
-        $this->address = trim((string)$this->address);
-        $this->zip = trim((string)$this->zip);
-        $this->town = trim((string)$this->town);
+        $this->address = trim((string) $this->address);
+        $this->zip = trim((string) $this->zip);
+        $this->town = trim((string) $this->town);
 
         $this->state_id = ($this->state_id > 0 ? $this->state_id : 0);
         $this->country_id = ($this->country_id > 0 ? $this->country_id : 0);
-        $this->office_phone = trim((string)$this->office_phone);
-        $this->office_fax = trim((string)$this->office_fax);
-        $this->user_mobile = trim((string)$this->user_mobile);
-        $this->personal_mobile = trim((string)$this->personal_mobile);
-        $this->email = trim((string)$this->email);
-        $this->personal_email = trim((string)$this->personal_email);
-
-        $this->job = trim((string)$this->job);
-        $this->signature = trim((string)$this->signature);
-        $this->note_public = trim((string)$this->note_public);
-        $this->note_private = trim((string)$this->note_private);
-        $this->openid = trim((string)$this->openid);
+        $this->office_phone = trim((string) $this->office_phone);
+        $this->office_fax = trim((string) $this->office_fax);
+        $this->user_mobile = trim((string) $this->user_mobile);
+        $this->personal_mobile = trim((string) $this->personal_mobile);
+        $this->email = trim((string) $this->email);
+        $this->personal_email = trim((string) $this->personal_email);
+
+        $this->job = trim((string) $this->job);
+        $this->signature = trim((string) $this->signature);
+        $this->note_public = trim((string) $this->note_public);
+        $this->note_private = trim((string) $this->note_private);
+        $this->openid = trim((string) $this->openid);
         $this->admin = ($this->admin > 0 ? $this->admin : 0);
 
-        $this->accountancy_code = trim((string)$this->accountancy_code);
-        $this->color = trim((string)$this->color);
+        $this->accountancy_code = trim((string) $this->accountancy_code);
+        $this->color = trim((string) $this->color);
         $this->dateemployment = empty($this->dateemployment) ? '' : $this->dateemployment;
         $this->dateemploymentend = empty($this->dateemploymentend) ? '' : $this->dateemploymentend;
 
         $this->birth = empty($this->birth) ? '' : $this->birth;
-        $this->fk_warehouse = (int)$this->fk_warehouse;
+        $this->fk_warehouse = (int) $this->fk_warehouse;
 
         $this->setUpperOrLowerCase();
 
@@ -1868,7 +1868,7 @@ 
 
         // Check if login already exists in same entity or into entity 0.
         if (!empty($this->oldcopy) && $this->oldcopy->login != $this->login) {
-            $sqltochecklogin = "SELECT COUNT(*) as nb FROM " . $dbPrefix . "user WHERE entity IN (" . $this->db->sanitize((int)$this->entity) . ", 0) AND login = '" . $this->db->escape($this->login) . "'";
+            $sqltochecklogin = "SELECT COUNT(*) as nb FROM " . $dbPrefix . "user WHERE entity IN (" . $this->db->sanitize((int) $this->entity) . ", 0) AND login = '" . $this->db->escape($this->login) . "'";
             $resqltochecklogin = $this->db->query($sqltochecklogin);
             if ($resqltochecklogin) {
                 $objtochecklogin = $this->db->fetch_object($resqltochecklogin);
@@ -1882,7 +1882,7 @@ 
             }
         }
         if (!empty($this->oldcopy) && !empty($this->email) && $this->oldcopy->email != $this->email) {
-            $sqltochecklogin = "SELECT COUNT(*) as nb FROM " . $dbPrefix . "user WHERE entity IN (" . $this->db->sanitize((int)$this->entity) . ", 0) AND email = '" . $this->db->escape($this->email) . "'";
+            $sqltochecklogin = "SELECT COUNT(*) as nb FROM " . $dbPrefix . "user WHERE entity IN (" . $this->db->sanitize((int) $this->entity) . ", 0) AND email = '" . $this->db->escape($this->email) . "'";
             $resqltochecklogin = $this->db->query($sqltochecklogin);
             if ($resqltochecklogin) {
                 $objtochecklogin = $this->db->fetch_object($resqltochecklogin);
@@ -1903,13 +1903,13 @@ 
         $sql .= ", firstname = '" . $this->db->escape($this->firstname) . "'";
         $sql .= ", ref_employee = '" . $this->db->escape($this->ref_employee) . "'";
         $sql .= ", national_registration_number = '" . $this->db->escape($this->national_registration_number) . "'";
-        $sql .= ", employee = " . (int)$this->employee;
+        $sql .= ", employee = " . (int) $this->employee;
         $sql .= ", login = '" . $this->db->escape($this->login) . "'";
         $sql .= ", api_key = " . ($this->api_key ? "'" . $this->db->escape(dolEncrypt($this->api_key, '', '', 'dolibarr')) . "'" : "null");
         $sql .= ", gender = " . ($this->gender != -1 ? "'" . $this->db->escape($this->gender) . "'" : "null"); // 'man' or 'woman'
         $sql .= ", birth=" . (strval($this->birth) != '' ? "'" . $this->db->idate($this->birth, 'tzserver') . "'" : 'null');
         if (!empty($user->admin)) {
-            $sql .= ", admin = " . (int)$this->admin; // admin flag can be set/unset only by an admin user
+            $sql .= ", admin = " . (int) $this->admin; // admin flag can be set/unset only by an admin user
         }
         $sql .= ", address = '" . $this->db->escape($this->address) . "'";
         $sql .= ", zip = '" . $this->db->escape($this->zip) . "'";
@@ -1952,13 +1952,13 @@ 
         }
         $sql .= ", weeklyhours= " . ($this->weeklyhours != '' ? "'" . $this->db->escape($this->weeklyhours) . "'" : "null");
         if (!empty($user->admin) && empty($user->entity) && $user->id != $this->id) {
-            $sql .= ", entity = " . ((int)$this->entity); // entity flag can be set/unset only by an another superadmin user
+            $sql .= ", entity = " . ((int) $this->entity); // entity flag can be set/unset only by an another superadmin user
         }
         $sql .= ", default_range = " . ($this->default_range > 0 ? $this->default_range : 'null');
         $sql .= ", default_c_exp_tax_cat = " . ($this->default_c_exp_tax_cat > 0 ? $this->default_c_exp_tax_cat : 'null');
         $sql .= ", fk_warehouse = " . ($this->fk_warehouse > 0 ? $this->fk_warehouse : "null");
         $sql .= ", lang = " . ($this->lang ? "'" . $this->db->escape($this->lang) . "'" : "null");
-        $sql .= " WHERE rowid = " . ((int)$this->id);
+        $sql .= " WHERE rowid = " . ((int) $this->id);
 
         dol_syslog(get_class($this) . "::update", LOG_DEBUG);
         $resql = $this->db->query($sql);
@@ -1979,7 +1979,7 @@ 
             // If user is linked to a member, remove old link to this member
             if ($this->fk_member > 0) {
                 dol_syslog(get_class($this) . "::update remove link with member. We will recreate it later", LOG_DEBUG);
-                $sql = "UPDATE " . $dbPrefix . "user SET fk_member = NULL where fk_member = " . ((int)$this->fk_member);
+                $sql = "UPDATE " . $dbPrefix . "user SET fk_member = NULL where fk_member = " . ((int) $this->fk_member);
                 $resql = $this->db->query($sql);
                 if (!$resql) {
                     $this->error = $this->db->error();
@@ -1989,7 +1989,7 @@ 
             }
             // Set link to user
             dol_syslog(get_class($this) . "::update set link with member", LOG_DEBUG);
-            $sql = "UPDATE " . $dbPrefix . "user SET fk_member =" . ($this->fk_member > 0 ? ((int)$this->fk_member) : 'null') . " where rowid = " . ((int)$this->id);
+            $sql = "UPDATE " . $dbPrefix . "user SET fk_member =" . ($this->fk_member > 0 ? ((int) $this->fk_member) : 'null') . " where rowid = " . ((int) $this->id);
             $resql = $this->db->query($sql);
             if (!$resql) {
                 $this->error = $this->db->error();
@@ -2216,7 +2216,7 @@ 
             } else {
                 $sql .= ", pass = '" . $this->db->escape($password) . "'";
             }
-            $sql .= " WHERE rowid = " . ((int)$this->id);
+            $sql .= " WHERE rowid = " . ((int) $this->id);
 
             dol_syslog(get_class($this) . "::setPassword", LOG_DEBUG);
             $result = $this->db->query($sql);
@@ -2274,7 +2274,7 @@ 
             // After receiving confirmation link, we will erase and store it in pass_crypted
             $sql = "UPDATE " . $dbPrefix . "user";
             $sql .= " SET pass_temp = '" . $this->db->escape($password) . "'";
-            $sql .= " WHERE rowid = " . ((int)$this->id);
+            $sql .= " WHERE rowid = " . ((int) $this->id);
 
             dol_syslog(get_class($this) . "::setPassword", LOG_DEBUG); // No log
             $result = $this->db->query($sql);
@@ -2351,7 +2351,7 @@ 
             } elseif (!empty($this->pass_crypted)) {    // If an encrypted password is already known, we save it directly into database because the previous create did not save it.
                 $sql = "UPDATE " . $dbPrefix . "user";
                 $sql .= " SET pass_crypted = '" . $this->db->escape($this->pass_crypted) . "'";
-                $sql .= " WHERE rowid=" . ((int)$this->id);
+                $sql .= " WHERE rowid=" . ((int) $this->id);
 
                 $resql = $this->db->query($sql);
                 if (!$resql) {
@@ -2361,8 +2361,8 @@ 
 
             if ($result > 0 && $member->socid) {    // If member is linked to a thirdparty
                 $sql = "UPDATE " . $dbPrefix . "user";
-                $sql .= " SET fk_soc=" . ((int)$member->socid);
-                $sql .= " WHERE rowid=" . ((int)$this->id);
+                $sql .= " SET fk_soc=" . ((int) $member->socid);
+                $sql .= " WHERE rowid=" . ((int) $this->id);
 
                 dol_syslog(get_class($this) . "::create_from_member", LOG_DEBUG);
                 $resql = $this->db->query($sql);
@@ -2413,7 +2413,7 @@ 
         $sql .= " datelastlogin = '" . $this->db->idate($now) . "',";
         $sql .= " iplastlogin = '" . $this->db->escape($userremoteip) . "',";
         $sql .= " tms = tms"; // The last update date must change because the last login date is updated
-        $sql .= " WHERE rowid = " . ((int)$this->id);
+        $sql .= " WHERE rowid = " . ((int) $this->id);
 
         dol_syslog(get_class($this) . "::update_last_login_date user->id=" . $this->id . " " . $sql, LOG_DEBUG);
         $resql = $this->db->query($sql);
@@ -2564,7 +2564,7 @@ 
         // phpcs:enable
         $sql = "SELECT url, login, pass, poste ";
         $sql .= " FROM " . $dbPrefix . "user_clicktodial as u";
-        $sql .= " WHERE u.fk_user = " . ((int)$this->id);
+        $sql .= " WHERE u.fk_user = " . ((int) $this->id);
 
         $resql = $this->db->query($sql);
         if ($resql) {
@@ -2603,7 +2603,7 @@ 
         $this->db->begin();
 
         $sql = "DELETE FROM " . $dbPrefix . "user_clicktodial";
-        $sql .= " WHERE fk_user = " . ((int)$this->id);
+        $sql .= " WHERE fk_user = " . ((int) $this->id);
 
         dol_syslog(get_class($this) . '::update_clicktodial', LOG_DEBUG);
         $result = $this->db->query($sql);
@@ -2649,14 +2649,14 @@ 
         $this->db->begin();
 
         $sql = "DELETE FROM " . $dbPrefix . "usergroup_user";
-        $sql .= " WHERE fk_user  = " . ((int)$this->id);
-        $sql .= " AND fk_usergroup = " . ((int)$group);
-        $sql .= " AND entity = " . ((int)$entity);
+        $sql .= " WHERE fk_user  = " . ((int) $this->id);
+        $sql .= " AND fk_usergroup = " . ((int) $group);
+        $sql .= " AND entity = " . ((int) $entity);
 
         $result = $this->db->query($sql);
 
         $sql = "INSERT INTO " . $dbPrefix . "usergroup_user (entity, fk_user, fk_usergroup)";
-        $sql .= " VALUES (" . ((int)$entity) . "," . ((int)$this->id) . "," . ((int)$group) . ")";
+        $sql .= " VALUES (" . ((int) $entity) . "," . ((int) $this->id) . "," . ((int) $group) . ")";
 
         $result = $this->db->query($sql);
         if ($result) {
@@ -2705,12 +2705,12 @@ 
         $this->db->begin();
 
         $sql = "DELETE FROM " . $dbPrefix . "usergroup_user";
-        $sql .= " WHERE fk_user  = " . ((int)$this->id);
-        $sql .= " AND fk_usergroup = " . ((int)$group);
+        $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);
+            $sql .= " AND entity = " . ((int) $entity);
         }
 
         $result = $this->db->query($sql);
@@ -2896,7 +2896,7 @@ 
      */
     public function getLibStatut($mode = 0)
     {
-        return $this->LibStatut(isset($this->statut) ? (int)$this->statut : (int)$this->status, $mode);
+        return $this->LibStatut(isset($this->statut) ? (int) $this->statut : (int) $this->status, $mode);
     }
 
     // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
@@ -3048,7 +3048,7 @@ 
         $result .= (($option == 'nolink') ? '' : $linkstart);
         if ($withpictoimg) {
             $paddafterimage = '';
-            if (abs((int)$withpictoimg) == 1) {
+            if (abs((int) $withpictoimg) == 1) {
                 $paddafterimage = 'style="margin-' . ($langs->trans("DIRECTION") == 'rtl' ? 'left' : 'right') . ': 3px;"';
             }
             // Only picto
@@ -3179,7 +3179,7 @@ 
         if (!empty($this->socid)) {
             if ($module = 'societe' && $permlevel1 == 'client' && $permlevel2 == 'voir') {
                 dump('return 0');
-                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)
             }
         }
 
@@ -3272,7 +3272,7 @@ 
 
         if ($option == 'xxx') {
             $linkstart = '<a href="' . DOL_URL_ROOT . '/user/card.php?id=' . $this->id . '">';
-            $linkend = '</a>';  // @phan-suppress-current-line PhanPluginRedundantAssignment
+            $linkend = '</a>'; // @phan-suppress-current-line PhanPluginRedundantAssignment
         }
 
         if ($option == 'nolink') {
@@ -3589,7 +3589,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;
@@ -3611,7 +3611,7 @@ 
         $sql = "SELECT u.rowid, u.login as ref, u.datec,";
         $sql .= " u.tms as date_modification, u.entity";
         $sql .= " FROM " . $dbPrefix . "user as u";
-        $sql .= " WHERE u.rowid = " . ((int)$id);
+        $sql .= " WHERE u.rowid = " . ((int) $id);
 
         $result = $this->db->query($sql);
         if ($result) {
@@ -3684,13 +3684,13 @@ 
             }
         }
         if ($admin >= 0) {
-            $sql .= " AND admin = " . (int)$admin;
+            $sql .= " AND admin = " . (int) $admin;
         }
 
         $resql = $this->db->query($sql);
         if ($resql) {
             $obj = $this->db->fetch_object($resql);
-            $nb = (int)$obj->nb;
+            $nb = (int) $obj->nb;
 
             $this->db->free($resql);
             return $nb;
@@ -3765,7 +3765,7 @@ 
     {
         // phpcs:enable
         $sql = "SELECT rowid FROM " . $dbPrefix . "user";
-        $sql .= " WHERE fk_user = " . ((int)$this->id);
+        $sql .= " WHERE fk_user = " . ((int) $this->id);
 
         dol_syslog(get_class($this) . "::get_children", LOG_DEBUG);
         $res = $this->db->query($sql);
@@ -4102,7 +4102,7 @@ 
 
         $sql = "SELECT rowid, email, user_mobile, civility, lastname, firstname";
         $sql .= " FROM " . $dbPrefix . "user";
-        $sql .= " WHERE rowid = " . ((int)$rowid);
+        $sql .= " WHERE rowid = " . ((int) $rowid);
 
         $resql = $this->db->query($sql);
         if ($resql) {
@@ -4142,7 +4142,7 @@ 
 
         $encodedsecurekey = dol_hash($conf->file->instance_unique_id . 'uservirtualcard' . $this->id . '-' . $this->login, 'md5');
         if (isModEnabled('multicompany')) {
-            $entity_qr = '&entity=' . ((int)$conf->entity);
+            $entity_qr = '&entity=' . ((int) $conf->entity);
         } else {
             $entity_qr = '';
         }
@@ -4196,7 +4196,7 @@ 
             return -1;
         }
 
-        $this->findUserIdByEmailCache[$email] = (int)$obj->rowid;
+        $this->findUserIdByEmailCache[$email] = (int) $obj->rowid;
 
         return $this->findUserIdByEmailCache[$email];
     }

Dolibarr/Lib/Admin.php

Spacing +13 added lines, -13 removed lines

@@ -325,8 +325,8 @@ 
             // with
             // 'INSERT INTO llx_accounting_account (entity, rowid, fk_pcg_version, pcg_type, account_number, account_parent, label, active) VALUES (__ENTITY__, 1401 + 200100000, 'PCG99-ABREGE','CAPIT', '1234', 1400 + 200100000,...'
             // Note: string with 'PCG99-ABREGE','CAPIT', 1234  instead of  'PCG99-ABREGE','CAPIT', '1234' is also supported
-            $newsql = preg_replace('/VALUES\s*\(__ENTITY__, \s*(\d+)\s*,(\s*\'[^\',]*\'\s*,\s*\'[^\',]*\'\s*,\s*\'?[^\',]*\'?\s*),\s*\'?([^\',]*)\'?/ims', 'VALUES (__ENTITY__, \1 + ' . ((int)$offsetforchartofaccount) . ', \2, \3 + ' . ((int)$offsetforchartofaccount), $newsql);
-            $newsql = preg_replace('/([,\s])0 \+ ' . ((int)$offsetforchartofaccount) . '/ims', '\1 0', $newsql);
+            $newsql = preg_replace('/VALUES\s*\(__ENTITY__, \s*(\d+)\s*,(\s*\'[^\',]*\'\s*,\s*\'[^\',]*\'\s*,\s*\'?[^\',]*\'?\s*),\s*\'?([^\',]*)\'?/ims', 'VALUES (__ENTITY__, \1 + ' . ((int) $offsetforchartofaccount) . ', \2, \3 + ' . ((int) $offsetforchartofaccount), $newsql);
+            $newsql = preg_replace('/([,\s])0 \+ ' . ((int) $offsetforchartofaccount) . '/ims', '\1 0', $newsql);
             //var_dump($newsql);
             $arraysql[$i] = $newsql;
 
@@ -488,7 +488,7 @@ 
                 }
 
                 // Define list of errors we accept (array $okerrors)
-                $okerrors = [  // By default
+                $okerrors = [// By default
                     'DB_ERROR_TABLE_ALREADY_EXISTS',
                     'DB_ERROR_COLUMN_ALREADY_EXISTS',
                     'DB_ERROR_KEY_NAME_ALREADY_EXISTS',
@@ -557,7 +557,7 @@ 
     if ($error == 0) {
         $ok = 1;
     } else {
-        $ok = 0;  // @phan-suppress-current-line PhanPluginRedundantAssignment
+        $ok = 0; // @phan-suppress-current-line PhanPluginRedundantAssignment
     }
 
     return $ok;
@@ -587,11 +587,11 @@ 
     $sql = "DELETE FROM " . MAIN_DB_PREFIX . "const";
     $sql .= " WHERE (" . $db->decrypt('name') . " = '" . $db->escape($name) . "'";
     if (is_numeric($name)) {
-        $sql .= " OR rowid = " . ((int)$name);
+        $sql .= " OR rowid = " . ((int) $name);
     }
     $sql .= ")";
     if ($entity >= 0) {
-        $sql .= " AND entity = " . ((int)$entity);
+        $sql .= " AND entity = " . ((int) $entity);
     }
 
     dol_syslog("admin.lib::dolibarr_del_const", LOG_DEBUG);
@@ -623,7 +623,7 @@ 
     $sql = "SELECT " . $db->decrypt('value') . " as value";
     $sql .= " FROM " . MAIN_DB_PREFIX . "const";
     $sql .= " WHERE name = " . $db->encrypt($name);
-    $sql .= " AND entity = " . ((int)$entity);
+    $sql .= " AND entity = " . ((int) $entity);
 
     dol_syslog("admin.lib::dolibarr_get_const", LOG_DEBUG);
     $resql = $db->query($sql);
@@ -674,7 +674,7 @@ 
     $sql = "DELETE FROM " . MAIN_DB_PREFIX . "const";
     $sql .= " WHERE name = " . $db->encrypt($name);
     if ($entity >= 0) {
-        $sql .= " AND entity = " . ((int)$entity);
+        $sql .= " AND entity = " . ((int) $entity);
     }
 
     dol_syslog("admin.lib::dolibarr_set_const", LOG_DEBUG);
@@ -696,7 +696,7 @@ 
         $sql .= " VALUES (";
         $sql .= $db->encrypt($name);
         $sql .= ", " . $db->encrypt($newvalue);
-        $sql .= ", '" . $db->escape($type) . "', " . ((int)$visible) . ", '" . $db->escape($note) . "', " . ((int)$entity) . ")";
+        $sql .= ", '" . $db->escape($type) . "', " . ((int) $visible) . ", '" . $db->escape($note) . "', " . ((int) $entity) . ")";
 
         //print "sql".$value."-".pg_escape_string($value)."-".$sql;exit;
         //print "xx".$db->escape($value);
@@ -859,7 +859,7 @@ 
     $sql = "SELECT COUNT(r.id) as nb";
     $sql .= " FROM " . MAIN_DB_PREFIX . "rights_def as r";
     $sql .= " WHERE r.libelle NOT LIKE 'tou%'"; // On ignore droits "tous"
-    $sql .= " AND entity = " . ((int)$conf->entity);
+    $sql .= " AND entity = " . ((int) $conf->entity);
     $sql .= " AND bydefault = 1";
     if (!getDolGlobalString('MAIN_USE_ADVANCED_PERMS')) {
         $sql .= " AND r.perms NOT LIKE '%_advance'"; // Hide advanced perms if option is not enabled
@@ -1755,7 +1755,7 @@ 
             $obj = $db->fetch_object($result); // Take first result of select
 
             if (empty($obj)) {  // If not yet into table
-                $obj = (object)['rowid' => '', 'name' => $const, 'value' => '', 'type' => $type, 'note' => ''];
+                $obj = (object) ['rowid' => '', 'name' => $const, 'value' => '', 'type' => $type, 'note' => ''];
             }
 
             if (empty($strictw3c)) {
@@ -1956,7 +1956,7 @@ 
     $db->begin();
 
     $sql = "INSERT INTO " . MAIN_DB_PREFIX . "document_model (nom, type, entity, libelle, description)";
-    $sql .= " VALUES ('" . $db->escape($name) . "','" . $db->escape($type) . "'," . ((int)$conf->entity) . ", ";
+    $sql .= " VALUES ('" . $db->escape($name) . "','" . $db->escape($type) . "'," . ((int) $conf->entity) . ", ";
     $sql .= ($label ? "'" . $db->escape($label) . "'" : 'null') . ", ";
     $sql .= (!empty($description) ? "'" . $db->escape($description) . "'" : "null");
     $sql .= ")";
@@ -1990,7 +1990,7 @@ 
     $sql = "DELETE FROM " . MAIN_DB_PREFIX . "document_model";
     $sql .= " WHERE nom = '" . $db->escape($name) . "'";
     $sql .= " AND type = '" . $db->escape($type) . "'";
-    $sql .= " AND entity = " . ((int)$conf->entity);
+    $sql .= " AND entity = " . ((int) $conf->entity);
 
     dol_syslog("admin.lib::delDocumentModel", LOG_DEBUG);
     $resql = $db->query($sql);