alxarafe / alixar

public/htdocs/core/ajax/vatrates.php

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
 require constant('DOL_DOCUMENT_ROOT') . '/main.inc.php';
 
 $id = GETPOSTINT('id');
-$action = GETPOST('action', 'aZ09');    // 'getSellerVATRates' or 'getBuyerVATRates'
+$action = GETPOST('action', 'aZ09'); // 'getSellerVATRates' or 'getBuyerVATRates'
 $htmlname   = GETPOST('htmlname', 'alpha');
 $selected   = (GETPOST('selected') ? GETPOST('selected') : '-1');
 $productid = (GETPOSTINT('productid') ? GETPOSTINT('productid') : 0);

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/DolibarrModules.class.php

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']);