alxarafe / alixar

Dolibarr/Code/Stripe/Classes/Stripe.php

Spacing +4 added lines, -4 removed lines

@@ -488,7 +488,7 @@ 
                 "confirmation_method" => $mode,
                 "amount" => $stripeamount,
                 "currency" => $currency_code,
-                "payment_method_types" => $paymentmethodtypes,  // When payment_method_types is set, return_url is not required but payment mode can't be managed from dashboard
+                "payment_method_types" => $paymentmethodtypes, // When payment_method_types is set, return_url is not required but payment mode can't be managed from dashboard
                 /*
                 'return_url' => $dolibarr_main_url_root.'/public/payment/paymentok.php',
                 'automatic_payment_methods' => array(
@@ -503,7 +503,7 @@ 
             );
             if ($descriptor) {
                 $dataforintent["statement_descriptor_suffix"] = $descriptor; // For card payment, 22 chars that appears on bank receipt (prefix into stripe setup + this suffix)
-                $dataforintent["statement_descriptor"] = $descriptor;   // For SEPA, it will take only statement_descriptor, not statement_descriptor_suffix
+                $dataforintent["statement_descriptor"] = $descriptor; // For SEPA, it will take only statement_descriptor, not statement_descriptor_suffix
             }
             if (!is_null($customer)) {
                 $dataforintent["customer"] = $customer;
@@ -712,7 +712,7 @@ 
 
             $dataforintent = array(
                 "confirm" => $confirmnow, // Do not confirm immediately during creation of intent
-                "payment_method_types" => $paymentmethodtypes,  // When payment_method_types is set, return_url is not required but payment mode can't be managed from dashboard
+                "payment_method_types" => $paymentmethodtypes, // When payment_method_types is set, return_url is not required but payment mode can't be managed from dashboard
                 /*
                  'return_url' => $dolibarr_main_url_root.'/public/payment/paymentok.php',
                  'automatic_payment_methods' => array(
@@ -1128,7 +1128,7 @@ 
                     } catch (Exception $e) {
                         $sepa = null;
                         $this->error = 'Stripe error: ' . $e->getMessage() . '. Check the BAN information.';
-                        dol_syslog($this->error, LOG_WARNING);  // Error from Stripe, so a warning on Dolibarr
+                        dol_syslog($this->error, LOG_WARNING); // Error from Stripe, so a warning on Dolibarr
                     }
                 }
             }

Dolibarr/Code/Ticket/Classes/CTicketCategory.php

Indentation +2 added lines, -2 removed lines

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

Dolibarr/Code/Ticket/Classes/Ticket.php

Indentation +6 added lines, -6 removed lines

@@ -1467,7 +1467,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return status label of object
      *
@@ -1479,7 +1479,7 @@ 
      */
     public function LibStatut($status, $mode = 0, $notooltip = 0, $progress = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $hookmanager;
 
         $labelStatus = (isset($status) && !empty($this->labelStatus[$status])) ? $this->labelStatus[$status] : '';
@@ -2433,7 +2433,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return if at least one photo is available
      *
@@ -2442,7 +2442,7 @@ 
      */
     public function is_photo_available($sdir)
     {
-		// phpcs:enable
+        // phpcs:enable
         include_once DOL_DOCUMENT_ROOT . '/core/lib/files.lib.php';
 
         global $conf;
@@ -3059,7 +3059,7 @@ 
         return $is_sent;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Load indicators for dashboard (this->nbtodo and this->nbtodolate)
      *
@@ -3069,7 +3069,7 @@ 
      */
     public function load_board($user, $mode)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user, $langs;
 
         $now = dol_now();

Spacing +13 added lines, -13 removed lines

@@ -263,14 +263,14 @@ 
     /**
      * Status
      */
-    const STATUS_NOT_READ = 0;          // Draft. Not take into account yet.
-    const STATUS_READ = 1;              // Ticket was read.
-    const STATUS_ASSIGNED = 2;          // Ticket was just assigned to someone. Not in progress yet.
-    const STATUS_IN_PROGRESS = 3;       // In progress
-    const STATUS_NEED_MORE_INFO = 5;    // Waiting requester feedback
-    const STATUS_WAITING = 7;           // On hold
-    const STATUS_CLOSED = 8;            // Closed - Solved
-    const STATUS_CANCELED = 9;          // Closed - Not solved
+    const STATUS_NOT_READ = 0; // Draft. Not take into account yet.
+    const STATUS_READ = 1; // Ticket was read.
+    const STATUS_ASSIGNED = 2; // Ticket was just assigned to someone. Not in progress yet.
+    const STATUS_IN_PROGRESS = 3; // In progress
+    const STATUS_NEED_MORE_INFO = 5; // Waiting requester feedback
+    const STATUS_WAITING = 7; // On hold
+    const STATUS_CLOSED = 8; // Closed - Solved
+    const STATUS_CANCELED = 9; // Closed - Not solved
 
 
     /**
@@ -1854,7 +1854,7 @@ 
         $actioncomm->socid = $this->socid;
         $actioncomm->label = $this->subject;
         $actioncomm->note_private = $this->message;
-        $actioncomm->userassigned = array($user->id => array('id' => $user->id,'transparency' => 0));
+        $actioncomm->userassigned = array($user->id => array('id' => $user->id, 'transparency' => 0));
         $actioncomm->userownerid = $user->id;
         $actioncomm->datep = $now;
         $actioncomm->percentage = -1; // percentage is not relevant for punctual events
@@ -2429,7 +2429,7 @@ 
                         'email' => $obj->email,
                         'rowid' => $obj->rowid,
                         'code' => $obj->code,
-                        'libelle' => $labelType,        // deprecated, replaced with labeltype
+                        'libelle' => $labelType, // deprecated, replaced with labeltype
                         'labeltype' => $labelType,
                         'status' => $obj->statuslink,
                         'statuscontact' => $obj->statuscontact,
@@ -2514,7 +2514,7 @@ 
             if (is_resource($handle)) {
                 while (($file = readdir($handle)) !== false) {
                     if (!utf8_check($file)) {
-                        $file = mb_convert_encoding($file, 'UTF-8', 'ISO-8859-1');  // To be sure data is stored in UTF8 in memory
+                        $file = mb_convert_encoding($file, 'UTF-8', 'ISO-8859-1'); // To be sure data is stored in UTF8 in memory
                     }
                     if (dol_is_file($dir . $file)) {
                         return true;
@@ -2552,7 +2552,7 @@ 
         $formmail->trackid = (is_null($forcetrackid) ? 'tic' . $this->id : '');
         $attachedfiles = $formmail->get_attached_files();
 
-        $filepath = $attachedfiles['paths'];    // path is for example user->dir_temp.'/'.$user->id.'/'...
+        $filepath = $attachedfiles['paths']; // path is for example user->dir_temp.'/'.$user->id.'/'...
         $filename = $attachedfiles['names'];
         $mimetype = $attachedfiles['mimes'];
 
@@ -2980,7 +2980,7 @@ 
                                     if ($result) {
                                         // update last_msg_sent date (for last message sent to external users)
                                         $this->date_last_msg_sent = dol_now();
-                                        $this->update($user, 1);    // disable trigger when updating date_last_msg_sent. sendTicketMessageByEmail already create an event in actioncomm table.
+                                        $this->update($user, 1); // disable trigger when updating date_last_msg_sent. sendTicketMessageByEmail already create an event in actioncomm table.
                                     }
                                 }
                             }

Dolibarr/Code/Ticket/Api/Tickets.php

Spacing +2 added lines, -2 removed lines

@@ -300,14 +300,14 @@
         $sql .= " LEFT JOIN " . MAIN_DB_PREFIX . "ticket_extrafields AS ef ON (ef.fk_object = t.rowid)"; // Modification VMR Global Solutions to include extrafields as search parameters in the API GET call, so we will be able to filter on extrafields
         $sql .= ' WHERE t.entity IN (' . getEntity('ticket', 1) . ')';
         if ($socid > 0) {
-            $sql .= " AND t.fk_soc = " . ((int)$socid);
+            $sql .= " AND t.fk_soc = " . ((int) $socid);
         }
         // Search on sale representative
         if ($search_sale && $search_sale != '-1') {
             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

Dolibarr/Code/User/Classes/UserGroup.php

Indentation +8 added lines, -8 removed lines

@@ -741,7 +741,7 @@ 
         return $this->LibStatut(0, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the label of a given status
      *
@@ -751,7 +751,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
         $langs->load('users');
         return '';
@@ -873,8 +873,8 @@ 
         return $result;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Retourne chaine DN complete dans l'annuaire LDAP pour l'objet
      *
@@ -886,7 +886,7 @@ 
      */
     public function _load_ldap_dn($info, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
         $dn = '';
         if ($mode == 0) {
@@ -902,8 +902,8 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Initialize the info array (array of LDAP values) that will be used to call LDAP functions
      *
@@ -911,7 +911,7 @@ 
      */
     public function _load_ldap_info()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $info = array();

Spacing +2 added lines, -2 removed lines

@@ -100,7 +100,7 @@ 
     public $members = array(); // Array of users
 
     public $nb_rights; // Number of rights granted to the user
-    public $nb_users;  // Number of users in the group
+    public $nb_users; // Number of users in the group
 
     public $rights; // Permissions of the group
 
@@ -174,7 +174,7 @@ 
 
         if ($result) {
             if ($load_members) {
-                $this->members = $this->listUsersForGroup();    // This make a lot of subrequests
+                $this->members = $this->listUsersForGroup(); // This make a lot of subrequests
             }
 
             return 1;

Dolibarr/Code/User/Classes/User.php

Indentation +36 added lines, -36 removed lines

@@ -1759,7 +1759,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Create a user from a contact object. User will be internal but if contact is linked to a third party, user will be external
      *
@@ -1770,7 +1770,7 @@ 
      */
     public function create_from_contact($contact, $login = '', $password = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user, $langs;
 
         $error = 0;
@@ -1845,7 +1845,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Create a user into database from a member object.
      *  If $member->fk_soc is set, it will be an external user.
@@ -1856,7 +1856,7 @@ 
      */
     public function create_from_member($member, $login = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user;
 
         // Set properties on new user
@@ -1934,7 +1934,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Assign rights by default
      *
@@ -1942,7 +1942,7 @@ 
      */
     public function set_default_rights()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $rd = array();
@@ -2335,7 +2335,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update the user's last login date in the database.
      *  Function called when a new connection is made by the user
@@ -2344,7 +2344,7 @@ 
      */
     public function update_last_login_date()
     {
-		// phpcs:enable
+        // phpcs:enable
         $now = dol_now();
 
         $userremoteip = getUserRemoteIP();
@@ -2517,7 +2517,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Send a new password (or instructions to reset it) by email
      *
@@ -2528,7 +2528,7 @@ 
      */
     public function send_password($user, $password = '', $changelater = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $mysoc;
         global $dolibarr_main_url_root;
 
@@ -2646,7 +2646,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Read clicktodial information for user
      *
@@ -2654,7 +2654,7 @@ 
      */
     public function fetch_clicktodial()
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "SELECT url, login, pass, poste ";
         $sql .= " FROM " . $this->db->prefix() . "user_clicktodial as u";
         $sql .= " WHERE u.fk_user = " . ((int) $this->id);
@@ -2680,7 +2680,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update clicktodial info
      *
@@ -2688,7 +2688,7 @@ 
      */
     public function update_clicktodial()
     {
-		// phpcs:enable
+        // phpcs:enable
         $this->db->begin();
 
         $sql = "DELETE FROM " . $this->db->prefix() . "user_clicktodial";
@@ -2718,7 +2718,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Add user into a group
      *
@@ -2729,7 +2729,7 @@ 
      */
     public function SetInGroup($group, $entity, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $user;
 
         $error = 0;
@@ -2774,7 +2774,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Remove a user from a group
      *
@@ -2785,7 +2785,7 @@ 
      */
     public function RemoveFromGroup($group, $entity, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs, $user;
 
         $error = 0;
@@ -3178,7 +3178,7 @@ 
         return $this->LibStatut(isset($this->statut) ? (int) $this->statut : (int) $this->status, $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return the label of a status of user (active, inactive)
      *
@@ -3188,7 +3188,7 @@ 
      */
     public function LibStatut($status, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
@@ -3280,8 +3280,8 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Retourne chaine DN complete dans l'annuaire LDAP pour l'objet
      *
@@ -3293,7 +3293,7 @@ 
      */
     public function _load_ldap_dn($info, $mode = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
         $dn = '';
         if ($mode == 0) {
@@ -3306,8 +3306,8 @@ 
         return $dn;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Initialize the info array (array of LDAP values) that will be used to call LDAP functions
      *
@@ -3315,7 +3315,7 @@ 
      */
     public function _load_ldap_info()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $info = array();
@@ -3621,7 +3621,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Update user using data from the LDAP
      *
@@ -3630,7 +3630,7 @@ 
      */
     public function update_ldap2dolibarr(&$ldapuser)
     {
-		// phpcs:enable
+        // phpcs:enable
         // TODO: Voir pourquoi le update met à jour avec toutes les valeurs vide (global $user écrase ?)
         global $user, $conf;
 
@@ -3675,7 +3675,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Return and array with all instantiated first level children users of current user
      *
@@ -3684,7 +3684,7 @@ 
      */
     public function get_children()
     {
-		// phpcs:enable
+        // phpcs:enable
         $sql = "SELECT rowid FROM " . $this->db->prefix() . "user";
         $sql .= " WHERE fk_user = " . ((int) $this->id);
 
@@ -3735,7 +3735,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Build the hierarchy/tree of users into an array.
      *  Set and return this->users that is an array sorted according to tree with arrays of:
@@ -3751,7 +3751,7 @@ 
      */
     public function get_full_tree($deleteafterid = 0, $filter = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
         global $hookmanager;
 
@@ -3880,7 +3880,7 @@ 
         return $childids;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  For user id_user and its children available in this->users, define property fullpath and fullname.
      *  Function called by get_full_tree().
@@ -3891,7 +3891,7 @@ 
      */
     public function build_path_from_id_user($id_user, $protection = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         //dol_syslog(get_class($this)."::build_path_from_id_user id_user=".$id_user." protection=".$protection, LOG_DEBUG);
 
         if (!empty($this->users[$id_user]['fullpath'])) {
@@ -4011,7 +4011,7 @@ 
         return $this->commonGenerateDocument($modelpath, $modele, $outputlangs, $hidedetails, $hidedesc, $hideref, $moreparams);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return property of user from its id
      *
@@ -4021,7 +4021,7 @@ 
      */
     public function user_get_property($rowid, $mode)
     {
-		// phpcs:enable
+        // phpcs:enable
         $user_property = '';
 
         if (empty($rowid)) {

Spacing +12 added lines, -12 removed lines

@@ -599,7 +599,7 @@ 
                 $this->note_public = $obj->note_public;
                 $this->note_private = $obj->note_private;
 
-                $this->statut       = $obj->status;         // deprecated
+                $this->statut       = $obj->status; // deprecated
                 $this->status       = $obj->status;
 
                 $this->photo        = $obj->photo;
@@ -728,7 +728,7 @@ 
             require_once constant('DOL_DOCUMENT_ROOT') . '/core/class/defaultvalues.class.php';
 
             $defaultValues = new DefaultValues($this->db);
-            $result = $defaultValues->fetchAll('', '', 0, 0, '(t.user_id:in:0,' . $this->id . ') AND (entity:in:' . (isset($this->entity) ? $this->entity : $conf->entity) . ',' . $conf->entity . ')');    // User 0 (all) + me (if defined)
+            $result = $defaultValues->fetchAll('', '', 0, 0, '(t.user_id:in:0,' . $this->id . ') AND (entity:in:' . (isset($this->entity) ? $this->entity : $conf->entity) . ',' . $conf->entity . ')'); // User 0 (all) + me (if defined)
             //$result = $defaultValues->fetchAll('', '', 0, 0, array('t.user_id'=>array(0, $this->id), 'entity'=>array((isset($this->entity) ? $this->entity : $conf->entity), $conf->entity)));    // User 0 (all) + me (if defined)
 
             if (!is_array($result) && $result < 0) {
@@ -842,16 +842,16 @@ 
         // Special case for external user
         if (!empty($this->socid)) {
             if ($module == 'societe' && ($permlevel1 == 'creer' || $permlevel1 == 'write')) {
-                return 0;   // An external user never has the permission ->societe->write to see all thirdparties (always restricted to himself)
+                return 0; // An external user never has the permission ->societe->write to see all thirdparties (always restricted to himself)
             }
             if ($module == 'societe' && $permlevel1 == 'client' && $permlevel2 == 'voir') {
-                return 0;   // An external user never has the permission ->societe->client->voir to see all thirdparties (always restricted to himself)
+                return 0; // An external user never has the permission ->societe->client->voir to see all thirdparties (always restricted to himself)
             }
             if ($module == 'societe' && $permlevel1 == 'export') {
-                return 0;   // An external user never has the permission ->societe->export to see all thirdparties (always restricted to himself)
+                return 0; // An external user never has the permission ->societe->export to see all thirdparties (always restricted to himself)
             }
             if ($module == 'societe' && ($permlevel1 == 'supprimer' || $permlevel1 == 'delete')) {
-                return 0;   // An external user never has the permission ->societe->delete to see all thirdparties (always restricted to himself)
+                return 0; // An external user never has the permission ->societe->delete to see all thirdparties (always restricted to himself)
             }
         }
 
@@ -999,7 +999,7 @@ 
             $sql .= " FROM " . $this->db->prefix() . "rights_def";
             $sql .= " WHERE entity = " . ((int) $entity);
             if (!empty($whereforadd) && $whereforadd != 'allmodules') {
-                $sql .= " AND (" . $whereforadd . ")";  // Note: parenthesis are important because whereforadd can contains OR. Also note that $whereforadd is already sanitized
+                $sql .= " AND (" . $whereforadd . ")"; // Note: parenthesis are important because whereforadd can contains OR. Also note that $whereforadd is already sanitized
             }
 
             $sqldelete = "DELETE FROM " . $this->db->prefix() . "user_rights";
@@ -1135,7 +1135,7 @@ 
             $sql .= " FROM " . $this->db->prefix() . "rights_def";
             $sql .= " WHERE entity IN (" . $this->db->sanitize($entity, 0, 0, 0, 0) . ")";
             if (!empty($wherefordel) && $wherefordel != 'allmodules') {
-                $sql .= " AND (" . $wherefordel . ")";  // Note: parenthesis are important because wherefordel can contains OR. Also note that $wherefordel is already sanitized
+                $sql .= " AND (" . $wherefordel . ")"; // Note: parenthesis are important because wherefordel can contains OR. Also note that $wherefordel is already sanitized
             }
 
             // avoid admin to remove his own important rights
@@ -1309,12 +1309,12 @@ 
                     $sql .= " AND r.entity = " . ((int) $conf->entity);
                 }
             } else {
-                $sql .= " AND gr.entity = " . ((int) $conf->entity);  // Only groups created in current entity
+                $sql .= " AND gr.entity = " . ((int) $conf->entity); // Only groups created in current entity
                 // The entity on the table gu=usergroup_user should be useless and should never be used because it is already into gr and r.
                 // but when using MULTICOMPANY_TRANSVERSE_MODE, we may have inserted record that make rubbish result here due to the duplicate record of
                 // other entities, so we are forced to add a filter on gu here
                 $sql .= " AND gu.entity IN (0," . $conf->entity . ")";
-                $sql .= " AND r.entity = " . ((int) $conf->entity);   // Only permission of modules enabled in current entity
+                $sql .= " AND r.entity = " . ((int) $conf->entity); // Only permission of modules enabled in current entity
             }
             // End of strange business rule
             $sql .= " AND gr.fk_usergroup = gu.fk_usergroup";
@@ -2796,7 +2796,7 @@ 
         $sql .= " WHERE fk_user  = " . ((int) $this->id);
         $sql .= " AND fk_usergroup = " . ((int) $group);
         if (empty($entity)) {
-            $sql .= " AND entity IN (0, 1)";    // group may be in entity 0 (so $entity=0) and link with user into entity 1.
+            $sql .= " AND entity IN (0, 1)"; // group may be in entity 0 (so $entity=0) and link with user into entity 1.
         } else {
             $sql .= " AND entity = " . ((int) $entity);
         }
@@ -3516,7 +3516,7 @@ 
         $this->iplastlogin = '127.0.0.1';
         $this->datepreviouslogin = $now;
         $this->ippreviouslogin = '127.0.0.1';
-        $this->statut = 1;      // deprecated
+        $this->statut = 1; // deprecated
         $this->status = 1;
 
         $this->entity = 1;

Dolibarr/Code/User/Api/Users.php

Indentation +2 added lines, -2 removed lines

@@ -664,7 +664,7 @@ 
         );
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Clean sensible object datas
      *
@@ -673,7 +673,7 @@ 
      */
     protected function _cleanObjectDatas($object)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $object = parent::_cleanObjectDatas($object);

Dolibarr/Code/WebHook/Classes/Target.php

Indentation +2 added lines, -2 removed lines

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

Dolibarr/Code/SupplierProposal/Classes/SupplierProposal.php

Indentation +16 added lines, -16 removed lines

@@ -243,7 +243,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Add line into array ->lines
      *
@@ -256,7 +256,7 @@ 
      */
     public function add_product($idproduct, $qty, $remise_percent = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $mysoc;
 
         if (!$qty) {
@@ -300,7 +300,7 @@ 
         return -1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Adding line of fixed discount in the proposal in DB
      *
@@ -309,7 +309,7 @@ 
      */
     public function insert_discount($idremise)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs;
 
         include_once DOL_DOCUMENT_ROOT . '/core/lib/price.lib.php';
@@ -1508,7 +1508,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set delivery date
      *
@@ -1519,7 +1519,7 @@ 
      */
     public function set_date_livraison($user, $delivery_date)
     {
-		// phpcs:enable
+        // phpcs:enable
         return $this->setDeliveryDate($user, $delivery_date);
     }
 
@@ -1549,7 +1549,7 @@ 
         return 0;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set an overall discount on the proposal
      *
@@ -1582,7 +1582,7 @@ 
     }
     */
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set an absolute overall discount on the proposal
      *
@@ -1880,7 +1880,7 @@ 
         return 1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Set draft status
      *
@@ -1889,7 +1889,7 @@ 
      */
     public function setDraft($user)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $error = 0;
@@ -1931,7 +1931,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *    Return list of askprice (eventually filtered on user) into an array
      *
@@ -1947,7 +1947,7 @@ 
      */
     public function liste_array($shortlist = 0, $draft = 0, $notcurrentuser = 0, $socid = 0, $limit = 0, $offset = 0, $sortfield = 'p.datec', $sortorder = 'DESC')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user;
 
         $ga = array();
@@ -2166,7 +2166,7 @@ 
         return $this->LibStatut((isset($this->statut) ? $this->statut : $this->status), $mode);
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return label of a status (draft, validated, ...)
      *
@@ -2176,7 +2176,7 @@ 
      */
     public function LibStatut($status, $mode = 1)
     {
-		// phpcs:enable
+        // phpcs:enable
 
         // Init/load array of translation of status
         if (empty($this->labelStatus) || empty($this->labelStatusShort)) {
@@ -2211,7 +2211,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *      Load indicators for dashboard (this->nbtodo and this->nbtodolate)
      *
@@ -2221,7 +2221,7 @@ 
      */
     public function load_board($user, $mode)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user, $langs;
 
         $now = dol_now();

Spacing +2 added lines, -2 removed lines

@@ -164,7 +164,7 @@ 
     public $total;
 
     public $cond_reglement_code;
-    public $cond_reglement_doc;     // label doc
+    public $cond_reglement_doc; // label doc
 
     public $mode_reglement_code;
     /**
@@ -1251,7 +1251,7 @@ 
                 $this->status               = (int) $obj->fk_statut;
                 $this->datec                = $this->db->jdate($obj->datec); // TODO deprecated
                 $this->datev                = $this->db->jdate($obj->datev); // TODO deprecated
-                $this->date_creation = $this->db->jdate($obj->datec);   // Creation date
+                $this->date_creation = $this->db->jdate($obj->datec); // Creation date
                 $this->date                 = $this->date_creation;
                 $this->date_validation = $this->db->jdate($obj->datev); // Validation date
                 $this->delivery_date        = $this->db->jdate($obj->delivery_date);