alxarafe / alixar

Dolibarr/Code/Core/Classes/FormMail.php

Indentation +12 added lines, -12 removed lines

@@ -231,7 +231,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)
      *
@@ -239,7 +239,7 @@ 
      */
     public function clear_attached_files()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
         require_once constant('DOL_DOCUMENT_ROOT') . '/core/lib/files.lib.php';
 
@@ -256,7 +256,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)
      *
@@ -267,7 +267,7 @@ 
      */
     public function add_attached_files($path, $file = '', $type = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         $listofpaths = array();
         $listofnames = array();
         $listofmimes = array();
@@ -299,7 +299,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)
      *
@@ -308,7 +308,7 @@ 
      */
     public function remove_attached_files($keytodelete)
     {
-		// phpcs:enable
+        // phpcs:enable
         $listofpaths = array();
         $listofnames = array();
         $listofmimes = array();
@@ -334,7 +334,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return list of attached files (stored in SECTION array)
      *
@@ -342,7 +342,7 @@ 
      */
     public function get_attached_files()
     {
-		// phpcs:enable
+        // phpcs:enable
         $listofpaths = array();
         $listofnames = array();
         $listofmimes = array();
@@ -360,7 +360,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
@@ -373,11 +373,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
@@ -390,7 +390,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')) {

Dolibarr/Code/Core/Classes/FormExpenseReport.php

Spacing +1 added lines, -1 removed lines

@@ -78,7 +78,7 @@
             if (!empty($selected) && $selected == $key) {
                 $html .= '<option value="' . $key . '" selected>';
             } else {
-                $html .=  '<option value="' . $key . '">';
+                $html .= '<option value="' . $key . '">';
             }
             $html .= $langs->trans($val);
             $html .= '</option>';

Dolibarr/Code/Core/Classes/HookManager.php

Spacing +1 added lines, -1 removed lines

@@ -290,7 +290,7 @@
                 '@phan-var-force array<string,CommonHookActions> $modules';
                 // Loop on each active hooks of module for this context
                 foreach ($modules as $module => $actionclassinstance) {
-                    $module = preg_replace('/^\d+:/', '', $module);     // $module string is 'priority:module'
+                    $module = preg_replace('/^\d+:/', '', $module); // $module string is 'priority:module'
                     //print "Before hook ".get_class($actionclassinstance)." method=".$method." module=".$module." hooktype=".$hooktype." results=".count($actionclassinstance->results)." resprints=".count($actionclassinstance->resprints)." resaction=".$resaction."<br>\n";
 
                     // test to avoid running twice a hook, when a module implements several active contexts

Dolibarr/Code/Core/Classes/ExtraFields.php

Indentation +8 added lines, -8 removed lines

@@ -364,7 +364,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Add description of a new optional attribute
      *
@@ -394,7 +394,7 @@ 
      */
     private function create_label($attrname, $label = '', $type = '', $pos = 0, $size = '', $elementtype = '', $unique = 0, $required = 0, $param = '', $alwayseditable = 0, $perms = '', $list = '-1', $help = '', $default = '', $computed = '', $entity = '', $langfile = '', $enabled = '1', $totalizable = 0, $printable = 0, $moreparams = array())
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
 
         if ($elementtype == 'thirdparty') {
@@ -575,7 +575,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Delete description of an optional attribute
      *
@@ -585,7 +585,7 @@ 
      */
     private function delete_label($attrname, $elementtype = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         if ($elementtype == 'thirdparty') {
@@ -755,7 +755,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Modify description of personalized attribute
      *  This is a private method. For public method, use updateExtraField.
@@ -786,7 +786,7 @@ 
      */
     private function update_label($attrname, $label, $type, $size, $elementtype, $unique = 0, $required = 0, $pos = 0, $param = array(), $alwayseditable = 0, $perms = '', $list = '0', $help = '', $default = '', $computed = '', $entity = '', $langfile = '', $enabled = '1', $totalizable = 0, $printable = 0, $moreparams = array())
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
         dol_syslog(get_class($this) . "::update_label " . $attrname . ", " . $label . ", " . $type . ", " . $size . ", " . $elementtype . ", " . $unique . ", " . $required . ", " . $pos . ", " . $alwayseditable . ", " . $perms . ", " . $list . ", " . $default . ", " . $computed . ", " . $entity . ", " . $langfile . ", " . $enabled . ", " . $totalizable . ", " . $printable);
 
@@ -929,7 +929,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load the array of extrafields definition $this->attributes
      *
@@ -940,7 +940,7 @@ 
      */
     public function fetch_name_optionals_label($elementtype, $forceload = false, $attrname = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         if (empty($elementtype)) {

Spacing +8 added lines, -8 removed lines

@@ -1239,7 +1239,7 @@ 
             } else {
                 $out = $form->selectyesno($keyprefix . $key . $keysuffix, $value, 1, false, 1);
             }
-            $out .= '<input type="hidden" name="' . $keyprefix . $key . $keysuffix . '_boolean" value="1">';    // A hidden field ending with "_boolean" that is always set to 1.
+            $out .= '<input type="hidden" name="' . $keyprefix . $key . $keysuffix . '_boolean" value="1">'; // A hidden field ending with "_boolean" that is always set to 1.
         } elseif ($type == 'price') {
             if (!empty($value)) {       // $value in memory is a php numeric, we format it into user number format.
                 $value = price($value);
@@ -1707,7 +1707,7 @@ 
 
             $tmparray = explode(':', $param_list[0]);
 
-            $element = $extrafieldsobjectkey;       // $extrafieldsobjectkey comes from $object->table_element but we need $object->element
+            $element = $extrafieldsobjectkey; // $extrafieldsobjectkey comes from $object->table_element but we need $object->element
             if ($element == 'socpeople') {
                 $element = 'contact';
             } elseif ($element == 'projet') {
@@ -1715,8 +1715,8 @@ 
             }
 
             //$objectdesc = $param_list[0];             // Example: 'ObjectName:classPath:1:(status:=:1)'   Replaced by next line: this was propagated also a filter by ajax call that was blocked by some WAF
-            $objectdesc = $tmparray[0];                 // Example: 'ObjectName:classPath'                  To not propagate any filter (selectForForms do ajax call and propagating SQL filter is blocked by some WAF). Also we should use the one into the definition in the ->fields of $elem if found.
-            $objectfield = $element . ':options_' . $key;   // Example: 'actioncomm:options_fff'                To be used in priority to know object linked with all its definition (including filters)
+            $objectdesc = $tmparray[0]; // Example: 'ObjectName:classPath'                  To not propagate any filter (selectForForms do ajax call and propagating SQL filter is blocked by some WAF). Also we should use the one into the definition in the ->fields of $elem if found.
+            $objectfield = $element . ':options_' . $key; // Example: 'actioncomm:options_fff'                To be used in priority to know object linked with all its definition (including filters)
 
             $out = $form->selectForForms($objectdesc, $keyprefix . $key . $keysuffix, $value, $showempty, '', '', $morecss, '', 0, 0, '', $objectfield);
         } elseif (in_array($type, ['point', 'multipts', 'linestrg', 'polygon'])) {
@@ -1781,7 +1781,7 @@ 
 
         $label = $this->attributes[$extrafieldsobjectkey]['label'][$key];
         $type = $this->attributes[$extrafieldsobjectkey]['type'][$key];
-        $size = $this->attributes[$extrafieldsobjectkey]['size'][$key];         // Can be '255', '24,8'...
+        $size = $this->attributes[$extrafieldsobjectkey]['size'][$key]; // Can be '255', '24,8'...
         $default = $this->attributes[$extrafieldsobjectkey]['default'][$key];
         $computed = $this->attributes[$extrafieldsobjectkey]['computed'][$key];
         $unique = $this->attributes[$extrafieldsobjectkey]['unique'][$key];
@@ -2109,7 +2109,7 @@ 
             } else {
                 $value = '';
             }
-        } elseif (in_array($type, ['multipts','linestrg', 'polygon'])) {
+        } elseif (in_array($type, ['multipts', 'linestrg', 'polygon'])) {
             if (!empty($value)) {
                 require_once constant('DOL_DOCUMENT_ROOT') . '/core/class/dolgeophp.class.php';
                 $dolgeophp = new DolGeoPHP($this->db);
@@ -2300,7 +2300,7 @@ 
                     continue;
                 }
 
-                if (!empty($onlykey) && $onlykey == '@GETPOSTISSET' && !GETPOSTISSET('options_' . $key) && (! in_array($this->attributes[$object->table_element]['type'][$key], array('boolean', 'checkbox', 'chkbxlst', 'point', 'multipts', 'linestrg', 'polygon')))) {
+                if (!empty($onlykey) && $onlykey == '@GETPOSTISSET' && !GETPOSTISSET('options_' . $key) && (!in_array($this->attributes[$object->table_element]['type'][$key], array('boolean', 'checkbox', 'chkbxlst', 'point', 'multipts', 'linestrg', 'polygon')))) {
                     //when unticking boolean field, it's not set in POST
                     continue;
                 }
@@ -2330,7 +2330,7 @@ 
                         $onlykey === '@GETPOSTISSET'
                         && in_array($this->attributes[$object->table_element]['type'][$key], array('boolean', 'checkbox', 'chkbxlst'))
                         && in_array(abs($enabled), array(2, 5))
-                        && ! GETPOSTISSET('options_' . $key) // Update hidden checkboxes and multiselect only if they are provided
+                        && !GETPOSTISSET('options_' . $key) // Update hidden checkboxes and multiselect only if they are provided
                     )
                 ) {
                     continue;

Dolibarr/Code/Core/Classes/FormAdmin.php

Indentation +10 added lines, -10 removed lines

@@ -53,7 +53,7 @@ 
         $this->db = $db;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return html select list with available languages (key='en_US', value='United States' for example)
      *
@@ -74,7 +74,7 @@ 
      */
     public function select_language($selected = '', $htmlname = 'lang_id', $showauto = 0, $filter = array(), $showempty = '', $showwarning = 0, $disabled = 0, $morecss = '', $showcode = 0, $forcecombo = 0, $multiselect = 0, $onlykeys = array(), $mainlangonly = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         if (getDolGlobalString('MAIN_DEFAULT_LANGUAGE_FILTER')) {
@@ -179,7 +179,7 @@ 
         return $out;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Return list of available menus (eldy_backoffice, ...)
      *
@@ -191,7 +191,7 @@ 
      */
     public function select_menu($selected, $htmlname, $dirmenuarray, $moreattrib = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf;
 
         // Clean parameters
@@ -297,7 +297,7 @@ 
         return;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return combo list of available menu families
      *
@@ -308,7 +308,7 @@ 
      */
     public function select_menu_families($selected, $htmlname, $dirmenuarray)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf;
 
         //$expdevmenu=array('smartphone_backoffice.php','smartphone_frontoffice.php');  // Menu to disable if $conf->global->MAIN_FEATURES_LEVEL is not set
@@ -375,7 +375,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return a HTML select list of timezones
      *
@@ -385,7 +385,7 @@ 
      */
     public function select_timezone($selected, $htmlname)
     {
-		// phpcs:enable
+        // phpcs:enable
         print '<select class="flat" id="' . $htmlname . '" name="' . $htmlname . '">';
         print '<option value="-1">&nbsp;</option>';
 
@@ -428,7 +428,7 @@ 
 
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return html select list with available languages (key='en_US', value='United States' for example)
      *
@@ -441,7 +441,7 @@ 
      */
     public function select_paper_format($selected = '', $htmlname = 'paperformat_id', $filter = '', $showempty = 0, $forcecombo = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         $langs->load("dict");

Spacing +1 added lines, -1 removed lines

@@ -129,7 +129,7 @@
             $out .= '>' . $langs->trans("AutoDetectLang") . '</option>';
         }
 
-        asort($langs_available);    // array('XX' => 'Language (Country)', ...)
+        asort($langs_available); // array('XX' => 'Language (Country)', ...)
 
         foreach ($langs_available as $key => $value) {
             $valuetoshow = $value;

Dolibarr/Code/Core/Classes/CommonInvoice.php

Indentation +10 added lines, -10 removed lines

@@ -536,7 +536,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return if an invoice can be deleted
      *  Rule is:
@@ -552,7 +552,7 @@ 
      */
     public function is_erasable()
     {
-		// phpcs:enable
+        // phpcs:enable
 
         // We check if invoice is a temporary number (PROVxxxx)
         $tmppart = substr($this->ref, 1, 4);
@@ -778,7 +778,7 @@ 
         return $this->LibStatut($this->paye, $this->statut, $mode, $alreadypaid, $this->type);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return label of a status
      *
@@ -791,7 +791,7 @@ 
      */
     public function LibStatut($paye, $status, $mode = 0, $alreadypaid = -1, $type = -1)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $hookmanager;
         $langs->load('bills');
 
@@ -861,7 +861,7 @@ 
         return dolGetStatus($labelStatus, $labelStatusShort, '', $statusType, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Returns an invoice payment deadline based on the invoice settlement
      *  conditions and billing date.
@@ -871,7 +871,7 @@ 
      */
     public function calculate_date_lim_reglement($cond_reglement = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (!$cond_reglement) {
             $cond_reglement = $this->cond_reglement_code;
         }
@@ -957,7 +957,7 @@ 
         return $datelim;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Create a withdrawal request for a direct debit order or a credit transfer order.
      *  Use the remain to pay excluding all existing open direct debit requests.
@@ -971,7 +971,7 @@ 
      */
     public function demande_prelevement($fuser, $amount = 0, $type = 'direct-debit', $sourcetype = 'facture', $checkduplicateamongall = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $error = 0;
@@ -1624,7 +1624,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Remove a direct debit request or a credit transfer request
      *
@@ -1634,7 +1634,7 @@ 
      */
     public function demande_prelevement_delete($fuser, $did)
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = 'DELETE FROM ' . $this->db->prefix() . 'prelevement_demande';
         $sql .= ' WHERE rowid = ' . ((int) $did);
         $sql .= ' AND traite = 0';

Spacing +40 added lines, -40 removed lines

@@ -99,9 +99,9 @@ 
      */
     public $revenuestamp;
 
-    public $totalpaid;          // duplicate with sumpayed
-    public $totaldeposits;      // duplicate with sumdeposit
-    public $totalcreditnotes;   // duplicate with sumcreditnote
+    public $totalpaid; // duplicate with sumpayed
+    public $totaldeposits; // duplicate with sumdeposit
+    public $totalcreditnotes; // duplicate with sumcreditnote
 
     public $sumpayed;
     public $sumpayed_multicurrency;
@@ -1138,7 +1138,7 @@ 
 
         if ($this->status > self::STATUS_DRAFT && $this->paye == 0) {
             // Get the default payment mode for BAN payment of the third party
-            $bac = new CompanyBankAccount($this->db);   // Table societe_rib
+            $bac = new CompanyBankAccount($this->db); // Table societe_rib
             $result = $bac->fetch(0, '', $this->socid, 1, 'ban');
             if ($result <= 0 || empty($bac->id)) {
                 $this->error = $langs->trans("ThirdpartyHasNoDefaultBanAccount");
@@ -1152,16 +1152,16 @@ 
             $sql .= " FROM " . $this->db->prefix() . "prelevement_demande";
             $sql .= " WHERE rowid = " . ((int) $did);
             if ($type != 'bank-transfer' && $type != 'credit-transfer') {
-                $sql .= " AND fk_facture = " . ((int) $this->id);             // Add a protection to not pay another invoice than current one
+                $sql .= " AND fk_facture = " . ((int) $this->id); // Add a protection to not pay another invoice than current one
             }
             if ($type != 'direct-debit') {
                 if ($sourcetype == 'salary') {
-                    $sql .= " AND fk_salary = " . ((int) $this->id);          // Add a protection to not pay another salary than current one
+                    $sql .= " AND fk_salary = " . ((int) $this->id); // Add a protection to not pay another salary than current one
                 } else {
-                    $sql .= " AND fk_facture_fourn = " . ((int) $this->id);   // Add a protection to not pay another invoice than current one
+                    $sql .= " AND fk_facture_fourn = " . ((int) $this->id); // Add a protection to not pay another invoice than current one
                 }
             }
-            $sql .= " AND traite = 0";  // To not process payment request that were already converted into a direct debit or credit transfer order (Note: fk_prelevement_bons is also empty when traite = 0)
+            $sql .= " AND traite = 0"; // To not process payment request that were already converted into a direct debit or credit transfer order (Note: fk_prelevement_bons is also empty when traite = 0)
 
             dol_syslog(get_class($this) . "::makeStripeSepaRequest load requests to process", LOG_DEBUG);
             $resql = $this->db->query($sql);
@@ -1176,7 +1176,7 @@ 
                 $amount = $obj->amount;
 
                 if (is_numeric($amount) && $amount != 0) {
-                    $companypaymentmode = new CompanyPaymentMode($this->db);    // table societe_rib
+                    $companypaymentmode = new CompanyPaymentMode($this->db); // table societe_rib
                     $companypaymentmode->fetch($bac->id);
 
                     $this->stripechargedone = 0;
@@ -1186,7 +1186,7 @@ 
 
                     $currency = $conf->currency;
 
-                    $errorforinvoice = 0;     // We reset the $errorforinvoice at each invoice loop
+                    $errorforinvoice = 0; // We reset the $errorforinvoice at each invoice loop
 
                     $this->fetch_thirdparty();
 
@@ -1205,7 +1205,7 @@ 
                         $amountstripe = $amountstripe * 100;
                     }
 
-                    $fk_bank_account = getDolGlobalInt('STRIPE_BANK_ACCOUNT_FOR_PAYMENTS');     // Bank account used for SEPA direct debit or credit transfer. Must be the Stripe account in Dolibarr.
+                    $fk_bank_account = getDolGlobalInt('STRIPE_BANK_ACCOUNT_FOR_PAYMENTS'); // Bank account used for SEPA direct debit or credit transfer. Must be the Stripe account in Dolibarr.
                     if (!($fk_bank_account > 0)) {
                         $error++;
                         $errorforinvoice++;
@@ -1265,7 +1265,7 @@ 
                                 $thirdparty = new Societe($this->db);
                                 $resultthirdparty = $thirdparty->fetch($this->socid);
 
-                                include_once DOL_DOCUMENT_ROOT . '/stripe/class/stripe.class.php';        // This include the include of htdocs/stripe/config.php
+                                include_once DOL_DOCUMENT_ROOT . '/stripe/class/stripe.class.php'; // This include the include of htdocs/stripe/config.php
                                 // So it inits or erases the $stripearrayofkeysbyenv
                                 $stripe = new Stripe($this->db);
 
@@ -1278,26 +1278,26 @@ 
                                 $foundalternativestripeaccount = '';
 
                                 // Force stripe to another value (by default this value is empty)
-                                if (! empty($forcestripe)) {
+                                if (!empty($forcestripe)) {
                                     dol_syslog("makeStripeSepaRequest A dedicated stripe account was forced, so we switch to it.");
 
                                     $tmparray = explode('@', $forcestripe);
-                                    if (! empty($tmparray[1])) {
+                                    if (!empty($tmparray[1])) {
                                         $tmparray2 = explode(':', $tmparray[1]);
-                                        if (! empty($tmparray2[1])) {
+                                        if (!empty($tmparray2[1])) {
                                             $stripearrayofkeysbyenv[$servicestatus]["publishable_key"] = $tmparray2[0];
                                             $stripearrayofkeysbyenv[$servicestatus]["secret_key"] = $tmparray2[1];
 
                                             $stripearrayofkeys = $stripearrayofkeysbyenv[$servicestatus];
                                             \Stripe\Stripe::setApiKey($stripearrayofkeys['secret_key']);
 
-                                            $foundalternativestripeaccount = $tmparray[0];    // Store the customer id
+                                            $foundalternativestripeaccount = $tmparray[0]; // Store the customer id
 
                                             dol_syslog("makeStripeSepaRequest We use now customer=" . $foundalternativestripeaccount . " publishable_key=" . $stripearrayofkeys['publishable_key'], LOG_DEBUG);
                                         }
                                     }
 
-                                    if (! $foundalternativestripeaccount) {
+                                    if (!$foundalternativestripeaccount) {
                                         $stripearrayofkeysbyenv = $savstripearrayofkeysbyenv;
 
                                         $stripearrayofkeys = $savstripearrayofkeysbyenv[$servicestatus];
@@ -1312,7 +1312,7 @@ 
                                     dol_syslog("makeStripeSepaRequest No dedicated Stripe Account requested, so we use global one, so " . $stripearrayofkeys['publishable_key'], LOG_DEBUG);
                                 }
 
-                                $stripeacc = $stripe->getStripeAccount($service, $this->socid);                             // Get Stripe OAuth connect account if it exists (no network access here)
+                                $stripeacc = $stripe->getStripeAccount($service, $this->socid); // Get Stripe OAuth connect account if it exists (no network access here)
 
                                 if ($foundalternativestripeaccount) {
                                     if (empty($stripeacc)) {                // If the Stripe connect account not set, we use common API usage
@@ -1322,7 +1322,7 @@ 
                                     }
                                 } else {
                                     $customer = $stripe->customerStripe($thirdparty, $stripeacc, $servicestatus, 0);
-                                    if (empty($customer) && ! empty($stripe->error)) {
+                                    if (empty($customer) && !empty($stripe->error)) {
                                         $this->errors[] = $stripe->error;
                                     }
                                     /*if (!empty($customer) && empty($customer->sources)) {
@@ -1378,7 +1378,7 @@ 
                                                 $charge->failure_message = $stripe->error;
                                                 $charge->failure_declinecode = $stripe->declinecode;
                                                 $stripefailurecode = $stripe->code;
-                                                $stripefailuremessage = 'Action required. Contact the support at ';// . $conf->global->SELLYOURSAAS_MAIN_EMAIL;
+                                                $stripefailuremessage = 'Action required. Contact the support at '; // . $conf->global->SELLYOURSAAS_MAIN_EMAIL;
                                                 $stripefailuredeclinecode = $stripe->declinecode;
                                             } else {
                                                 dol_syslog(var_export($paymentintent, true), LOG_DEBUG);
@@ -1518,18 +1518,18 @@ 
                                     // Insert record of payment (success or error)
                                     $actioncomm = new ActionComm($this->db);
 
-                                    $actioncomm->type_code = 'AC_OTH_AUTO';     // Type of event ('AC_OTH', 'AC_OTH_AUTO', 'AC_XXX'...)
+                                    $actioncomm->type_code = 'AC_OTH_AUTO'; // Type of event ('AC_OTH', 'AC_OTH_AUTO', 'AC_XXX'...)
                                     $actioncomm->code = 'AC_' . $actioncode;
                                     $actioncomm->label = $description;
                                     $actioncomm->note_private = implode(",\n", $postactionmessages);
                                     $actioncomm->fk_project = $this->fk_project;
                                     $actioncomm->datep = $now;
                                     $actioncomm->datef = $now;
-                                    $actioncomm->percentage = -1;   // Not applicable
+                                    $actioncomm->percentage = -1; // Not applicable
                                     $actioncomm->socid = $thirdparty->id;
                                     $actioncomm->contactid = 0;
-                                    $actioncomm->authorid = $user->id;   // User saving action
-                                    $actioncomm->userownerid = $user->id;   // Owner of action
+                                    $actioncomm->authorid = $user->id; // User saving action
+                                    $actioncomm->userownerid = $user->id; // Owner of action
                                     // Fields when action is a real email (content is already into note)
                                     /*$actioncomm->email_msgid = $object->email_msgid;
                                      $actioncomm->email_from  = $object->email_from;
@@ -1665,10 +1665,10 @@ 
 
         // Add the standard elements to the QR code
         $lines = [
-            'BCD',  // Service Tag (optional)
-            '002',  // Version (optional)
-            '1',    // Character set (optional)
-            'SCT',  // Identification (optional)
+            'BCD', // Service Tag (optional)
+            '002', // Version (optional)
+            '1', // Character set (optional)
+            'SCT', // Identification (optional)
         ];
 
         // Add the bank account information
@@ -1741,10 +1741,10 @@ 
         $s .= pack('C1', 3) . pack('C1', strlen($datestring)) . $datestring;
         $s .= pack('C1', 4) . pack('C1', strlen($pricewithtaxstring)) . $pricewithtaxstring;
         $s .= pack('C1', 5) . pack('C1', strlen($pricetaxstring)) . $pricetaxstring;
-        $s .= '';                   // Hash of xml invoice
-        $s .= '';                   // ecda signature
-        $s .= '';                   // ecda public key
-        $s .= '';                   // ecda signature of public key stamp
+        $s .= ''; // Hash of xml invoice
+        $s .= ''; // ecda signature
+        $s .= ''; // ecda public key
+        $s .= ''; // ecda signature of public key stamp
 
         $s = base64_encode($s);
 
@@ -1811,8 +1811,8 @@ 
             $s .= "S\n";
             $s .= dol_trunc($bankaccount->proprio, 70, 'right', 'UTF-8', 1) . "\n";
             $addresslinearray = explode("\n", $bankaccount->owner_address);
-            $s .= dol_trunc(empty($addresslinearray[1]) ? '' : $addresslinearray[1], 70, 'right', 'UTF-8', 1) . "\n";     // address line 1
-            $s .= dol_trunc(empty($addresslinearray[2]) ? '' : $addresslinearray[2], 70, 'right', 'UTF-8', 1) . "\n";     // address line 2
+            $s .= dol_trunc(empty($addresslinearray[1]) ? '' : $addresslinearray[1], 70, 'right', 'UTF-8', 1) . "\n"; // address line 1
+            $s .= dol_trunc(empty($addresslinearray[2]) ? '' : $addresslinearray[2], 70, 'right', 'UTF-8', 1) . "\n"; // address line 2
             /*$s .= dol_trunc($mysoc->zip, 16, 'right', 'UTF-8', 1)."\n";
             $s .= dol_trunc($mysoc->town, 35, 'right', 'UTF-8', 1)."\n";
             $s .= dol_trunc($mysoc->country_code, 2, 'right', 'UTF-8', 1)."\n";*/
@@ -1820,8 +1820,8 @@ 
             $s .= "S\n";
             $s .= dol_trunc($mysoc->name, 70, 'right', 'UTF-8', 1) . "\n";
             $addresslinearray = explode("\n", $mysoc->address);
-            $s .= dol_trunc(empty($addresslinearray[1]) ? '' : $addresslinearray[1], 70, 'right', 'UTF-8', 1) . "\n";     // address line 1
-            $s .= dol_trunc(empty($addresslinearray[2]) ? '' : $addresslinearray[2], 70, 'right', 'UTF-8', 1) . "\n";     // address line 2
+            $s .= dol_trunc(empty($addresslinearray[1]) ? '' : $addresslinearray[1], 70, 'right', 'UTF-8', 1) . "\n"; // address line 1
+            $s .= dol_trunc(empty($addresslinearray[2]) ? '' : $addresslinearray[2], 70, 'right', 'UTF-8', 1) . "\n"; // address line 2
             $s .= dol_trunc($mysoc->zip, 16, 'right', 'UTF-8', 1) . "\n";
             $s .= dol_trunc($mysoc->town, 35, 'right', 'UTF-8', 1) . "\n";
             $s .= dol_trunc($mysoc->country_code, 2, 'right', 'UTF-8', 1) . "\n";
@@ -1841,14 +1841,14 @@ 
         $s .= "S\n";
         $s .= dol_trunc($this->thirdparty->name, 70, 'right', 'UTF-8', 1) . "\n";
         $addresslinearray = explode("\n", $this->thirdparty->address);
-        $s .= dol_trunc(empty($addresslinearray[1]) ? '' : $addresslinearray[1], 70, 'right', 'UTF-8', 1) . "\n";     // address line 1
-        $s .= dol_trunc(empty($addresslinearray[2]) ? '' : $addresslinearray[2], 70, 'right', 'UTF-8', 1) . "\n";     // address line 2
+        $s .= dol_trunc(empty($addresslinearray[1]) ? '' : $addresslinearray[1], 70, 'right', 'UTF-8', 1) . "\n"; // address line 1
+        $s .= dol_trunc(empty($addresslinearray[2]) ? '' : $addresslinearray[2], 70, 'right', 'UTF-8', 1) . "\n"; // address line 2
         $s .= dol_trunc($this->thirdparty->zip, 16, 'right', 'UTF-8', 1) . "\n";
         $s .= dol_trunc($this->thirdparty->town, 35, 'right', 'UTF-8', 1) . "\n";
         $s .= dol_trunc($this->thirdparty->country_code, 2, 'right', 'UTF-8', 1) . "\n";
         // ID of payment
-        $s .= "NON\n";          // NON or QRR
-        $s .= "\n";             // QR Code reference if previous field is QRR
+        $s .= "NON\n"; // NON or QRR
+        $s .= "\n"; // QR Code reference if previous field is QRR
         // Free text
         if ($complementaryinfo) {
             $s .= $complementaryinfo . "\n";

Dolibarr/Code/Core/Classes/FormTicket.php

Indentation +2 added lines, -2 removed lines

@@ -1308,7 +1308,7 @@ 
         print ajax_combobox('select' . $htmlname);
     }
 
-	// 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)
      *
@@ -1316,7 +1316,7 @@ 
      */
     public function clear_attached_files()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
         require_once constant('DOL_DOCUMENT_ROOT') . '/core/lib/files.lib.php';
 

Spacing +9 added lines, -9 removed lines

@@ -86,17 +86,17 @@ 
 
     public $backtopage;
 
-    public $ispublic;  // to show information or not into public form
+    public $ispublic; // to show information or not into public form
 
     public $withtitletopic;
     public $withtopicreadonly;
     public $withreadid;
 
-    public $withcompany;  // to show company drop-down list
+    public $withcompany; // to show company drop-down list
     public $withfromsocid;
     public $withfromcontactid;
     public $withnotifytiersatcreate;
-    public $withusercreate;  // to show name of creating user in form
+    public $withusercreate; // to show name of creating user in form
     public $withcreatereadonly;
 
     /**
@@ -104,7 +104,7 @@ 
      */
     public $withextrafields;
 
-    public $withref;  // to show ref field
+    public $withref; // to show ref field
     public $withcancel;
 
     public $type_code;
@@ -149,7 +149,7 @@ 
         $this->withcreatereadonly = 1;
         $this->withemail = 0;
         $this->withref = 0;
-        $this->withextrafields = 0;  // to show extrafields or not
+        $this->withextrafields = 0; // to show extrafields or not
         //$this->withtopicreadonly=0;
     }
 
@@ -565,7 +565,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
                 }
                 $out .= '<input type="file" class="flat" id="addedfile" name="addedfile" value="' . $langs->trans("Upload") . '" />';
                 $out .= ' ';
@@ -881,7 +881,7 @@ 
         $publicgroups = ($filtertype == 'public=1' || $filtertype == '(public:=:1)');
 
         $ticketstat = new Ticket($this->db);
-        $ticketstat->loadCacheCategoriesTickets($publicgroups ? 1 : -1);    // get list of active ticket groups
+        $ticketstat->loadCacheCategoriesTickets($publicgroups ? 1 : -1); // get list of active ticket groups
 
         if ($use_multilevel <= 0) {
             print '<select id="select' . $htmlname . '" class="flat minwidth100' . ($morecss ? ' ' . $morecss : '') . '" name="' . $htmlname . '">';
@@ -1053,7 +1053,7 @@ 
             }
             $stringtoprint .= '</select>&nbsp;';
 
-            $levelid = 1;   // The first combobox
+            $levelid = 1; // The first combobox
             while ($levelid <= $use_multilevel) {   // Loop to take the child of the combo
                 $tabscript = array();
                 $stringtoprint .= '<select id="' . $htmlname . '_child_' . $levelid . '" class="maxwidth500 minwidth400 groupticketchild" child_id="' . $levelid . '">';
@@ -1129,7 +1129,7 @@ 
                 $stringtoprint .= '</select>';
 
                 $stringtoprint .= '<script nonce="' . getNonce() . '">';
-                $stringtoprint .= 'arraynotparents = ' . json_encode($arraycodenotparent) . ';';    // when the last visible combo list is number x, this is the array of group
+                $stringtoprint .= 'arraynotparents = ' . json_encode($arraycodenotparent) . ';'; // when the last visible combo list is number x, this is the array of group
                 $stringtoprint .= 'if (arraynotparents.includes($("#' . $htmlname . ($levelid > 1 ? '_child_' . ($levelid - 1) : '') . '").val())){
 					console.log("' . $htmlname . '_child_' . $levelid . '")
 					if($("#' . $htmlname . '_child_' . $levelid . '").val() == "" && ($("#' . $htmlname . '_child_' . $levelid . '").attr("child_id")>' . $child_id . ')){

Braces +3 added lines, -1 removed lines

@@ -236,7 +236,9 @@
         print '<form method="POST" ' . ($withdolfichehead ? '' : 'style="margin-bottom: 30px;" ') . 'name="ticket" id="form_create_ticket" enctype="multipart/form-data" action="' . (!empty($this->param["returnurl"]) ? $this->param["returnurl"] : $_SERVER['PHP_SELF']) . '">';
         print '<input type="hidden" name="token" value="' . newToken() . '">';
         print '<input type="hidden" name="action" value="' . $this->action . '">';
-        if (!empty($object->id)) print '<input type="hidden" name="id" value="' . $object->id . '">';
+        if (!empty($object->id)) {
+            print '<input type="hidden" name="id" value="' . $object->id . '">';
+        }
         print '<input type="hidden" name="trackid" value="' . $this->trackid . '">';
         foreach ($this->param as $key => $value) {
             print '<input type="hidden" name="' . $key . '" value="' . $value . '">';

Dolibarr/Code/Core/Classes/DiscountAbsolute.php

Indentation +4 added lines, -4 removed lines

@@ -415,7 +415,7 @@ 
 
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Link the discount to a particular invoice line or a particular invoice.
      *  When discount is a global discount used as an invoice line, we link using rowidline.
@@ -427,7 +427,7 @@ 
      */
     public function link_to_invoice($rowidline, $rowidinvoice)
     {
-		// phpcs:enable
+        // phpcs:enable
         // Check parameters
         if (!$rowidline && !$rowidinvoice) {
             $this->error = 'ErrorBadParameters';
@@ -474,7 +474,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Link the discount to a particular invoice line or a particular invoice.
      *  Do not call this if discount is linked to a reconcialiated invoice
@@ -483,7 +483,7 @@ 
      */
     public function unlink_invoice()
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "UPDATE " . $this->db->prefix() . "societe_remise_except";
         if (!empty($this->discount_type)) {
             $sql .= " SET fk_invoice_supplier_line = NULL, fk_invoice_supplier = NULL";

Spacing +4 added lines, -4 removed lines

@@ -47,16 +47,16 @@
     public $total_ht;
     public $total_tva;
     public $total_ttc;
-    public $amount_ht;  // deprecated
+    public $amount_ht; // deprecated
     public $amount_tva; // deprecated
     public $amount_ttc; // deprecated
 
     public $multicurrency_total_ht;
     public $multicurrency_total_tva;
     public $multicurrency_total_ttc;
-    public $multicurrency_amount_ht;    // deprecated
-    public $multicurrency_amount_tva;   // deprecated
-    public $multicurrency_amount_ttc;   // deprecated
+    public $multicurrency_amount_ht; // deprecated
+    public $multicurrency_amount_tva; // deprecated
+    public $multicurrency_amount_ttc; // deprecated
 
     /**
      * @var double

Dolibarr/Code/Core/Classes/Fiscalyear.php

Indentation +2 added lines, -2 removed lines

@@ -400,7 +400,7 @@ 
         return $this->LibStatut($this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Give a label from a status
      *
@@ -410,7 +410,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
             global $langs;
             //$langs->load("mymodule@mymodule");

Spacing +1 added lines, -1 removed lines

@@ -240,7 +240,7 @@
 
             $this->id = $obj->rowid;
             $this->ref = $obj->rowid;
-            $this->date_start   = $this->db->jdate($obj->date_start);
+            $this->date_start = $this->db->jdate($obj->date_start);
             $this->date_end = $this->db->jdate($obj->date_end);
             $this->label = $obj->label;
             $this->statut = $obj->status;