Inspection of "Daily Inspection: css" - Dolibarr/dolibarr - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — develop (d71e39)

unknown

created 2023-11-07 16:05 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/tools/update.php 1 patch

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

@@ -22,8 +22,8 @@  discard block
 block discarded – undo
  *		\brief      Page to make a Dolibarr online upgrade
  */
 
-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
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
 	$result = getURLContent('https://sourceforge.net/projects/dolibarr/rss');
 	//var_dump($result['content']);
 	if (function_exists('simplexml_load_string')) {
-		$sfurl = simplexml_load_string($result['content'], 'SimpleXMLElement', LIBXML_NOCDATA|LIBXML_NONET);
+		$sfurl = simplexml_load_string($result['content'], 'SimpleXMLElement', LIBXML_NOCDATA | LIBXML_NONET);
 	} else {
 		$sfurl = 'xml_not_available';
 	}

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.

		@@ -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	}

		@@ -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	}

		@@ -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>';

		@@ -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	/*

		@@ -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

		@@ -22,8 +22,8 @@ discard block
		block discarded – undo
22	22	* \brief Page to make a Dolibarr online upgrade
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
		@@ -63,7 +63,7 @@ discard block
		block discarded – undo
63	63	$result = getURLContent('https://sourceforge.net/projects/dolibarr/rss');
64	64	//var_dump($result['content']);
65	65	if (function_exists('simplexml_load_string')) {
66		- $sfurl = simplexml_load_string($result['content'], 'SimpleXMLElement', LIBXML_NOCDATA\|LIBXML_NONET);
	66	+ $sfurl = simplexml_load_string($result['content'], 'SimpleXMLElement', LIBXML_NOCDATA \| LIBXML_NONET);
67	67	} else {
68	68	$sfurl = 'xml_not_available';
69	69	}

Dolibarr / dolibarr

Branch — develop (d71e39)

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 +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