alxarafe / alixar

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

@@ -107,7 +107,7 @@ 
             $posy = 0;
             $lines = explode("\n", $content);
             $nblines = count($lines);
-            for ($i = 0 ; $i < $nblines ; $i++) {
+            for ($i = 0; $i < $nblines; $i++) {
                 if (preg_match('/' . $poscursor['find'] . '/', $lines[$i])) {
                     $posy = $i;
                     break;
@@ -176,7 +176,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')) {
@@ -277,7 +277,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

Braces +3 added lines, -1 removed lines

@@ -113,7 +113,9 @@
                     break;
                 }
             }
-            if ($posy != 0) $poscursor['y'] = $posy;
+            if ($posy != 0) {
+                $poscursor['y'] = $posy;
+            }
         }
 
         // Define some properties

Dolibarr/Code/Core/Classes/FormAccounting.php

Indentation +14 added lines, -14 removed lines

@@ -71,7 +71,7 @@ 
         $this->db = $db;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return list of journals with label by nature
      *
@@ -88,7 +88,7 @@ 
      */
     public function select_journal($selectid, $htmlname = 'journal', $nature = 0, $showempty = 0, $select_in = 0, $select_out = 0, $morecss = 'maxwidth300 maxwidthonsmartphone', $usecache = '', $disabledajaxcombo = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $out = '';
@@ -151,7 +151,7 @@ 
         return $out;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return list of journals with label by nature
      *
@@ -168,7 +168,7 @@ 
      */
     public function multi_select_journal($selectedIds = array(), $htmlname = 'journal', $nature = 0, $showempty = 0, $select_in = 0, $select_out = 0, $morecss = '', $usecache = '', $disabledajaxcombo = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $out = '';
@@ -230,7 +230,7 @@ 
         return $out;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of accounting category.
      *  Use mysoc->country_id or mysoc->country_code so they must be defined.
@@ -245,7 +245,7 @@ 
      */
     public function select_accounting_category($selected = 0, $htmlname = 'account_category', $useempty = 0, $maxlen = 0, $help = 1, $allcountries = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $mysoc;
 
         if (empty($mysoc->country_id) && empty($mysoc->country_code) && empty($allcountries)) {
@@ -322,7 +322,7 @@ 
         return $out;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return select filter with date of transaction
      *
@@ -332,7 +332,7 @@ 
      */
     public function select_bookkeeping_importkey($htmlname = 'importkey', $selectedkey = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         $options = array();
 
         $sql = "SELECT DISTINCT import_key FROM " . $this->db->prefix() . "accounting_bookkeeping";
@@ -355,7 +355,7 @@ 
         return Form::selectarray($htmlname, $options, $selectedkey);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return list of accounts with label by chart of accounts
      *
@@ -372,7 +372,7 @@ 
      */
     public function select_account($selectid, $htmlname = 'account', $showempty = 0, $event = array(), $select_in = 0, $select_out = 0, $morecss = 'minwidth100 maxwidth300 maxwidthonsmartphone', $usecache = '', $active = '1')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         require_once constant('DOL_DOCUMENT_ROOT') . '/core/lib/accounting.lib.php';
@@ -466,7 +466,7 @@ 
         return $out;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return list of auxiliary accounts. Cumulate list from customers, suppliers and users.
      *
@@ -480,7 +480,7 @@ 
      */
     public function select_auxaccount($selectid, $htmlname = 'account_num_aux', $showempty = 0, $morecss = 'minwidth100 maxwidth300 maxwidthonsmartphone', $usecache = '', $labelhtmlname = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $aux_account = array();
@@ -561,7 +561,7 @@ 
         return $out;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return HTML combo list of years existing into book keepping
      *
@@ -573,7 +573,7 @@ 
      */
     public function selectyear_accountancy_bookkepping($selected = '', $htmlname = 'yearid', $useempty = 0, $output_format = 'html')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $out_array = array();

Dolibarr/Code/Core/Traits/CommonIncoterm.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 = '';

Dolibarr/Code/Core/Traits/DolDeprecationHandler.php

Spacing +3 added lines, -3 removed lines

@@ -58,12 +58,12 @@ 
             return $this->$newProperty;
         }
         if ($this->isDynamicPropertiesEnabled()) {
-            return null;  // If the property is set, then __get is not called.
+            return null; // If the property is set, then __get is not called.
         }
         $msg = "DolDeprecationHandler: Undefined property '" . $name . "'" . self::getCallerInfoString();
         dol_syslog($msg);
         trigger_error($msg, E_USER_NOTICE);
-        return $this->$name;  // Returning value anyway (graceful degradation)
+        return $this->$name; // Returning value anyway (graceful degradation)
     }
 
     /**
@@ -93,7 +93,7 @@ 
         if (!$this->isDynamicPropertiesEnabled()) {
             $msg = "DolDeprecationHandler: Undefined property '" . $name . "'" . self::getCallerInfoString();
             trigger_error($msg, E_USER_NOTICE);
-            $this->$name = $value;  // Setting anyway for graceful degradation
+            $this->$name = $value; // Setting anyway for graceful degradation
         } else {
             $this->$name = $value;
         }

Dolibarr/Code/Commande/Classes/Commande.php

Indentation +28 added lines, -28 removed lines

@@ -607,7 +607,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set draft status
      *
@@ -617,7 +617,7 @@ 
      */
     public function setDraft($user, $idwarehouse = -1)
     {
-		//phpcs:enable
+        //phpcs:enable
         global $conf, $langs;
 
         $error = 0;
@@ -697,7 +697,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Tag the order as validated (opened)
      *  Function used when order is reopend after being closed.
@@ -707,7 +707,7 @@ 
      */
     public function set_reopen($user)
     {
-		// phpcs:enable
+        // phpcs:enable
         $error = 0;
 
         if ($this->statut != self::STATUS_CANCELED && $this->statut != self::STATUS_CLOSED) {
@@ -1737,7 +1737,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Add line into array
      *  $this->client must be loaded
@@ -1753,7 +1753,7 @@ 
      */
     public function add_product($idproduct, $qty, $remise_percent = 0.0, $date_start = '', $date_end = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $mysoc;
 
         if (!$qty) {
@@ -1989,7 +1989,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Add line of fixed discount in the order in DB
      *
@@ -1998,7 +1998,7 @@ 
      */
     public function insert_discount($idremise)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         include_once DOL_DOCUMENT_ROOT . '/core/lib/price.lib.php';
@@ -2058,7 +2058,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load array lines
      *
@@ -2068,7 +2068,7 @@ 
      */
     public function fetch_lines($only_product = 0, $loadalsotranslation = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf;
 
         $this->lines = array();
@@ -2335,7 +2335,7 @@ 
         return 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return a array with the pending stock by product
      *
@@ -2426,7 +2426,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Applique une remise relative
      *
@@ -2439,7 +2439,7 @@ 
      */
     public function set_remise($user, $remise, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         dol_syslog(get_class($this) . "::set_remise is deprecated, use setDiscount instead", LOG_NOTICE);
         // @phan-suppress-next-line PhanDeprecatedFunction
         return $this->setDiscount($user, $remise, $notrigger);
@@ -2507,7 +2507,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Set a fixed amount discount
      *
@@ -2574,7 +2574,7 @@ 
     }
     */
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set the order date
      *
@@ -2585,7 +2585,7 @@ 
      */
     public function set_date($user, $date, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if ($user->hasRight('commande', 'creer')) {
             $error = 0;
 
@@ -2632,7 +2632,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set delivery date
      *
@@ -2644,7 +2644,7 @@ 
      */
     public function set_date_livraison($user, $delivery_date, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         return $this->setDeliveryDate($user, $delivery_date, $notrigger);
     }
 
@@ -2704,7 +2704,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of orders (eventuelly filtered on a user) into an array
      *
@@ -2720,7 +2720,7 @@ 
      */
     public function liste_array($shortlist = 0, $draft = 0, $excluser = null, $socid = 0, $limit = 0, $offset = 0, $sortfield = 'c.date_commande', $sortorder = 'DESC')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user;
 
         $ga = array();
@@ -2840,7 +2840,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update order demand_reason
      *
@@ -2850,7 +2850,7 @@ 
      */
     public function demand_reason($demand_reason_id, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user;
 
         dol_syslog('Commande::demand_reason(' . $demand_reason_id . ')');
@@ -2904,7 +2904,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set customer ref
      *
@@ -2915,7 +2915,7 @@ 
      */
     public function set_ref_client($user, $ref_client, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if ($user->hasRight('commande', 'creer')) {
             $error = 0;
 
@@ -3546,7 +3546,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load indicators for dashboard (this->nbtodo and this->nbtodolate)
      *
@@ -3556,7 +3556,7 @@ 
      */
     public function load_board($user, $mode)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $clause = " WHERE";
@@ -3665,7 +3665,7 @@ 
         return $this->LibStatut($this->statut, $this->billed, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return label of status
      *
@@ -3677,7 +3677,7 @@ 
      */
     public function LibStatut($status, $billed, $mode, $donotshowbilled = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $hookmanager;
 
         $billedtext = '';

Spacing +3 added lines, -3 removed lines

@@ -374,8 +374,8 @@ 
     /**
      * Shipment on process
      */
-    const STATUS_SHIPMENTONPROCESS = 2;     // We set this status when a shipment is validated
-    const STATUS_ACCEPTED = 2;              // For backward compatibility. Use key STATUS_SHIPMENTONPROCESS instead.
+    const STATUS_SHIPMENTONPROCESS = 2; // We set this status when a shipment is validated
+    const STATUS_ACCEPTED = 2; // For backward compatibility. Use key STATUS_SHIPMENTONPROCESS instead.
 
     /**
      * Closed (Sent, billed or not)
@@ -1951,7 +1951,7 @@ 
                 //Incoterms
                 $this->fk_incoterms         = $obj->fk_incoterms;
                 $this->location_incoterms   = $obj->location_incoterms;
-                $this->label_incoterms    = $obj->label_incoterms;
+                $this->label_incoterms = $obj->label_incoterms;
 
                 // Multicurrency
                 $this->fk_multicurrency         = $obj->fk_multicurrency;

Dolibarr/Code/Commande/Classes/OrderLine.php

Indentation +2 added lines, -2 removed lines

@@ -624,7 +624,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update DB line fields total_xxx
      *  Used by migration
@@ -633,7 +633,7 @@ 
      */
     public function update_total()
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->db->begin();
 
         // Clean parameters

Dolibarr/Code/Commande/Api/Orders.php

Spacing +5 added lines, -5 removed lines

@@ -201,7 +201,7 @@ 
             if ($search_sale == -2) {
                 $sql .= " AND NOT EXISTS (SELECT sc.fk_soc FROM " . MAIN_DB_PREFIX . "societe_commerciaux as sc WHERE sc.fk_soc = t.fk_soc)";
             } elseif ($search_sale > 0) {
-                $sql .= " AND EXISTS (SELECT sc.fk_soc FROM " . MAIN_DB_PREFIX . "societe_commerciaux as sc WHERE sc.fk_soc = t.fk_soc AND sc.fk_user = " . ((int)$search_sale) . ")";
+                $sql .= " AND EXISTS (SELECT sc.fk_soc FROM " . MAIN_DB_PREFIX . "societe_commerciaux as sc WHERE sc.fk_soc = t.fk_soc AND sc.fk_user = " . ((int) $search_sale) . ")";
             }
         }
         // Add sql filters
@@ -300,7 +300,7 @@ 
             throw new RestException(500, "Error creating order", array_merge(array($this->commande->error), $this->commande->errors));
         }
 
-        return ((int)$this->commande->id);
+        return ((int) $this->commande->id);
     }
 
     /**
@@ -378,7 +378,7 @@ 
             throw new RestException(403, 'Access not allowed for login ' . DolibarrApiAccess::$user->login);
         }
 
-        $request_data = (object)$request_data;
+        $request_data = (object) $request_data;
 
         $request_data->desc = sanitizeVal($request_data->desc, 'restricthtml');
         $request_data->label = sanitizeVal($request_data->label);
@@ -445,7 +445,7 @@ 
             throw new RestException(403, 'Access not allowed for login ' . DolibarrApiAccess::$user->login);
         }
 
-        $request_data = (object)$request_data;
+        $request_data = (object) $request_data;
 
         $request_data->desc = sanitizeVal($request_data->desc, 'restricthtml');
         $request_data->label = sanitizeVal($request_data->label);
@@ -1055,7 +1055,7 @@ 
         $sql .= " ON edet.fk_elementdet = cdet.rowid";
         $sql .= " JOIN " . MAIN_DB_PREFIX . "commande as c";
         $sql .= " ON cdet.fk_commande = c.rowid";
-        $sql .= " WHERE c.rowid = " . ((int)$id);
+        $sql .= " WHERE c.rowid = " . ((int) $id);
         $sql .= " GROUP BY e.rowid";
         $sql .= $this->db->order("e.rowid", "ASC");
 

Dolibarr/Code/Compta/Classes/LignePrelevement.php

Indentation +2 added lines, -2 removed lines

@@ -152,7 +152,7 @@ 
         return $this->LibStatut($this->statut, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Return status label for a status
      *
@@ -162,7 +162,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         if ($mode == 0) {

Spacing +2 added lines, -2 removed lines

@@ -71,8 +71,8 @@
 
     const STATUS_DRAFT = 0;
     const STATUS_NOT_USED = 1;
-    const STATUS_CREDITED = 2;      // STATUS_CREDITED and STATUS_DEBITED is same. Difference is in ->type
-    const STATUS_DEBITED = 2;       // STATUS_CREDITED and STATUS_DEBITED is same. Difference is in ->type
+    const STATUS_CREDITED = 2; // STATUS_CREDITED and STATUS_DEBITED is same. Difference is in ->type
+    const STATUS_DEBITED = 2; // STATUS_CREDITED and STATUS_DEBITED is same. Difference is in ->type
     const STATUS_REJECTED = 3;
 
 

Dolibarr/Code/Compta/Classes/BonPrelevement.php

Indentation +28 added lines, -28 removed lines

@@ -232,7 +232,7 @@ 
         $this->fetched = 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Add invoice to withdrawal
      *
@@ -250,7 +250,7 @@ 
      */
     public function AddFacture($invoice_id, $client_id, $client_nom, $amount, $code_banque, $code_guichet, $number, $number_key, $type = 'debit-order', $sourcetype = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         $result = 0;
         $line_id = 0;
 
@@ -470,7 +470,7 @@ 
         return $this->updateCommon($user, $notrigger);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set direct debit or credit transfer order to "paid" status.
      *  Then create the payment for each invoice or salary of the prelemevement_bon.
@@ -482,7 +482,7 @@ 
      */
     public function set_infocredit($user, $date, $type = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $error = 0;
@@ -682,7 +682,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set withdrawal to transmitted status
      *
@@ -693,7 +693,7 @@ 
      */
     public function set_infotrans($user, $date, $method)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $error = 0;
@@ -818,7 +818,7 @@ 
         return $arr;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Returns amount waiting for direct debit payment or credit transfer payment
      *
@@ -828,7 +828,7 @@ 
      */
     public function SommeAPrelever($mode = 'direct-debit', $type = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "SELECT sum(pd.amount) as nb";
         if ($type !== 'salary') {
             if ($mode != 'bank-transfer') {
@@ -890,7 +890,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Get number of invoices to pay
      *
@@ -900,7 +900,7 @@ 
      */
     public function NbFactureAPrelever($type = 'direct-debit', $forsalary = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if ($forsalary == 1) {
             $sql = "SELECT count(s.rowid) as nb";
             $sql .= " FROM " . MAIN_DB_PREFIX . "salary as s";
@@ -957,7 +957,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Create a BAN payment order:
      *  - Select waiting requests from prelevement_demande (or use $did if provided)
@@ -980,7 +980,7 @@ 
      */
     public function create($banque = 0, $agence = 0, $mode = 'real', $format = 'ALL', $executiondate = '', $notrigger = 0, $type = 'direct-debit', $did = 0, $fk_bank_account = 0, $sourcetype = 'invoice')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $user;
 
         dol_syslog(__METHOD__ . " Bank=" . $banque . " Office=" . $agence . " mode=" . $mode . " format=" . $format, LOG_DEBUG);
@@ -1640,7 +1640,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Add a notification
      *
@@ -1651,7 +1651,7 @@ 
      */
     public function addNotification($db, $user, $action)
     {
-		// phpcs:enable
+        // phpcs:enable
         $result = 0;
 
         if (is_object($user)) {
@@ -2073,7 +2073,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Write recipient of request (customer)
      *
@@ -2092,7 +2092,7 @@ 
      */
     public function EnregDestinataire($rowid, $client_nom, $rib_banque, $rib_guichet, $rib_number, $amount, $ref, $facid, $rib_dom = '', $type = 'direct-debit')
     {
-		// phpcs:enable
+        // phpcs:enable
         fwrite($this->file, "06");
         fwrite($this->file, "08"); // Prelevement ordinaire
 
@@ -2148,7 +2148,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Write recipient (thirdparty concerned by request)
      *
@@ -2176,7 +2176,7 @@ 
      */
     public function EnregDestinataireSEPA($row_code_client, $row_nom, $row_address, $row_zip, $row_town, $row_country_code, $row_cb, $row_cg, $row_cc, $row_somme, $row_ref, $row_idfac, $row_iban, $row_bic, $row_datec, $row_drum, $row_rum, $type = 'direct-debit', $row_comment = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         if (getDolGlobalString('SEPA_FORCE_TWO_DECIMAL')) {
@@ -2316,7 +2316,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Write sender of request (me).
      *
@@ -2326,7 +2326,7 @@ 
      */
     public function EnregEmetteur($type = 'direct-debit')
     {
-		// phpcs:enable
+        // phpcs:enable
         fwrite($this->file, "03");
         fwrite($this->file, "08"); // Prelevement ordinaire
 
@@ -2386,7 +2386,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Write sender of request (me).
      *  Note: The tag PmtInf is opened here but closed into caller
@@ -2404,7 +2404,7 @@ 
      */
     public function EnregEmetteurSEPA($configuration, $ladate, $nombre, $total, $CrLf = '\n', $format = 'FRST', $type = 'direct-debit', $fk_bank_account = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
 
         // Clean parameters
         $dateTime_YMD = dol_print_date($ladate, '%Y%m%d');
@@ -2586,7 +2586,7 @@ 
         return $XML_SEPA_INFO;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Write end
      *
@@ -2595,7 +2595,7 @@ 
      */
     public function EnregTotal($total)
     {
-		// phpcs:enable
+        // phpcs:enable
         fwrite($this->file, "08");
         fwrite($this->file, "08"); // Prelevement ordinaire
 
@@ -2660,7 +2660,7 @@ 
         return $this->LibStatut($this->statut, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return status label for a status
      *
@@ -2670,7 +2670,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
             global $langs;
             //$langs->load("mymodule");
@@ -2698,7 +2698,7 @@ 
         return dolGetStatus($this->labelStatus[$status], $this->labelStatusShort[$status], '', $statusType, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Load indicators for dashboard (this->nbtodo and this->nbtodolate)
      *
@@ -2708,7 +2708,7 @@ 
      */
     public function load_board($user, $mode)
     {
-		// phpcs:enable
+        // phpcs:enable
         if ($user->socid) {
             return -1; // protection pour eviter appel par utilisateur externe
         }

Spacing +15 added lines, -15 removed lines

@@ -106,8 +106,8 @@ 
 
     const STATUS_DRAFT = 0;
     const STATUS_TRANSFERED = 1;
-    const STATUS_CREDITED = 2;      // STATUS_CREDITED and STATUS_DEBITED is same. Difference is in ->type
-    const STATUS_DEBITED = 2;       // STATUS_CREDITED and STATUS_DEBITED is same. Difference is in ->type
+    const STATUS_CREDITED = 2; // STATUS_CREDITED and STATUS_DEBITED is same. Difference is in ->type
+    const STATUS_DEBITED = 2; // STATUS_CREDITED and STATUS_DEBITED is same. Difference is in ->type
 
 
     /**
@@ -313,7 +313,7 @@ 
     public function addline(&$line_id, $client_id, $client_nom, $amount, $code_banque, $code_guichet, $number, $number_key, $sourcetype = '')
     {
         $result = -1;
-        $concat = 0;    // ??? what is this for. Seems not used.
+        $concat = 0; // ??? what is this for. Seems not used.
 
         if ($concat == 1) {
             /*
@@ -353,7 +353,7 @@ 
             $sql .= ($sourcetype == 'salary' ? ", fk_user" : "");
             $sql .= ") VALUES (";
             $sql .= $this->id;
-            $sql .= ", " . (($sourcetype != 'salary') ? ((int) $client_id) : "0");  // fk_soc can't be null
+            $sql .= ", " . (($sourcetype != 'salary') ? ((int) $client_id) : "0"); // fk_soc can't be null
             $sql .= ", '" . $this->db->escape($client_nom) . "'";
             $sql .= ", " . ((float) price2num($amount));
             $sql .= ", '" . $this->db->escape($code_banque) . "'";
@@ -444,7 +444,7 @@ 
                 if (empty($this->status)) {     // Value is sometimes null in database
                     $this->status = 0;
                 }
-                $this->statut         = $this->status; // For backward compatibility
+                $this->statut = $this->status; // For backward compatibility
 
                 $this->fetched = 1;
 
@@ -633,7 +633,7 @@ 
                         } else {
                             $modeforaddpayment = 'payment';
                             $labelforaddpayment = '(CustomerInvoicePayment)';
-                            $addbankurl = 'direct-debit';   // = 'directdebit'
+                            $addbankurl = 'direct-debit'; // = 'directdebit'
                         }
 
                         $result = $paiement->addPaymentToBank($user, $modeforaddpayment, $labelforaddpayment, $fk_bank_account, '', '', 0, '', $addbankurl);
@@ -667,7 +667,7 @@ 
 
             // End of procedure
             if ($error == 0) {
-                $this->date_credit = $date;     // date credit or debit
+                $this->date_credit = $date; // date credit or debit
                 $this->statut = self::STATUS_CREDITED;
                 $this->status = self::STATUS_CREDITED;
 
@@ -1053,7 +1053,7 @@ 
                 $sql .= " FROM " . MAIN_DB_PREFIX . "salary as f";
                 $sql .= " LEFT JOIN " . MAIN_DB_PREFIX . "prelevement_demande as pd ON f.rowid = pd.fk_salary";
                 $sql .= " LEFT JOIN " . MAIN_DB_PREFIX . "user as s ON s.rowid = f.fk_user";
-                $sql .= " LEFT JOIN " . MAIN_DB_PREFIX . "user_rib as sr ON s.rowid = sr.fk_user";  // TODO Add AND sr.default_rib = 1 here
+                $sql .= " LEFT JOIN " . MAIN_DB_PREFIX . "user_rib as sr ON s.rowid = sr.fk_user"; // TODO Add AND sr.default_rib = 1 here
             }
             if ($sourcetype != 'salary') {
                 if ($type != 'bank-transfer') {
@@ -1076,7 +1076,7 @@ 
             $sql .= " AND pd.traite = 0";
             $sql .= " AND pd.ext_payment_id IS NULL";
             if ($sourcetype != 'salary') {
-                $sql .= " AND sr.type = 'ban'";     // TODO Add AND sr.type = 'ban' for users too
+                $sql .= " AND sr.type = 'ban'"; // TODO Add AND sr.type = 'ban' for users too
             }
             if ($did > 0) {
                 $sql .= " AND pd.rowid = " . ((int) $did);
@@ -1088,7 +1088,7 @@ 
                 $i = 0;
 
                 while ($i < $num) {
-                    $row = $this->db->fetch_row($resql);    // TODO Replace with fetch_object()
+                    $row = $this->db->fetch_row($resql); // TODO Replace with fetch_object()
                     $factures[$i] = $row; // All fields
 
                     if ($row[7] == 0) {
@@ -1239,7 +1239,7 @@ 
                 $ref = substr($year, -2) . $month;
 
                 // Get next free number for the ref of bon prelevement
-                $sql = "SELECT substring(ref from char_length(ref) - 1)";   // To extract "YYMMXX" from "TYYMMXX"
+                $sql = "SELECT substring(ref from char_length(ref) - 1)"; // To extract "YYMMXX" from "TYYMMXX"
                 $sql .= " FROM " . MAIN_DB_PREFIX . "prelevement_bons";
                 $sql .= " WHERE ref LIKE '_" . $this->db->escape($ref) . "%'";
                 $sql .= " AND entity = " . ((int) $conf->entity);
@@ -1369,7 +1369,7 @@ 
                         $this->emetteur_iban               = $account->iban;
                         $this->emetteur_bic                = $account->bic;
 
-                        $this->emetteur_ics = (($type == 'bank-transfer' && getDolGlobalString("SEPA_USE_IDS")) ? $account->ics_transfer : $account->ics);  // Example "FR78ZZZ123456"
+                        $this->emetteur_ics = (($type == 'bank-transfer' && getDolGlobalString("SEPA_USE_IDS")) ? $account->ics_transfer : $account->ics); // Example "FR78ZZZ123456"
 
                         $this->raison_sociale = $account->proprio;
                     }
@@ -1424,7 +1424,7 @@ 
 
             if (!$error) {
                 $this->db->commit();
-                return count($factures_prev);   // The error of failed lines are into $this->invoice_in_error and $this->thirdparty_in_error
+                return count($factures_prev); // The error of failed lines are into $this->invoice_in_error and $this->thirdparty_in_error
             } else {
                 $this->db->rollback();
                 return -1;
@@ -2427,7 +2427,7 @@ 
             $this->emetteur_iban = $account->iban;
             $this->emetteur_bic = $account->bic;
 
-            $this->emetteur_ics = (($type == 'bank-transfer' && getDolGlobalString("SEPA_USE_IDS")) ? $account->ics_transfer : $account->ics);  // Ex: PRELEVEMENT_ICS = "FR78ZZZ123456";
+            $this->emetteur_ics = (($type == 'bank-transfer' && getDolGlobalString("SEPA_USE_IDS")) ? $account->ics_transfer : $account->ics); // Ex: PRELEVEMENT_ICS = "FR78ZZZ123456";
 
             $this->raison_sociale = $account->proprio;
         }
@@ -2823,7 +2823,7 @@ 
         if ($id) {
             $sql = "SELECT COUNT(*) AS nb FROM " . MAIN_DB_PREFIX . "prelevement_lignes";
             $sql .= " WHERE fk_prelevement_bons = " . ((int) $id);
-            $sql .= " AND fk_soc = 0";  // fk_soc can't be NULL
+            $sql .= " AND fk_soc = 0"; // fk_soc can't be NULL
             $sql .= " AND fk_user IS NOT NULL";
 
             $num = 0;