alxarafe / alixar

Dolibarr/Code/WebPortal/Classes/Controller.php

Indentation +5 added lines, -5 removed lines

@@ -22,11 +22,11 @@
 
 use DoliDB;
 
- /**
- * \file       htdocs/webportal/class/controller.class.php
- * \ingroup    webportal
- * \brief      File of controller class for WebPortal
- */
+    /**
+     * \file       htdocs/webportal/class/controller.class.php
+     * \ingroup    webportal
+     * \brief      File of controller class for WebPortal
+     */
 
 /**
  *  Class to manage pages

Dolibarr/Code/Website/Classes/WebsitePage.php

Indentation +2 added lines, -2 removed lines

@@ -124,8 +124,8 @@
      */
     public $newid;         // offline
         /**
-     * @var array<string,array{type:string,label:string,enabled:int<0,2>|string,position:int,notnull?:int,visible:int,noteditable?:int,default?:string,index?:int,foreignkey?:string,searchall?:int,isameasure?:int,css?:string,csslist?:string,help?:string,showoncombobox?:int,disabled?:int,arrayofkeyval?:array<int,string>,comment?:string}>  Array with all fields and their property. Do not use it as a static var. It may be modified by constructor.
-     */
+         * @var array<string,array{type:string,label:string,enabled:int<0,2>|string,position:int,notnull?:int,visible:int,noteditable?:int,default?:string,index?:int,foreignkey?:string,searchall?:int,isameasure?:int,css?:string,csslist?:string,help?:string,showoncombobox?:int,disabled?:int,arrayofkeyval?:array<int,string>,comment?:string}>  Array with all fields and their property. Do not use it as a static var. It may be modified by constructor.
+         */
     public $fields = array(
         'rowid' => array('type' => 'integer', 'label' => 'TechnicalID', 'enabled' => 1, 'visible' => -1, 'notnull' => 1, 'index' => 1, 'position' => 1, 'comment' => 'Id'),
         'pageurl' => array('type' => 'varchar(16)', 'label' => 'WEBSITE_PAGENAME', 'enabled' => 1, 'visible' => 1, 'notnull' => 1, 'index' => 1, 'position' => 10, 'searchall' => 1, 'comment' => 'Ref/alias of page'),

Spacing +9 added lines, -9 removed lines

@@ -64,7 +64,7 @@ 
     /**
      * @var int ID
      */
-    public $fk_website;        // If translation of another page
+    public $fk_website; // If translation of another page
 public $fk_page;
     public $pageurl;
     public $aliasalt;
@@ -122,7 +122,7 @@ 
     /**
      * @var int         Another ID that is the $id but with an offset so that ID of pages of the website start at 1
      */
-    public $newid;         // offline
+    public $newid; // offline
         /**
      * @var array<string,array{type:string,label:string,enabled:int<0,2>|string,position:int,notnull?:int,visible:int,noteditable?:int,default?:string,index?:int,foreignkey?:string,searchall?:int,isameasure?:int,css?:string,csslist?:string,help?:string,showoncombobox?:int,disabled?:int,arrayofkeyval?:array<int,string>,comment?:string}>  Array with all fields and their property. Do not use it as a static var. It may be modified by constructor.
      */
@@ -153,7 +153,7 @@ 
         'import_key' => array('type' => 'varchar(14)', 'label' => 'ImportId', 'enabled' => 1, 'visible' => -1, 'index' => 1, 'position' => 1000, 'notnull' => -1),
         'object_type' => array('type' => 'varchar(255)', 'label' => 'ObjectType', 'enabled' => 1, 'visible' => 0, 'position' => 46, 'searchall' => 0, 'help' => ''),
         'fk_object' => array('type' => 'varchar(255)', 'label' => 'ObjectId', 'enabled' => 1, 'visible' => 0, 'position' => 47, 'searchall' => 0, 'help' => '')
-    );     // online
+    ); // online
 
 
     /**
@@ -243,7 +243,7 @@ 
         $sql .= " t.fk_object";
         $sql .= ' FROM ' . MAIN_DB_PREFIX . $this->table_element . ' as t';
         if (!empty($websiteid)) {
-            $sql .= ' WHERE t.fk_website = ' . ((int)$websiteid);
+            $sql .= ' WHERE t.fk_website = ' . ((int) $websiteid);
         }
 
         // Deprecated. If we receive an array, we use it. Prefer using the USF syntax.
@@ -253,7 +253,7 @@ 
             if (count($filter) > 0) {
                 foreach ($filter as $key => $value) {
                     if ($key == 't.rowid' || $key == 'rowid' || $key == 't.fk_website' || $key == 'fk_website' || $key == 'status' || $key == 't.status') {
-                        $sqlwhere[] = $key . " = " . ((int)$value);
+                        $sqlwhere[] = $key . " = " . ((int) $value);
                     } elseif ($key == 'type_container' || $key == 't.type_container') {
                         $sqlwhere[] = $key . " = '" . $this->db->escape($value) . "'";
                     } elseif ($key == 'lang' || $key == 't.lang') {
@@ -364,14 +364,14 @@ 
 
         $sql = 'SELECT COUNT(t.rowid) as nb';
         $sql .= ' FROM ' . MAIN_DB_PREFIX . $this->table_element . ' as t';
-        $sql .= ' WHERE t.fk_website = ' . ((int)$websiteid);
+        $sql .= ' WHERE t.fk_website = ' . ((int) $websiteid);
 
         // Manage filter (same than into fetchAll)
         $sqlwhere = array();
         if (count($filter) > 0) {
             foreach ($filter as $key => $value) {
                 if ($key == 't.rowid' || $key == 't.fk_website' || $key == 'status') {
-                    $sqlwhere[] = $key . " = " . ((int)$value);
+                    $sqlwhere[] = $key . " = " . ((int) $value);
                 } elseif ($key == 'type_container') {
                     $sqlwhere[] = $key . " = '" . $this->db->escape($value) . "'";
                 } elseif ($key == 'lang' || $key == 't.lang') {
@@ -499,7 +499,7 @@ 
         //$sql .= ' WHERE entity IN ('.getEntity('website').')';       // entity is on website level
         $sql .= ' WHERE 1 = 1';
         if ($id > 0) {
-            $sql .= ' AND t.rowid = ' . ((int)$id);
+            $sql .= ' AND t.rowid = ' . ((int) $id);
         } else {
             if ($id < 0) {
                 $sql .= ' AND t.rowid <> ' . abs($id);
@@ -595,7 +595,7 @@ 
         if (!$error) {
             foreach ($this->childtablesoncascade as $table) {
                 $sql = "DELETE FROM " . MAIN_DB_PREFIX . $table;
-                $sql .= " WHERE fk_website_page = " . (int)$this->id;
+                $sql .= " WHERE fk_website_page = " . (int) $this->id;
 
                 $result = $this->db->query($sql);
                 if (!$result) {

Dolibarr/Code/Zapier/Classes/Hook.php

Spacing +2 added lines, -2 removed lines

@@ -403,7 +403,7 @@ 
         $sql .= ' t.rowid';
         // TODO Get all fields
         $sql .= ' FROM ' . MAIN_DB_PREFIX . $this->table_element . ' as t';
-        $sql .= ' WHERE t.entity = ' . ((int)$conf->entity);
+        $sql .= ' WHERE t.entity = ' . ((int) $conf->entity);
 
         // Manage filter
         $errormessage = '';
@@ -611,7 +611,7 @@ 
         $sql = 'SELECT rowid, date_creation as datec, tms as datem,';
         $sql .= ' fk_user_creat, fk_user_modif';
         $sql .= ' FROM ' . MAIN_DB_PREFIX . $this->table_element . ' as t';
-        $sql .= ' WHERE t.rowid = ' . ((int)$id);
+        $sql .= ' WHERE t.rowid = ' . ((int) $id);
         $result = $this->db->query($sql);
         if ($result) {
             if ($this->db->num_rows($result)) {

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/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();

public/htdocs/index.php

Spacing +11 added lines, -11 removed lines

@@ -228,7 +228,7 @@ 
     }
 
     // Number of supplier proposals open (expired)
-    if (isModEnabled('supplier_proposal')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_SUPPLIER') && $user->hasRight('supplier_proposal', 'lire')) {
+    if (isModEnabled('supplier_proposal') && !getDolGlobalString('MAIN_DISABLE_BLOCK_SUPPLIER') && $user->hasRight('supplier_proposal', 'lire')) {
         $langs->load("supplier_proposal");
         $board = new SupplierProposal($db);
         $dashboardlines[$board->element . '_opened'] = $board->load_board($user, "opened");
@@ -237,7 +237,7 @@ 
     }
 
     // Number of sales orders
-    if (isModEnabled('order')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_CUSTOMER') && $user->hasRight('commande', 'lire')) {
+    if (isModEnabled('order') && !getDolGlobalString('MAIN_DISABLE_BLOCK_CUSTOMER') && $user->hasRight('commande', 'lire')) {
         $board = new Commande($db);
         // Number of customer orders to be shipped (validated and in progress)
         $dashboardlines[$board->element . '_toship'] = $board->load_board($user, 'toship');
@@ -250,14 +250,14 @@ 
     }
 
     // Number of suppliers orders
-    if (isModEnabled('supplier_order')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_SUPPLIER') && $user->hasRight('fournisseur', 'commande', 'lire')) {
+    if (isModEnabled('supplier_order') && !getDolGlobalString('MAIN_DISABLE_BLOCK_SUPPLIER') && $user->hasRight('fournisseur', 'commande', 'lire')) {
         $board = new CommandeFournisseur($db);
         $dashboardlines[$board->element . '_opened'] = $board->load_board($user, "opened");
         $dashboardlines[$board->element . '_awaiting'] = $board->load_board($user, 'awaiting');
     }
 
     // Number of contract / services enabled (delayed)
-    if (isModEnabled('contract')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_CONTRACT') && $user->hasRight('contrat', 'lire')) {
+    if (isModEnabled('contract') && !getDolGlobalString('MAIN_DISABLE_BLOCK_CONTRACT') && $user->hasRight('contrat', 'lire')) {
         $board = new Contrat($db);
         $dashboardlines[$board->element . '_inactive'] = $board->load_board($user, "inactive");
         // Number of active services (expired)
@@ -265,7 +265,7 @@ 
     }
 
     // Number of tickets open
-    if (isModEnabled('ticket')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_TICKET') && $user->hasRight('ticket', 'read')) {
+    if (isModEnabled('ticket') && !getDolGlobalString('MAIN_DISABLE_BLOCK_TICKET') && $user->hasRight('ticket', 'read')) {
         $board = new Ticket($db);
         $dashboardlines[$board->element . '_opened'] = $board->load_board($user, "opened");
         // Number of active services (expired)
@@ -285,7 +285,7 @@ 
     }
 
     // Number of transactions to conciliate
-    if (isModEnabled('bank')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_BANK') && $user->hasRight('banque', 'lire') && !$user->socid) {
+    if (isModEnabled('bank') && !getDolGlobalString('MAIN_DISABLE_BLOCK_BANK') && $user->hasRight('banque', 'lire') && !$user->socid) {
         $board = new Account($db);
         $nb = $board->countAccountToReconcile(); // Get nb of account to reconciliate
         if ($nb > 0) {
@@ -295,7 +295,7 @@ 
 
 
     // Number of cheque to send
-    if (isModEnabled('bank')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_BANK') && $user->hasRight('banque', 'lire') && !$user->socid) {
+    if (isModEnabled('bank') && !getDolGlobalString('MAIN_DISABLE_BLOCK_BANK') && $user->hasRight('banque', 'lire') && !$user->socid) {
         if (!getDolGlobalString('BANK_DISABLE_CHECK_DEPOSIT')) {
             $board = new RemiseCheque($db);
             $dashboardlines[$board->element] = $board->load_board($user);
@@ -311,26 +311,26 @@ 
     }
 
     // Number of foundation members
-    if (isModEnabled('member')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_ADHERENT') && $user->hasRight('adherent', 'lire') && !$user->socid) {
+    if (isModEnabled('member') && !getDolGlobalString('MAIN_DISABLE_BLOCK_ADHERENT') && $user->hasRight('adherent', 'lire') && !$user->socid) {
         $board = new Adherent($db);
         $dashboardlines[$board->element . '_shift'] = $board->load_board($user, 'shift');
         $dashboardlines[$board->element . '_expired'] = $board->load_board($user, 'expired');
     }
 
     // Number of expense reports to approve
-    if (isModEnabled('expensereport')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_EXPENSEREPORT') && $user->hasRight('expensereport', 'approve')) {
+    if (isModEnabled('expensereport') && !getDolGlobalString('MAIN_DISABLE_BLOCK_EXPENSEREPORT') && $user->hasRight('expensereport', 'approve')) {
         $board = new ExpenseReport($db);
         $dashboardlines[$board->element . '_toapprove'] = $board->load_board($user, 'toapprove');
     }
 
     // Number of expense reports to pay
-    if (isModEnabled('expensereport')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_EXPENSEREPORT') && $user->hasRight('expensereport', 'to_paid')) {
+    if (isModEnabled('expensereport') && !getDolGlobalString('MAIN_DISABLE_BLOCK_EXPENSEREPORT') && $user->hasRight('expensereport', 'to_paid')) {
         $board = new ExpenseReport($db);
         $dashboardlines[$board->element . '_topay'] = $board->load_board($user, 'topay');
     }
 
     // Number of holidays to approve
-    if (isModEnabled('holiday')  && !getDolGlobalString('MAIN_DISABLE_BLOCK_HOLIDAY') && $user->hasRight('holiday', 'approve')) {
+    if (isModEnabled('holiday') && !getDolGlobalString('MAIN_DISABLE_BLOCK_HOLIDAY') && $user->hasRight('holiday', 'approve')) {
         $board = new Holiday($db);
         $dashboardlines[$board->element] = $board->load_board($user);
     }

public/htdocs/adherents/list.php

Spacing +13 added lines, -13 removed lines

@@ -74,7 +74,7 @@ 
 $search_address = GETPOST("search_address", 'alpha');
 $search_zip = GETPOST("search_zip", 'alpha');
 $search_town = GETPOST("search_town", 'alpha');
-$search_state = GETPOST("search_state", 'alpha');  // county / departement / federal state
+$search_state = GETPOST("search_state", 'alpha'); // county / departement / federal state
 $search_country = GETPOST("search_country", 'alpha');
 $search_phone = GETPOST("search_phone", 'alpha');
 $search_phone_perso = GETPOST("search_phone_perso", 'alpha');
@@ -91,7 +91,7 @@ 
 }
 
 $search_filter = GETPOST("search_filter", 'alpha');
-$search_status = GETPOST("search_status", 'intcomma');  // status
+$search_status = GETPOST("search_status", 'intcomma'); // status
 $search_datec_start = dol_mktime(0, 0, 0, GETPOSTINT('search_datec_start_month'), GETPOSTINT('search_datec_start_day'), GETPOSTINT('search_datec_start_year'));
 $search_datec_end = dol_mktime(23, 59, 59, GETPOSTINT('search_datec_end_month'), GETPOSTINT('search_datec_end_day'), GETPOSTINT('search_datec_end_year'));
 $search_datem_start = dol_mktime(0, 0, 0, GETPOSTINT('search_datem_start_month'), GETPOSTINT('search_datem_start_day'), GETPOSTINT('search_datem_start_year'));
@@ -205,18 +205,18 @@ 
     // If $val['visible']==0, then we never show the field
 
     if (!empty($val['visible'])) {
-        $visible = (int)dol_eval($val['visible'], 1);
+        $visible = (int) dol_eval($val['visible'], 1);
         $arrayfields[$tableprefix . '.' . $key] = array(
             'label' => $val['label'],
             'checked' => (($visible < 0) ? 0 : 1),
-            'enabled' => (abs($visible) != 3 && (int)dol_eval($val['enabled'], 1)),
+            'enabled' => (abs($visible) != 3 && (int) dol_eval($val['enabled'], 1)),
             'position' => $val['position'],
             'help' => isset($val['help']) ? $val['help'] : ''
         );
     }
 }
 $arrayfields = dol_sort_array($arrayfields, 'position');
-'@phan-var-force array<string,array{label:string,checked?:int<0,1>,position?:int,help?:string}> $arrayfields';  // dol_sort_array looses type for Phan
+'@phan-var-force array<string,array{label:string,checked?:int<0,1>,position?:int,help?:string}> $arrayfields'; // dol_sort_array looses type for Phan
 //var_dump($arrayfields);exit;
 
 // Security check
@@ -444,7 +444,7 @@ 
 $sqlfields = $sql; // $sql fields to remove for count total
 
 // SQL Alias adherent
-$sql .= " FROM " . MAIN_DB_PREFIX . "adherent as d";  // maybe better to use ad (adh) instead of d
+$sql .= " FROM " . MAIN_DB_PREFIX . "adherent as d"; // maybe better to use ad (adh) instead of d
 if (!empty($extrafields->attributes[$object->table_element]['label']) && count($extrafields->attributes[$object->table_element]['label'])) {
     $sql .= " LEFT JOIN " . MAIN_DB_PREFIX . $object->table_element . "_extrafields as ef on (d.rowid = ef.fk_object)";
 }
@@ -467,9 +467,9 @@ 
             $searchCategoryContactSqlList[] = "NOT EXISTS (SELECT ck.fk_categorie FROM " . MAIN_DB_PREFIX . "categorie_member as ck WHERE d.rowid = ck.fk_member)";
         } elseif (intval($searchCategoryContact) > 0) {
             if ($searchCategoryContactOperator == 0) {
-                $searchCategoryContactSqlList[] = " EXISTS (SELECT ck.fk_categorie FROM " . MAIN_DB_PREFIX . "categorie_member as ck WHERE d.rowid = ck.fk_member AND ck.fk_categorie = " . ((int)$searchCategoryContact) . ")";
+                $searchCategoryContactSqlList[] = " EXISTS (SELECT ck.fk_categorie FROM " . MAIN_DB_PREFIX . "categorie_member as ck WHERE d.rowid = ck.fk_member AND ck.fk_categorie = " . ((int) $searchCategoryContact) . ")";
             } else {
-                $listofcategoryid .= ($listofcategoryid ? ', ' : '') . ((int)$searchCategoryContact);
+                $listofcategoryid .= ($listofcategoryid ? ', ' : '') . ((int) $searchCategoryContact);
             }
         }
     }
@@ -492,7 +492,7 @@ 
     $sql .= natural_search(array_keys($fieldstosearchall), $search_all);
 }
 if ($search_type > 0) {
-    $sql .= " AND t.rowid=" . ((int)$search_type);
+    $sql .= " AND t.rowid=" . ((int) $search_type);
 }
 if ($search_filter == 'withoutsubscription') {
     $sql .= " AND (datefin IS NULL)";
@@ -636,7 +636,7 @@ 
 // Output page
 // --------------------------------------------------------------------
 
-llxHeader('', $title, $help_url, '', 0, 0, $morejs, $morecss, '', 'bodyforlist');   // Can use also classforhorizontalscrolloftabs instead of bodyforlist for no horizontal scroll
+llxHeader('', $title, $help_url, '', 0, 0, $morejs, $morecss, '', 'bodyforlist'); // Can use also classforhorizontalscrolloftabs instead of bodyforlist for no horizontal scroll
 
 $arrayofselected = is_array($toselect) ? $toselect : array();
 
@@ -656,7 +656,7 @@ 
     $param .= '&contextpage=' . urlencode($contextpage);
 }
 if ($limit > 0 && $limit != $conf->liste_limit) {
-    $param .= '&limit=' . ((int)$limit);
+    $param .= '&limit=' . ((int) $limit);
 }
 if ($optioncss != '') {
     $param .= '&optioncss=' . urlencode($optioncss);
@@ -686,7 +686,7 @@ 
     $param .= "&search_email=" . urlencode($search_email);
 }
 if ($search_categ > 0 || $search_categ == -2) {
-    $param .= "&search_categ=" . urlencode((string)($search_categ));
+    $param .= "&search_categ=" . urlencode((string) ($search_categ));
 }
 if ($search_company) {
     $param .= "&search_company=" . urlencode($search_company);
@@ -832,7 +832,7 @@ 
 }
 
 $varpage = empty($contextpage) ? $_SERVER["PHP_SELF"] : $contextpage;
-$htmlofselectarray = $form->multiSelectArrayWithCheckbox('selectedfields', $arrayfields, $varpage, getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN'));  // This also change content of $arrayfields with user setup
+$htmlofselectarray = $form->multiSelectArrayWithCheckbox('selectedfields', $arrayfields, $varpage, getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN')); // This also change content of $arrayfields with user setup
 $selectedfields = ($mode != 'kanban' ? $htmlofselectarray : '');
 $selectedfields .= (count($arrayofmassactions) ? $form->showCheckAddButtons('checkforselect', 1) : '');
 

public/htdocs/modulebuilder/template/myobject_list.php

Indentation +2 added lines, -2 removed lines

@@ -625,7 +625,7 @@ 
             print $form->selectDate($search[$key . '_dtend'] ? $search[$key . '_dtend'] : '', "search_" . $key . "_dtend", 0, 0, 1, '', 1, 0, 0, '', '', '', '', 1, '', $langs->trans('to'));
             print '</div>';
         } elseif ($key == 'lang') {
-			$formadmin = new FormAdmin($db);
+            $formadmin = new FormAdmin($db);
             print $formadmin->select_language((isset($search[$key]) ? $search[$key] : ''), 'search_lang', 0, null, 1, 0, 0, 'minwidth100imp maxwidth125', 2);
         } else {
             print '<input type="text" class="flat maxwidth75" name="search_' . $key . '" value="' . dol_escape_htmltag(isset($search[$key]) ? $search[$key] : '') . '">';
@@ -893,7 +893,7 @@ 
         $hidegeneratedfilelistifempty = 0;
     }
 
-	$formfile = new FormFile($db);
+    $formfile = new FormFile($db);
 
     // Show list of available documents
     $urlsource = $_SERVER['PHP_SELF'] . '?sortfield=' . $sortfield . '&sortorder=' . $sortorder;

public/htdocs/modulebuilder/template/core/modules/mymodule/doc/pdf_standard_myobject.modules.php

Indentation +9 added lines, -9 removed lines

@@ -150,7 +150,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Function to build pdf onto disk
      *
@@ -164,7 +164,7 @@ 
      */
     public function write_file($object, $outputlangs, $srctemplatepath = '', $hidedetails = 0, $hidedesc = 0, $hideref = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $user, $langs, $conf, $mysoc, $db, $hookmanager, $nblines;
 
         dol_syslog("write_file outputlangs->defaultlang=" . (is_object($outputlangs) ? $outputlangs->defaultlang : 'null'));
@@ -280,7 +280,7 @@ 
             if (file_exists($dir)) {
                 // Add pdfgeneration hook
                 if (!is_object($hookmanager)) {
-					$hookmanager = new HookManager($this->db);
+                    $hookmanager = new HookManager($this->db);
                 }
                 $hookmanager->initHooks(array('pdfgeneration'));
                 $parameters = array('file' => $file, 'object' => $object, 'outputlangs' => $outputlangs);
@@ -824,7 +824,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Return list of active generation modules
      *
@@ -834,11 +834,11 @@ 
      */
     public static function liste_modeles($db, $maxfilenamelength = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         return parent::liste_modeles($db, $maxfilenamelength); // TODO: Change the autogenerated stub
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *   Show table for lines
      *
@@ -899,7 +899,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *  Show top header of page.
      *
@@ -912,7 +912,7 @@ 
      */
     protected function _pagehead(&$pdf, $object, $showaddress, $outputlangs, $outputlangsbis = null)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $ltrdirection = 'L';
@@ -1163,7 +1163,7 @@ 
         return $top_shift;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *      Show footer of page. Need this->emetteur object
      *