Inspection of "Daily Inspection: Better structure of payment page..." - Dolibarr/dolibarr - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — develop (bd76a0)

unknown

created 2024-03-20 16:49 UTC

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -39,7 +39,7 @@
 block discarded – undo
 
 // Get Parameters
 $action = GETPOST('action', 'aZ09');
-$modulepart = GETPOST('modulepart', 'aZ09');	// Used by actions_setmoduleoptions.inc.php
+$modulepart = GETPOST('modulepart', 'aZ09'); // Used by actions_setmoduleoptions.inc.php
 
 
 /*

Please login to merge, or discard this patch.

htdocs/admin/commande.php 1 patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -43,7 +43,7 @@  discard block
 block discarded – undo
 
 $action = GETPOST('action', 'aZ09');
 $value = GETPOST('value', 'alpha');
-$modulepart = GETPOST('modulepart', 'aZ09');	// Used by actions_setmoduleoptions.inc.php
+$modulepart = GETPOST('modulepart', 'aZ09'); // Used by actions_setmoduleoptions.inc.php
 
 $label = GETPOST('label', 'alpha');
 $scandir = GETPOST('scan_dir', 'alpha');
@@ -211,7 +211,7 @@  discard block
 block discarded – undo
 		setEventMessages($langs->trans('Error'), null, 'errors');
 	} else {
 		setEventMessages($langs->trans('SetupSaved'), null, 'mesgs');
-		header("Location: " . $_SERVER["PHP_SELF"]);
+		header("Location: ".$_SERVER["PHP_SELF"]);
 		exit();
 	}
 } elseif (preg_match('/del_(.*)/', $action, $reg)) {
@@ -226,7 +226,7 @@  discard block
 block discarded – undo
 		setEventMessages($langs->trans('Error'), null, 'errors');
 	} else {
 		setEventMessages($langs->trans('SetupSaved'), null, 'mesgs');
-		header("Location: " . $_SERVER["PHP_SELF"]);
+		header("Location: ".$_SERVER["PHP_SELF"]);
 		exit();
 	}
 }

Please login to merge, or discard this patch.

htdocs/admin/tools/dolibarr_import.php 1 patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -22,8 +22,8 @@
 block discarded – undo
  * 		\brief      Page to import database
  */
 
-if (! defined('CSRFCHECK_WITH_TOKEN')) {
-	define('CSRFCHECK_WITH_TOKEN', '1');		// Force use of CSRF protection with tokens even for GET
+if (!defined('CSRFCHECK_WITH_TOKEN')) {
+	define('CSRFCHECK_WITH_TOKEN', '1'); // Force use of CSRF protection with tokens even for GET
 }
 
 // Load Dolibarr environment

Please login to merge, or discard this patch.

htdocs/admin/tools/purge.php 1 patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -21,8 +21,8 @@
 block discarded – undo
  *		\brief      Page to purge files (temporary or not)
  */
 
-if (! defined('CSRFCHECK_WITH_TOKEN')) {
-	define('CSRFCHECK_WITH_TOKEN', '1');		// Force use of CSRF protection with tokens even for GET
+if (!defined('CSRFCHECK_WITH_TOKEN')) {
+	define('CSRFCHECK_WITH_TOKEN', '1'); // Force use of CSRF protection with tokens even for GET
 }
 
 // Load Dolibarr environment

Please login to merge, or discard this patch.

htdocs/admin/tools/listsessions.php 1 patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -22,8 +22,8 @@
 block discarded – undo
  *      \brief      List of PHP sessions
  */
 
-if (! defined('CSRFCHECK_WITH_TOKEN')) {
-	define('CSRFCHECK_WITH_TOKEN', '1');		// Force use of CSRF protection with tokens even for GET
+if (!defined('CSRFCHECK_WITH_TOKEN')) {
+	define('CSRFCHECK_WITH_TOKEN', '1'); // Force use of CSRF protection with tokens even for GET
 }
 
 // Load Dolibarr environment

Please login to merge, or discard this patch.

htdocs/admin/tools/export_files.php 1 patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -23,8 +23,8 @@
 block discarded – undo
  *		\brief      Page to export documents into a compressed file
  */
 
-if (! defined('CSRFCHECK_WITH_TOKEN')) {
-	define('CSRFCHECK_WITH_TOKEN', '1');		// Force use of CSRF protection with tokens even for GET
+if (!defined('CSRFCHECK_WITH_TOKEN')) {
+	define('CSRFCHECK_WITH_TOKEN', '1'); // Force use of CSRF protection with tokens even for GET
 }
 
 // Load Dolibarr environment

Please login to merge, or discard this patch.

htdocs/admin/supplier_proposal.php 1 patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

@@ -43,7 +43,7 @@  discard block
 block discarded – undo
 
 $action = GETPOST('action', 'aZ09');
 $value = GETPOST('value', 'alpha');
-$modulepart = GETPOST('modulepart', 'aZ09');	// Used by actions_setmoduleoptions.inc.php
+$modulepart = GETPOST('modulepart', 'aZ09'); // Used by actions_setmoduleoptions.inc.php
 
 $label = GETPOST('label', 'alpha');
 $scandir = GETPOST('scan_dir', 'alpha');
@@ -211,7 +211,7 @@  discard block
 block discarded – undo
 		setEventMessages($langs->trans('Error'), null, 'errors');
 	} else {
 		setEventMessages($langs->trans('SetupSaved'), null, 'mesgs');
-		header("Location: " . $_SERVER["PHP_SELF"]);
+		header("Location: ".$_SERVER["PHP_SELF"]);
 		exit();
 	}
 } elseif (preg_match('/del_(.*)/', $action, $reg)) {
@@ -226,7 +226,7 @@  discard block
 block discarded – undo
 		setEventMessages($langs->trans('Error'), null, 'errors');
 	} else {
 		setEventMessages($langs->trans('SetupSaved'), null, 'mesgs');
-		header("Location: " . $_SERVER["PHP_SELF"]);
+		header("Location: ".$_SERVER["PHP_SELF"]);
 		exit();
 	}
 }

Please login to merge, or discard this patch.

htdocs/admin/company.php 1 patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -489,7 +489,7 @@  discard block
 block discarded – undo
 print '<tr class="oddeven"><td><label for="logo">'.$form->textwithpicto($langs->trans("Logo"), $tooltiplogo).'</label></td><td>';
 print '<div class="centpercent nobordernopadding valignmiddle "><div class="inline-block marginrightonly">';
 if ($maxmin > 0) {
-	print '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">';	// MAX_FILE_SIZE must precede the field type=file
+	print '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">'; // MAX_FILE_SIZE must precede the field type=file
 }
 print '<input type="file" class="flat minwidth100 maxwidthinputfileonsmartphone" name="logo" id="logo" accept="image/*">';
 print '</div>';
@@ -531,7 +531,7 @@  discard block
 block discarded – undo
 $maxfilesizearray = getMaxFileSizeArray();
 $maxmin = $maxfilesizearray['maxmin'];
 if ($maxmin > 0) {
-	print '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">';	// MAX_FILE_SIZE must precede the field type=file
+	print '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">'; // MAX_FILE_SIZE must precede the field type=file
 }
 print '<input type="file" class="flat minwidth100 maxwidthinputfileonsmartphone" name="logo_squarred" id="logo_squarred" accept="image/*">';
 print '</div>';

Please login to merge, or discard this patch.

htdocs/core/tpl/commonfields_view.tpl.php 1 patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -155,7 +155,7 @@  discard block
 block discarded – undo
 	if ($val['type'] == 'text' || $val['type'] == 'html') {
 		$rightpart .= ' tdtop';
 	}
-	$rightpart.= '">';
+	$rightpart .= '">';
 	$labeltoshow = '';
 	if (!empty($val['help'])) {
 		$labeltoshow .= $form->textwithpicto($langs->trans($val['label']), $langs->trans($val['help']));
@@ -195,7 +195,7 @@  discard block
 block discarded – undo
 				$out = $object->showOutputField($val, $key, $value, '', '', '', 0);
 				$rightpart .= showValueWithClipboardCPButton($out, 0, $out);
 			} else {
-				$rightpart.= $object->showOutputField($val, $key, $value, '', '', '', 0);
+				$rightpart .= $object->showOutputField($val, $key, $value, '', '', '', 0);
 			}
 		}
 		if (preg_match('/^(text|html)/', $val['type'])) {

Please login to merge, or discard this patch.

		@@ -489,7 +489,7 @@ discard block
		block discarded – undo
489	489	print '<tr class="oddeven"><td><label for="logo">'.$form->textwithpicto($langs->trans("Logo"), $tooltiplogo).'</label></td><td>';
490	490	print '<div class="centpercent nobordernopadding valignmiddle "><div class="inline-block marginrightonly">';
491	491	if ($maxmin > 0) {
492		- print '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">'; // MAX_FILE_SIZE must precede the field type=file
	492	+ print '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">'; // MAX_FILE_SIZE must precede the field type=file
493	493	}
494	494	print '<input type="file" class="flat minwidth100 maxwidthinputfileonsmartphone" name="logo" id="logo" accept="image/*">';
495	495	print '</div>';
		@@ -531,7 +531,7 @@ discard block
		block discarded – undo
531	531	$maxfilesizearray = getMaxFileSizeArray();
532	532	$maxmin = $maxfilesizearray['maxmin'];
533	533	if ($maxmin > 0) {
534		- print '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">'; // MAX_FILE_SIZE must precede the field type=file
	534	+ print '<input type="hidden" name="MAX_FILE_SIZE" value="'.($maxmin * 1024).'">'; // MAX_FILE_SIZE must precede the field type=file
535	535	}
536	536	print '<input type="file" class="flat minwidth100 maxwidthinputfileonsmartphone" name="logo_squarred" id="logo_squarred" accept="image/*">';
537	537	print '</div>';

		@@ -155,7 +155,7 @@ discard block
		block discarded – undo
155	155	if ($val['type'] == 'text' \|\| $val['type'] == 'html') {
156	156	$rightpart .= ' tdtop';
157	157	}
158		- $rightpart.= '">';
	158	+ $rightpart .= '">';
159	159	$labeltoshow = '';
160	160	if (!empty($val['help'])) {
161	161	$labeltoshow .= $form->textwithpicto($langs->trans($val['label']), $langs->trans($val['help']));
		@@ -195,7 +195,7 @@ discard block
		block discarded – undo
195	195	$out = $object->showOutputField($val, $key, $value, '', '', '', 0);
196	196	$rightpart .= showValueWithClipboardCPButton($out, 0, $out);
197	197	} else {
198		- $rightpart.= $object->showOutputField($val, $key, $value, '', '', '', 0);
	198	+ $rightpart .= $object->showOutputField($val, $key, $value, '', '', '', 0);
199	199	}
200	200	}
201	201	if (preg_match('/^(text\|html)/', $val['type'])) {

		@@ -39,7 +39,7 @@
		block discarded – undo
39	39
40	40	// Get Parameters
41	41	$action = GETPOST('action', 'aZ09');
42		-$modulepart = GETPOST('modulepart', 'aZ09'); // Used by actions_setmoduleoptions.inc.php
	42	+$modulepart = GETPOST('modulepart', 'aZ09'); // Used by actions_setmoduleoptions.inc.php
43	43
44	44
45	45	/*

		@@ -43,7 +43,7 @@ discard block
		block discarded – undo
43	43
44	44	$action = GETPOST('action', 'aZ09');
45	45	$value = GETPOST('value', 'alpha');
46		-$modulepart = GETPOST('modulepart', 'aZ09'); // Used by actions_setmoduleoptions.inc.php
	46	+$modulepart = GETPOST('modulepart', 'aZ09'); // Used by actions_setmoduleoptions.inc.php
47	47
48	48	$label = GETPOST('label', 'alpha');
49	49	$scandir = GETPOST('scan_dir', 'alpha');
		@@ -211,7 +211,7 @@ discard block
		block discarded – undo
211	211	setEventMessages($langs->trans('Error'), null, 'errors');
212	212	} else {
213	213	setEventMessages($langs->trans('SetupSaved'), null, 'mesgs');
214		- header("Location: " . $_SERVER["PHP_SELF"]);
	214	+ header("Location: ".$_SERVER["PHP_SELF"]);
215	215	exit();
216	216	}
217	217	} elseif (preg_match('/del_(.*)/', $action, $reg)) {
		@@ -226,7 +226,7 @@ discard block
		block discarded – undo
226	226	setEventMessages($langs->trans('Error'), null, 'errors');
227	227	} else {
228	228	setEventMessages($langs->trans('SetupSaved'), null, 'mesgs');
229		- header("Location: " . $_SERVER["PHP_SELF"]);
	229	+ header("Location: ".$_SERVER["PHP_SELF"]);
230	230	exit();
231	231	}
232	232	}

		@@ -22,8 +22,8 @@
		block discarded – undo
22	22	* \brief Page to import database
23	23	*/
24	24
25		-if (! defined('CSRFCHECK_WITH_TOKEN')) {
26		- define('CSRFCHECK_WITH_TOKEN', '1'); // Force use of CSRF protection with tokens even for GET
	25	+if (!defined('CSRFCHECK_WITH_TOKEN')) {
	26	+ define('CSRFCHECK_WITH_TOKEN', '1'); // Force use of CSRF protection with tokens even for GET
27	27	}
28	28
29	29	// Load Dolibarr environment

		@@ -21,8 +21,8 @@
		block discarded – undo
21	21	* \brief Page to purge files (temporary or not)
22	22	*/
23	23
24		-if (! defined('CSRFCHECK_WITH_TOKEN')) {
25		- define('CSRFCHECK_WITH_TOKEN', '1'); // Force use of CSRF protection with tokens even for GET
	24	+if (!defined('CSRFCHECK_WITH_TOKEN')) {
	25	+ define('CSRFCHECK_WITH_TOKEN', '1'); // Force use of CSRF protection with tokens even for GET
26	26	}
27	27
28	28	// Load Dolibarr environment

		@@ -22,8 +22,8 @@
		block discarded – undo
22	22	* \brief List of PHP sessions
23	23	*/
24	24
25		-if (! defined('CSRFCHECK_WITH_TOKEN')) {
26		- define('CSRFCHECK_WITH_TOKEN', '1'); // Force use of CSRF protection with tokens even for GET
	25	+if (!defined('CSRFCHECK_WITH_TOKEN')) {
	26	+ define('CSRFCHECK_WITH_TOKEN', '1'); // Force use of CSRF protection with tokens even for GET
27	27	}
28	28
29	29	// Load Dolibarr environment

		@@ -23,8 +23,8 @@
		block discarded – undo
23	23	* \brief Page to export documents into a compressed file
24	24	*/
25	25
26		-if (! defined('CSRFCHECK_WITH_TOKEN')) {
27		- define('CSRFCHECK_WITH_TOKEN', '1'); // Force use of CSRF protection with tokens even for GET
	26	+if (!defined('CSRFCHECK_WITH_TOKEN')) {
	27	+ define('CSRFCHECK_WITH_TOKEN', '1'); // Force use of CSRF protection with tokens even for GET
28	28	}
29	29
30	30	// Load Dolibarr environment

Dolibarr / dolibarr

Branch — develop (bd76a0)

Category

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Spacing +3 added lines, -3 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch

Spacing +2 added lines, -2 removed lines patch added patch discarded remove patch