alxarafe / alixar

htdocs/core/class/html.formcompany.class.php

Indentation +25 added lines, -25 removed lines

@@ -40,7 +40,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
      *
@@ -50,7 +50,7 @@ 
      */
     public function typent_array($mode = 0, $filter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $mysoc;
 
         $effs = array();
@@ -91,7 +91,7 @@ 
         return $effs;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Renvoie la liste des types d'effectifs possibles (pas de traduction car nombre)
      *
@@ -101,7 +101,7 @@ 
      */
     public function effectif_array($mode = 0, $filter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         $effs = array();
 
         $sql = "SELECT id, code, libelle as label";
@@ -136,7 +136,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Affiche formulaire de selection des modes de reglement
      *
@@ -148,7 +148,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 . '">';
@@ -240,7 +240,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.
@@ -254,11 +254,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.
@@ -274,7 +274,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);
@@ -370,7 +370,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.
@@ -397,7 +397,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
@@ -410,7 +410,7 @@ 
      */
     public function select_region($selected = '', $htmlname = 'region_id')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
         $langs->load("dict");
 
@@ -456,7 +456,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return combo list with people title
      *
@@ -468,7 +468,7 @@ 
      */
     public function select_civility($selected = '', $htmlname = 'civility_id', $morecss = 'maxwidth150', $addjscombo = 1)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $user;
         $langs->load("dict");
 
@@ -515,7 +515,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.
@@ -529,11 +529,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.
@@ -547,7 +547,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");
 
@@ -893,7 +893,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
      *
@@ -908,7 +908,7 @@ 
      */
     public function select_ziptown($selected = '', $htmlname = 'zipcode', $fields = array(), $fieldsize = 0, $disableautocomplete = 0, $moreattrib = '', $morecss = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $out = '';
@@ -927,7 +927,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...)
      *
@@ -940,7 +940,7 @@ 
      */
     public function get_input_id_prof($idprof, $htmlname, $preselected, $country_code, $morecss = 'maxwidth200')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $hookmanager;
 
         $formlength = 0;
@@ -1005,7 +1005,7 @@ 
         return $out;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return a HTML select with localtax values for thirdparties
      *
@@ -1016,7 +1016,7 @@ 
      */
     public function select_localtax($local, $selected, $htmlname)
     {
-		// phpcs:enable
+        // phpcs:enable
         $tax = get_localtax_by_third($local);
 
         if ($tax) {

htdocs/core/class/interfaces.class.php

Indentation +2 added lines, -2 removed lines

@@ -59,7 +59,7 @@ 
         $this->db = $db;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *   Function called when a Dolibarr business event occurs
      *   This function call all qualified triggers.
@@ -73,7 +73,7 @@ 
      */
     public function run_triggers($action, $object, $user, $langs, $conf)
     {
-		// phpcs:enable
+        // phpcs:enable
 
         if (getDolGlobalInt('MAIN_TRIGGER_DEBUG')) {
             // This his too much verbose, enabled if const enabled only

htdocs/core/class/html.formactions.class.php

Indentation +4 added lines, -4 removed lines

@@ -52,7 +52,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Show list of action status
      *
@@ -67,7 +67,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(
@@ -338,7 +338,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output html select list of type of event
      *
@@ -354,7 +354,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, $conf;
 
         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, ' ');

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/commonobject.class.php

Indentation +62 added lines, -62 removed lines

@@ -1153,7 +1153,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Add a link between element $this->element and a contact
      *
@@ -1165,7 +1165,7 @@ 
      */
     public function add_contact($fk_socpeople, $type_contact, $source = 'external', $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user, $langs;
 
 
@@ -1263,7 +1263,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Copy contact from one element to current
      *
@@ -1273,7 +1273,7 @@ 
      */
     public function copy_linked_contact($objFrom, $source = 'internal')
     {
-		// phpcs:enable
+        // phpcs:enable
         $contacts = $objFrom->liste_contact(-1, $source);
         foreach ($contacts as $contact) {
             if ($this->add_contact($contact['id'], $contact['fk_c_type_contact'], $contact['source']) < 0) {
@@ -1283,7 +1283,7 @@ 
         return 1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Update a link to contact line
      *
@@ -1295,7 +1295,7 @@ 
      */
     public function update_contact($rowid, $statut, $type_contact_id = 0, $fk_socpeople = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         // Insert into database
         $sql = "UPDATE " . $this->db->prefix() . "element_contact set";
         $sql .= " statut = " . $statut;
@@ -1315,7 +1315,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Delete a link to contact line
      *
@@ -1325,7 +1325,7 @@ 
      */
     public function delete_contact($rowid, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user;
 
         $error = 0;
@@ -1365,7 +1365,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Delete all links between an object $this and all its contacts in llx_element_contact
      *
@@ -1375,7 +1375,7 @@ 
      */
     public function delete_linked_contact($source = '', $code = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         $listId = '';
         $temp = array();
         $typeContact = $this->liste_type_contact($source, '', 0, 0, $code);
@@ -1406,7 +1406,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Get array of all contacts for an object
      *
@@ -1420,7 +1420,7 @@ 
      */
     public function liste_contact($statusoflink = -1, $source = 'external', $list = 0, $code = '', $status = -1, $arrayoftcids = array())
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $tab = array();
@@ -1542,7 +1542,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Return array with list of possible values for type of contacts
      *
@@ -1555,7 +1555,7 @@ 
      */
     public function liste_type_contact($source = 'internal', $order = 'position', $option = 0, $activeonly = 0, $code = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         if (empty($order)) {
@@ -1759,7 +1759,7 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Load object contact with id=$this->contact_id into $this->contact
      *
@@ -1768,7 +1768,7 @@ 
      */
     public function fetch_contact($contactid = null)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (empty($contactid)) {
             $contactid = $this->contact_id;
         }
@@ -1784,7 +1784,7 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Load the third party of object, from id $this->socid or $this->fk_soc, into this->thirdparty
      *
@@ -1793,7 +1793,7 @@ 
      */
     public function fetch_thirdparty($force_thirdparty_id = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (empty($this->socid) && empty($this->fk_soc) && empty($force_thirdparty_id)) {
             return 0;
         }
@@ -1860,7 +1860,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load data for barcode into properties ->barcode_type*
      *  Properties ->barcode_type that is id of barcode. Type is used to find other properties, but
@@ -1870,7 +1870,7 @@ 
      */
     public function fetch_barcode()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         dol_syslog(get_class($this) . '::fetch_barcode this->element=' . $this->element . ' this->barcode_type=' . $this->barcode_type);
@@ -1909,7 +1909,7 @@ 
         return 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Load the project with id $this->fk_project into this->project
      *
@@ -1917,11 +1917,11 @@ 
      */
     public function fetch_project()
     {
-		// phpcs:enable
+        // phpcs:enable
         return $this->fetch_projet();
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Load the project with id $this->fk_project into this->project
      *
@@ -1929,7 +1929,7 @@ 
      */
     public function fetch_projet()
     {
-		// phpcs:enable
+        // phpcs:enable
         include_once DOL_DOCUMENT_ROOT . '/projet/class/project.class.php';
 
         if (empty($this->fk_project) && !empty($this->fk_projet)) {
@@ -1947,7 +1947,7 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Load the product with id $this->fk_product into this->product
      *
@@ -1955,7 +1955,7 @@ 
      */
     public function fetch_product()
     {
-		// phpcs:enable
+        // phpcs:enable
         include_once DOL_DOCUMENT_ROOT . '/product/class/product.class.php';
 
         if (empty($this->fk_product)) {
@@ -1969,7 +1969,7 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Load the user with id $userid into this->user
      *
@@ -1978,14 +1978,14 @@ 
      */
     public function fetch_user($userid)
     {
-		// phpcs:enable
+        // phpcs:enable
         $user = new User($this->db);
         $result = $user->fetch($userid);
         $this->user = $user;
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Read linked origin object.
      *  Set ->origin_object
@@ -1995,7 +1995,7 @@ 
      */
     public function fetch_origin()
     {
-		// phpcs:enable
+        // phpcs:enable
         if ($this->origin == 'shipping') {
             $this->origin = 'expedition';
         }
@@ -2222,7 +2222,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Load properties id_previous and id_next by comparing $fieldid with $this->ref
      *
@@ -2234,7 +2234,7 @@ 
      */
     public function load_previous_next_ref($filter, $fieldid, $nodbprefix = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
 
         if (!$this->table_element) {
@@ -3174,7 +3174,7 @@ 
 
     // TODO: Move line related operations to CommonObjectLine?
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Save a new position (field rang) for details lines.
      *  You can choose to set position for lines with already a position or lines without any position defined.
@@ -3186,7 +3186,7 @@ 
      */
     public function line_order($renum = false, $rowidorder = 'ASC', $fk_parent_line = true)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (!$this->table_element_line) {
             dol_syslog(get_class($this) . "::line_order was called on object with property table_element_line not defined", LOG_ERR);
             return -1;
@@ -3298,7 +3298,7 @@ 
         return $rows;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update a line to have a lower rank
      *
@@ -3308,7 +3308,7 @@ 
      */
     public function line_up($rowid, $fk_parent_line = true)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->line_order(false, 'ASC', $fk_parent_line);
 
         // Get rang of line
@@ -3318,7 +3318,7 @@ 
         $this->updateLineUp($rowid, $rang);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update a line to have a higher rank
      *
@@ -3328,7 +3328,7 @@ 
      */
     public function line_down($rowid, $fk_parent_line = true)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->line_order(false, 'ASC', $fk_parent_line);
 
         // Get rang of line
@@ -3371,7 +3371,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update position of line with ajax (rang)
      *
@@ -3380,7 +3380,7 @@ 
      */
     public function line_ajaxorder($rows)
     {
-		// phpcs:enable
+        // phpcs:enable
         $num = count($rows);
         for ($i = 0; $i < $num; $i++) {
             $this->updateRangOfLine($rows[$i], ($i + 1));
@@ -3499,7 +3499,7 @@ 
         return 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Get max value used for position of line (rang)
      *
@@ -3508,7 +3508,7 @@ 
      */
     public function line_max($fk_parent_line = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $positionfield = 'rang';
         if (in_array($this->table_element, array('bom_bom', 'product_attribute'))) {
             $positionfield = 'position';
@@ -3546,7 +3546,7 @@ 
         return 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update external ref of element
      *
@@ -3555,7 +3555,7 @@ 
      */
     public function update_ref_ext($ref_ext)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (!$this->table_element) {
             dol_syslog(get_class($this) . "::update_ref_ext was called on object with property table_element not defined", LOG_ERR);
             return -1;
@@ -3575,7 +3575,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update note of element
      *
@@ -3586,7 +3586,7 @@ 
      */
     public function update_note($note, $suffix = '', $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user;
 
         if (!$this->table_element) {
@@ -3663,7 +3663,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update public note (kept for backward compatibility)
      *
@@ -3674,11 +3674,11 @@ 
      */
     public function update_note_public($note)
     {
-		// phpcs:enable
+        // phpcs:enable
         return $this->update_note($note, '_public');
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update total_ht, total_ttc, total_vat, total_localtax1, total_localtax2 for an object (sum of lines).
      *  Must be called at end of methods addline or updateline.
@@ -3691,7 +3691,7 @@ 
      */
     public function update_price($exclspec = 0, $roundingadjust = 'none', $nodatabaseupdate = 0, $seller = null)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $hookmanager, $action;
 
         $parameters = array('exclspec' => $exclspec, 'roundingadjust' => $roundingadjust, 'nodatabaseupdate' => $nodatabaseupdate, 'seller' => $seller);
@@ -3970,7 +3970,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Add an object link into llx_element_element.
      *
@@ -3983,7 +3983,7 @@ 
      */
     public function add_object_linked($origin = null, $origin_id = null, $f_user = null, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user, $hookmanager, $action;
         $origin = (!empty($origin) ? $origin : $this->origin);
         $origin_id = (!empty($origin_id) ? $origin_id : $this->origin_id);
@@ -5455,7 +5455,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Add resources to the current object : add entry into llx_element_resources
      *  Need $this->element & $this->id
@@ -5468,7 +5468,7 @@ 
      */
     public function add_element_resource($resource_id, $resource_type, $busy = 0, $mandatory = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->db->begin();
 
         $sql = "INSERT INTO " . $this->db->prefix() . "element_resources (";
@@ -5498,7 +5498,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Delete a link to resource line
      *
@@ -5509,7 +5509,7 @@ 
      */
     public function delete_resource($rowid, $element, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user;
 
         $this->db->begin();
@@ -5979,7 +5979,7 @@ 
     /* For triggers */
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Call trigger based on this instance.
      * Some context information may also be provided into array property this->context.
@@ -5992,7 +5992,7 @@ 
      */
     public function call_trigger($triggerName, $user)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf;
 
         if (!empty(self::TRIGGER_PREFIX) && strpos($triggerName, self::TRIGGER_PREFIX . '_') !== 0) {
@@ -6197,7 +6197,7 @@ 
         return $ret;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Function to get extra fields of an object into $this->array_options
      *  This method is in most cases called by method fetch of objects but you can call it separately.
@@ -6209,7 +6209,7 @@ 
      */
     public function fetch_optionals($rowid = null, $optionsArray = null)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $extrafields;
 
         if (empty($rowid)) {
@@ -9064,7 +9064,7 @@ 
         return array('dir' => $dir, 'file' => $file, 'originalfile' => $originalfile, 'altfile' => $altfile, 'email' => $email, 'capture' => $capture);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Show photos of an object (nbmax maximum), into several columns
      *
@@ -9086,7 +9086,7 @@ 
      */
     public function show_photos($modulepart, $sdir, $size = 0, $nbmax = 0, $nbbyrow = 5, $showfilename = 0, $showaction = 0, $maxHeight = 120, $maxWidth = 160, $nolink = 0, $overwritetitle = 0, $usesharelink = 0, $cache = '', $addphotorefcss = 'photoref')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user, $langs;
 
         include_once DOL_DOCUMENT_ROOT . '/core/lib/files.lib.php';

Spacing +27 added lines, -27 removed lines

@@ -508,12 +508,12 @@ 
     /**
      * @var float Multicurrency total localta1
      */
-    public $multicurrency_total_localtax1;  // not in database
+    public $multicurrency_total_localtax1; // not in database
 
     /**
      * @var float Multicurrency total localtax2
      */
-    public $multicurrency_total_localtax2;  // not in database
+    public $multicurrency_total_localtax2; // not in database
 
     /**
      * @var string
@@ -3607,7 +3607,7 @@ 
             $newsuffix = '';
         }
         if (in_array($this->table_element, array('actioncomm', 'adherent', 'advtargetemailing', 'cronjob', 'establishment'))) {
-            $fieldusermod =  "fk_user_mod";
+            $fieldusermod = "fk_user_mod";
         } elseif ($this->table_element == 'ecm_files') {
             $fieldusermod = "fk_user_m";
         } else {
@@ -4554,14 +4554,14 @@ 
             $sql .= ", date_validation = '" . $this->db->idate(dol_now()) . "'";
         }
         $sql .= " WHERE rowid = " . ((int) $elementId);
-        $sql .= " AND " . $fieldstatus . " <> " . ((int) $status);    // We avoid update if status already correct
+        $sql .= " AND " . $fieldstatus . " <> " . ((int) $status); // We avoid update if status already correct
 
         dol_syslog(get_class($this) . "::setStatut", LOG_DEBUG);
         $resql = $this->db->query($sql);
         if ($resql) {
             $error = 0;
 
-            $nb_rows_affected = $this->db->affected_rows($resql);   // should be 1 or 0 if status was already correct
+            $nb_rows_affected = $this->db->affected_rows($resql); // should be 1 or 0 if status was already correct
 
             if ($nb_rows_affected > 0) {
                 if (empty($trigkey)) {
@@ -4606,7 +4606,7 @@ 
                     if ($fieldstatus == 'tosell') {
                         $this->status = $status;
                     } elseif ($fieldstatus == 'tobuy') {
-                        $this->status_buy = $status;    // @phpstan-ignore-line
+                        $this->status_buy = $status; // @phpstan-ignore-line
                     } else {
                         $this->statut = $status;
                         $this->status = $status;
@@ -4715,7 +4715,7 @@ 
             return -1;
         }
 
-        $arraytoscan = $this->childtables;      // array('tablename'=>array('fk_element'=>'parentfield'), ...) or array('tablename'=>array('parent'=>table_parent, 'parentkey'=>'nameoffieldforparentfkkey'), ...)
+        $arraytoscan = $this->childtables; // array('tablename'=>array('fk_element'=>'parentfield'), ...) or array('tablename'=>array('parent'=>table_parent, 'parentkey'=>'nameoffieldforparentfkkey'), ...)
         // For backward compatibility, we check if array is old format array('tablename1', 'tablename2', ...)
         $tmparray = array_keys($this->childtables);
         if (is_numeric($tmparray[0])) {
@@ -5783,7 +5783,7 @@ 
             $setsharekey = false;
             if ($this->element == 'propal' || $this->element == 'proposal') {
                 if (getDolGlobalInt("PROPOSAL_ALLOW_ONLINESIGN")) {
-                    $setsharekey = true;    // feature to make online signature is not set or set to on (default)
+                    $setsharekey = true; // feature to make online signature is not set or set to on (default)
                 }
                 if (getDolGlobalInt("PROPOSAL_ALLOW_EXTERNAL_DOWNLOAD")) {
                     $setsharekey = true;
@@ -5841,7 +5841,7 @@ 
                 $ecmfile->gen_or_uploaded = 'generated';
                 $ecmfile->description = ''; // indexed content
                 $ecmfile->keywords = ''; // keyword content
-                $ecmfile->src_object_type = $this->table_element;   // $this->table_name is 'myobject' or 'mymodule_myobject'.
+                $ecmfile->src_object_type = $this->table_element; // $this->table_name is 'myobject' or 'mymodule_myobject'.
                 $ecmfile->src_object_id   = $this->id;
 
                 $result = $ecmfile->create($user);
@@ -5893,7 +5893,7 @@ 
             $maxwidthmini = $tmparraysize['maxwidthmini'];
             $maxheightmini = $tmparraysize['maxheightmini'];
             //$quality = $tmparraysize['quality'];
-            $quality = 50;  // For thumbs, we force quality to 50
+            $quality = 50; // For thumbs, we force quality to 50
 
             // Create small thumbs for company (Ratio is near 16/9)
             // Used on logon for example
@@ -6190,7 +6190,7 @@ 
         $savDisableCompute = $conf->disable_compute;
         $conf->disable_compute = 1;
 
-        $ret = $this->fetch($id);   /* @phpstan-ignore-line */
+        $ret = $this->fetch($id); /* @phpstan-ignore-line */
 
         $conf->disable_compute = $savDisableCompute;
 
@@ -6294,8 +6294,8 @@ 
                         if (!empty($extrafields->attributes[$this->table_element]) && !empty($extrafields->attributes[$this->table_element]['computed'][$key])) {
                             //var_dump($conf->disable_compute);
                             if (empty($conf->disable_compute)) {
-                                global $objectoffield;        // We set a global variable to $objectoffield so
-                                $objectoffield = $this;        // we can use it inside computed formula
+                                global $objectoffield; // We set a global variable to $objectoffield so
+                                $objectoffield = $this; // we can use it inside computed formula
                                 $this->array_options['options_' . $key] = dol_eval($extrafields->attributes[$this->table_element]['computed'][$key], 1, 0, '2');
                             }
                         }
@@ -6477,7 +6477,7 @@ 
                                         // If old value encrypted in database is same than submitted new value, it means we don't change it, so we don't update.
                                         if ($algo == 'dolcrypt') {  // dolibarr reversible encryption
                                             if (!preg_match('/^dolcrypt:/', $this->array_options[$key])) {
-                                                $new_array_options[$key] = dolEncrypt($this->array_options[$key]);  // warning, must be called when on the master
+                                                $new_array_options[$key] = dolEncrypt($this->array_options[$key]); // warning, must be called when on the master
                                             } else {
                                                 $new_array_options[$key] = $this->array_options[$key]; // Value is kept
                                             }
@@ -6488,7 +6488,7 @@ 
                                         // If value has changed
                                         if ($algo == 'dolcrypt') {  // dolibarr reversible encryption
                                             if (!preg_match('/^dolcrypt:/', $this->array_options[$key])) {
-                                                $new_array_options[$key] = dolEncrypt($this->array_options[$key]);  // warning, must be called when on the master
+                                                $new_array_options[$key] = dolEncrypt($this->array_options[$key]); // warning, must be called when on the master
                                             } else {
                                                 $new_array_options[$key] = $this->array_options[$key]; // Value is kept
                                             }
@@ -6500,7 +6500,7 @@ 
                                     //var_dump('jjj'.$algo.' '.$this->oldcopy->array_options[$key].' -> '.$this->array_options[$key]);
                                     // If this->oldcopy is not defined, we can't know if we change attribute or not, so we must keep value
                                     if ($algo == 'dolcrypt' && !preg_match('/^dolcrypt:/', $this->array_options[$key])) {   // dolibarr reversible encryption
-                                        $new_array_options[$key] = dolEncrypt($this->array_options[$key]);  // warning, must be called when on the master
+                                        $new_array_options[$key] = dolEncrypt($this->array_options[$key]); // warning, must be called when on the master
                                     } else {
                                         $new_array_options[$key] = $this->array_options[$key]; // Value is kept
                                     }
@@ -6897,7 +6897,7 @@ 
                                 if (isset($this->oldcopy->array_options["options_" . $key]) && $this->array_options["options_" . $key] == $this->oldcopy->array_options["options_" . $key]) { // If old value encrypted in database is same than submitted new value, it means we don't change it, so we don't update.
                                     if ($algo == 'dolcrypt') {  // dolibarr reversible encryption
                                         if (!preg_match('/^dolcrypt:/', $this->array_options["options_" . $key])) {
-                                            $new_array_options["options_" . $key] = dolEncrypt($this->array_options["options_" . $key]);    // warning, must be called when on the master
+                                            $new_array_options["options_" . $key] = dolEncrypt($this->array_options["options_" . $key]); // warning, must be called when on the master
                                         } else {
                                             $new_array_options["options_" . $key] = $this->array_options["options_" . $key]; // Value is kept
                                         }
@@ -6917,7 +6917,7 @@ 
                                 }
                             } else {
                                 if ($algo == 'dolcrypt' && !preg_match('/^dolcrypt:/', $this->array_options["options_" . $key])) {    // dolibarr reversible encryption
-                                    $new_array_options["options_" . $key] = dolEncrypt($this->array_options["options_" . $key]);    // warning, must be called when on the master
+                                    $new_array_options["options_" . $key] = dolEncrypt($this->array_options["options_" . $key]); // warning, must be called when on the master
                                 } else {
                                     $new_array_options["options_" . $key] = $this->array_options["options_" . $key]; // Value is kept
                                 }
@@ -7301,7 +7301,7 @@ 
             }
             $out = '<input type="text" class="flat ' . $morecss . ' maxwidthonsmartphone" name="' . $keyprefix . $key . $keysuffix . '" id="' . $keyprefix . $key . $keysuffix . '" value="' . $value . '" ' . ($moreparam ? $moreparam : '') . '> ';
         } elseif ($type == 'select') {  // combo list
-            $out = '';  // @phan-suppress-current-line PhanPluginRedundantAssignment
+            $out = ''; // @phan-suppress-current-line PhanPluginRedundantAssignment
             if (!empty($conf->use_javascript_ajax) && !getDolGlobalString('MAIN_EXTRAFIELDS_DISABLE_SELECT2')) {
                 include_once DOL_DOCUMENT_ROOT . '/core/lib/ajax.lib.php';
                 $out .= ajax_combobox($keyprefix . $key . $keysuffix, array(), 0);
@@ -7333,7 +7333,7 @@ 
             $out .= $tmpselect;
             $out .= '</select>';
         } elseif ($type == 'sellist') {
-            $out = '';  // @phan-suppress-current-line PhanPluginRedundantAssignment
+            $out = ''; // @phan-suppress-current-line PhanPluginRedundantAssignment
             if (!empty($conf->use_javascript_ajax) && !getDolGlobalString('MAIN_EXTRAFIELDS_DISABLE_SELECT2')) {
                 include_once DOL_DOCUMENT_ROOT . '/core/lib/ajax.lib.php';
                 $out .= ajax_combobox($keyprefix . $key . $keysuffix, array(), 0);
@@ -7343,7 +7343,7 @@ 
             if (is_array($param['options'])) {
                 $param_list = array_keys($param['options']);
                 $InfoFieldList = explode(":", $param_list[0], 5);
-                if (! empty($InfoFieldList[4])) {
+                if (!empty($InfoFieldList[4])) {
                     $pos = 0;
                     $parenthesisopen = 0;
                     while (substr($InfoFieldList[4], $pos, 1) !== '' && ($parenthesisopen || $pos == 0 || substr($InfoFieldList[4], $pos, 1) != ':')) {
@@ -7527,7 +7527,7 @@ 
             $value_arr = explode(',', $value);
             $out = $form->multiselectarray($keyprefix . $key . $keysuffix, (empty($param['options']) ? null : $param['options']), $value_arr, '', 0, $morecss, 0, '100%');
         } elseif ($type == 'radio') {
-            $out = '';  // @phan-suppress-current-line PhanPluginRedundantAssignment
+            $out = ''; // @phan-suppress-current-line PhanPluginRedundantAssignment
             foreach ($param['options'] as $keyopt => $valopt) {
                 $out .= '<input class="flat ' . $morecss . '" type="radio" name="' . $keyprefix . $key . $keysuffix . '" id="' . $keyprefix . $key . $keysuffix . '" ' . ($moreparam ? $moreparam : '');
                 $out .= ' value="' . $keyopt . '"';
@@ -7748,7 +7748,7 @@ 
             $newval = $val;
             $newval['type'] = 'varchar(256)';
 
-            $out = '';  // @phan-suppress-current-line PhanPluginRedundantAssignment
+            $out = ''; // @phan-suppress-current-line PhanPluginRedundantAssignment
             if (!empty($value)) {
                 foreach ($value as $option) {
                     $out .= '<span><a class="' . dol_escape_htmltag($keyprefix . $key . $keysuffix) . '_del" href="javascript:;"><span class="fa fa-minus-circle valignmiddle"></span></a> ';
@@ -9758,7 +9758,7 @@ 
         if (!$error) {
             $sql = "INSERT INTO " . $this->db->prefix() . $this->table_element;
             $sql .= " (" . implode(", ", $keys) . ')';
-            $sql .= " VALUES (" . implode(", ", $values) . ")";     // $values can contains 'abc' or 123
+            $sql .= " VALUES (" . implode(", ", $values) . ")"; // $values can contains 'abc' or 123
 
             $res = $this->db->query($sql);
             if (!$res) {
@@ -10051,7 +10051,7 @@ 
 
         // Update extrafield
         if (!$error) {
-            $result = $this->insertExtraFields();   // This delete and reinsert extrafields
+            $result = $this->insertExtraFields(); // This delete and reinsert extrafields
             if ($result < 0) {
                 $error++;
             }
@@ -10252,7 +10252,7 @@ 
                 $error++;
             } else {
                 while ($obj = $this->db->fetch_object($resql)) {
-                    $result = $this->fetch($obj->rowid);    // @phpstan-ignore-line
+                    $result = $this->fetch($obj->rowid); // @phpstan-ignore-line
                     if ($result < 0) {
                         $error++;
                         $this->errors[] = $this->error;
@@ -10469,7 +10469,7 @@ 
         );
         foreach ($fields as $key => $value) {
             if (array_key_exists($key, $this->fields)) {
-                $this->{$key} = $value;     // @phpstan-ignore-line
+                $this->{$key} = $value; // @phpstan-ignore-line
             }
         }
 

htdocs/core/class/html.formmail.class.php

Indentation +12 added lines, -12 removed lines

@@ -227,7 +227,7 @@ 
         $this->withfckeditor = -1; // -1 = Auto
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Clear list of attached files in send mail form (also stored in session)
      *
@@ -235,7 +235,7 @@ 
      */
     public function clear_attached_files()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
         require_once DOL_DOCUMENT_ROOT . '/core/lib/files.lib.php';
 
@@ -252,7 +252,7 @@ 
         unset($_SESSION["listofmimes" . $keytoavoidconflict]);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Add a file into the list of attached files (stored in SECTION array)
      *
@@ -263,7 +263,7 @@ 
      */
     public function add_attached_files($path, $file = '', $type = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         $listofpaths = array();
         $listofnames = array();
         $listofmimes = array();
@@ -295,7 +295,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Remove a file from the list of attached files (stored in SECTION array)
      *
@@ -304,7 +304,7 @@ 
      */
     public function remove_attached_files($keytodelete)
     {
-		// phpcs:enable
+        // phpcs:enable
         $listofpaths = array();
         $listofnames = array();
         $listofmimes = array();
@@ -330,7 +330,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return list of attached files (stored in SECTION array)
      *
@@ -338,7 +338,7 @@ 
      */
     public function get_attached_files()
     {
-		// phpcs:enable
+        // phpcs:enable
         $listofpaths = array();
         $listofnames = array();
         $listofmimes = array();
@@ -356,7 +356,7 @@ 
         return array('paths' => $listofpaths, 'names' => $listofnames, 'mimes' => $listofmimes);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Show the form to input an email
      *  this->withfile: 0=No attaches files, 1=Show attached files, 2=Can add new attached files
@@ -369,11 +369,11 @@ 
      */
     public function show_form($addfileaction = 'addfile', $removefileaction = 'removefile')
     {
-		// phpcs:enable
+        // phpcs:enable
         print $this->get_form($addfileaction, $removefileaction);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Get the form to input an email
      *  this->withfile: 0=No attaches files, 1=Show attached files, 2=Can add new attached files
@@ -386,7 +386,7 @@ 
      */
     public function get_form($addfileaction = 'addfile', $removefileaction = 'removefile')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $user, $hookmanager, $form;
 
         // Required to show preview wof mail attachments

Spacing +1 added lines, -1 removed lines

@@ -886,7 +886,7 @@
                         $maxfilesizearray = getMaxFileSizeArray();
                         $maxmin = $maxfilesizearray['maxmin'];
                         if ($maxmin > 0) {
-                            $out .= '<input type="hidden" name="MAX_FILE_SIZE" value="' . ($maxmin * 1024) . '">';  // MAX_FILE_SIZE must precede the field type=file
+                            $out .= '<input type="hidden" name="MAX_FILE_SIZE" value="' . ($maxmin * 1024) . '">'; // MAX_FILE_SIZE must precede the field type=file
                         }
                         // Can add other files
                         if (!getDolGlobalString('FROM_MAIL_DONT_USE_INPUT_FILE_MULTIPLE')) {

htdocs/core/class/commonincoterm.class.php

Indentation +2 added lines, -2 removed lines

@@ -47,7 +47,7 @@ 
     public $location_incoterms;
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Return incoterms information
      *    TODO Use a cache for label get
@@ -56,7 +56,7 @@ 
      */
     public function display_incoterms()
     {
-		// phpcs:enable
+        // phpcs:enable
         $out = '';
 
         $this->label_incoterms = '';

htdocs/core/class/openid.class.php

Indentation +36 added lines, -36 removed lines

@@ -45,7 +45,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * SetOpenIDServer
      *
@@ -54,11 +54,11 @@ 
      */
     public function SetOpenIDServer($a)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->URLs['openid_server'] = $a;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * SetOpenIDServer
      *
@@ -67,11 +67,11 @@ 
      */
     public function SetTrustRoot($a)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->URLs['trust_root'] = $a;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * SetOpenIDServer
      *
@@ -80,11 +80,11 @@ 
      */
     public function SetCancelURL($a)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->URLs['cancel'] = $a;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * SetApprovedURL
      *
@@ -93,11 +93,11 @@ 
      */
     public function SetApprovedURL($a)
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->URLs['approved'] = $a;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * SetRequiredFields
      *
@@ -106,7 +106,7 @@ 
      */
     public function SetRequiredFields($a)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (is_array($a)) {
             $this->fields['required'] = $a;
         } else {
@@ -114,7 +114,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * SetOptionalFields
      *
@@ -123,7 +123,7 @@ 
      */
     public function SetOptionalFields($a)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (is_array($a)) {
             $this->fields['optional'] = $a;
         } else {
@@ -131,7 +131,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * SetIdentity
      *
@@ -140,7 +140,7 @@ 
      */
     public function SetIdentity($a)
     {
-		// phpcs:enable
+        // phpcs:enable
         // Set Identity URL
         if (
             (stripos($a, 'http://') === false)
@@ -164,7 +164,7 @@ 
         $this->openid_url_identity = $a;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * GetIdentity
      *
@@ -172,12 +172,12 @@ 
      */
     public function GetIdentity()
     {
-		// phpcs:enable
+        // phpcs:enable
         // Get Identity
         return $this->openid_url_identity;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * SetOpenIDServer
      *
@@ -185,12 +185,12 @@ 
      */
     public function GetError()
     {
-		// phpcs:enable
+        // phpcs:enable
         $e = $this->error;
         return array('code' => $e[0], 'description' => $e[1]);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * ErrorStore
      *
@@ -200,7 +200,7 @@ 
      */
     public function ErrorStore($code, $desc = null)
     {
-		// phpcs:enable
+        // phpcs:enable
         $errs = array();
         $errs['OPENID_NOSERVERSFOUND'] = 'Cannot find OpenID Server TAG on Identity page.';
         if ($desc == null) {
@@ -209,7 +209,7 @@ 
         $this->error = array($code, $desc);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * IsError
      *
@@ -217,7 +217,7 @@ 
      */
     public function IsError()
     {
-		// phpcs:enable
+        // phpcs:enable
         if (count($this->error) > 0) {
             return true;
         } else {
@@ -245,7 +245,7 @@ 
         return $r;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * OpenID_Standarize
      *
@@ -254,7 +254,7 @@ 
      */
     public function OpenID_Standarize($openid_identity = null)
     {
-		// phpcs:enable
+        // phpcs:enable
         if ($openid_identity === null) {
             $openid_identity = $this->openid_url_identity;
         }
@@ -293,7 +293,7 @@ 
         return $query;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * FSOCK_Request
      *
@@ -304,7 +304,7 @@ 
      */
     public function FSOCK_Request($url, $method = "GET", $params = "")
     {
-		// phpcs:enable
+        // phpcs:enable
         $fp = fsockopen("ssl://www.myopenid.com", 443, $errno, $errstr, 3); // Connection timeout is 3 seconds
         if (!$fp) {
             $this->ErrorStore('OPENID_SOCKETERROR', $errstr);
@@ -327,7 +327,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * HTML2OpenIDServer
      *
@@ -336,7 +336,7 @@ 
      */
     public function HTML2OpenIDServer($content)
     {
-		// phpcs:enable
+        // phpcs:enable
         $get = array();
 
         $matches1 = array();
@@ -358,7 +358,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Get openid server
      *
@@ -367,7 +367,7 @@ 
      */
     public function GetOpenIDServer($url = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         include_once DOL_DOCUMENT_ROOT . '/core/lib/geturl.lib.php';
@@ -392,7 +392,7 @@ 
         return $servers[0];
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * GetRedirectURL
      *
@@ -400,7 +400,7 @@ 
      */
     public function GetRedirectURL()
     {
-		// phpcs:enable
+        // phpcs:enable
         $params = array();
         $params['openid.return_to'] = urlencode($this->URLs['approved']);
         $params['openid.mode'] = 'checkid_setup';
@@ -422,7 +422,7 @@ 
         return $this->URLs['openid_server'] . "?" . $this->array2url($params);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Redirect
      *
@@ -430,7 +430,7 @@ 
      */
     public function Redirect()
     {
-		// phpcs:enable
+        // phpcs:enable
         $redirect_to = $this->GetRedirectURL();
         if (headers_sent()) { // Use JavaScript to redirect if content has been previously sent (not recommended, but safe)
             echo '<script nonce="' . getNonce() . '" type="text/javascript">window.location=\'';
@@ -441,7 +441,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * ValidateWithServer
      *
@@ -449,7 +449,7 @@ 
      */
     public function ValidateWithServer()
     {
-		// phpcs:enable
+        // phpcs:enable
         $params = array(
             'openid.assoc_handle' => urlencode($_GET['openid_assoc_handle']),
             'openid.signed' => urlencode($_GET['openid_signed']),