Test Failed
Push — main ( c8394f...8477f1 )
by Rafael
66:21
created
public/htdocs/admin/dolistore/class/dolistore.class.php 2 patches
Indentation   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -198,7 +198,7 @@  discard block
 block discarded – undo
198 198
         }
199 199
     }
200 200
 
201
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
201
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
202 202
     /**
203 203
      * Return tree of Dolistore categories. $this->categories must have been loaded before.
204 204
      *
@@ -207,7 +207,7 @@  discard block
 block discarded – undo
207 207
      */
208 208
     public function get_categories($parent = 0)
209 209
     {
210
-		// phpcs:enable
210
+        // phpcs:enable
211 211
         if (!isset($this->categories)) {
212 212
             die('not possible');
213 213
         }
@@ -245,7 +245,7 @@  discard block
 block discarded – undo
245 245
         }
246 246
     }
247 247
 
248
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
248
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
249 249
     /**
250 250
      * Return list of product formatted for output
251 251
      *
@@ -253,7 +253,7 @@  discard block
 block discarded – undo
253 253
      */
254 254
     public function get_products()
255 255
     {
256
-		// phpcs:enable
256
+        // phpcs:enable
257 257
         global $langs, $conf;
258 258
         $html       = "";
259 259
         $last_month = time() - (30 * 24 * 60 * 60);
@@ -342,7 +342,7 @@  discard block
 block discarded – undo
342 342
         return $html;
343 343
     }
344 344
 
345
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
345
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
346 346
     /**
347 347
      * get previous link
348 348
      *
@@ -351,11 +351,11 @@  discard block
 block discarded – undo
351 351
      */
352 352
     public function get_previous_link($text = '<<')
353 353
     {
354
-		// phpcs:enable
354
+        // phpcs:enable
355 355
         return '<a href="' . $this->get_previous_url() . '" class="button">' . dol_escape_htmltag($text) . '</a>';
356 356
     }
357 357
 
358
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
358
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
359 359
     /**
360 360
      * get next link
361 361
      *
@@ -364,11 +364,11 @@  discard block
 block discarded – undo
364 364
      */
365 365
     public function get_next_link($text = '>>')
366 366
     {
367
-		// phpcs:enable
367
+        // phpcs:enable
368 368
         return '<a href="' . $this->get_next_url() . '" class="button">' . dol_escape_htmltag($text) . '</a>';
369 369
     }
370 370
 
371
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
371
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
372 372
     /**
373 373
      * get previous url
374 374
      *
@@ -376,7 +376,7 @@  discard block
 block discarded – undo
376 376
      */
377 377
     public function get_previous_url()
378 378
     {
379
-		// phpcs:enable
379
+        // phpcs:enable
380 380
         $param_array = array();
381 381
         if ($this->start < $this->per_page) {
382 382
             $sub = 0;
@@ -392,7 +392,7 @@  discard block
 block discarded – undo
392 392
         return $this->url . "&" . $param;
393 393
     }
394 394
 
395
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
395
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
396 396
     /**
397 397
      * get next url
398 398
      *
@@ -400,7 +400,7 @@  discard block
 block discarded – undo
400 400
      */
401 401
     public function get_next_url()
402 402
     {
403
-		// phpcs:enable
403
+        // phpcs:enable
404 404
         $param_array = array();
405 405
         if (count($this->products) < $this->per_page) {
406 406
             $add = 0;
@@ -416,7 +416,7 @@  discard block
 block discarded – undo
416 416
         return $this->url . "&" . $param;
417 417
     }
418 418
 
419
-	// phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
419
+    // phpcs:disable PEAR.NamingConventions.ValidFunctionName.ScopeNotCamelCaps
420 420
     /**
421 421
      * version compare
422 422
      *
@@ -426,7 +426,7 @@  discard block
 block discarded – undo
426 426
      */
427 427
     public function version_compare($v1, $v2)
428 428
     {
429
-		// phpcs:enable
429
+        // phpcs:enable
430 430
         $v1       = explode('.', $v1);
431 431
         $v2       = explode('.', $v2);
432 432
         $ret      = 0;
Please login to merge, or discard this patch.
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -324,12 +324,12 @@
 block discarded – undo
324 324
             //output template
325 325
             $html .= '<tr class="app oddeven ' . dol_escape_htmltag($compatible) . '">';
326 326
             $html .= '<td class="center" width="210"><div class="newAppParent">';
327
-            $html .= $newapp . $images;   // No dol_escape_htmltag, it is already escape html
327
+            $html .= $newapp . $images; // No dol_escape_htmltag, it is already escape html
328 328
             $html .= '</div></td>';
329 329
             $html .= '<td class="margeCote"><h2 class="appTitle">';
330 330
             $html .= dol_escape_htmltag($product->name->language[$this->lang - 1]);
331 331
             $html .= '<br><small>';
332
-            $html .= $version;          // No dol_escape_htmltag, it is already escape html
332
+            $html .= $version; // No dol_escape_htmltag, it is already escape html
333 333
             $html .= '</small></h2>';
334 334
             $html .= '<small> ' . dol_print_date(dol_stringtotime($product->date_upd), 'dayhour') . ' - ' . $langs->trans('Ref') . ': ' . dol_escape_htmltag($product->reference) . ' - ' . dol_escape_htmltag($langs->trans('Id')) . ': ' . ((int) $product->id) . '</small><br><br>' . dol_escape_htmltag($product->description_short->language[$this->lang - 1]) . '</td>';
335 335
             // do not load if display none
Please login to merge, or discard this patch.
public/htdocs/admin/user.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -42,7 +42,7 @@
 block discarded – undo
42 42
 
43 43
 $action = GETPOST('action', 'aZ09');
44 44
 $backtopage = GETPOST('backtopage', 'alpha');
45
-$modulepart = GETPOST('modulepart', 'aZ09');    // Used by actions_setmoduleoptions.inc.php
45
+$modulepart = GETPOST('modulepart', 'aZ09'); // Used by actions_setmoduleoptions.inc.php
46 46
 
47 47
 $value = GETPOST('value', 'alpha');
48 48
 $label = GETPOST('label', 'alpha');
Please login to merge, or discard this patch.
public/htdocs/admin/reception_setup.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@
 block discarded – undo
49 49
 
50 50
 $action = GETPOST('action', 'aZ09');
51 51
 $value = GETPOST('value', 'alpha');
52
-$modulepart = GETPOST('modulepart', 'aZ09');    // Used by actions_setmoduleoptions.inc.php
52
+$modulepart = GETPOST('modulepart', 'aZ09'); // Used by actions_setmoduleoptions.inc.php
53 53
 
54 54
 $label = GETPOST('label', 'alpha');
55 55
 $scandir = GETPOST('scan_dir', 'alpha');
Please login to merge, or discard this patch.
public/htdocs/admin/mails_senderprofile_list.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
75 75
 
76 76
 // Default sort order (if not yet defined by previous GETPOST)
77 77
 if (!$sortfield) {
78
-    reset($object->fields);                 // Reset is required to avoid key() to return null.
78
+    reset($object->fields); // Reset is required to avoid key() to return null.
79 79
     $sortfield = "t." . key($object->fields); // Set here default search field. By default 1st field in definition.
80 80
 }
81 81
 if (!$sortorder) {
@@ -540,7 +540,7 @@  discard block
 block discarded – undo
540 540
 }
541 541
 
542 542
 $varpage = empty($contextpage) ? $_SERVER["PHP_SELF"] : $contextpage;
543
-$htmlofselectarray = $form->multiSelectArrayWithCheckbox('selectedfields', $arrayfields, $varpage, getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN'));  // This also change content of $arrayfields with user setup
543
+$htmlofselectarray = $form->multiSelectArrayWithCheckbox('selectedfields', $arrayfields, $varpage, getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN')); // This also change content of $arrayfields with user setup
544 544
 $selectedfields = ($mode != 'kanban' ? $htmlofselectarray : '');
545 545
 $selectedfields .= (count($arrayofmassactions) ? $form->showCheckAddButtons('checkforselect', 1) : '');
546 546
 
@@ -630,7 +630,7 @@  discard block
 block discarded – undo
630 630
     } elseif (in_array($val['type'], array('double(24,8)', 'double(6,3)', 'integer', 'real', 'price')) && !in_array($key, array('id', 'rowid', 'ref', 'status')) && $val['label'] != 'TechnicalID' && empty($val['arrayofkeyval'])) {
631 631
         $cssforfield .= ($cssforfield ? ' ' : '') . 'right';
632 632
     }
633
-    $cssforfield = preg_replace('/small\s*/', '', $cssforfield);    // the 'small' css must not be used for the title label
633
+    $cssforfield = preg_replace('/small\s*/', '', $cssforfield); // the 'small' css must not be used for the title label
634 634
     if (!empty($arrayfields['t.' . $key]['checked'])) {
635 635
         print getTitleFieldOfList($arrayfields['t.' . $key]['label'], 0, $_SERVER['PHP_SELF'], 't.' . $key, '', $param, ($cssforfield ? 'class="' . $cssforfield . '"' : ''), $sortfield, $sortorder, ($cssforfield ? $cssforfield . ' ' : ''), 0, (empty($val['helplist']) ? '' : $val['helplist'])) . "\n";
636 636
         $totalarray['nbfield']++;
Please login to merge, or discard this patch.
public/htdocs/admin/limits.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -99,7 +99,7 @@
 block discarded – undo
99 99
         setEventMessages($langs->trans("ErrorValueCantBeNull", dol_trunc(dol_string_nohtmltag($langs->transnoentitiesnoconv("MAIN_MAX_DECIMALS_SHOWN")), 40)), null, 'errors');
100 100
         $action = 'edit';
101 101
     }
102
-    if (! $error && ((float) $valmainmaxdecimalsshown < $valmainmaxdecimalsunit || (float) $valmainmaxdecimalsshown < $valmainmaxdecimalstot)) {
102
+    if (!$error && ((float) $valmainmaxdecimalsshown < $valmainmaxdecimalsunit || (float) $valmainmaxdecimalsshown < $valmainmaxdecimalstot)) {
103 103
         $langs->load("errors");
104 104
         $error++;
105 105
         setEventMessages($langs->trans("ErrorValueForTooLow", dol_trunc(dol_string_nohtmltag($langs->transnoentitiesnoconv("MAIN_MAX_DECIMALS_SHOWN")), 40)), null, 'errors');
Please login to merge, or discard this patch.
public/htdocs/admin/expedition.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@
 block discarded – undo
49 49
 
50 50
 $action = GETPOST('action', 'aZ09');
51 51
 $value = GETPOST('value', 'alpha');
52
-$modulepart = GETPOST('modulepart', 'aZ09');    // Used by actions_setmoduleoptions.inc.php
52
+$modulepart = GETPOST('modulepart', 'aZ09'); // Used by actions_setmoduleoptions.inc.php
53 53
 
54 54
 $label = GETPOST('label', 'alpha');
55 55
 $scandir = GETPOST('scan_dir', 'alpha');
Please login to merge, or discard this patch.
public/htdocs/admin/mrp.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@
 block discarded – undo
49 49
 
50 50
 $action = GETPOST('action', 'aZ09');
51 51
 $value = GETPOST('value', 'alpha');
52
-$modulepart = GETPOST('modulepart', 'aZ09');    // Used by actions_setmoduleoptions.inc.php
52
+$modulepart = GETPOST('modulepart', 'aZ09'); // Used by actions_setmoduleoptions.inc.php
53 53
 
54 54
 $label = GETPOST('label', 'alpha');
55 55
 $scandir = GETPOST('scan_dir', 'alpha');
Please login to merge, or discard this patch.
public/htdocs/admin/expensereport_rules.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -37,7 +37,7 @@
 block discarded – undo
37 37
 $langs->loadLangs(array("admin", "other", "trips", "errors", "dict"));
38 38
 
39 39
 // Initialize technical object to manage hooks of page. Note that conf->hooks_modules contains array of hook context
40
-$hookmanager->initHooks(array('admin', 'dictionaryadmin','expensereport_rules'));
40
+$hookmanager->initHooks(array('admin', 'dictionaryadmin', 'expensereport_rules'));
41 41
 
42 42
 $object = new ExpenseReportRule($db);
43 43
 
Please login to merge, or discard this patch.
public/htdocs/admin/supplier_invoice.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@
 block discarded – undo
44 44
 
45 45
 $type = GETPOST('type', 'alpha');
46 46
 $value = GETPOST('value', 'alpha');
47
-$modulepart = GETPOST('modulepart', 'aZ09');    // Used by actions_setmoduleoptions.inc.php
47
+$modulepart = GETPOST('modulepart', 'aZ09'); // Used by actions_setmoduleoptions.inc.php
48 48
 
49 49
 $label = GETPOST('label', 'alpha');
50 50
 $scandir = GETPOST('scan_dir', 'alpha');
Please login to merge, or discard this patch.