alxarafe / alixar

public/htdocs/core/lib/functions.lib.php

Spacing +132 added lines, -132 removed lines

@@ -168,7 +168,7 @@ 
 function getDolGlobalString($key, $default = '')
 {
     global $conf;
-    return (string)(isset($conf->global->$key) ? $conf->global->$key : $default);
+    return (string) (isset($conf->global->$key) ? $conf->global->$key : $default);
 }
 
 /**
@@ -183,7 +183,7 @@ 
 function getDolGlobalInt($key, $default = 0)
 {
     global $conf;
-    return (int)(isset($conf->global->$key) ? $conf->global->$key : $default);
+    return (int) (isset($conf->global->$key) ? $conf->global->$key : $default);
 }
 
 /**
@@ -201,7 +201,7 @@ 
         $tmpuser = $user;
     }
 
-    return (string)(empty($tmpuser->conf->$key) ? $default : $tmpuser->conf->$key);
+    return (string) (empty($tmpuser->conf->$key) ? $default : $tmpuser->conf->$key);
 }
 
 /**
@@ -219,7 +219,7 @@ 
         $tmpuser = $user;
     }
 
-    return (int)(empty($tmpuser->conf->$key) ? $default : $tmpuser->conf->$key);
+    return (int) (empty($tmpuser->conf->$key) ? $default : $tmpuser->conf->$key);
 }
 
 
@@ -236,23 +236,23 @@ 
     'MODULE_MAPPING',
     array(
         // Map deprecated names to new names
-        'adherent' => 'member',  // Has new directory
-        'member_type' => 'adherent_type',   // No directory, but file called adherent_type
-        'banque' => 'bank',   // Has new directory
+        'adherent' => 'member', // Has new directory
+        'member_type' => 'adherent_type', // No directory, but file called adherent_type
+        'banque' => 'bank', // Has new directory
         'contrat' => 'contract', // Has new directory
-        'entrepot' => 'stock',   // Has new directory
+        'entrepot' => 'stock', // Has new directory
         'projet' => 'project', // Has new directory
         'categorie' => 'category', // Has old directory
-        'commande' => 'order',    // Has old directory
+        'commande' => 'order', // Has old directory
         'expedition' => 'shipping', // Has old directory
         'facture' => 'invoice', // Has old directory
         'fichinter' => 'intervention', // Has old directory
-        'ficheinter' => 'intervention',  // Backup for 'fichinter'
+        'ficheinter' => 'intervention', // Backup for 'fichinter'
         'propale' => 'propal', // Has old directory
         'socpeople' => 'contact', // Has old directory
-        'fournisseur' => 'supplier',  // Has old directory
+        'fournisseur' => 'supplier', // Has old directory
 
-        'actioncomm' => 'agenda',  // NO module directory (public dir agenda)
+        'actioncomm' => 'agenda', // NO module directory (public dir agenda)
         'product_price' => 'productprice', // NO directory
         'product_fournisseur_price' => 'productsupplierprice', // NO directory
     )
@@ -390,7 +390,7 @@ 
         if (in_array($element, $addzero)) {
             $out .= '0,';
         }
-        $out .= ((int)$conf->entity);
+        $out .= ((int) $conf->entity);
     }
 
     // Manipulate entities to query on the fly
@@ -483,7 +483,7 @@ 
     $os = 'unknown';
     $phone = '';
 
-    $user_agent = substr($user_agent, 0, 512);  // Avoid to process too large user agent
+    $user_agent = substr($user_agent, 0, 512); // Avoid to process too large user agent
 
     $detectmobile = new Mobile_Detect(null, $user_agent);
     $tablet = $detectmobile->isTablet();
@@ -572,8 +572,8 @@ 
         'browserversion' => $version,
         'browseros' => $os,
         'browserua' => $user_agent,
-        'layout' => $layout,    // tablet, phone, classic
-        'phone' => $phone,      // deprecated
+        'layout' => $layout, // tablet, phone, classic
+        'phone' => $phone, // deprecated
         'tablet' => $tablet     // deprecated
     );
 }
@@ -989,13 +989,13 @@ 
     // Sanitizing for special parameters.
     // Note: There is no reason to allow the backtopage, backtolist or backtourl parameter to contains an external URL. Only relative URLs are allowed.
     if ($paramname == 'backtopage' || $paramname == 'backtolist' || $paramname == 'backtourl') {
-        $out = str_replace('\\', '/', $out);                                // Can be before the loop because only 1 char is replaced. No risk to get it after other replacements.
-        $out = str_replace(array(':', ';', '@', "\t", ' '), '', $out);      // Can be before the loop because only 1 char is replaced. No risk to retrieve it after other replacements.
+        $out = str_replace('\\', '/', $out); // Can be before the loop because only 1 char is replaced. No risk to get it after other replacements.
+        $out = str_replace(array(':', ';', '@', "\t", ' '), '', $out); // Can be before the loop because only 1 char is replaced. No risk to retrieve it after other replacements.
         do {
             $oldstringtoclean = $out;
             $out = str_ireplace(array('javascript', 'vbscript', '&colon', '&#'), '', $out);
-            $out = preg_replace(array('/^[^\?]*%/'), '', $out);             // We remove any % chars before the ?. Example in url: '/product/stock/card.php?action=create&backtopage=%2Fdolibarr_dev%2Fhtdocs%2Fpro%25duct%2Fcard.php%3Fid%3Dabc'
-            $out = preg_replace(array('/^[a-z]*\/\s*\/+/i'), '', $out);     // We remove schema*// to remove external URL
+            $out = preg_replace(array('/^[^\?]*%/'), '', $out); // We remove any % chars before the ?. Example in url: '/product/stock/card.php?action=create&backtopage=%2Fdolibarr_dev%2Fhtdocs%2Fpro%25duct%2Fcard.php%3Fid%3Dabc'
+            $out = preg_replace(array('/^[a-z]*\/\s*\/+/i'), '', $out); // We remove schema*// to remove external URL
         } while ($oldstringtoclean != $out);
     }
 
@@ -1029,7 +1029,7 @@ 
  */
 function GETPOSTINT($paramname, $method = 0)
 {
-    return (int)GETPOST($paramname, 'int', $method, null, null, 0);
+    return (int) GETPOST($paramname, 'int', $method, null, null, 0);
 }
 
 
@@ -1044,7 +1044,7 @@ 
 function GETPOSTFLOAT($paramname, $rounding = '')
 {
     // price2num() is used to sanitize any valid user input (such as "1 234.5", "1 234,5", "1'234,5", "1·234,5", "1,234.5", etc.)
-    return (float)price2num(GETPOST($paramname), $rounding, 2);
+    return (float) price2num(GETPOST($paramname), $rounding, 2);
 }
 
 
@@ -1235,7 +1235,7 @@ 
         }
 
         // If prefix is for session (no need to have $conf loaded)
-        global $dolibarr_main_instance_unique_id, $dolibarr_main_cookie_cryptkey;   // This is loaded by filefunc.inc.php
+        global $dolibarr_main_instance_unique_id, $dolibarr_main_cookie_cryptkey; // This is loaded by filefunc.inc.php
         $tmp_instance_unique_id = empty($dolibarr_main_instance_unique_id) ? (empty($dolibarr_main_cookie_cryptkey) ? '' : $dolibarr_main_cookie_cryptkey) : $dolibarr_main_instance_unique_id; // Unique id of instance
 
         // The recommended value (may be not defined for old versions)
@@ -1437,7 +1437,7 @@ 
         $tmpsavdb = null;
         if (isset($object->db) && isset($object->db->db) && is_object($object->db->db) && get_class($object->db->db) == 'PgSql\Connection') {
             $tmpsavdb = $object->db;
-            unset($object->db);     // Such property can not be serialized with pgsl (when object->db->db = 'PgSql\Connection')
+            unset($object->db); // Such property can not be serialized with pgsl (when object->db->db = 'PgSql\Connection')
         }
 
         $myclone = unserialize(serialize($object)); // serialize then unserialize is a hack to be sure to have a new object for all fields
@@ -1448,7 +1448,7 @@ 
     } elseif ($native == 2) {
         // recommended method to have a full isolated cloned object
         $myclone = new stdClass();
-        $tmparray = get_object_vars($object);   // return only public properties
+        $tmparray = get_object_vars($object); // return only public properties
 
         if (is_array($tmparray)) {
             foreach ($tmparray as $propertykey => $propertyval) {
@@ -1893,9 +1893,9 @@ 
 
     // escape quotes and backslashes, newlines, etc.
     if ($escapeonlyhtmltags) {
-        $tmp = htmlspecialchars_decode((string)$stringtoescape, ENT_COMPAT);
+        $tmp = htmlspecialchars_decode((string) $stringtoescape, ENT_COMPAT);
     } else {
-        $tmp = html_entity_decode((string)$stringtoescape, ENT_COMPAT, 'UTF-8');
+        $tmp = html_entity_decode((string) $stringtoescape, ENT_COMPAT, 'UTF-8');
     }
     if (!$keepb) {
         $tmp = strtr($tmp, array("<b>" => '', '</b>' => '', '<strong>' => '', '</strong>' => ''));
@@ -1925,7 +1925,7 @@ 
                 // For case of tag with attribute
                 $reg = array();
                 if (preg_match('/<' . preg_quote($tagtoreplace, '/') . '\s+([^>]+)>/', $tmp, $reg)) {
-                    $tmpattributes = str_ireplace(array('[', ']'), '_', $reg[1]);   // We must never have [ ] inside the attribute string
+                    $tmpattributes = str_ireplace(array('[', ']'), '_', $reg[1]); // We must never have [ ] inside the attribute string
                     $tmpattributes = str_ireplace('href="http:', '__HREFHTTPA', $tmpattributes);
                     $tmpattributes = str_ireplace('href="https:', '__HREFHTTPSA', $tmpattributes);
                     $tmpattributes = str_ireplace('src="http:', '__SRCHTTPIMG', $tmpattributes);
@@ -1935,7 +1935,7 @@ 
                     $tmp = preg_replace('/<' . preg_quote($tagtoreplace, '/') . '\s+([^>]+)>/', '__BEGINTAGTOREPLACE' . $tagtoreplace . '[' . $tmpattributes . ']__', $tmp);
                 }
                 if (preg_match('/<' . preg_quote($tagtoreplace, '/') . '\s+([^>]+)> \/>/', $tmp, $reg)) {
-                    $tmpattributes = str_ireplace(array('[', ']'), '_', $reg[1]);   // We must not have [ ] inside the attribute string
+                    $tmpattributes = str_ireplace(array('[', ']'), '_', $reg[1]); // We must not have [ ] inside the attribute string
                     $tmpattributes = str_ireplace('"', '__DOUBLEQUOTE', $tmpattributes);
                     $tmpattributes = preg_replace('/[^a-z0-9_\/\?\;\s=&]/i', '', $tmpattributes);
                     $tmp = preg_replace('/<' . preg_quote($tagtoreplace, '/') . '\s+([^>]+) \/>/', '__BEGINENDTAGTOREPLACE' . $tagtoreplace . '[' . $tmpattributes . ']__', $tmp);
@@ -2378,7 +2378,7 @@ 
                 $to_gmt = false;
                 $offsettzstring = @date_default_timezone_get(); // Example 'Europe/Berlin' or 'Indian/Reunion'
                 // @phan-suppress-next-line PhanPluginRedundantAssignment
-                $offsettz = 0;  // Timezone offset with server timezone (because to_gmt is false), so 0
+                $offsettz = 0; // Timezone offset with server timezone (because to_gmt is false), so 0
                 // @phan-suppress-next-line PhanPluginRedundantAssignment
                 $offsetdst = 0; // Dst offset with server timezone (because to_gmt is false), so 0
             } elseif ($tzoutput == 'tzuser' || $tzoutput == 'tzuserrel') {
@@ -2389,8 +2389,8 @@ 
                     $user_date_tz = new DateTimeZone($offsettzstring);
                     $user_dt = new DateTime();
                     $user_dt->setTimezone($user_date_tz);
-                    $user_dt->setTimestamp($tzoutput == 'tzuser' ? dol_now() : (int)$time);
-                    $offsettz = $user_dt->getOffset();  // should include dst ?
+                    $user_dt->setTimestamp($tzoutput == 'tzuser' ? dol_now() : (int) $time);
+                    $offsettz = $user_dt->getOffset(); // should include dst ?
                 } else {    // with old method (The 'tzuser' was processed like the 'tzuserrel')
                     $offsettz = (empty($_SESSION['dol_tz']) ? 0 : $_SESSION['dol_tz']) * 60 * 60; // Will not be used anymore
                     $offsetdst = (empty($_SESSION['dol_dst']) ? 0 : $_SESSION['dol_dst']) * 60 * 60; // Will not be used anymore
@@ -2407,7 +2407,7 @@ 
 
     // Do we have to reduce the length of date (year on 2 chars) to save space.
     // Note: dayinputnoreduce is same than day but no reduction of year length will be done
-    $reduceformat = (!empty($conf->dol_optimize_smallscreen) && in_array($format, array('day', 'dayhour', 'dayhoursec'))) ? 1 : 0;  // Test on original $format param.
+    $reduceformat = (!empty($conf->dol_optimize_smallscreen) && in_array($format, array('day', 'dayhour', 'dayhoursec'))) ? 1 : 0; // Test on original $format param.
     $format = preg_replace('/inputnoreduce/', '', $format); // so format 'dayinputnoreduce' is processed like day
     $formatwithoutreduce = preg_replace('/reduceformat/', '', $format);
     if ($formatwithoutreduce != $format) {
@@ -2474,10 +2474,10 @@ 
 
     // Analyze date
     $reg = array();
-    if (preg_match('/^([0-9][0-9][0-9][0-9])([0-9][0-9])([0-9][0-9])([0-9][0-9])([0-9][0-9])([0-9][0-9])$/i', (string)$time, $reg)) {  // Deprecated. Ex: 1970-01-01, 1970-01-01 01:00:00, 19700101010000
+    if (preg_match('/^([0-9][0-9][0-9][0-9])([0-9][0-9])([0-9][0-9])([0-9][0-9])([0-9][0-9])([0-9][0-9])$/i', (string) $time, $reg)) {  // Deprecated. Ex: 1970-01-01, 1970-01-01 01:00:00, 19700101010000
         dol_print_error(null, "Functions.lib::dol_print_date function called with a bad value from page " . (empty($_SERVER["PHP_SELF"]) ? 'unknown' : $_SERVER["PHP_SELF"]));
         return '';
-    } elseif (preg_match('/^([0-9]+)\-([0-9]+)\-([0-9]+) ?([0-9]+)?:?([0-9]+)?:?([0-9]+)?/i', (string)$time, $reg)) {    // Still available to solve problems in extrafields of type date
+    } elseif (preg_match('/^([0-9]+)\-([0-9]+)\-([0-9]+) ?([0-9]+)?:?([0-9]+)?:?([0-9]+)?/i', (string) $time, $reg)) {    // Still available to solve problems in extrafields of type date
         // This part of code should not be used anymore.
         dol_syslog("Functions.lib::dol_print_date function called with a bad value from page " . (empty($_SERVER["PHP_SELF"]) ? 'unknown' : $_SERVER["PHP_SELF"]), LOG_WARNING);
         //if (function_exists('debug_print_backtrace')) debug_print_backtrace();
@@ -2494,7 +2494,7 @@ 
         if ($to_gmt) {
             $tzo = new DateTimeZone('UTC'); // when to_gmt is true, base for offsettz and offsetdst (so timetouse) is UTC
         } else {
-            $tzo = new DateTimeZone(date_default_timezone_get());   // when to_gmt is false, base for offsettz and offsetdst (so timetouse) is PHP server
+            $tzo = new DateTimeZone(date_default_timezone_get()); // when to_gmt is false, base for offsettz and offsetdst (so timetouse) is PHP server
         }
         $dtts = new DateTime();
         $dtts->setTimestamp($time);
@@ -2518,7 +2518,7 @@ 
             if ($to_gmt) {
                 $tzo = new DateTimeZone('UTC'); // when to_gmt is true, base for offsettz and offsetdst (so timetouse) is UTC
             } else {
-                $tzo = new DateTimeZone(date_default_timezone_get());   // when to_gmt is false, base for offsettz and offsetdst (so timetouse) is PHP server
+                $tzo = new DateTimeZone(date_default_timezone_get()); // when to_gmt is false, base for offsettz and offsetdst (so timetouse) is PHP server
             }
             $dtts = new DateTime();
             $dtts->setTimestamp($timetouse);
@@ -2546,12 +2546,12 @@ 
         if ($to_gmt) {
             $tzo = new DateTimeZone('UTC'); // when to_gmt is true, base for offsettz and offsetdst (so timetouse) is UTC
         } else {
-            $tzo = new DateTimeZone(date_default_timezone_get());   // when to_gmt is false, base for offsettz and offsetdst (so timetouse) is PHP server
+            $tzo = new DateTimeZone(date_default_timezone_get()); // when to_gmt is false, base for offsettz and offsetdst (so timetouse) is PHP server
         }
         $dtts = new DateTime();
         $dtts->setTimestamp($timetouse);
         $dtts->setTimezone($tzo);
-        $month = (int)$dtts->format("m");
+        $month = (int) $dtts->format("m");
         $month = sprintf("%02d", $month); // $month may be return with format '06' on some installation and '6' on other, so we force it to '06'.
         if ($encodetooutput) {
             $monthtext = $outputlangs->transnoentities('Month' . $month);
@@ -2621,14 +2621,14 @@ 
         $datetimeobj->setTimezone(new DateTimeZone($forcetimezone == 'gmt' ? 'UTC' : $forcetimezone)); //  (add timezone relative to the date entered)
     }
     $arrayinfo = array(
-        'year' => ((int)date_format($datetimeobj, 'Y')),
-        'mon' => ((int)date_format($datetimeobj, 'm')),
-        'mday' => ((int)date_format($datetimeobj, 'd')),
-        'wday' => ((int)date_format($datetimeobj, 'w')),
-        'yday' => ((int)date_format($datetimeobj, 'z')),
-        'hours' => ((int)date_format($datetimeobj, 'H')),
-        'minutes' => ((int)date_format($datetimeobj, 'i')),
-        'seconds' => ((int)date_format($datetimeobj, 's')),
+        'year' => ((int) date_format($datetimeobj, 'Y')),
+        'mon' => ((int) date_format($datetimeobj, 'm')),
+        'mday' => ((int) date_format($datetimeobj, 'd')),
+        'wday' => ((int) date_format($datetimeobj, 'w')),
+        'yday' => ((int) date_format($datetimeobj, 'z')),
+        'hours' => ((int) date_format($datetimeobj, 'H')),
+        'minutes' => ((int) date_format($datetimeobj, 'i')),
+        'seconds' => ((int) date_format($datetimeobj, 's')),
         '0' => $timestamp
     );
 
@@ -2726,8 +2726,8 @@ 
     //var_dump($localtz);
     //var_dump($year.'-'.$month.'-'.$day.'-'.$hour.'-'.$minute);
     $dt = new DateTime('now', $localtz);
-    $dt->setDate((int)$year, (int)$month, (int)$day);
-    $dt->setTime((int)$hour, (int)$minute, (int)$second);
+    $dt->setDate((int) $year, (int) $month, (int) $day);
+    $dt->setTime((int) $hour, (int) $minute, (int) $second);
     $date = $dt->getTimestamp(); // should include daylight saving time
     //var_dump($date);
     return $date;
@@ -2757,7 +2757,7 @@ 
     } elseif ($mode == 'tzserver') {        // Time for now with PHP server timezone added
         require_once constant('DOL_DOCUMENT_ROOT') . '/core/lib/date.lib.php';
         $tzsecond = getServerTimeZoneInt('now'); // Contains tz+dayling saving time
-        $ret = (int)(dol_now('gmt') + ($tzsecond * 3600));
+        $ret = (int) (dol_now('gmt') + ($tzsecond * 3600));
         //} elseif ($mode == 'tzref') {// Time for now with parent company timezone is added
         //  require_once DOL_DOCUMENT_ROOT.'/core/lib/date.lib.php';
         //  $tzsecond=getParentCompanyTimeZoneInt();    // Contains tz+dayling saving time
@@ -2768,7 +2768,7 @@ 
         //print 'time: '.time();
         $offsettz = (empty($_SESSION['dol_tz']) ? 0 : $_SESSION['dol_tz']) * 60 * 60;
         $offsetdst = (empty($_SESSION['dol_dst']) ? 0 : $_SESSION['dol_dst']) * 60 * 60;
-        $ret = (int)(dol_now('gmt') + ($offsettz + $offsetdst));
+        $ret = (int) (dol_now('gmt') + ($offsettz + $offsetdst));
     }
 
     return $ret;
@@ -2945,9 +2945,9 @@ 
     if (empty($_SERVER['HTTP_X_FORWARDED_FOR']) || preg_match('/[^0-9\.\:,\[\]]/', $_SERVER['HTTP_X_FORWARDED_FOR'])) {
         if (empty($_SERVER['HTTP_CLIENT_IP']) || preg_match('/[^0-9\.\:,\[\]]/', $_SERVER['HTTP_CLIENT_IP'])) {
             if (empty($_SERVER["HTTP_CF_CONNECTING_IP"])) {
-                $ip = (empty($_SERVER['REMOTE_ADDR']) ? '' : $_SERVER['REMOTE_ADDR']);  // value may have been the IP of the proxy and not the client
+                $ip = (empty($_SERVER['REMOTE_ADDR']) ? '' : $_SERVER['REMOTE_ADDR']); // value may have been the IP of the proxy and not the client
             } else {
-                $ip = $_SERVER["HTTP_CF_CONNECTING_IP"];    // value here may have been forged by client
+                $ip = $_SERVER["HTTP_CF_CONNECTING_IP"]; // value here may have been forged by client
             }
         } else {
             $ip = $_SERVER['HTTP_CLIENT_IP']; // value is clean here but may have been forged by proxy
@@ -3147,7 +3147,7 @@ 
             if (count($mxhosts) > 1) {
                 return 1;
             }
-            if (count($mxhosts) == 1 && !in_array((string)$mxhosts[0], array('', '.'))) {
+            if (count($mxhosts) == 1 && !in_array((string) $mxhosts[0], array('', '.'))) {
                 return 1;
             }
 
@@ -4173,7 +4173,7 @@ 
     }
     // Si on depasse max
     $max_nbdecimal = getDolGlobalString('MAIN_MAX_DECIMALS_SHOWN');
-    if ($trunc && $nbdecimal > (int)$max_nbdecimal) {
+    if ($trunc && $nbdecimal > (int) $max_nbdecimal) {
         $nbdecimal = $max_nbdecimal;
         if (preg_match('/\.\.\./i', $nbdecimal)) {
             // Si un affichage est tronque, on montre des ...
@@ -4182,7 +4182,7 @@ 
     }
 
     // If force rounding
-    if ((string)$forcerounding != '-1') {
+    if ((string) $forcerounding != '-1') {
         if ($forcerounding === 'MU') {
             $nbdecimal = getDolGlobalString('MAIN_MAX_DECIMALS_UNIT');
         } elseif ($forcerounding === 'MT') {
@@ -4193,7 +4193,7 @@ 
     }
 
     // Format number
-    $output = number_format((float)$amount, $nbdecimal, $dec, $thousand);
+    $output = number_format((float) $amount, $nbdecimal, $dec, $thousand);
     if ($form) {
         $output = preg_replace('/\s/', '&nbsp;', $output);
         $output = preg_replace('/\'/', '&#039;', $output);
@@ -4282,7 +4282,7 @@ 
             $amount = preg_replace('/[a-zA-Z\/\\\*\(\)\<\>\_]/', '', $amount);
         }
 
-        if ($option == 2 && $thousand == '.' && preg_match('/\.(\d\d\d)$/', (string)$amount)) {    // It means the . is used as a thousand separator and string come from input data, so 1.123 is 1123
+        if ($option == 2 && $thousand == '.' && preg_match('/\.(\d\d\d)$/', (string) $amount)) {    // It means the . is used as a thousand separator and string come from input data, so 1.123 is 1123
             $amount = str_replace($thousand, '', $amount);
         }
 
@@ -4307,7 +4307,7 @@ 
         $amount = str_replace($thousand, '', $amount); // Replace of thousand before replace of dec to avoid pb if thousand is .
         $amount = str_replace($dec, '.', $amount);
 
-        $amount = preg_replace('/[^0-9\-\.]/', '', $amount);    // Clean non numeric chars (so it clean some UTF8 spaces for example.
+        $amount = preg_replace('/[^0-9\-\.]/', '', $amount); // Clean non numeric chars (so it clean some UTF8 spaces for example.
     }
     //print ' XX'.$amount.' '.$rounding;
 
@@ -4323,14 +4323,14 @@ 
         } elseif ($rounding == 'CU') {
             $nbofdectoround = max(getDolGlobalString('MAIN_MAX_DECIMALS_UNIT'), 8); // TODO Use param of currency
         } elseif ($rounding == 'CT') {
-            $nbofdectoround = max(getDolGlobalString('MAIN_MAX_DECIMALS_TOT'), 8);      // TODO Use param of currency
+            $nbofdectoround = max(getDolGlobalString('MAIN_MAX_DECIMALS_TOT'), 8); // TODO Use param of currency
         } elseif (is_numeric($rounding)) {
-            $nbofdectoround = (int)$rounding;
+            $nbofdectoround = (int) $rounding;
         }
 
         //print " RR".$amount.' - '.$nbofdectoround.'<br>';
         if (dol_strlen($nbofdectoround)) {
-            $amount = round(is_string($amount) ? (float)$amount : $amount, $nbofdectoround); // $nbofdectoround can be 0.
+            $amount = round(is_string($amount) ? (float) $amount : $amount, $nbofdectoround); // $nbofdectoround can be 0.
         } else {
             return 'ErrorBadParameterProvidedToFunction';
         }
@@ -4357,7 +4357,7 @@ 
         $amount = str_replace($thousand, '', $amount); // Replace of thousand before replace of dec to avoid pb if thousand is .
         $amount = str_replace($dec, '.', $amount);
 
-        $amount = preg_replace('/[^0-9\-\.]/', '', $amount);    // Clean non numeric chars (so it clean some UTF8 spaces for example.
+        $amount = preg_replace('/[^0-9\-\.]/', '', $amount); // Clean non numeric chars (so it clean some UTF8 spaces for example.
     }
 
     return $amount;
@@ -4437,7 +4437,7 @@ 
 
     $vatratecleaned = $vatrate;
     $reg = array();
-    if (preg_match('/^(.*)\s*\((.*)\)$/', (string)$vatrate, $reg)) {     // If vat is "xx (yy)"
+    if (preg_match('/^(.*)\s*\((.*)\)$/', (string) $vatrate, $reg)) {     // If vat is "xx (yy)"
         $vatratecleaned = trim($reg[1]);
         $vatratecode = $reg[2];
     }
@@ -4450,7 +4450,7 @@ 
     // Some test to guess with no need to make database access
     if ($mysoc->country_code == 'ES') { // For spain localtaxes 1 and 2, tax is qualified if buyer use local tax
         if ($local == 1) {
-            if (!$mysoc->localtax1_assuj || (string)$vatratecleaned == "0") {
+            if (!$mysoc->localtax1_assuj || (string) $vatratecleaned == "0") {
                 return 0;
             }
             if ($thirdparty_seller->id == $mysoc->id) {
@@ -4526,7 +4526,7 @@ 
     $sql = "SELECT t.localtax1, t.localtax2, t.localtax1_type, t.localtax2_type";
     $sql .= " FROM " . MAIN_DB_PREFIX . "c_tva as t, " . MAIN_DB_PREFIX . "c_country as c";
     $sql .= " WHERE t.fk_pays = c.rowid AND c.code = '" . $db->escape($thirdparty_seller->country_code) . "'";
-    $sql .= " AND t.taux = " . ((float)$vatratecleaned) . " AND t.active = 1";
+    $sql .= " AND t.taux = " . ((float) $vatratecleaned) . " AND t.active = 1";
     $sql .= " AND t.entity IN (" . getEntity('c_tva') . ")";
     if (!empty($vatratecode)) {
         $sql .= " AND t.code ='" . $db->escape($vatratecode) . "'"; // If we have the code, we use it in priority
@@ -4626,7 +4626,7 @@ 
     $sql .= " t.localtax1, t.localtax1_type, t.localtax2, t.localtax2_type";
     $sql .= " FROM " . MAIN_DB_PREFIX . "c_tva as t";
     if ($firstparamisid) {
-        $sql .= " WHERE t.rowid = " . (int)$vatrate;
+        $sql .= " WHERE t.rowid = " . (int) $vatrate;
     } else {
         $vatratecleaned = $vatrate;
         $vatratecode = '';
@@ -4640,7 +4640,7 @@ 
         /*if ($mysoc->country_code == 'ES') $sql.= " WHERE t.fk_pays = c.rowid AND c.code = '".$db->escape($buyer->country_code)."'";    // vat in spain use the buyer country ??
         else $sql.= " WHERE t.fk_pays = c.rowid AND c.code = '".$db->escape($seller->country_code)."'";*/
         $sql .= " WHERE t.fk_pays = c.rowid AND c.code = '" . $db->escape($seller->country_code) . "'";
-        $sql .= " AND t.taux = " . ((float)$vatratecleaned) . " AND t.active = 1";
+        $sql .= " AND t.taux = " . ((float) $vatratecleaned) . " AND t.active = 1";
         $sql .= " AND t.entity IN (" . getEntity('c_tva') . ")";
         if ($vatratecode) {
             $sql .= " AND t.code = '" . $db->escape($vatratecode) . "'";
@@ -4699,7 +4699,7 @@ 
     $sql = "SELECT t.taux as rate, t.code, t.localtax1, t.localtax1_type, t.localtax2, t.localtax2_type, t.accountancy_code_sell, t.accountancy_code_buy";
     $sql .= " FROM " . MAIN_DB_PREFIX . "c_tva as t";
     if ($firstparamisid) {
-        $sql .= " WHERE t.rowid = " . (int)$vatrate;
+        $sql .= " WHERE t.rowid = " . (int) $vatrate;
     } else {
         $vatratecleaned = $vatrate;
         $vatratecode = '';
@@ -4717,7 +4717,7 @@ 
             $countrycodetouse = ((empty($seller) || empty($seller->country_code)) ? $mysoc->country_code : $seller->country_code);
             $sql .= " WHERE t.fk_pays = c.rowid AND c.code = '" . $db->escape($countrycodetouse) . "'";
         }
-        $sql .= " AND t.taux = " . ((float)$vatratecleaned) . " AND t.active = 1";
+        $sql .= " AND t.taux = " . ((float) $vatratecleaned) . " AND t.active = 1";
         if ($vatratecode) {
             $sql .= " AND t.code = '" . $db->escape($vatratecode) . "'";
         }
@@ -4783,7 +4783,7 @@ 
                 }
             }
             if (!$found) {
-                $ret = $product->tva_tx;    // Default sales vat of product
+                $ret = $product->tva_tx; // Default sales vat of product
                 if ($product->default_vat_code) {
                     $ret .= ' (' . $product->default_vat_code . ')';
                 }
@@ -4934,7 +4934,7 @@ 
     $buyer_country_code = $thirdparty_buyer->country_code;
     $buyer_in_cee = isInEEC($thirdparty_buyer);
 
-    dol_syslog("get_default_tva: seller use vat=" . $seller_use_vat . ", seller country=" . $seller_country_code . ", seller in cee=" . ((string)(int)$seller_in_cee) . ", buyer vat number=" . $thirdparty_buyer->tva_intra . " buyer country=" . $buyer_country_code . ", buyer in cee=" . ((string)(int)$buyer_in_cee) . ", idprod=" . $idprod . ", idprodfournprice=" . $idprodfournprice . ", SERVICE_ARE_ECOMMERCE_200238EC=" . (getDolGlobalString('SERVICES_ARE_ECOMMERCE_200238EC') ? $conf->global->SERVICES_ARE_ECOMMERCE_200238EC : ''));
+    dol_syslog("get_default_tva: seller use vat=" . $seller_use_vat . ", seller country=" . $seller_country_code . ", seller in cee=" . ((string) (int) $seller_in_cee) . ", buyer vat number=" . $thirdparty_buyer->tva_intra . " buyer country=" . $buyer_country_code . ", buyer in cee=" . ((string) (int) $buyer_in_cee) . ", idprod=" . $idprod . ", idprodfournprice=" . $idprodfournprice . ", SERVICE_ARE_ECOMMERCE_200238EC=" . (getDolGlobalString('SERVICES_ARE_ECOMMERCE_200238EC') ? $conf->global->SERVICES_ARE_ECOMMERCE_200238EC : ''));
 
     // If services are eServices according to EU Council Directive 2002/38/EC (http://ec.europa.eu/taxation_customs/taxation/vat/traders/e-commerce/article_1610_en.htm)
     // we use the buyer VAT.
@@ -5177,7 +5177,7 @@ 
         // In a future, we may distribute directories on several levels depending on setup and object.
         // Here, $object->id, $object->ref and $modulepart are required.
         //var_dump($modulepart);
-        $path = dol_sanitizeFileName(empty($object->ref) ? (string)((is_object($object) && property_exists($object, 'id')) ? $object->id : '') : $object->ref);
+        $path = dol_sanitizeFileName(empty($object->ref) ? (string) ((is_object($object) && property_exists($object, 'id')) ? $object->id : '') : $object->ref);
     }
 
     if (empty($withoutslash) && !empty($path)) {
@@ -5237,7 +5237,7 @@ 
                 dol_syslog("functions.lib::dol_mkdir: Directory '" . $ccdir . "' does not exists or is outside open_basedir PHP setting.", LOG_DEBUG);
 
                 umask(0);
-                $dirmaskdec = octdec((string)$newmask);
+                $dirmaskdec = octdec((string) $newmask);
                 if (empty($newmask)) {
                     $dirmaskdec = !getDolGlobalString('MAIN_UMASK') ? octdec('0755') : octdec($conf->global->MAIN_UMASK);
                 }
@@ -5456,7 +5456,7 @@ 
     }
     $a = preg_split($pattern, $text, -1, PREG_SPLIT_DELIM_CAPTURE | PREG_SPLIT_NO_EMPTY);
 
-    $nblines = (int)floor((count($a) + 1) / 2);
+    $nblines = (int) floor((count($a) + 1) / 2);
     // count possible auto line breaks
     if ($maxlinesize) {
         foreach ($a as $line) {
@@ -5592,26 +5592,26 @@ 
         $emailsendersignature = $user->signature; //  dy default, we use the signature of current user. We must complete substitution with signature in c_email_senderprofile of array after calling getCommonSubstitutionArray()
         $usersignature = $user->signature;
         $substitutionarray = array_merge($substitutionarray, array(
-            '__SENDEREMAIL_SIGNATURE__' => (string)((!getDolGlobalString('MAIN_MAIL_DO_NOT_USE_SIGN')) ? ($onlykey == 2 ? dol_trunc('SignatureFromTheSelectedSenderProfile', 30) : $emailsendersignature) : ''),
-            '__USER_SIGNATURE__' => (string)(($usersignature && !getDolGlobalString('MAIN_MAIL_DO_NOT_USE_SIGN')) ? ($onlykey == 2 ? dol_trunc(dol_string_nohtmltag($usersignature), 30) : $usersignature) : '')
+            '__SENDEREMAIL_SIGNATURE__' => (string) ((!getDolGlobalString('MAIN_MAIL_DO_NOT_USE_SIGN')) ? ($onlykey == 2 ? dol_trunc('SignatureFromTheSelectedSenderProfile', 30) : $emailsendersignature) : ''),
+            '__USER_SIGNATURE__' => (string) (($usersignature && !getDolGlobalString('MAIN_MAIL_DO_NOT_USE_SIGN')) ? ($onlykey == 2 ? dol_trunc(dol_string_nohtmltag($usersignature), 30) : $usersignature) : '')
         ));
 
         if (is_object($user) && ($user instanceof User)) {
             $substitutionarray = array_merge($substitutionarray, array(
-                '__USER_ID__' => (string)$user->id,
-                '__USER_LOGIN__' => (string)$user->login,
-                '__USER_EMAIL__' => (string)$user->email,
-                '__USER_PHONE__' => (string)dol_print_phone($user->office_phone, '', 0, 0, '', " ", '', '', -1),
-                '__USER_PHONEPRO__' => (string)dol_print_phone($user->user_mobile, '', 0, 0, '', " ", '', '', -1),
-                '__USER_PHONEMOBILE__' => (string)dol_print_phone($user->personal_mobile, '', 0, 0, '', " ", '', '', -1),
-                '__USER_FAX__' => (string)$user->office_fax,
-                '__USER_LASTNAME__' => (string)$user->lastname,
-                '__USER_FIRSTNAME__' => (string)$user->firstname,
-                '__USER_FULLNAME__' => (string)$user->getFullName($outputlangs),
-                '__USER_SUPERVISOR_ID__' => (string)($user->fk_user ? $user->fk_user : '0'),
-                '__USER_JOB__' => (string)$user->job,
-                '__USER_REMOTE_IP__' => (string)getUserRemoteIP(),
-                '__USER_VCARD_URL__' => (string)$user->getOnlineVirtualCardUrl('', 'external')
+                '__USER_ID__' => (string) $user->id,
+                '__USER_LOGIN__' => (string) $user->login,
+                '__USER_EMAIL__' => (string) $user->email,
+                '__USER_PHONE__' => (string) dol_print_phone($user->office_phone, '', 0, 0, '', " ", '', '', -1),
+                '__USER_PHONEPRO__' => (string) dol_print_phone($user->user_mobile, '', 0, 0, '', " ", '', '', -1),
+                '__USER_PHONEMOBILE__' => (string) dol_print_phone($user->personal_mobile, '', 0, 0, '', " ", '', '', -1),
+                '__USER_FAX__' => (string) $user->office_fax,
+                '__USER_LASTNAME__' => (string) $user->lastname,
+                '__USER_FIRSTNAME__' => (string) $user->firstname,
+                '__USER_FULLNAME__' => (string) $user->getFullName($outputlangs),
+                '__USER_SUPERVISOR_ID__' => (string) ($user->fk_user ? $user->fk_user : '0'),
+                '__USER_JOB__' => (string) $user->job,
+                '__USER_REMOTE_IP__' => (string) getUserRemoteIP(),
+                '__USER_VCARD_URL__' => (string) $user->getOnlineVirtualCardUrl('', 'external')
             ));
         }
     }
@@ -5632,7 +5632,7 @@ 
             '__MYCOMPANY_PROFID9__' => $mysoc->idprof9,
             '__MYCOMPANY_PROFID10__' => $mysoc->idprof10,
             '__MYCOMPANY_CAPITAL__' => $mysoc->capital,
-            '__MYCOMPANY_FULLADDRESS__' => (method_exists($mysoc, 'getFullAddress') ? $mysoc->getFullAddress(1, ', ') : ''),    // $mysoc may be stdClass
+            '__MYCOMPANY_FULLADDRESS__' => (method_exists($mysoc, 'getFullAddress') ? $mysoc->getFullAddress(1, ', ') : ''), // $mysoc may be stdClass
             '__MYCOMPANY_ADDRESS__' => $mysoc->address,
             '__MYCOMPANY_ZIP__' => $mysoc->zip,
             '__MYCOMPANY_TOWN__' => $mysoc->town,
@@ -6004,7 +6004,7 @@ 
                                 $substitutionarray['__EXTRAFIELD_' . strtoupper($key) . '__'] = dol_print_phone($object->array_options['options_' . $key]);
                             } elseif ($extrafields->attributes[$object->table_element]['type'][$key] == 'price') {
                                 $substitutionarray['__EXTRAFIELD_' . strtoupper($key) . '__'] = $object->array_options['options_' . $key];
-                                $substitutionarray['__EXTRAFIELD_' . strtoupper($key) . '_FORMATED__'] = price($object->array_options['options_' . $key]);    // For compatibility
+                                $substitutionarray['__EXTRAFIELD_' . strtoupper($key) . '_FORMATED__'] = price($object->array_options['options_' . $key]); // For compatibility
                                 $substitutionarray['__EXTRAFIELD_' . strtoupper($key) . '_FORMATTED__'] = price($object->array_options['options_' . $key]);
                             } elseif ($extrafields->attributes[$object->table_element]['type'][$key] != 'separator') {
                                 $substitutionarray['__EXTRAFIELD_' . strtoupper($key) . '__'] = !empty($object->array_options['options_' . $key]) ? $object->array_options['options_' . $key] : '';
@@ -6219,26 +6219,26 @@ 
         $daytext = $outputlangs->trans('Day' . $tmp['wday']);
 
         $substitutionarray = array_merge($substitutionarray, array(
-            '__NOW_TMS__' => (string)$now,     // Must be the string that represent the int
+            '__NOW_TMS__' => (string) $now, // Must be the string that represent the int
             '__NOW_TMS_YMD__' => dol_print_date($now, 'day', 'auto', $outputlangs),
-            '__DAY__' => (string)$tmp['mday'],
+            '__DAY__' => (string) $tmp['mday'],
             '__DAY_TEXT__' => $daytext, // Monday
             '__DAY_TEXT_SHORT__' => dol_trunc($daytext, 3, 'right', 'UTF-8', 1), // Mon
             '__DAY_TEXT_MIN__' => dol_trunc($daytext, 1, 'right', 'UTF-8', 1), // M
-            '__MONTH__' => (string)$tmp['mon'],
+            '__MONTH__' => (string) $tmp['mon'],
             '__MONTH_TEXT__' => $outputlangs->trans('Month' . sprintf("%02d", $tmp['mon'])),
             '__MONTH_TEXT_SHORT__' => $outputlangs->trans('MonthShort' . sprintf("%02d", $tmp['mon'])),
             '__MONTH_TEXT_MIN__' => $outputlangs->trans('MonthVeryShort' . sprintf("%02d", $tmp['mon'])),
-            '__YEAR__' => (string)$tmp['year'],
-            '__PREVIOUS_DAY__' => (string)$tmp2['day'],
-            '__PREVIOUS_MONTH__' => (string)$tmp3['month'],
-            '__PREVIOUS_YEAR__' => (string)($tmp['year'] - 1),
-            '__NEXT_DAY__' => (string)$tmp4['day'],
-            '__NEXT_MONTH__' => (string)$tmp5['month'],
+            '__YEAR__' => (string) $tmp['year'],
+            '__PREVIOUS_DAY__' => (string) $tmp2['day'],
+            '__PREVIOUS_MONTH__' => (string) $tmp3['month'],
+            '__PREVIOUS_YEAR__' => (string) ($tmp['year'] - 1),
+            '__NEXT_DAY__' => (string) $tmp4['day'],
+            '__NEXT_MONTH__' => (string) $tmp5['month'],
             '__NEXT_MONTH_TEXT__' => $outputlangs->trans('Month' . sprintf("%02d", $tmp5['month'])),
             '__NEXT_MONTH_TEXT_SHORT__' => $outputlangs->trans('MonthShort' . sprintf("%02d", $tmp5['month'])),
             '__NEXT_MONTH_TEXT_MIN__' => $outputlangs->trans('MonthVeryShort' . sprintf("%02d", $tmp5['month'])),
-            '__NEXT_YEAR__' => (string)($tmp['year'] + 1),
+            '__NEXT_YEAR__' => (string) ($tmp['year'] + 1),
         ));
     }
 
@@ -6406,7 +6406,7 @@ 
                         $path = $param_arr[0];
                         $class = $param_arr[1];
                         $method = $param_arr[2];
-                        $id = (int)$param_arr[3];
+                        $id = (int) $param_arr[3];
 
                         // load class file and init object list in memory
                         if (!isset($memory_object_list[$class])) {
@@ -6468,7 +6468,7 @@ 
     // Note: substitution key for each extrafields, using key __EXTRA_XXX__ is already available into the getCommonSubstitutionArray used to build the substitution array.
 
     // Check if there is external substitution to do, requested by plugins
-    $dirsubstitutions = array_merge(array(), (array)$conf->modules_parts['substitutions']);
+    $dirsubstitutions = array_merge(array(), (array) $conf->modules_parts['substitutions']);
 
     foreach ($dirsubstitutions as $reldir) {
         $dir = dol_buildpath($reldir, 0);
@@ -6611,7 +6611,7 @@ 
 {
     //dol_syslog(__FUNCTION__ . " is deprecated", LOG_WARNING);     This is not deprecated, it is used by setEventMessages function
     if (!is_array($mesgs)) {
-        $mesgs = trim((string)$mesgs);
+        $mesgs = trim((string) $mesgs);
         // If mesgs is a not an empty string
         if ($mesgs) {
             if (!empty($noduplicate) && isset($_SESSION['dol_events'][$style]) && in_array($mesgs, $_SESSION['dol_events'][$style])) {
@@ -6622,7 +6622,7 @@ 
     } else {
         // If mesgs is an array
         foreach ($mesgs as $mesg) {
-            $mesg = trim((string)$mesg);
+            $mesg = trim((string) $mesg);
             if ($mesg) {
                 if (!empty($noduplicate) && isset($_SESSION['dol_events'][$style]) && in_array($mesg, $_SESSION['dol_events'][$style])) {
                     return;
@@ -6656,7 +6656,7 @@ 
             $mesg .= '';
         }
         if (empty($messagekey) || empty($_COOKIE["DOLHIDEMESSAGE" . $messagekey])) {
-            if (!in_array((string)$style, array('mesgs', 'warnings', 'errors'))) {
+            if (!in_array((string) $style, array('mesgs', 'warnings', 'errors'))) {
                 dol_print_error(null, 'Bad parameter style=' . $style . ' for setEventMessages');
             }
             if (empty($mesgs)) {
@@ -6701,7 +6701,7 @@ 
                     $temp[$key] = empty($array[$key][$index]) ? 0 : $array[$key][$index];
                 }
                 if ($natsort == -1) {
-                    $temp[$key] = '___' . $temp[$key];        // We add a string at begin of value to force an alpha order when using asort.
+                    $temp[$key] = '___' . $temp[$key]; // We add a string at begin of value to force an alpha order when using asort.
                 }
             }
 
@@ -6744,7 +6744,7 @@ 
  */
 function utf8_check($str)
 {
-    $str = (string)$str;   // Sometimes string is an int.
+    $str = (string) $str; // Sometimes string is an int.
 
     // We must use here a binary strlen function (so not dol_strlen)
     $strLength = strlen($str);
@@ -6939,7 +6939,7 @@ 
     if (isset($strToEvaluate) && $strToEvaluate !== '') {
         //var_dump($strToEvaluate);
         //$rep = dol_eval($strToEvaluate, 1, 0, '1'); // to show the error
-        $rep = (int)dol_eval($strToEvaluate, 1, 1, $onlysimplestring); // The dol_eval() must contains all the "global $xxx;" for all variables $xxx found into the string condition
+        $rep = (int) dol_eval($strToEvaluate, 1, 1, $onlysimplestring); // The dol_eval() must contains all the "global $xxx;" for all variables $xxx found into the string condition
         $rights = $rep && (!is_string($rep) || strpos($rep, 'Bad string syntax to evaluate') === false);
         //var_dump($rights);
     }
@@ -6963,17 +6963,17 @@ 
 function dol_eval($s, $returnvalue = 1, $hideerrors = 1, $onlysimplestring = '1')
 {
     // Only this global variables can be read by eval function and returned to caller
-    global $conf;   // Read of const is done with getDolGlobalString() but we need $conf->currency for example
+    global $conf; // Read of const is done with getDolGlobalString() but we need $conf->currency for example
     global $db, $langs, $user, $website, $websitepage;
     global $action, $mainmenu, $leftmenu;
     global $mysoc;
-    global $objectoffield;  // To allow the use of $objectoffield in computed fields
+    global $objectoffield; // To allow the use of $objectoffield in computed fields
 
     // Old variables used
     global $object;
     global $obj; // To get $obj used into list when dol_eval() is used for computed fields and $obj is not yet $object
 
-    $isObBufferActive = false;  // When true, the ObBuffer must be cleaned in the exception handler
+    $isObBufferActive = false; // When true, the ObBuffer must be cleaned in the exception handler
     if (!in_array($onlysimplestring, array('0', '1', '2'))) {
         return "Bad call of dol_eval. Parameter onlysimplestring must be '0' (deprecated), '1' or '2'";
     }
@@ -7004,11 +7004,11 @@ 
             while ($scheck && $savescheck != $scheck) {
                 $savescheck = $scheck;
                 $scheck = preg_replace('/->[a-zA-Z0-9_]+\(/', '->__METHOD__', $scheck); // accept parenthesis in '...->method(...'
-                $scheck = preg_replace('/^\(/', '__PARENTHESIS__ ', $scheck);   // accept parenthesis in '(...'. Must replace with __PARENTHESIS__ with a space after to allow following substitutions
-                $scheck = preg_replace('/\s\(/', '__PARENTHESIS__ ', $scheck);  // accept parenthesis in '... ('. Must replace with __PARENTHESIS__ with a space after to allow following substitutions
+                $scheck = preg_replace('/^\(/', '__PARENTHESIS__ ', $scheck); // accept parenthesis in '(...'. Must replace with __PARENTHESIS__ with a space after to allow following substitutions
+                $scheck = preg_replace('/\s\(/', '__PARENTHESIS__ ', $scheck); // accept parenthesis in '... ('. Must replace with __PARENTHESIS__ with a space after to allow following substitutions
                 $scheck = preg_replace('/^!?[a-zA-Z0-9_]+\(/', '__FUNCTION__', $scheck); // accept parenthesis in 'function(' and '!function('
                 $scheck = preg_replace('/\s!?[a-zA-Z0-9_]+\(/', '__FUNCTION__', $scheck); // accept parenthesis in '... function(' and '... !function('
-                $scheck = preg_replace('/(\^|\')\(/', '__REGEXSTART__', $scheck);   // To allow preg_match('/^(aaa|bbb)/'...  or  isStringVarMatching('leftmenu', '(aaa|bbb)')
+                $scheck = preg_replace('/(\^|\')\(/', '__REGEXSTART__', $scheck); // To allow preg_match('/^(aaa|bbb)/'...  or  isStringVarMatching('leftmenu', '(aaa|bbb)')
             }
             //print 'scheck='.$scheck." : ".strpos($scheck, '(')."<br>\n";
             if (strpos($scheck, '(') !== false) {
@@ -7065,7 +7065,7 @@ 
         $forbiddenphpfunctions = array_merge($forbiddenphpfunctions, array("eval", "create_function", "assert", "mb_ereg_replace")); // function with eval capabilities
         $forbiddenphpfunctions = array_merge($forbiddenphpfunctions, array("dol_compress_dir", "dol_decode", "dol_delete_file", "dol_delete_dir", "dol_delete_dir_recursive", "dol_copy", "archiveOrBackupFile")); // more dolibarr functions
 
-        $forbiddenphpmethods = array('invoke', 'invokeArgs');   // Method of ReflectionFunction to execute a function
+        $forbiddenphpmethods = array('invoke', 'invokeArgs'); // Method of ReflectionFunction to execute a function
 
         $forbiddenphpregex = 'global\s+\$|\b(' . implode('|', $forbiddenphpfunctions) . ')\b';
 
@@ -7283,8 +7283,8 @@ 
         'en-GB',
         'en-IE',
         'en-IN',
-        'as-IN',    // as-IN must be after en-IN (en in priority if country is IN)
-        'bn-IN',    // bn-IN must be after en-IN (en in priority if country is IN)
+        'as-IN', // as-IN must be after en-IN (en in priority if country is IN)
+        'bn-IN', // bn-IN must be after en-IN (en in priority if country is IN)
         'en-JM',
         'en-MY',
         'en-NZ',
@@ -7486,7 +7486,7 @@ 
     if (!function_exists('getmypid')) {
         return mt_rand(99900000, 99965535);
     } else {
-        return getmypid();  // May be a number on 64 bits (depending on OS)
+        return getmypid(); // May be a number on 64 bits (depending on OS)
     }
 }
 
@@ -7533,15 +7533,15 @@ 
         $fields = array($fields);
     }
 
-    $i1 = 0;    // count the nb of and criteria added (all fields / criteria)
+    $i1 = 0; // count the nb of and criteria added (all fields / criteria)
     foreach ($crits as $crit) {     // Loop on each AND criteria
         $crit = trim($crit);
-        $i2 = 0;    // count the nb of valid criteria added for this this first criteria
+        $i2 = 0; // count the nb of valid criteria added for this this first criteria
         $newres = '';
         foreach ($fields as $field) {
             if ($mode == 1) {
                 $tmpcrits = explode('|', $crit);
-                $i3 = 0;    // count the nb of valid criteria added for this current field
+                $i3 = 0; // count the nb of valid criteria added for this current field
                 foreach ($tmpcrits as $tmpcrit) {
                     if ($tmpcrit !== '0' && empty($tmpcrit)) {
                         continue;
@@ -7561,7 +7561,7 @@ 
                     if ($newcrit != '') {
                         $numnewcrit = price2num($newcrit);
                         if (is_numeric($numnewcrit)) {
-                            $newres .= $field . ' ' . $operator . ' ' . ((float)$numnewcrit); // should be a numeric
+                            $newres .= $field . ' ' . $operator . ' ' . ((float) $numnewcrit); // should be a numeric
                         } else {
                             $newres .= '1 = 2'; // force false, we received a corrupted data
                         }
@@ -7612,7 +7612,7 @@ 
                 }
             } else { // $mode=0
                 $tmpcrits = explode('|', $crit);
-                $i3 = 0;    // count the nb of valid criteria added for the current couple criteria/field
+                $i3 = 0; // count the nb of valid criteria added for the current couple criteria/field
                 foreach ($tmpcrits as $tmpcrit) {   // loop on each OR criteria
                     if ($tmpcrit !== '0' && empty($tmpcrit)) {
                         continue;
@@ -7626,7 +7626,7 @@ 
                     }
 
                     if (preg_match('/\.(id|rowid)$/', $field)) {    // Special case for rowid that is sometimes a ref so used as a search field
-                        $newres .= $field . " = " . (is_numeric($tmpcrit) ? ((float)$tmpcrit) : '0');
+                        $newres .= $field . " = " . (is_numeric($tmpcrit) ? ((float) $tmpcrit) : '0');
                     } else {
                         $tmpcrit2 = $tmpcrit;
                         $tmpbefore = '%';
@@ -8131,7 +8131,7 @@ 
     }
 
     // Return mimetype string
-    switch ((int)$mode) {
+    switch ((int) $mode) {
         case 1:
             $tmp = explode('/', $mime);
             return (!empty($tmp[1]) ? $tmp[1] : $tmp[0]);
@@ -8160,7 +8160,7 @@ 
 {
     global $conf, $db;
 
-    $tablename = preg_replace('/^' . preg_quote(MAIN_DB_PREFIX, '/') . '/', '', $tablename);    // Clean name of table for backward compatibility.
+    $tablename = preg_replace('/^' . preg_quote(MAIN_DB_PREFIX, '/') . '/', '', $tablename); // Clean name of table for backward compatibility.
 
     $dictvalues = (isset($conf->cache['dictvalues_' . $tablename]) ? $conf->cache['dictvalues_' . $tablename] : null);
 
@@ -8175,7 +8175,7 @@ 
         $resql = $db->query($sql);
         if ($resql) {
             while ($obj = $db->fetch_object($resql)) {
-                $dictvalues[$obj->$rowidfield] = $obj;  // $obj is stdClass
+                $dictvalues[$obj->$rowidfield] = $obj; // $obj is stdClass
             }
         } else {
             dol_print_error($db);
@@ -8279,7 +8279,7 @@ 
 function roundUpToNextMultiple($n, $x = 5)
 {
     $result = (ceil($n) % $x === 0) ? ceil($n) : (round(($n + $x / 2) / $x) * $x);
-    return (int)$result;
+    return (int) $result;
 }
 
 /**
@@ -8356,7 +8356,7 @@ 
                     return $objecttmp;
                 }
             } else {
-                return $objecttmp;  // returned an object without fetch
+                return $objecttmp; // returned an object without fetch
             }
         } else {
             return -1;

public/htdocs/core/lib/data.lib.php

Spacing +9 added lines, -9 removed lines

@@ -59,7 +59,7 @@ 
 
     $sql = 'SELECT ecm.rowid as id, ecm.src_object_type, ecm.src_object_id, ecm.filepath, ecm.filename';
     $sql .= ' FROM ' . MAIN_DB_PREFIX . 'ecm_files ecm';
-    $sql .= " WHERE ecm.filepath = 'agenda/" . ((int)$object->id) . "'";
+    $sql .= " WHERE ecm.filepath = 'agenda/" . ((int) $object->id) . "'";
     //$sql.= " ecm.src_object_type = '".$db->escape($object->element)."' AND ecm.src_object_id = ".((int) $object->id); // Old version didn't add object_type during upload
     $sql .= ' ORDER BY ecm.position ASC';
 
@@ -211,9 +211,9 @@ 
         if (strtoupper($tmpescaped) == 'NULL') {
             $tmpescaped = 'NULL';
         } elseif (is_int($tmpescaped)) {
-            $tmpescaped = (int)$tmpescaped;
+            $tmpescaped = (int) $tmpescaped;
         } else {
-            $tmpescaped = (float)$tmpescaped;
+            $tmpescaped = (float) $tmpescaped;
         }
     }
 
@@ -248,20 +248,20 @@ 
         $filter = '(' . $filter . ')';
     }
 
-    $regexstring = '\(([a-zA-Z0-9_\.]+:[<>!=insotlke]+:[^\(\)]+)\)';    // Must be  (aaa:bbb:...) with aaa is a field name (with alias or not) and bbb is one of this operator '=', '<', '>', '<=', '>=', '!=', 'in', 'notin', 'like', 'notlike', 'is', 'isnot'
+    $regexstring = '\(([a-zA-Z0-9_\.]+:[<>!=insotlke]+:[^\(\)]+)\)'; // Must be  (aaa:bbb:...) with aaa is a field name (with alias or not) and bbb is one of this operator '=', '<', '>', '<=', '>=', '!=', 'in', 'notin', 'like', 'notlike', 'is', 'isnot'
     $firstandlastparenthesis = 0;
 
     if (!dolCheckFilters($filter, $errorstr, $firstandlastparenthesis)) {
         if ($noerror) {
             return '1 = 2';
         } else {
-            return 'Filter syntax error - ' . $errorstr;      // Bad balance of parenthesis, we return an error message or force a SQL not found
+            return 'Filter syntax error - ' . $errorstr; // Bad balance of parenthesis, we return an error message or force a SQL not found
         }
     }
 
     // Test the filter syntax
     $t = preg_replace_callback('/' . $regexstring . '/i', 'dolForgeDummyCriteriaCallback', $filter);
-    $t = str_replace(array('and', 'or', 'AND', 'OR', ' '), '', $t);     // Remove the only strings allowed between each () criteria
+    $t = str_replace(array('and', 'or', 'AND', 'OR', ' '), '', $t); // Remove the only strings allowed between each () criteria
     // If the string result contains something else than '()', the syntax was wrong
 
     if (preg_match('/[^\(\)]/', $t)) {
@@ -270,7 +270,7 @@ 
         if ($noerror) {
             return '1 = 2';
         } else {
-            return 'Filter error - ' . $tmperrorstr;      // Bad syntax of the search string, we return an error message or force a SQL not found
+            return 'Filter error - ' . $tmperrorstr; // Bad syntax of the search string, we return an error message or force a SQL not found
         }
     }
 
@@ -280,7 +280,7 @@ 
         $ret = str_replace('__NOW__', $db->idate(dol_now()), $ret);
     }
     if (is_object($user)) {
-        $ret = str_replace('__USER_ID__', (string)$user->id, $ret);
+        $ret = str_replace('__USER_ID__', (string) $user->id, $ret);
     }
 
     return $ret;
@@ -431,7 +431,7 @@ 
         return '';
     }
 
-    return '()';    // An empty criteria
+    return '()'; // An empty criteria
 }
 
 /**

public/htdocs/core/lib/html.lib.php

Spacing +42 added lines, -42 removed lines

@@ -62,7 +62,7 @@ 
 
     $form = new Form($db);
 
-    $templatenameforexport = $website->name_template;   // Example 'website_template-corporate'
+    $templatenameforexport = $website->name_template; // Example 'website_template-corporate'
     if (empty($templatenameforexport)) {
         $templatenameforexport = 'website_' . $website->ref;
     }
@@ -658,7 +658,7 @@ 
                 } else {
                     $width = 14;
                     $cssclass = 'photorefcenter';
-                    $picto = $object->picto;  // @phan-suppress-current-line PhanUndeclaredProperty
+                    $picto = $object->picto; // @phan-suppress-current-line PhanUndeclaredProperty
                     $prefix = 'object_';
                     if ($object->element == 'project' && !$object->public) {  // @phan-suppress-current-line PhanUndeclaredProperty
                         $picto = 'project'; // instead of projectpub
@@ -768,7 +768,7 @@ 
     }
     // Show address and email
     if (method_exists($object, 'getBannerAddress') && !in_array($object->element, array('product', 'bookmark', 'ecm_directories', 'ecm_files'))) {
-        $moreaddress = $object->getBannerAddress('refaddress', $object);    // address, email, url, social networks
+        $moreaddress = $object->getBannerAddress('refaddress', $object); // address, email, url, social networks
         if ($moreaddress) {
             $morehtmlref .= '<div class="refidno refaddress">';
             $morehtmlref .= $moreaddress;
@@ -778,7 +778,7 @@ 
     if (getDolGlobalString('MAIN_SHOW_TECHNICAL_ID') && (getDolGlobalString('MAIN_SHOW_TECHNICAL_ID') == '1' || preg_match('/' . preg_quote($object->element, '/') . '/i', $conf->global->MAIN_SHOW_TECHNICAL_ID)) && !empty($object->id)) {
         $morehtmlref .= '<div style="clear: both;"></div>';
         $morehtmlref .= '<div class="refidno opacitymedium">';
-        $morehtmlref .= $langs->trans("TechnicalID") . ': ' . ((int)$object->id);
+        $morehtmlref .= $langs->trans("TechnicalID") . ': ' . ((int) $object->id);
         $morehtmlref .= '</div>';
     }
 
@@ -936,7 +936,7 @@ 
             $type = 'AC_EMAIL';
             $linktoaddaction = '';
             if (getDolGlobalString('AGENDA_ADDACTIONFOREMAIL')) {
-                $linktoaddaction = '<a href="' . constant('BASE_URL') . '/comm/action/card.php?action=create&amp;backtopage=1&amp;actioncode=' . urlencode($type) . '&amp;contactid=' . ((int)$cid) . '&amp;socid=' . ((int)$socid) . '">' . img_object($langs->trans("AddAction"), "calendar") . '</a>';
+                $linktoaddaction = '<a href="' . constant('BASE_URL') . '/comm/action/card.php?action=create&amp;backtopage=1&amp;actioncode=' . urlencode($type) . '&amp;contactid=' . ((int) $cid) . '&amp;socid=' . ((int) $socid) . '">' . img_object($langs->trans("AddAction"), "calendar") . '</a>';
             }
             if ($linktoaddaction) {
                 $newemail = '<div>' . $newemail . ' ' . $linktoaddaction . '</div>';
@@ -1316,7 +1316,7 @@ 
             $url = make_substitutions($url, $substitarray);
             if (!getDolGlobalString('CLICKTODIAL_DO_NOT_USE_AJAX_CALL')) {
                 // Default and recommended: New method using ajax without submitting a page making a javascript history.go(-1) back
-                $newphoneastart = '<a href="' . $url . '" class="cssforclicktodial">';  // Call of ajax is handled by the lib_foot.js.php on class 'cssforclicktodial'
+                $newphoneastart = '<a href="' . $url . '" class="cssforclicktodial">'; // Call of ajax is handled by the lib_foot.js.php on class 'cssforclicktodial'
                 $newphoneaend = '</a>';
             } else {
                 // Old method
@@ -1347,7 +1347,7 @@ 
 
     if (getDolGlobalString('CONTACT_PHONEMOBILE_SHOW_LINK_TO_WHATSAPP') && $withpicto == 'mobile') {
         // Link to Whatsapp
-        $newphone .= ' <a href="https://wa.me/' . $newphonewa . '" target="_blank"';// Use api to whatasapp contacts
+        $newphone .= ' <a href="https://wa.me/' . $newphonewa . '" target="_blank"'; // Use api to whatasapp contacts
         $newphone .= '><span class="paddingright fab fa-whatsapp" style="color:#25D366;" title="WhatsApp"></span></a>';
     }
 
@@ -1846,7 +1846,7 @@ 
             top_httphead();
         }
         //http_response_code(500);      // If we use 500, message is not output with some command line tools
-        http_response_code(202);        // If we use 202, this is not really an error message, but this allow to output message on command line tools
+        http_response_code(202); // If we use 202, this is not really an error message, but this allow to output message on command line tools
     }
 
     if (empty($dolibarr_main_prod)) {
@@ -2116,7 +2116,7 @@ 
     }
     $return .= '<td class="nobordernopadding valignmiddle col-title">';
     $return .= '<div class="titre inline-block">';
-    $return .= $title;  // $title is already HTML sanitized content
+    $return .= $title; // $title is already HTML sanitized content
     $return .= '</div>';
     $return .= '</td>';
     if (dol_strlen($morehtmlcenter)) {
@@ -2163,7 +2163,7 @@ 
         $totalnboflines = abs($totalnboflines);
     }
 
-    $page = (int)$page;
+    $page = (int) $page;
 
     if ($picto == 'setup') {
         $picto = 'title_setup.png';
@@ -2194,8 +2194,8 @@ 
 
     print '<td class="nobordernopadding valignmiddle col-title">';
     print '<div class="titre inline-block">';
-    print $title;   // $title may contains HTML
-    if (!empty($title) && $savtotalnboflines >= 0 && (string)$savtotalnboflines != '') {
+    print $title; // $title may contains HTML
+    if (!empty($title) && $savtotalnboflines >= 0 && (string) $savtotalnboflines != '') {
         print '<span class="opacitymedium colorblack paddingleft">(' . $totalnboflines . ')</span>';
     }
     print '</div></td>';
@@ -2207,7 +2207,7 @@ 
 
     // Right
     print '<td class="nobordernopadding valignmiddle right col-right">';
-    print '<input type="hidden" name="pageplusoneold" value="' . ((int)$page + 1) . '">';
+    print '<input type="hidden" name="pageplusoneold" value="' . ((int) $page + 1) . '">';
     if ($sortfield) {
         $options .= "&sortfield=" . urlencode($sortfield);
     }
@@ -2326,7 +2326,7 @@ 
     }
 
     if (empty($hidenavigation)) {
-        if ((int)$limit > 0 && empty($hideselectlimit)) {
+        if ((int) $limit > 0 && empty($hideselectlimit)) {
             $pagesizechoices = '10:10,15:15,20:20,25:25,50:50,100:100,250:250,500:500,1000:1000';
             $pagesizechoices .= ',5000:5000,10000:10000,20000:20000';
             //$pagesizechoices.=',0:'.$langs->trans("All");     // Not yet supported
@@ -2359,7 +2359,7 @@ 
                 $key = $tmp[0];
                 $val = $tmp[1];
                 if ($key != '' && $val != '') {
-                    if ((int)$key == (int)$limit) {
+                    if ((int) $key == (int) $limit) {
                         $selected = ' selected="selected"';
                     }
                     print '<option name="' . $key . '"' . $selected . '>' . dol_escape_htmltag($val) . '</option>' . "\n";
@@ -2561,7 +2561,7 @@ 
         if (is_object($objcon) && $objcon->id > 0) {
             $force_filter_contact = true;
             $sql .= " INNER JOIN " . MAIN_DB_PREFIX . "actioncomm_resources as r ON a.id = r.fk_actioncomm";
-            $sql .= " AND r.element_type = '" . $db->escape($objcon->table_element) . "' AND r.fk_element = " . ((int)$objcon->id);
+            $sql .= " AND r.element_type = '" . $db->escape($objcon->table_element) . "' AND r.fk_element = " . ((int) $objcon->id);
         }
 
         if (is_object($filterobj) && get_class($filterobj) == 'Societe') {
@@ -2570,7 +2570,7 @@ 
             $sql .= " INNER JOIN " . MAIN_DB_PREFIX . "element_resources as er";
             $sql .= " ON er.resource_type = 'dolresource'";
             $sql .= " AND er.element_id = a.id";
-            $sql .= " AND er.resource_id = " . ((int)$filterobj->id);
+            $sql .= " AND er.resource_id = " . ((int) $filterobj->id);
         } elseif (is_object($filterobj) && get_class($filterobj) == 'Adherent') {
             $sql .= ", " . MAIN_DB_PREFIX . "adherent as m";
         } elseif (is_object($filterobj) && get_class($filterobj) == 'CommandeFournisseur') {
@@ -2588,42 +2588,42 @@ 
         $sql .= " WHERE a.entity IN (" . getEntity('agenda') . ")";
         if (!$force_filter_contact) {
             if (is_object($filterobj) && in_array(get_class($filterobj), array('Societe', 'Client', 'Fournisseur')) && $filterobj->id) {
-                $sql .= " AND a.fk_soc = " . ((int)$filterobj->id);
+                $sql .= " AND a.fk_soc = " . ((int) $filterobj->id);
             } elseif (is_object($filterobj) && get_class($filterobj) == 'Project' && $filterobj->id) {
-                $sql .= " AND a.fk_project = " . ((int)$filterobj->id);
+                $sql .= " AND a.fk_project = " . ((int) $filterobj->id);
             } elseif (is_object($filterobj) && get_class($filterobj) == 'Adherent') {
                 $sql .= " AND a.fk_element = m.rowid AND a.elementtype = 'member'";
                 if ($filterobj->id) {
-                    $sql .= " AND a.fk_element = " . ((int)$filterobj->id);
+                    $sql .= " AND a.fk_element = " . ((int) $filterobj->id);
                 }
             } elseif (is_object($filterobj) && get_class($filterobj) == 'CommandeFournisseur') {
                 $sql .= " AND a.fk_element = o.rowid AND a.elementtype = 'order_supplier'";
                 if ($filterobj->id) {
-                    $sql .= " AND a.fk_element = " . ((int)$filterobj->id);
+                    $sql .= " AND a.fk_element = " . ((int) $filterobj->id);
                 }
             } elseif (is_object($filterobj) && get_class($filterobj) == 'Product') {
                 $sql .= " AND a.fk_element = o.rowid AND a.elementtype = 'product'";
                 if ($filterobj->id) {
-                    $sql .= " AND a.fk_element = " . ((int)$filterobj->id);
+                    $sql .= " AND a.fk_element = " . ((int) $filterobj->id);
                 }
             } elseif (is_object($filterobj) && get_class($filterobj) == 'Ticket') {
                 $sql .= " AND a.fk_element = o.rowid AND a.elementtype = 'ticket'";
                 if ($filterobj->id) {
-                    $sql .= " AND a.fk_element = " . ((int)$filterobj->id);
+                    $sql .= " AND a.fk_element = " . ((int) $filterobj->id);
                 }
             } elseif (is_object($filterobj) && get_class($filterobj) == 'BOM') {
                 $sql .= " AND a.fk_element = o.rowid AND a.elementtype = 'bom'";
                 if ($filterobj->id) {
-                    $sql .= " AND a.fk_element = " . ((int)$filterobj->id);
+                    $sql .= " AND a.fk_element = " . ((int) $filterobj->id);
                 }
             } elseif (is_object($filterobj) && get_class($filterobj) == 'Contrat') {
                 $sql .= " AND a.fk_element = o.rowid AND a.elementtype = 'contract'";
                 if ($filterobj->id) {
-                    $sql .= " AND a.fk_element = " . ((int)$filterobj->id);
+                    $sql .= " AND a.fk_element = " . ((int) $filterobj->id);
                 }
             }
         } else {
-            $sql .= " AND u.rowid = " . ((int)$filterobj->id);
+            $sql .= " AND u.rowid = " . ((int) $filterobj->id);
         }
 
         // Condition on actioncode
@@ -3311,7 +3311,7 @@ 
         $showonlyonhover = 0;
     }*/
 
-    $tag = 'span';  // Using div (like any style of type 'block') does not work when using the js copy code.
+    $tag = 'span'; // Using div (like any style of type 'block') does not work when using the js copy code.
     if ($texttoshow === 'none') {
         $result = '<span class="clipboardCP' . ($showonlyonhover ? ' clipboardCPShowOnHover' : '') . '"><' . $tag . ' class="clipboardCPValue hidewithsize">' . dol_escape_htmltag($valuetocopy, 1, 1) . '</' . $tag . '><span class="clipboardCPValueToPrint"></span><span class="clipboardCPButton far fa-clipboard opacitymedium paddingleft paddingright"></span><span class="clipboardCPText"></span></span>';
     } elseif ($texttoshow) {
@@ -3652,7 +3652,7 @@ 
         $table_element = 'adherent_type';
     } elseif ($elementType == 'bank_account') {
         $classpath = 'compta/bank/class';
-        $module = 'bank';   // We need $conf->bank->dir_output and not $conf->banque->dir_output
+        $module = 'bank'; // We need $conf->bank->dir_output and not $conf->banque->dir_output
         $classfile = 'account';
         $classname = 'Account';
     } elseif ($elementType == 'category') {
@@ -4305,8 +4305,8 @@ 
 
 
     $parameters = array(
-        'TCompiledAttr' => $TCompiledAttr,              // array
-        'compiledAttributes' => $compiledAttributes,    // string
+        'TCompiledAttr' => $TCompiledAttr, // array
+        'compiledAttributes' => $compiledAttributes, // string
         'attr' => $attr,
         'tag' => $tag,
         'label' => $label,
@@ -4422,7 +4422,7 @@ 
                     if ($posstart > 0) {
                         $res1 = substr($str, $posstart + 1, $posend - $posstart - 1);
                         if (is_numeric($res1)) {
-                            $postab = (int)$res1;
+                            $postab = (int) $res1;
                             $values[1] = '+' . substr($str, $posend + 1);
                         }
                     }
@@ -4998,7 +4998,7 @@ 
 
             if (!empty($out) && getDolGlobalString('MAIN_RESTRICTHTML_ONLY_VALID_HTML') && $check != 'restricthtmlallowunvalid') {
                 try {
-                    libxml_use_internal_errors(false);  // Avoid to fill memory with xml errors
+                    libxml_use_internal_errors(false); // Avoid to fill memory with xml errors
                     if (LIBXML_VERSION < 20900) {
                         // Avoid load of external entities (security problem).
                         // Required only if LIBXML_VERSION < 20900
@@ -5039,7 +5039,7 @@ 
                         // See options at https://tidy.sourceforge.net/docs/quickref.html
                         $config = array(
                             'clean' => false,
-                            'quote-marks' => false,     // do not replace " that are used for real text content (not a string symbol for html attribute) into &quot;
+                            'quote-marks' => false, // do not replace " that are used for real text content (not a string symbol for html attribute) into &quot;
                             'doctype' => 'strict',
                             'show-body-only' => true,
                             "indent-attributes" => false,
@@ -5082,7 +5082,7 @@ 
                  * @param string[] $m
                  * @return string
                  */
-                static function ($m) {
+                static function($m) {
                     return realCharForNumericEntities($m);
                 },
                 $out
@@ -5215,7 +5215,7 @@ 
  */
 function dol_htmlcleanlastbr($stringtodecode)
 {
-    $ret = preg_replace('/&nbsp;$/i', "", $stringtodecode);     // Because wysiwyg editor may add a &nbsp; at end of last line
+    $ret = preg_replace('/&nbsp;$/i', "", $stringtodecode); // Because wysiwyg editor may add a &nbsp; at end of last line
     $ret = preg_replace('/(<br>|<br(\s[\sa-zA-Z_="]*)?\/?>|' . "\n" . '|' . "\r" . ')+$/i', "", $ret);
     return $ret;
 }
@@ -5235,7 +5235,7 @@ 
     if ($keepsomeentities) {
         $newstring = strtr($newstring, array('&amp;' => '__andamp__', '&lt;' => '__andlt__', '&gt;' => '__andgt__', '"' => '__dquot__'));
     }
-    $newstring = html_entity_decode((string)$newstring, (int)$b, (string)$c);
+    $newstring = html_entity_decode((string) $newstring, (int) $b, (string) $c);
     if ($keepsomeentities) {
         $newstring = strtr($newstring, array('__andamp__' => '&amp;', '__andlt__' => '&lt;', '__andgt__' => '&gt;', '__dquot__' => '"'));
     }
@@ -5311,7 +5311,7 @@ 
                             if (isset($valuetoclean)) {
                                 do {
                                     $oldvaluetoclean = $valuetoclean;
-                                    $valuetoclean = preg_replace('/\/\*.*\*\//m', '', $valuetoclean);   // clean css comments
+                                    $valuetoclean = preg_replace('/\/\*.*\*\//m', '', $valuetoclean); // clean css comments
                                     $valuetoclean = preg_replace('/position\s*:\s*[a-z]+/mi', '', $valuetoclean);
                                     if ($els->item($i)->tagName == 'a') {   // more paranoiac cleaning for clickable tags.
                                         $valuetoclean = preg_replace('/display\s*:/mi', '', $valuetoclean);
@@ -5399,7 +5399,7 @@ 
         $tempbis = $temp;
         do {
             $temp = $tempbis;
-            $tempbis = str_replace('<>', '', $temp);    // No reason to have this into a text, except if value is to try bypass the next html cleaning
+            $tempbis = str_replace('<>', '', $temp); // No reason to have this into a text, except if value is to try bypass the next html cleaning
             $tempbis = preg_replace($pattern, '', $tempbis);
             //$idowhile++; print $temp.'-'.$tempbis."\n"; if ($idowhile > 100) break;
         } while ($tempbis != $temp);
@@ -5454,7 +5454,7 @@ 
             "header", "footer", "nav", "section", "menu", "menuitem"    // html5 tags
         );
     }
-    $allowed_tags[] = "comment";        // this tags is added to manage comment <!--...--> that are replaced into <comment>...</comment>
+    $allowed_tags[] = "comment"; // this tags is added to manage comment <!--...--> that are replaced into <comment>...</comment>
     if ($allowiframe) {
         if (!in_array('iframe', $allowed_tags)) {
             $allowed_tags[] = "iframe";
@@ -5469,7 +5469,7 @@ 
     $allowed_tags_string = implode("><", $allowed_tags);
     $allowed_tags_string = '<' . $allowed_tags_string . '>';
 
-    $stringtoclean = str_replace('<!DOCTYPE html>', '__!DOCTYPE_HTML__', $stringtoclean);   // Replace DOCTYPE to avoid to have it removed by the strip_tags
+    $stringtoclean = str_replace('<!DOCTYPE html>', '__!DOCTYPE_HTML__', $stringtoclean); // Replace DOCTYPE to avoid to have it removed by the strip_tags
 
     $stringtoclean = dol_string_nounprintableascii($stringtoclean, 0);
 
@@ -5479,7 +5479,7 @@ 
     $stringtoclean = preg_replace('/&colon;/i', ':', $stringtoclean);
     $stringtoclean = preg_replace('/&#58;|&#0+58|&#x3A/i', '', $stringtoclean); // refused string ':' encoded (no reason to have a : encoded like this) to disable 'javascript:...'
 
-    $temp = strip_tags($stringtoclean, $allowed_tags_string);   // Warning: This remove also undesired </>, so may changes string obfuscated with </> that pass the injection detection into a harmfull string
+    $temp = strip_tags($stringtoclean, $allowed_tags_string); // Warning: This remove also undesired </>, so may changes string obfuscated with </> that pass the injection detection into a harmfull string
 
     if ($cleanalsosomestyles) { // Clean for remaining html tags
         $temp = preg_replace('/position\s*:\s*(absolute|fixed)\s*!\s*important/i', '', $temp); // Note: If hacker try to introduce css comment into string to bypass this regex, the string must also be encoded by the dol_htmlentitiesbr during output so it become harmless
@@ -5496,7 +5496,7 @@ 
 
     $temp = str_replace('__!DOCTYPE_HTML__', '<!DOCTYPE html>', $temp); // Restore the DOCTYPE
 
-    $temp = preg_replace('/<comment>([^>]*)<\/comment>/', '<!--\1-->', $temp);  // Restore html comments
+    $temp = preg_replace('/<comment>([^>]*)<\/comment>/', '<!--\1-->', $temp); // Restore html comments
 
 
     return $temp;