alxarafe / alixar

Dolibarr/Code/Core/Classes/FormCompany.php

Indentation +25 added lines, -25 removed lines

@@ -41,7 +41,7 @@ 
  */
 class FormCompany extends Form
 {
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Return list of labels (translated) of third parties type
      *
@@ -51,7 +51,7 @@ 
      */
     public function typent_array($mode = 0, $filter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $mysoc;
 
         $effs = array();
@@ -92,7 +92,7 @@ 
         return $effs;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the list of entries for staff (no translation, it is number ranges)
      *
@@ -102,7 +102,7 @@ 
      */
     public function effectif_array($mode = 0, $filter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         $effs = array();
 
         $sql = "SELECT id, code, libelle as label";
@@ -137,7 +137,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Affiche formulaire de selection des modes de reglement
      *
@@ -149,7 +149,7 @@ 
      */
     public function form_prospect_level($page, $selected = 0, $htmlname = 'prospect_level_id', $empty = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user, $langs;
 
         print '<form method="post" action="' . $page . '">';
@@ -241,7 +241,7 @@ 
         print '</form>';
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *   Returns the drop-down list of departments/provinces/cantons for all countries or for a given country.
      *   In the case of an all-country list, the display breaks on the country.
@@ -255,11 +255,11 @@ 
      */
     public function select_departement($selected = '', $country_codeid = 0, $htmlname = 'state_id')
     {
-		// phpcs:enable
+        // phpcs:enable
         print $this->select_state($selected, $country_codeid, $htmlname);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *   Returns the drop-down list of departments/provinces/cantons for all countries or for a given country.
      *   In the case of an all-country list, the display breaks on the country.
@@ -275,7 +275,7 @@ 
      */
     public function select_state($selected = 0, $country_codeid = 0, $htmlname = 'state_id', $morecss = 'maxwidth200onsmartphone  minwidth300')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $user;
 
         dol_syslog(get_class($this) . "::select_departement selected=" . $selected . ", country_codeid=" . $country_codeid, LOG_DEBUG);
@@ -371,7 +371,7 @@ 
         return $out;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *   Returns the drop-down list of departments/provinces/cantons for all countries or for a given country.
      *   In the case of an all-country list, the display breaks on the country.
@@ -398,7 +398,7 @@ 
         return $html . '</script><span id="target_' . $htmlname . '">' . $this->select_state($selected, $country_codeid, $htmlname, $morecss) . '</span>';
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *   Retourne la liste deroulante des regions actives don't le pays est actif
      *   La cle de la liste est le code (il peut y avoir plusieurs entree pour
@@ -411,7 +411,7 @@ 
      */
     public function select_region($selected = '', $htmlname = 'region_id')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
         $langs->load("dict");
 
@@ -457,7 +457,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return combo list with people title
      *
@@ -469,7 +469,7 @@ 
      */
     public function select_civility($selected = '', $htmlname = 'civility_id', $morecss = 'maxwidth150', $addjscombo = 1)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $user;
         $langs->load("dict");
 
@@ -516,7 +516,7 @@ 
         return $out;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Return the list of all juridical entity types for all countries or a specific country.
      *    A country separator is included in case the list for all countries is returned.
@@ -530,11 +530,11 @@ 
      */
     public function select_forme_juridique($selected = '', $country_codeid = 0, $filter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         print $this->select_juridicalstatus($selected, $country_codeid, $filter);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Return the list of all juridical entity types for all countries or a specific country.
      *    A country separator is included in case the list for all countries is returned.
@@ -548,7 +548,7 @@ 
      */
     public function select_juridicalstatus($selected = '', $country_codeid = 0, $filter = '', $htmlname = 'forme_juridique_code', $morecss = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $user;
         $langs->load("dict");
 
@@ -894,7 +894,7 @@ 
         return 'ErrorBadValueForParameterRenderMode'; // Should not happened
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Return a select list with zip codes and their town
      *
@@ -909,7 +909,7 @@ 
      */
     public function select_ziptown($selected = '', $htmlname = 'zipcode', $fields = array(), $fieldsize = 0, $disableautocomplete = 0, $moreattrib = '', $morecss = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $out = '';
@@ -928,7 +928,7 @@ 
         return $out;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return HTML string to use as input of professional id into a HTML page (siren, siret, etc...)
      *
@@ -941,7 +941,7 @@ 
      */
     public function get_input_id_prof($idprof, $htmlname, $preselected, $country_code, $morecss = 'maxwidth200')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $hookmanager;
 
         $formlength = 0;
@@ -1006,7 +1006,7 @@ 
         return $out;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return a HTML select with localtax values for thirdparties
      *
@@ -1017,7 +1017,7 @@ 
      */
     public function select_localtax($local, $selected, $htmlname)
     {
-		// phpcs:enable
+        // phpcs:enable
         $tax = get_localtax_by_third($local);
 
         if ($tax) {

Dolibarr/Code/Core/Classes/FormSms.php

Indentation +2 added lines, -2 removed lines

@@ -122,7 +122,7 @@ 
         $this->withbodyreadonly = 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Show the form to input an sms.
      *
@@ -132,7 +132,7 @@ 
      */
     public function show_form($morecss = 'titlefield', $showform = 1)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $form;
 
         if (!is_object($form)) {

Spacing +3 added lines, -3 removed lines

@@ -209,10 +209,10 @@
             } else {
                 print '<tr><td class="' . $morecss . '">' . $langs->trans("SmsFrom") . "</td><td>";
                 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);

Dolibarr/Code/Core/Classes/Translate.php

Spacing +2 added lines, -2 removed lines

@@ -319,7 +319,7 @@ 
             $file_lang_osencoded = dol_osencode($file_lang);
 
             //$filelangexists = is_file($file_lang_osencoded);
-            $filelangexists = @is_file($file_lang_osencoded);   // avoid [php:warn]
+            $filelangexists = @is_file($file_lang_osencoded); // avoid [php:warn]
 
             //dol_syslog(get_class($this).'::Load Try to read for alt='.$alt.' langofdir='.$langofdir.' domain='.$domain.' newdomain='.$newdomain.' modulename='.$modulename.' file_lang='.$file_lang." => filelangexists=".$filelangexists);
             //print 'Try to read for alt='.$alt.' langofdir='.$langofdir.' domain='.$domain.' newdomain='.$newdomain.' modulename='.$modulename.' this->_tab_loaded[newdomain]='.$this->_tab_loaded[$newdomain].' file_lang='.$file_lang." => filelangexists=".$filelangexists."\n";
@@ -1186,7 +1186,7 @@ 
                 if ($obj) {
                     // If a translation exists, we use it lese we use the default label
                     $this->cache_currencies[$obj->code_iso]['label'] = ($obj->code_iso && $this->trans("Currency" . $obj->code_iso) != "Currency" . $obj->code_iso ? $this->trans("Currency" . $obj->code_iso) : ($obj->label != '-' ? $obj->label : ''));
-                    $this->cache_currencies[$obj->code_iso]['unicode'] = (array)json_decode((empty($obj->unicode) ? '' : $obj->unicode), true);
+                    $this->cache_currencies[$obj->code_iso]['unicode'] = (array) json_decode((empty($obj->unicode) ? '' : $obj->unicode), true);
                     $label[$obj->code_iso] = $this->cache_currencies[$obj->code_iso]['label'];
                 }
                 $i++;

Dolibarr/Code/Core/Classes/FormActions.php

Indentation +4 added lines, -4 removed lines

@@ -53,7 +53,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Show list of action status
      *
@@ -68,7 +68,7 @@ 
      */
     public function form_select_status_action($formname, $selected, $canedit = 1, $htmlname = 'complete', $showempty = 0, $onlyselect = 0, $morecss = 'maxwidth100')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf;
 
         $listofstatus = array(
@@ -365,7 +365,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output html select list of type of event
      *
@@ -381,7 +381,7 @@ 
      */
     public function select_type_actions($selected = '', $htmlname = 'actioncode', $excludetype = '', $onlyautoornot = 0, $hideinfohelp = 0, $multiselect = 0, $nooutput = 0, $morecss = 'minwidth300')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $user, $form;
 
         if (!is_object($form)) {

Spacing +1 added lines, -1 removed lines

@@ -366,7 +366,7 @@
         $caction = new CActionComm($this->db);
 
         // Suggest a list with manual events or all auto events
-        $arraylist = $caction->liste_array(1, 'code', $excludetype, $onlyautoornot, '', 0);     // If we use param 'all' instead of 'code', there is no group by include in answer but the key 'type' of answer array contains the key for the group by.
+        $arraylist = $caction->liste_array(1, 'code', $excludetype, $onlyautoornot, '', 0); // If we use param 'all' instead of 'code', there is no group by include in answer but the key 'type' of answer array contains the key for the group by.
         if (empty($multiselect)) {
             // Add empty line at start only if no multiselect
             array_unshift($arraylist, ' ');

Dolibarr/Code/Core/Classes/FormProjets.php

Spacing +1 added lines, -1 removed lines

@@ -792,7 +792,7 @@
         if (is_array($selected)) {
             $selectedarray = $selected;
         } elseif ($selected == 99) {
-            $selectedarray = array(0,1);
+            $selectedarray = array(0, 1);
         } else {
             $selectedarray = explode(',', $selected);
         }

Indentation +6 added lines, -6 removed lines

@@ -64,7 +64,7 @@ 
         $this->db = $db;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
 
     /**
      * Output a combo list with projects qualified for a third party / user
@@ -89,7 +89,7 @@ 
      */
     public function select_projects($socid = -1, $selected = '', $htmlname = 'projectid', $maxlength = 16, $option_only = 0, $show_empty = 1, $discard_closed = 0, $forcefocus = 0, $disabled = 0, $mode = 0, $filterkey = '', $nooutput = 0, $forceaddid = 0, $morecss = '', $htmlid = '', $morefilter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf, $form;
 
         $selected_input_value = '';
@@ -132,7 +132,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
 
     /**
      * Returns an array with projects qualified for a third party
@@ -157,7 +157,7 @@ 
      */
     public function select_projects_list($socid = -1, $selected = 0, $htmlname = 'projectid', $maxlength = 24, $option_only = 0, $show_empty = 1, $discard_closed = 0, $forcefocus = 0, $disabled = 0, $mode = 0, $filterkey = '', $nooutput = 0, $forceaddid = 0, $htmlid = '', $morecss = 'maxwidth500', $morefilter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user, $conf, $langs;
 
 
@@ -529,7 +529,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
 
     /**
      *    Build a HTML select list of element of same thirdparty to suggest to link them to project
@@ -544,7 +544,7 @@ 
      */
     public function select_element($table_element, $socid = 0, $morecss = '', $limitonstatus = -2, $projectkey = "fk_projet", $placeholder = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         if ($table_element == 'projet_task') {

Dolibarr/Code/Core/Classes/CMailFile.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 +10 added lines, -10 removed lines

@@ -365,7 +365,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;
@@ -559,7 +559,7 @@ 
                     $this->buildCSS();
                 }
                 $msg = $this->html;
-                $msg = $this->checkIfHTML($msg);        // This add a header and a body including custom CSS to the HTML content
+                $msg = $this->checkIfHTML($msg); // This add a header and a body including custom CSS to the HTML content
             }
 
             // Replace . alone on a new line with .. to avoid to have SMTP interpret this as end of message
@@ -706,7 +706,7 @@ 
                     $this->buildCSS();
                 }
                 $msg = $this->html;
-                $msg = $this->checkIfHTML($msg);        // This add a header and a body including custom CSS to the HTML content
+                $msg = $this->checkIfHTML($msg); // This add a header and a body including custom CSS to the HTML content
             }
 
             if ($this->atleastoneimage) {
@@ -1149,8 +1149,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
@@ -1168,7 +1168,7 @@ 
                         }
                     }
 
-                    $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;
@@ -1418,7 +1418,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);
@@ -1683,11 +1683,11 @@ 
             // 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
-            $strContent = $this->checkIfHTML($strContent);      // This add a header and a body including custom CSS to the HTML content
+            $strContent = $this->checkIfHTML($strContent); // This add a header and a body including custom CSS to the HTML content
         }
 
         // Make RFC2045 Compliant, split lines
@@ -2075,7 +2075,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;
 

Dolibarr/Code/Core/Classes/CommonDocGenerator.php

Indentation +20 added lines, -20 removed lines

@@ -193,7 +193,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Define array with couple substitution key => substitution value
      *
@@ -203,7 +203,7 @@ 
      */
     public function get_substitutionarray_user($user, $outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $extrafields;
 
         $logotouse = $conf->user->dir_output . '/' . get_exdir(0, 0, 0, 0, $user, 'user') . 'photos/' . getImageFileNameForSize($user->photo, '_small');
@@ -289,7 +289,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Define array with couple substitution key => substitution value
      *
@@ -299,7 +299,7 @@ 
      */
     public function get_substitutionarray_mysoc($mysoc, $outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         if (empty($mysoc->forme_juridique) && !empty($mysoc->forme_juridique_code)) {
@@ -346,7 +346,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Define array with couple substitution key => substitution value
      * For example {company_name}, {company_name_alias}
@@ -358,7 +358,7 @@ 
      */
     public function get_substitutionarray_thirdparty($object, $outputlangs, $array_key = 'company')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $extrafields;
 
         if (empty($object->country) && !empty($object->country_code)) {
@@ -412,7 +412,7 @@ 
         return $array_thirdparty;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Define array with couple substitution key => substitution value
      *
@@ -423,7 +423,7 @@ 
      */
     public function get_substitutionarray_contact($object, $outputlangs, $array_key = 'object')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $extrafields;
 
         if (empty($object->country) && !empty($object->country_code)) {
@@ -472,7 +472,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Define array with couple substitution key => substitution value
      *
@@ -481,7 +481,7 @@ 
      */
     public function get_substitutionarray_other($outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $now = dol_now('gmt'); // gmt
@@ -512,7 +512,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Define array with couple substitution key => substitution value
      * Note that vars into substitutions array are formatted.
@@ -524,7 +524,7 @@ 
      */
     public function get_substitutionarray_object($object, $outputlangs, $array_key = 'object')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $extrafields;
 
         $sumpayed = $sumdeposit = $sumcreditnote = '';
@@ -715,7 +715,7 @@ 
         return $resarray;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Define array with couple substitution key => substitution value
      *  Note that vars into substitutions array are formatted.
@@ -727,7 +727,7 @@ 
      */
     public function get_substitutionarray_lines($line, $outputlangs, $linenumber = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $resarray = array(
             'line_pos' => $linenumber,
             'line_fulldesc' => doc_getlinedesc($line, $outputlangs),
@@ -839,7 +839,7 @@ 
         return $resarray;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Define array with couple substitution key => substitution value
      * Note that vars into substitutions array are formatted.
@@ -851,7 +851,7 @@ 
      */
     public function get_substitutionarray_shipment($object, $outputlangs, $array_key = 'object')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $extrafields;
 
         include_once DOL_DOCUMENT_ROOT . '/core/lib/product.lib.php';
@@ -908,7 +908,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Define array with couple substitution key => substitution value
      *
@@ -919,7 +919,7 @@ 
      */
     public function get_substitutionarray_each_var_object(&$object, $outputlangs, $recursive = 1)
     {
-		// phpcs:enable
+        // phpcs:enable
         $array_other = array();
         if (!empty($object)) {
             foreach ($object as $key => $value) {
@@ -948,7 +948,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Fill array with couple extrafield key => extrafield value
      *  Note that vars into substitutions array are formatted.
@@ -962,7 +962,7 @@ 
      */
     public function fill_substitutionarray_with_extrafields($object, $array_to_fill, $extrafields, $array_key, $outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         if ($extrafields->attributes[$object->table_element]['count'] > 0) {

Spacing +4 added lines, -4 removed lines

@@ -166,8 +166,8 @@ 
     public $posxlabel;
     public $posxup;
     public $posxref;
-    public $posxpicture;    // For picture
-    public $posxdesc;       // For description
+    public $posxpicture; // For picture
+    public $posxdesc; // For description
     public $posxqty;
     public $posxpuht;
     public $posxtva;
@@ -546,7 +546,7 @@ 
 
         if (get_class($object) == 'CommandeFournisseur') {
             /** @var CommandeFournisseur $object*/
-            $object->date_validation =  $object->date_valid;
+            $object->date_validation = $object->date_valid;
             $object->date_commande = $object->date;
         }
         $resarray = array(
@@ -1500,7 +1500,7 @@ 
                  * @param stdClass $b
                  * @return int<-1,1>
                  */
-                static function ($a, $b) {
+                static function($a, $b) {
                     return  ($a->rank > $b->rank) ? 1 : -1;
                 }
             );

Dolibarr/Code/Core/Classes/FormSocialContrib.php

Indentation +2 added lines, -2 removed lines

@@ -49,7 +49,7 @@ 
         $this->db = $db;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of social contributions.
      *  Use mysoc->country_id or mysoc->country_code so they must be defined.
@@ -65,7 +65,7 @@ 
      */
     public function select_type_socialcontrib($selected = '', $htmlname = 'actioncode', $useempty = 0, $maxlen = 40, $help = 1, $morecss = 'minwidth300', $noerrorifempty = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $db, $langs, $user, $mysoc;
 
         if (empty($mysoc->country_id) && empty($mysoc->country_code)) {

Dolibarr/Code/Core/Classes/ExtraLanguages.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)) {