alxarafe / alixar

public/htdocs/core/ajax/selectsearchbox.php

Spacing +1 added lines, -1 removed lines

@@ -118,7 +118,7 @@
 if (isModEnabled('supplier_proposal') && !getDolGlobalString('MAIN_SEARCHFORM_SUPPLIER_PROPAL_DISABLED') && $user->hasRight('supplier_proposal', 'lire')) {
     $arrayresult['searchintosupplierpropal'] = array('position' => 100, 'img' => 'object_supplier_proposal', 'label' => $langs->trans("SearchIntoSupplierProposals", $search_boxvalue), 'text' => img_picto('', 'object_supplier_proposal', 'class="pictofixedwidth"') . ' ' . $langs->trans("SearchIntoSupplierProposals", $search_boxvalue), 'url' => constant('BASE_URL') . '/supplier_proposal/list.php' . ($search_boxvalue ? '?search_all=' . urlencode($search_boxvalue) : ''));
 }
-if (((isModEnabled('fournisseur') && !getDolGlobalString('MAIN_USE_NEW_SUPPLIERMOD') && $user->hasRight('fournisseur', 'commande', 'lire')) || (isModEnabled('supplier_order') &&  $user->hasRight('supplier_order', 'lire'))) && !getDolGlobalString('MAIN_SEARCHFORM_SUPPLIER_ORDER_DISABLED')) {
+if (((isModEnabled('fournisseur') && !getDolGlobalString('MAIN_USE_NEW_SUPPLIERMOD') && $user->hasRight('fournisseur', 'commande', 'lire')) || (isModEnabled('supplier_order') && $user->hasRight('supplier_order', 'lire'))) && !getDolGlobalString('MAIN_SEARCHFORM_SUPPLIER_ORDER_DISABLED')) {
     $arrayresult['searchintosupplierorder'] = array('position' => 110, 'img' => 'object_supplier_order', 'label' => $langs->trans("SearchIntoSupplierOrders", $search_boxvalue), 'text' => img_picto('', 'object_supplier_order', 'class="pictofixedwidth"') . ' ' . $langs->trans("SearchIntoSupplierOrders", $search_boxvalue), 'url' => constant('BASE_URL') . '/fourn/commande/list.php' . ($search_boxvalue ? '?search_all=' . urlencode($search_boxvalue) : ''));
 }
 if (isModEnabled('reception') && !getDolGlobalString('MAIN_SEARCHFORM_SUPPLIER_RECEPTION_DISABLED') && $user->hasRight('reception', 'lire')) {

public/htdocs/core/ajax/locationincoterms.php

Spacing +1 added lines, -1 removed lines

@@ -46,7 +46,7 @@
 
 // Security check
 if (!isModEnabled('incoterm')) {
-    httponly_accessforbidden("Module incoterm not enabled");    // This includes the exit.
+    httponly_accessforbidden("Module incoterm not enabled"); // This includes the exit.
 }
 // There is no other permission on this component. Everybody connected can read content of the incoterm table
 

public/htdocs/core/ajax/row.php

Spacing +2 added lines, -2 removed lines

@@ -139,7 +139,7 @@ 
     if ($reshook > 0) {
         $perm = $hookmanager->resArray['perm'];
     }
-    if (! $perm) {
+    if (!$perm) {
         // We should not be here. If we are not allowed to reorder rows, feature should not be visible on script.
         // If we are here, it is a hack attempt, so we report a warning.
         print 'Bad permission to modify position of lines for object in table ' . $table_element_line;
@@ -161,7 +161,7 @@ 
 
     // Reorder line to have position of children lines sharing same counter than parent lines
     // This should be useless because there is no need to have children sharing same counter than parent, but well, it's cleaner into database.
-    if (in_array($fk_element, array('fk_facture', 'fk_propal', 'fk_commande','fk_contrat'))) {
+    if (in_array($fk_element, array('fk_facture', 'fk_propal', 'fk_commande', 'fk_contrat'))) {
         $result = $row->line_order(true);
     }
 } else {

public/htdocs/core/ajax/check_notifications.php

Spacing +1 added lines, -1 removed lines

@@ -131,7 +131,7 @@
     $sql .= ' INNER JOIN ' . MAIN_DB_PREFIX . 'actioncomm_reminder as ar ON a.id = ar.fk_actioncomm AND ar.fk_user = ' . ((int) $user->id);
     $sql .= " AND ar.typeremind = 'browser' AND ar.dateremind < '" . $db->idate(dol_now()) . "'";
     $sql .= " AND ar.status = 0";
-    $sql .= " AND ar.entity = " . ((int) $conf->entity);  // No sharing of entity for alerts
+    $sql .= " AND ar.entity = " . ((int) $conf->entity); // No sharing of entity for alerts
     $sql .= $db->order('datep', 'ASC');
     $sql .= $db->plimit(10); // Avoid too many notification at once
 

public/htdocs/core/ajax/objectonoff.php

Spacing +3 added lines, -3 removed lines

@@ -54,7 +54,7 @@ 
 // Load object according to $id and $element
 $object = fetchObjectByElement($id, $element);
 if (!is_object($object)) {
-    httponly_accessforbidden("Bad value for combination of parameters element/field: Object not found.");   // This includes the exit.
+    httponly_accessforbidden("Bad value for combination of parameters element/field: Object not found."); // This includes the exit.
 }
 
 $object->fields[$field] = array('type' => $format, 'enabled' => 1);
@@ -72,7 +72,7 @@ 
 if (!empty($user->socid)) {
     $socid = $user->socid;
     if (!empty($object->socid) && $socid != $object->socid) {
-        httponly_accessforbidden("Access on object not allowed for this external user.");   // This includes the exit.
+        httponly_accessforbidden("Access on object not allowed for this external user."); // This includes the exit.
     }
 }
 
@@ -83,7 +83,7 @@ 
 } elseif ($element == 'product' && in_array($field, array('tosell', 'tobuy', 'tobatch'))) { // Special case for products
     restrictedArea($user, 'produit|service', $object, 'product&product', '', '', 'rowid');
 } else {
-    httponly_accessforbidden("Bad value for combination of parameters element/field: Field not supported.");    // This includes the exit.
+    httponly_accessforbidden("Bad value for combination of parameters element/field: Field not supported."); // This includes the exit.
 }
 
 

public/htdocs/core/bookmarks_page.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
 $title = $langs->trans("Bookmarks");
 
 // URL http://mydolibarr/core/bookmarks_page?dol_use_jmobile=1 can be used for tests
-$head = '<!-- Bookmarks -->' . "\n";  // This is used by DoliDroid to know page is a bookmark selection page
+$head = '<!-- Bookmarks -->' . "\n"; // This is used by DoliDroid to know page is a bookmark selection page
 $arrayofjs = array();
 $arrayofcss = array();
 top_htmlhead($head, $title, 0, 0, $arrayofjs, $arrayofcss);

public/htdocs/core/actions_sendmails.inc.php

Spacing +1 added lines, -1 removed lines

@@ -398,7 +398,7 @@
                         $object->actionmsg = $message; // Long text
                         $object->actionmsg2 = $actionmsg2; // Short text ($langs->transnoentities('MailSentByTo')...);
                         if (getDolGlobalString('MAIN_MAIL_REPLACE_EVENT_TITLE_BY_EMAIL_SUBJECT')) {
-                            $object->actionmsg2     = $subject; // Short text
+                            $object->actionmsg2 = $subject; // Short text
                         }
 
                         $object->trackid = $trackid;

public/htdocs/core/modules/rapport/pdf_paiement.class.php

Indentation +6 added lines, -6 removed lines

@@ -114,7 +114,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Fonction generant la rapport sur le disque
      *
@@ -126,7 +126,7 @@ 
      */
     public function write_file($_dir, $month, $year, $outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         include_once DOL_DOCUMENT_ROOT . '/core/lib/date.lib.php';
 
         global $conf, $hookmanager, $langs, $user;
@@ -382,7 +382,7 @@ 
         return 1;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      *  Show top header of page.
      *
@@ -394,7 +394,7 @@ 
      */
     protected function _pagehead(&$pdf, $page, $showaddress, $outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
 
         // Do not add the BACKGROUND as this is a report
         //pdf_pagehead($pdf,$outputlangs,$this->page_hauteur);
@@ -457,7 +457,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      *  Output body
      *
@@ -469,7 +469,7 @@ 
      */
     public function Body(&$pdf, $page, $lines, $outputlangs)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf;
         $default_font_size = pdf_getPDFFontSize($outputlangs);
 

public/htdocs/core/modules/DolibarrModules.class.php

Indentation +44 added lines, -44 removed lines

@@ -426,7 +426,7 @@ 
     // We need constructor into function unActivateModule into admin.lib.php
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Enables a module.
      * Inserts all information into database.
@@ -439,7 +439,7 @@ 
      */
     protected function _init($array_sql, $options = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
         $err = 0;
 
@@ -538,7 +538,7 @@ 
         }
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Disable function. Deletes the module constants and boxes from the database.
      *
@@ -550,7 +550,7 @@ 
     protected function _remove($array_sql, $options = '')
     {
         global $conf;
-		// phpcs:enable
+        // phpcs:enable
         $err = 0;
 
         $this->db->begin();
@@ -1061,7 +1061,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Insert constants for module activation
      *
@@ -1069,7 +1069,7 @@ 
      */
     protected function _active()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
 
         $err = 0;
@@ -1111,7 +1111,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Module deactivation
      *
@@ -1119,7 +1119,7 @@ 
      */
     protected function _unactive()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $err = 0;
@@ -1138,7 +1138,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps,PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps,PEAR.NamingConventions.ValidFunctionName.PublicUnderscore
     /**
      * Create tables and keys required by module:
      * - Files table.sql or table-module.sql with create table instructions
@@ -1156,7 +1156,7 @@ 
      */
     protected function _load_tables($reldir, $onlywithsuffix = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $error = 0;
@@ -1327,7 +1327,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Adds boxes
      *
@@ -1337,7 +1337,7 @@ 
      */
     public function insert_boxes($option = '')
     {
-		// phpcs:enable
+        // phpcs:enable
         include_once DOL_DOCUMENT_ROOT . '/core/class/infobox.class.php';
 
         global $conf;
@@ -1427,7 +1427,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Removes boxes
      *
@@ -1435,7 +1435,7 @@ 
      */
     public function delete_boxes()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $err = 0;
@@ -1501,7 +1501,7 @@ 
         return $err;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Adds cronjobs
      *
@@ -1509,7 +1509,7 @@ 
      */
     public function insert_cronjobs()
     {
-		// phpcs:enable
+        // phpcs:enable
         include_once DOL_DOCUMENT_ROOT . '/core/class/infobox.class.php';
 
         global $conf;
@@ -1635,7 +1635,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Removes boxes
      *
@@ -1643,7 +1643,7 @@ 
      */
     public function delete_cronjobs()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $err = 0;
@@ -1666,7 +1666,7 @@ 
         return $err;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Removes tabs
      *
@@ -1674,7 +1674,7 @@ 
      */
     public function delete_tabs()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $err = 0;
@@ -1692,7 +1692,7 @@ 
         return $err;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Adds tabs
      *
@@ -1700,7 +1700,7 @@ 
      */
     public function insert_tabs()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $err = 0;
@@ -1759,7 +1759,7 @@ 
         return $err;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Adds constants
      *
@@ -1767,7 +1767,7 @@ 
      */
     public function insert_const()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $err = 0;
@@ -1828,7 +1828,7 @@ 
         return $err;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Removes constants tagged 'deleteonunactive'
      *
@@ -1836,7 +1836,7 @@ 
      */
     public function delete_const()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $err = 0;
@@ -1864,7 +1864,7 @@ 
         return $err;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Adds access rights
      *
@@ -1875,7 +1875,7 @@ 
      */
     public function insert_permissions($reinitadminperms = 0, $force_entity = null, $notrigger = 0)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
 
         $err = 0;
@@ -2034,7 +2034,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Removes access rights
      *
@@ -2042,7 +2042,7 @@ 
      */
     public function delete_permissions()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $err = 0;
@@ -2067,7 +2067,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Adds menu entries
      *
@@ -2075,7 +2075,7 @@ 
      */
     public function insert_menus()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $user;
 
         if (!is_array($this->menu) || empty($this->menu)) {
@@ -2168,7 +2168,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Removes menu entries
      *
@@ -2176,7 +2176,7 @@ 
      */
     public function delete_menus()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $err = 0;
@@ -2199,7 +2199,7 @@ 
         return $err;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Creates directories
      *
@@ -2207,7 +2207,7 @@ 
      */
     public function create_dirs()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $langs, $conf;
 
         $err = 0;
@@ -2265,7 +2265,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Adds directories definitions
      *
@@ -2276,7 +2276,7 @@ 
      */
     public function insert_dirs($name, $dir)
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $err = 0;
@@ -2307,7 +2307,7 @@ 
     }
 
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Removes directories
      *
@@ -2315,7 +2315,7 @@ 
      */
     public function delete_dirs()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $err = 0;
@@ -2333,7 +2333,7 @@ 
         return $err;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Save configuration for generic features.
      * This also generate website templates if the module provide some.
@@ -2342,7 +2342,7 @@ 
      */
     public function insert_module_parts()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf, $langs;
 
         $error = 0;
@@ -2447,7 +2447,7 @@ 
         return $error;
     }
 
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
     /**
      * Removes generic parts
      *
@@ -2455,7 +2455,7 @@ 
      */
     public function delete_module_parts()
     {
-		// phpcs:enable
+        // phpcs:enable
         global $conf;
 
         $err = 0;

Spacing +28 added lines, -28 removed lines

@@ -626,7 +626,7 @@ 
                 $moduleNameInConf = 'supplier_proposal';
             }
 
-            unset($conf->modules[$moduleNameInConf]);   // Add this module in list of enabled modules so isModEnabled() will work (conf->module->enabled must no more be used)
+            unset($conf->modules[$moduleNameInConf]); // Add this module in list of enabled modules so isModEnabled() will work (conf->module->enabled must no more be used)
 
             return 1;
         } else {
@@ -1487,7 +1487,7 @@ 
 
                 $sql = "DELETE FROM " . MAIN_DB_PREFIX . "boxes_def";
                 $sql .= " WHERE file = '" . $this->db->escape($file) . "'";
-                $sql .= " AND entity = " . $conf->entity;     // Do not use getEntity here, we want to delete only in current company
+                $sql .= " AND entity = " . $conf->entity; // Do not use getEntity here, we want to delete only in current company
 
                 dol_syslog(get_class($this) . "::delete_boxes", LOG_DEBUG);
                 $resql = $this->db->query($sql);
@@ -1910,9 +1910,9 @@ 
 
                 // If the module is active
                 foreach ($this->rights as $key => $value) {
-                    $r_id = $this->rights[$key][self::KEY_ID];  // permission id in llx_rights_def (not unique because primary key is couple id-entity)
+                    $r_id = $this->rights[$key][self::KEY_ID]; // permission id in llx_rights_def (not unique because primary key is couple id-entity)
                     $r_label = $this->rights[$key][self::KEY_LABEL];
-                    $r_type = $this->rights[$key][self::KEY_TYPE] ?? 'w';   // TODO deprecated
+                    $r_type = $this->rights[$key][self::KEY_TYPE] ?? 'w'; // TODO deprecated
                     $r_default = $this->rights[$key][self::KEY_DEFAULT] ?? 0;
                     $r_perms = $this->rights[$key][self::KEY_FIRST_LEVEL] ?? '';
                     $r_subperms = $this->rights[$key][self::KEY_SECOND_LEVEL] ?? '';
@@ -1936,7 +1936,7 @@ 
                     }
 
                     // condition to show or hide a user right (default: 1) (eg isModEnabled('anothermodule') or ($conf->global->MAIN_FEATURES_LEVEL > 0) or etc..)
-                    $r_enabled  = $this->rights[$key][self::KEY_ENABLED] ?? '1';
+                    $r_enabled = $this->rights[$key][self::KEY_ENABLED] ?? '1';
 
                     // Search if perm already present
                     $sql = "SELECT count(*) as nb FROM " . MAIN_DB_PREFIX . "rights_def";
@@ -1953,7 +1953,7 @@ 
                             $sql .= ", libelle";
                             $sql .= ", module";
                             $sql .= ", module_origin";
-                            $sql .= ", type";   // TODO deprecated
+                            $sql .= ", type"; // TODO deprecated
                             $sql .= ", bydefault";
                             $sql .= ", perms";
                             $sql .= ", subperms";
@@ -1964,7 +1964,7 @@ 
                             $sql .= ", '" . $this->db->escape($r_label) . "'";
                             $sql .= ", '" . $this->db->escape($r_module) . "'";
                             $sql .= ", '" . $this->db->escape($r_module_origin) . "'";
-                            $sql .= ", '" . $this->db->escape($r_type) . "'";   // TODO deprecated
+                            $sql .= ", '" . $this->db->escape($r_type) . "'"; // TODO deprecated
                             $sql .= ", " . ((int) $r_default);
                             $sql .= ", '" . $this->db->escape($r_perms) . "'";
                             $sql .= ", '" . $this->db->escape($r_subperms) . "'";
@@ -2186,7 +2186,7 @@ 
 
         $sql = "DELETE FROM " . MAIN_DB_PREFIX . "menu";
         $sql .= " WHERE module = '" . $this->db->escape($module) . "'";
-        $sql .= " AND menu_handler = 'all'";    // We delete only lines that were added manually or by the module activation. We keep entry added by menuhandler like 'auguria'
+        $sql .= " AND menu_handler = 'all'"; // We delete only lines that were added manually or by the module activation. We keep entry added by menuhandler like 'auguria'
         $sql .= " AND entity IN (0, " . $conf->entity . ")";
 
         dol_syslog(get_class($this) . "::delete_menus", LOG_DEBUG);
@@ -2570,36 +2570,36 @@ 
         }
 
         if ($this->isCoreOrExternalModule() == 'external' || preg_match('/development|experimental|deprecated/i', $version)) {
-            $versionTitle =  $langs->trans("Version") . ' ' . $this->getVersion(1);
+            $versionTitle = $langs->trans("Version") . ' ' . $this->getVersion(1);
             if ($this->needUpdate) {
                 $versionTitle .= '<br>' . $langs->trans('ModuleUpdateAvailable') . ' : ' . $this->lastVersion;
             }
 
-            $return .=  '<span class="info-box-icon-version' . ($versiontrans ? ' ' . $versiontrans : '') . ' classfortooltip" title="' . dol_escape_js($versionTitle) . '" >';
-            $return .=  $this->getVersion(1);
-            $return .=  '</span>';
+            $return .= '<span class="info-box-icon-version' . ($versiontrans ? ' ' . $versiontrans : '') . ' classfortooltip" title="' . dol_escape_js($versionTitle) . '" >';
+            $return .= $this->getVersion(1);
+            $return .= '</span>';
         }
 
-        $return .=  '</div>
+        $return .= '</div>
 	    <div class="info-box-content info-box-text-module' . (!getDolGlobalString($const_name) ? '' : ' info-box-module-enabled' . ($versiontrans ? ' info-box-content-warning' : '')) . '">
 	    <span class="info-box-title">' . $this->getName() . '</span>
 	    <span class="info-box-desc twolinesmax opacitymedium" title="' . dol_escape_htmltag($this->getDesc()) . '">' . nl2br($this->getDesc()) . '</span>';
 
-        $return .=  '<div class="valignmiddle inline-block info-box-more">';
+        $return .= '<div class="valignmiddle inline-block info-box-more">';
         //if ($versiontrans) print img_warning($langs->trans("Version").' '.$this->getVersion(1)).' ';
-        $return .=  '<a class="valignmiddle inline-block" href="javascript:document_preview(\'' . constant('BASE_URL') . '/admin/modulehelp.php?id=' . ((int) $this->numero) . '\',\'text/html\',\'' . dol_escape_js($langs->trans("Module")) . '\')">' . img_picto(($this->isCoreOrExternalModule() == 'external' ? $langs->trans("ExternalModule") . ' - ' : '') . $langs->trans("ClickToShowDescription"), $imginfo) . '</a>';
-        $return .=  '</div><br>';
-
-        $return .=  '<div class="valignmiddle inline-block info-box-actions">';
-        $return .=  '<div class="valignmiddle inline-block info-box-setup">';
-        $return .=  $codetoconfig;
-        $return .=  '</div>';
-        $return .=  '<div class="valignmiddle inline-block marginleftonly marginrightonly">';
-        $return .=  $codeenabledisable;
-        $return .=  '</div>';
-        $return .=  '</div>';
-
-        $return .=  '
+        $return .= '<a class="valignmiddle inline-block" href="javascript:document_preview(\'' . constant('BASE_URL') . '/admin/modulehelp.php?id=' . ((int) $this->numero) . '\',\'text/html\',\'' . dol_escape_js($langs->trans("Module")) . '\')">' . img_picto(($this->isCoreOrExternalModule() == 'external' ? $langs->trans("ExternalModule") . ' - ' : '') . $langs->trans("ClickToShowDescription"), $imginfo) . '</a>';
+        $return .= '</div><br>';
+
+        $return .= '<div class="valignmiddle inline-block info-box-actions">';
+        $return .= '<div class="valignmiddle inline-block info-box-setup">';
+        $return .= $codetoconfig;
+        $return .= '</div>';
+        $return .= '<div class="valignmiddle inline-block marginleftonly marginrightonly">';
+        $return .= $codeenabledisable;
+        $return .= '</div>';
+        $return .= '</div>';
+
+        $return .= '
 	    </div><!-- /.info-box-content -->
 	    </div><!-- /.info-box -->
 	    </div>';
@@ -2618,7 +2618,7 @@ 
     {
         require_once constant('DOL_DOCUMENT_ROOT') . '/core/lib/geturl.lib.php';
         if (!empty($this->url_last_version)) {
-            $lastVersion = getURLContent($this->url_last_version, 'GET', '', 1, array(), array('http', 'https'), 0);    // Accept http or https links on external remote server only
+            $lastVersion = getURLContent($this->url_last_version, 'GET', '', 1, array(), array('http', 'https'), 0); // Accept http or https links on external remote server only
             if (isset($lastVersion['content']) && strlen($lastVersion['content']) < 30) {
                 // Security warning :  be careful with remote data content, the module editor could be hacked (or evil) so limit to a-z A-Z 0-9 _ . -
                 $this->lastVersion = preg_replace("/[^a-zA-Z0-9_\.\-]+/", "", $lastVersion['content']);

Braces +3 added lines, -1 removed lines

@@ -2663,6 +2663,8 @@
                 $this->dictionaries['tab' . $field][] = $dictionaryArray[$field];
             }
         }
-        if ($langs && !in_array($langs, $this->dictionaries[$langs])) $this->dictionaries['langs'][] = $langs;
+        if ($langs && !in_array($langs, $this->dictionaries[$langs])) {
+            $this->dictionaries['langs'][] = $langs;
+        }
     }
 }