alxarafe / alixar

htdocs/core/class/stats.class.php

Indentation +13 added lines, -13 removed lines

@@ -381,7 +381,7 @@ 
     // Here we have low level of shared code called by XxxStats.class.php
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *  Return nb of elements by year
      *
@@ -390,7 +390,7 @@ 
      */
     protected function _getNbByYear($sql)
     {
-		// phpcs:enable
+        // phpcs:enable
         $result = array();
 
         dol_syslog(get_class($this) . '::' . __FUNCTION__, LOG_DEBUG);
@@ -410,7 +410,7 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *  Return nb of elements, total amount and avg amount each year
      *
@@ -419,7 +419,7 @@ 
      */
     protected function _getAllByYear($sql)
     {
-		// phpcs:enable
+        // phpcs:enable
         $result = array();
 
         dol_syslog(get_class($this) . '::' . __FUNCTION__, LOG_DEBUG);
@@ -464,7 +464,7 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *     Renvoie le nombre de documents par mois pour une annee donnee
      *     Return number of documents per month for a given year
@@ -476,7 +476,7 @@ 
      */
     protected function _getNbByMonth($year, $sql, $format = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $result = array();
@@ -522,7 +522,7 @@ 
         return $data;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *     Return the amount per month for a given year
      *
@@ -533,7 +533,7 @@ 
      */
     protected function _getAmountByMonth($year, $sql, $format = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $result = array();
@@ -579,7 +579,7 @@ 
         return $data;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *  Renvoie le montant moyen par mois pour une annee donnee
      *  Return the amount average par month for a given year
@@ -591,7 +591,7 @@ 
      */
     protected function _getAverageByMonth($year, $sql, $format = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $result = array();
@@ -638,7 +638,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *  Return number or total of product refs
      *
@@ -648,7 +648,7 @@ 
      */
     protected function _getAllByProduct($sql, $limit = 10)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $result = array();
@@ -679,7 +679,7 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *  Returns the summed amounts per year for a given number of past years ending now
      *  @param  string  $sql    SQL

htdocs/core/class/antivir.class.php

Indentation +2 added lines, -2 removed lines

@@ -60,7 +60,7 @@ 
         $this->db = $db;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Scan a file with antivirus.
      *  This function runs the command defined in setup. This antivirus command must return 0 if OK.
@@ -71,7 +71,7 @@ 
      */
     public function dol_avscan_file($file)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         if (preg_match('/\.virus$/i', $file)) {

htdocs/core/class/commonstickergenerator.class.php

Indentation +18 added lines, -18 removed lines

@@ -70,7 +70,7 @@ 
 
     public $code; // Code of format
 
-	// phpcs:disable PEAR.NamingConventions.ValidVariableName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidVariableName.PublicUnderscore
     // protected
     // Name of stick
     protected $_Avery_Name = '';
@@ -109,7 +109,7 @@ 
      * @var array
      */
     public $_Avery_Labels;
-	// phpcs:enable
+    // phpcs:enable
 
     /**
      *  Constructor
@@ -121,7 +121,7 @@ 
         $this->db = $db;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Function to build PDF on disk, then output on HTTP stream.
      *
@@ -132,7 +132,7 @@ 
      *  @return int                             1=OK, 0=KO
      */
     abstract public function write_file($arrayofrecords, $outputlangs, $srctemplatepath, $outputdir = '');
-	// phpcs:enable
+    // phpcs:enable
 
     /**
      * Output a sticker on page at position _COUNTX, _COUNTY (_COUNTX and _COUNTY start from 0)
@@ -144,7 +144,7 @@ 
      */
     abstract public function addSticker(&$pdf, $outputlangs, $param);
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Methode qui permet de modifier la taille des caracteres
      * Cela modiera aussi l'espace entre chaque ligne
@@ -155,7 +155,7 @@ 
      */
     public function Set_Char_Size(&$pdf, $pt)
     {
-		// phpcs:enable
+        // phpcs:enable
         if ($pt > 3) {
             $this->_Char_Size = $pt;
             $this->_Line_Height = $this->_Get_Height_Chars($pt);
@@ -163,8 +163,8 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * protected Print dot line
      *
@@ -179,7 +179,7 @@ 
      */
     protected function _Pointille(&$pdf, $x1 = 0, $y1 = 0, $x2 = 210, $y2 = 297, $epaisseur = 1, $nbPointilles = 15)
     {
-		// phpcs:enable
+        // phpcs:enable
         $pdf->SetLineWidth($epaisseur);
         $length = abs($x1 - $x2);
         $hauteur = abs($y1 - $y2);
@@ -210,8 +210,8 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * protected Function realisant une croix aux 4 coins des cartes
      *
@@ -228,7 +228,7 @@ 
      */
     protected function _Croix(&$pdf, $x1 = 0, $y1 = 0, $x2 = 210, $y2 = 297, $epaisseur = 1, $taille = 4)
     {
-		// phpcs:enable
+        // phpcs:enable
         $pdf->SetDrawColor(192, 192, 192);
 
         $pdf->SetLineWidth($epaisseur);
@@ -271,8 +271,8 @@ 
         return $value;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * protected Give the height for a char size given.
      *
@@ -281,7 +281,7 @@ 
      */
     protected function _Get_Height_Chars($pt)
     {
-		// phpcs:enable
+        // phpcs:enable
         // Array for link between height of characters and space between lines
         $_Table_Hauteur_Chars = array(6 => 2, 7 => 2.5, 8 => 3, 9 => 3.5, 10 => 4, 11 => 6, 12 => 7, 13 => 8, 14 => 9, 15 => 10);
         if (in_array($pt, array_keys($_Table_Hauteur_Chars))) {
@@ -291,8 +291,8 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * protected Set format
      *
@@ -302,7 +302,7 @@ 
      */
     protected function _Set_Format(&$pdf, $format)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->_Metric = $format['metric'];
         $this->_Avery_Name = $format['name'];
         $this->_Avery_Code = empty($format['code']) ? '' : $format['code'];

htdocs/core/class/extralanguages.class.php

Indentation +2 added lines, -2 removed lines

@@ -68,7 +68,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load array this->attributes with list of fields per object that need an alternate translation. The object and field must be managed with
      *  the widgetForTranslation() method.
@@ -81,7 +81,7 @@ 
      */
     public function fetch_name_extralanguages($elementtype, $forceload = false)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         if (empty($elementtype)) {

htdocs/core/class/CSMSFile.class.php

Indentation +4 added lines, -4 removed lines

@@ -222,7 +222,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Write content of a SendSms request into a dump file (mode = all)
      *  Used for debugging.
@@ -231,7 +231,7 @@ 
      */
     public function dump_sms()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $dolibarr_main_data_root;
 
         if (@is_writable($dolibarr_main_data_root)) {   // Avoid fatal error on fopen with open_basedir
@@ -252,7 +252,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Write content of a SendSms result into a dump file (mode = all)
      *  Used for debugging.
@@ -262,7 +262,7 @@ 
      */
     public function dump_sms_result($result)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $dolibarr_main_data_root;
 
         if (@is_writable($dolibarr_main_data_root)) {    // Avoid fatal error on fopen with open_basedir

Spacing +4 added lines, -4 removed lines

@@ -150,10 +150,10 @@ 
         if (!getDolGlobalString('MAIN_DISABLE_ALL_SMS')) {
             // Action according to the chose sending method
             if (getDolGlobalString('MAIN_SMS_SENDMODE')) {
-                $sendmode = getDolGlobalString('MAIN_SMS_SENDMODE');    // $conf->global->MAIN_SMS_SENDMODE looks like a value 'module'
-                $classmoduleofsender = getDolGlobalString('MAIN_MODULE_' . strtoupper($sendmode) . '_SMS', $sendmode);  // $conf->global->MAIN_MODULE_XXX_SMS looks like a value 'class@module'
+                $sendmode = getDolGlobalString('MAIN_SMS_SENDMODE'); // $conf->global->MAIN_SMS_SENDMODE looks like a value 'module'
+                $classmoduleofsender = getDolGlobalString('MAIN_MODULE_' . strtoupper($sendmode) . '_SMS', $sendmode); // $conf->global->MAIN_MODULE_XXX_SMS looks like a value 'class@module'
                 if ($classmoduleofsender == 'ovh') {
-                    $classmoduleofsender = 'ovhsms@ovh';    // For backward compatibility
+                    $classmoduleofsender = 'ovhsms@ovh'; // For backward compatibility
                 }
 
                 $tmp = explode('@', $classmoduleofsender);
@@ -209,7 +209,7 @@ 
                 // Send sms method not correctly defined
                 // --------------------------------------
                 $this->error = 'Bad value for MAIN_SMS_SENDMODE constant';
-                $res = false;  // @phan-suppress-current-line PhanPluginRedundantAssignment
+                $res = false; // @phan-suppress-current-line PhanPluginRedundantAssignment
             }
         } else {
             $this->error = 'No sms sent. Feature is disabled by option MAIN_DISABLE_ALL_SMS';

htdocs/core/class/canvas.class.php

Indentation +4 added lines, -4 removed lines

@@ -139,7 +139,7 @@ 
         //print ' => template_dir='.$this->template_dir.'<br>';
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Shared method for canvas to assign values for templates
      *
@@ -150,7 +150,7 @@ 
      */
     public function assign_values(&$action = 'view', $id = 0, $ref = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         if (is_object($this->control) && method_exists($this->control, 'assign_values')) {
             $this->control->assign_values($action, $id, $ref);
         }
@@ -176,7 +176,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Display a canvas page. This will include the template for output.
      *  Variables used by templates may have been defined or loaded before into the assign_values function.
@@ -186,7 +186,7 @@ 
      */
     public function display_canvas($action)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $db, $conf, $langs, $user, $canvas;
         global $form, $formfile;
 

htdocs/core/class/timespent.class.php

Indentation +2 added lines, -2 removed lines

@@ -847,7 +847,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the status
      *
@@ -857,7 +857,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
             global $langs;
             //$langs->load("timespent@timespent");

htdocs/core/class/doleditor.class.php

Indentation +2 added lines, -2 removed lines

@@ -117,7 +117,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output edit area inside the HTML stream.
      *  Output depends on this->tool (fckeditor, ckeditor, textarea, ...)
@@ -133,7 +133,7 @@ 
      */
     public function Create($noprint = 0, $morejs = '', $disallowAnyContent = true, $titlecontent = '', $option = '', $moreparam = '', $morecss = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $fullpage = false;

Spacing +2 added lines, -2 removed lines

@@ -158,7 +158,7 @@ 
                     define('REQUIRE_CKEDITOR', '1');
                 }
 
-                $skin = getDolGlobalString('FCKEDITOR_SKIN', 'moono-lisa');     // default with ckeditor 4.6 : moono-lisa
+                $skin = getDolGlobalString('FCKEDITOR_SKIN', 'moono-lisa'); // default with ckeditor 4.6 : moono-lisa
 
                 $pluginstodisable = 'elementspath,save,flash,div,anchor';
                 if (!getDolGlobalString('FCKEDITOR_ENABLE_SPECIALCHAR')) {
@@ -259,7 +259,7 @@ 
                                filebrowserImageWindowWidth : \'900\',
                                filebrowserImageWindowHeight : \'500\'';
                 }
-                $out .= '	})' . $morejs;    // end CKEditor.replace
+                $out .= '	})' . $morejs; // end CKEditor.replace
                 // Show the CKEditor javascript object once loaded is ready 'For debug)
                 //$out .= '; CKEDITOR.on(\'instanceReady\', function(ck) { ck.editor.removeMenuItem(\'maximize\'); ck.editor.removeMenuItem(\'Undo\'); ck.editor.removeMenuItem(\'undo\'); console.log(ck.editor); console.log(ck.editor.toolbar[0]); }); ';
                 $out .= '});' . "\n"; // end document.ready

htdocs/core/class/CMailFile.class.php

Indentation +19 added lines, -19 removed lines

@@ -1337,7 +1337,7 @@ 
         return '=?' . $conf->file->character_set_client . '?B?' . base64_encode($stringtoencode) . '?=';
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Read a file on disk and return encoded content for emails (mode = 'mail')
      *
@@ -1346,7 +1346,7 @@ 
      */
     private function _encode_file($sourcefile)
     {
-		// phpcs:enable
+        // phpcs:enable
         $newsourcefile = dol_osencode($sourcefile);
 
         if (is_readable($newsourcefile)) {
@@ -1361,7 +1361,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Write content of a SMTP request into a dump file (mode = all)
      *  Used for debugging.
@@ -1371,7 +1371,7 @@ 
      */
     public function dump_mail()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $dolibarr_main_data_root;
 
         if (@is_writable($dolibarr_main_data_root)) {   // Avoid fatal error on fopen with open_basedir
@@ -1402,7 +1402,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Save content if mail is in error
      *  Used for debugging.
@@ -1499,7 +1499,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Create SMTP headers (mode = 'mail')
      *
@@ -1507,7 +1507,7 @@ 
      */
     public function write_smtpheaders()
     {
-		// phpcs:enable
+        // phpcs:enable
         $out = "";
         $host = dol_getprefix('email');
 
@@ -1571,7 +1571,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Create header MIME (mode = 'mail')
      *
@@ -1581,7 +1581,7 @@ 
      */
     public function write_mimeheaders($filename_list, $mimefilename_list)
     {
-		// phpcs:enable
+        // phpcs:enable
         $mimedone = 0;
         $out = "";
 
@@ -1601,7 +1601,7 @@ 
         return $out;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return email content (mode = 'mail')
      *
@@ -1610,7 +1610,7 @@ 
      */
     public function write_body($msgtext)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $out = '';
@@ -1694,7 +1694,7 @@ 
         return $out;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Attach file to email (mode = 'mail')
      *
@@ -1706,7 +1706,7 @@ 
      */
     private function write_files($filename_list, $mimetype_list, $mimefilename_list, $cidlist)
     {
-		// phpcs:enable
+        // phpcs:enable
         $out = '';
 
         $filename_list_size = count($filename_list);
@@ -1745,7 +1745,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Attach an image to email (mode = 'mail')
      *
@@ -1754,7 +1754,7 @@ 
      */
     public function write_images($images_list)
     {
-		// phpcs:enable
+        // phpcs:enable
         $out = '';
 
         if (is_array($images_list)) {
@@ -1776,7 +1776,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Try to create a socket connection
      *
@@ -1786,7 +1786,7 @@ 
      */
     public function check_server_port($host, $port)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $_retVal = 0;
@@ -1857,7 +1857,7 @@ 
         return $_retVal;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * This function has been modified as provided by SirSir to allow multiline responses when
      * using SMTP Extensions.
@@ -1868,7 +1868,7 @@ 
      */
     public function server_parse($socket, $response)
     {
-		// phpcs:enable
+        // phpcs:enable
         $_retVal = true; // Indicates if Object was created or not
         $server_response = '';
 

Spacing +8 added lines, -8 removed lines

@@ -350,7 +350,7 @@ 
                     }
                 }
                 if ($emailtoadd && preg_match('/' . preg_quote($emailtoadd, '/') . '/i', $to)) {
-                    $emailtoadd = '';   // Email already in the "To"
+                    $emailtoadd = ''; // Email already in the "To"
                 }
                 if ($emailtoadd) {
                     $listofemailstoadd[$key] = $emailtoadd;
@@ -1107,8 +1107,8 @@ 
                     }
 
                     $smtperrorcode = 0;
-                    if (! $result) {
-                        $smtperrorcode = $this->smtps->lastretval;  // SMTP error code
+                    if (!$result) {
+                        $smtperrorcode = $this->smtps->lastretval; // SMTP error code
                         dol_syslog("CMailFile::sendfile: mail SMTP error code " . $smtperrorcode, LOG_WARNING);
 
                         if ($smtperrorcode == '421') {  // Try later
@@ -1126,10 +1126,10 @@ 
                         }
                     }
 
-                    $result = $this->smtps->getErrors();    // applicative error code (not SMTP error code)
+                    $result = $this->smtps->getErrors(); // applicative error code (not SMTP error code)
                     if (empty($this->error) && empty($result)) {
                         dol_syslog("CMailFile::sendfile: mail end success", LOG_DEBUG);
-                        $res = true;  // @phan-suppress-current-line PhanPluginRedundantAssignment
+                        $res = true; // @phan-suppress-current-line PhanPluginRedundantAssignment
                     } else {
                         if (empty($this->error)) {
                             $this->error = $result;
@@ -1376,7 +1376,7 @@ 
 
         if (@is_writable($dolibarr_main_data_root)) {   // Avoid fatal error on fopen with open_basedir
             $outputfile = $dolibarr_main_data_root . "/dolibarr_mail.log";
-            $fp = fopen($outputfile, "w");  // overwrite
+            $fp = fopen($outputfile, "w"); // overwrite
 
             if ($this->sendmode == 'mail') {
                 fwrite($fp, $this->headers);
@@ -1634,7 +1634,7 @@ 
             // Similar code to forge a text from html is also in smtps.class.php
             $strContentAltText = preg_replace("/<br\s*[^>]*>/", " ", $strContent);
             // TODO We could replace <img ...> with [Filename.ext] like Gmail do.
-            $strContentAltText = html_entity_decode(strip_tags($strContentAltText));    // Remove any HTML tags
+            $strContentAltText = html_entity_decode(strip_tags($strContentAltText)); // Remove any HTML tags
             $strContentAltText = trim(wordwrap($strContentAltText, 75, !getDolGlobalString('MAIN_FIX_FOR_BUGGED_MTA') ? "\r\n" : "\n"));
 
             // Check if html header already in message, if not complete the message
@@ -2019,7 +2019,7 @@ 
                 // We save the image to send in disk
                 $filecontent = $matches[2][$key];
 
-                $cid = 'cid000' . dol_hash($filecontent, 'md5');      // The id must not change if image is same
+                $cid = 'cid000' . dol_hash($filecontent, 'md5'); // The id must not change if image is same
 
                 $destfiletmp = $images_dir . '/' . $cid . '.' . $ext;