Dolibarr / dolibarr

htdocs/accountancy/bookkeeping/balance.php

Spacing +37 added lines, -37 removed lines

@@ -83,7 +83,7 @@ 
 
 // Initialize technical object to manage hooks of page. Note that conf->hooks_modules contains array of hook context
 $object = new BookKeeping($db);
-$hookmanager->initHooks(array($contextpage));  // Note that conf->hooks_modules contains array
+$hookmanager->initHooks(array($contextpage)); // Note that conf->hooks_modules contains array
 
 $formaccounting = new FormAccounting($db);
 $formother = new FormOther($db);
@@ -161,15 +161,15 @@ 
 
 	if (!empty($search_date_start)) {
 		$filter['t.doc_date>='] = $search_date_start;
-		$param .= '&date_startmonth=' . GETPOSTINT('date_startmonth') . '&date_startday=' . GETPOSTINT('date_startday') . '&date_startyear=' . GETPOSTINT('date_startyear');
+		$param .= '&date_startmonth='.GETPOSTINT('date_startmonth').'&date_startday='.GETPOSTINT('date_startday').'&date_startyear='.GETPOSTINT('date_startyear');
 	}
 	if (!empty($search_date_end)) {
 		$filter['t.doc_date<='] = $search_date_end;
-		$param .= '&date_endmonth=' . GETPOSTINT('date_endmonth') . '&date_endday=' . GETPOSTINT('date_endday') . '&date_endyear=' . GETPOSTINT('date_endyear');
+		$param .= '&date_endmonth='.GETPOSTINT('date_endmonth').'&date_endday='.GETPOSTINT('date_endday').'&date_endyear='.GETPOSTINT('date_endyear');
 	}
 	if (!empty($search_doc_date)) {
 		$filter['t.doc_date'] = $search_doc_date;
-		$param .= '&doc_datemonth=' . GETPOSTINT('doc_datemonth') . '&doc_dateday=' . GETPOSTINT('doc_dateday') . '&doc_dateyear=' . GETPOSTINT('doc_dateyear');
+		$param .= '&doc_datemonth='.GETPOSTINT('doc_datemonth').'&doc_dateday='.GETPOSTINT('doc_dateday').'&doc_dateyear='.GETPOSTINT('doc_dateyear');
 	}
 	if (!empty($search_accountancy_code_start)) {
 		if ($type == 'sub') {
@@ -177,7 +177,7 @@ 
 		} else {
 			$filter['t.numero_compte>='] = $search_accountancy_code_start;
 		}
-		$param .= '&search_accountancy_code_start=' . urlencode($search_accountancy_code_start);
+		$param .= '&search_accountancy_code_start='.urlencode($search_accountancy_code_start);
 	}
 	if (!empty($search_accountancy_code_end)) {
 		if ($type == 'sub') {
@@ -185,12 +185,12 @@ 
 		} else {
 			$filter['t.numero_compte<='] = $search_accountancy_code_end;
 		}
-		$param .= '&search_accountancy_code_end=' . urlencode($search_accountancy_code_end);
+		$param .= '&search_accountancy_code_end='.urlencode($search_accountancy_code_end);
 	}
 	if (!empty($search_ledger_code)) {
 		$filter['t.code_journal'] = $search_ledger_code;
 		foreach ($search_ledger_code as $code) {
-			$param .= '&search_ledger_code[]=' . urlencode($code);
+			$param .= '&search_ledger_code[]='.urlencode($code);
 		}
 	}
 	if (!empty($search_not_reconciled)) {
@@ -201,7 +201,7 @@ 
 	// param with type of list
 	$url_param = substr($param, 1); // remove first "&"
 	if (!empty($type)) {
-		$param = '&type=' . $type . $param;
+		$param = '&type='.$type.$param;
 	}
 }
 
@@ -223,11 +223,11 @@ 
 
 	foreach ($object->lines as $line) {
 		if ($type == 'sub') {
-			print '"' . length_accounta($line->subledger_account) . '"' . $sep;
-			print '"' . $line->subledger_label . '"' . $sep;
+			print '"'.length_accounta($line->subledger_account).'"'.$sep;
+			print '"'.$line->subledger_label.'"'.$sep;
 		} else {
-			print '"' . length_accountg($line->numero_compte) . '"' . $sep;
-			print '"' . $object->get_compte_desc($line->numero_compte) . '"' . $sep;
+			print '"'.length_accountg($line->numero_compte).'"'.$sep;
+			print '"'.$object->get_compte_desc($line->numero_compte).'"'.$sep;
 		}
 		print '"'.price($line->debit).'"'.$sep;
 		print '"'.price($line->credit).'"'.$sep;
@@ -302,7 +302,7 @@ 
 	$newcardbutton = empty($hookmanager->resPrint) ? '' : $hookmanager->resPrint;
 
 	if (empty($reshook)) {
-		$newcardbutton = '<input type="button" id="exportcsvbutton" name="exportcsvbutton" class="butAction" value="'.$langs->trans("Export").' (' . getDolGlobalString('ACCOUNTING_EXPORT_FORMAT').')" />';
+		$newcardbutton = '<input type="button" id="exportcsvbutton" name="exportcsvbutton" class="butAction" value="'.$langs->trans("Export").' ('.getDolGlobalString('ACCOUNTING_EXPORT_FORMAT').')" />';
 
 		print '<script type="text/javascript">
 		jQuery(document).ready(function() {
@@ -317,11 +317,11 @@ 
 		</script>';
 
 		if ($type == 'sub') {
-			$newcardbutton .= dolGetButtonTitle($langs->trans('AccountBalance')." - ".$langs->trans('GroupByAccountAccounting'), '', 'fa fa-stream paddingleft imgforviewmode', DOL_URL_ROOT . '/accountancy/bookkeeping/balance.php?' . $url_param, '', 1, array('morecss' => 'marginleftonly'));
-			$newcardbutton .= dolGetButtonTitle($langs->trans('AccountBalance')." - ".$langs->trans('GroupBySubAccountAccounting'), '', 'fa fa-align-left vmirror paddingleft imgforviewmode', DOL_URL_ROOT . '/accountancy/bookkeeping/balance.php?type=sub&' . $url_param, '', 1, array('morecss' => 'marginleftonly btnTitleSelected'));
+			$newcardbutton .= dolGetButtonTitle($langs->trans('AccountBalance')." - ".$langs->trans('GroupByAccountAccounting'), '', 'fa fa-stream paddingleft imgforviewmode', DOL_URL_ROOT.'/accountancy/bookkeeping/balance.php?'.$url_param, '', 1, array('morecss' => 'marginleftonly'));
+			$newcardbutton .= dolGetButtonTitle($langs->trans('AccountBalance')." - ".$langs->trans('GroupBySubAccountAccounting'), '', 'fa fa-align-left vmirror paddingleft imgforviewmode', DOL_URL_ROOT.'/accountancy/bookkeeping/balance.php?type=sub&'.$url_param, '', 1, array('morecss' => 'marginleftonly btnTitleSelected'));
 		} else {
-			$newcardbutton .= dolGetButtonTitle($langs->trans('AccountBalance')." - ".$langs->trans('GroupByAccountAccounting'), '', 'fa fa-stream paddingleft imgforviewmode', DOL_URL_ROOT . '/accountancy/bookkeeping/balance.php?' . $url_param, '', 1, array('morecss' => 'marginleftonly btnTitleSelected'));
-			$newcardbutton .= dolGetButtonTitle($langs->trans('AccountBalance')." - ".$langs->trans('GroupBySubAccountAccounting'), '', 'fa fa-align-left vmirror paddingleft imgforviewmode', DOL_URL_ROOT . '/accountancy/bookkeeping/balance.php?type=sub&' . $url_param, '', 1, array('morecss' => 'marginleftonly'));
+			$newcardbutton .= dolGetButtonTitle($langs->trans('AccountBalance')." - ".$langs->trans('GroupByAccountAccounting'), '', 'fa fa-stream paddingleft imgforviewmode', DOL_URL_ROOT.'/accountancy/bookkeeping/balance.php?'.$url_param, '', 1, array('morecss' => 'marginleftonly btnTitleSelected'));
+			$newcardbutton .= dolGetButtonTitle($langs->trans('AccountBalance')." - ".$langs->trans('GroupBySubAccountAccounting'), '', 'fa fa-align-left vmirror paddingleft imgforviewmode', DOL_URL_ROOT.'/accountancy/bookkeeping/balance.php?type=sub&'.$url_param, '', 1, array('morecss' => 'marginleftonly'));
 		}
 		$newcardbutton .= dolGetButtonTitleSeparator();
 		$newcardbutton .= dolGetButtonTitle($langs->trans('NewAccountingMvt'), '', 'fa fa-plus-circle paddingleft', DOL_URL_ROOT.'/accountancy/bookkeeping/card.php?action=create');
@@ -465,9 +465,9 @@ 
 	// without time and class limits (Class 6 and 7 accounts ???) and does not take into account the "a-nouveau" journal.
 	if (getDolGlobalString('ACCOUNTANCY_SHOW_OPENING_BALANCE')) {
 		$sql = "SELECT t.numero_compte, (SUM(t.debit) - SUM(t.credit)) as opening_balance";
-		$sql .= " FROM " . MAIN_DB_PREFIX . "accounting_bookkeeping as t";
-		$sql .= " WHERE t.entity = " . $conf->entity;        // Never do sharing into accounting features
-		$sql .= " AND t.doc_date < '" . $db->idate($search_date_start) . "'";
+		$sql .= " FROM ".MAIN_DB_PREFIX."accounting_bookkeeping as t";
+		$sql .= " WHERE t.entity = ".$conf->entity; // Never do sharing into accounting features
+		$sql .= " AND t.doc_date < '".$db->idate($search_date_start)."'";
 		$sql .= " GROUP BY t.numero_compte";
 
 		$resql = $db->query($sql);
@@ -475,7 +475,7 @@ 
 		$opening_balances = array();
 		for ($i = 0; $i < $nrows; $i++) {
 			$arr = $resql->fetch_array();
-			$opening_balances["'" . $arr['numero_compte'] . "'"] = $arr['opening_balance'];
+			$opening_balances["'".$arr['numero_compte']."'"] = $arr['opening_balance'];
 		}
 	}
 
@@ -507,9 +507,9 @@ 
 		//var_dump($accounting_account);
 		//var_dump($accountingaccountstatic);
 		if (empty($accountingaccountstatic->label) && $accountingaccountstatic->id > 0) {
-			$link = '<a class="editfielda reposition" href="' . DOL_URL_ROOT . '/accountancy/admin/card.php?action=update&token=' . newToken() . '&id=' . $accountingaccountstatic->id . '">' . img_edit() . '</a>';
+			$link = '<a class="editfielda reposition" href="'.DOL_URL_ROOT.'/accountancy/admin/card.php?action=update&token='.newToken().'&id='.$accountingaccountstatic->id.'">'.img_edit().'</a>';
 		} elseif ($accounting_account == 'NotDefined') {
-			$link = '<a href="' . DOL_URL_ROOT . '/accountancy/admin/card.php?action=create&token=' . newToken() . '&accountingaccount=' . length_accountg($line->numero_compte) . '">' . img_edit_add() . '</a>';
+			$link = '<a href="'.DOL_URL_ROOT.'/accountancy/admin/card.php?action=create&token='.newToken().'&accountingaccount='.length_accountg($line->numero_compte).'">'.img_edit_add().'</a>';
 		} elseif (empty($tmparrayforrootaccount['label'])) {
 			// $tmparrayforrootaccount['label'] not defined = the account has not parent with a parent.
 			// This is useless, we should not create a new account when an account has no parent, we must edit it to fix its parent.
@@ -546,7 +546,7 @@ 
 
 				// Show first line of a break
 				print '<tr class="trforbreak">';
-				print '<td colspan="'.($colspan+1).'" class="tdforbreak">'.$root_account_number.($root_account_description ? ' - '.$root_account_description : '').'</td>';
+				print '<td colspan="'.($colspan + 1).'" class="tdforbreak">'.$root_account_number.($root_account_description ? ' - '.$root_account_description : '').'</td>';
 				print '</tr>';
 
 				$displayed_account = $root_account_number;
@@ -585,22 +585,22 @@ 
 		$urlzoom = '';
 		if ($type == 'sub') {
 			if ($line->subledger_account) {
-				$urlzoom = DOL_URL_ROOT . '/accountancy/bookkeeping/listbyaccount.php?type=sub&search_accountancy_code_start=' . urlencode($line->subledger_account) . '&search_accountancy_code_end=' . urlencode($line->subledger_account);
+				$urlzoom = DOL_URL_ROOT.'/accountancy/bookkeeping/listbyaccount.php?type=sub&search_accountancy_code_start='.urlencode($line->subledger_account).'&search_accountancy_code_end='.urlencode($line->subledger_account);
 				if (GETPOSTISSET('date_startmonth')) {
-					$urlzoom .= '&search_date_startmonth=' . GETPOSTINT('date_startmonth') . '&search_date_startday=' . GETPOSTINT('date_startday') . '&search_date_startyear=' . GETPOSTINT('date_startyear');
+					$urlzoom .= '&search_date_startmonth='.GETPOSTINT('date_startmonth').'&search_date_startday='.GETPOSTINT('date_startday').'&search_date_startyear='.GETPOSTINT('date_startyear');
 				}
 				if (GETPOSTISSET('date_endmonth')) {
-					$urlzoom .= '&search_date_endmonth=' . GETPOSTINT('date_endmonth') . '&search_date_endday=' . GETPOSTINT('date_endday') . '&search_date_endyear=' . GETPOSTINT('date_endyear');
+					$urlzoom .= '&search_date_endmonth='.GETPOSTINT('date_endmonth').'&search_date_endday='.GETPOSTINT('date_endday').'&search_date_endyear='.GETPOSTINT('date_endyear');
 				}
 			}
 		} else {
 			if ($line->numero_compte) {
-				$urlzoom = DOL_URL_ROOT . '/accountancy/bookkeeping/listbyaccount.php?search_accountancy_code_start=' . urlencode($line->numero_compte) . '&search_accountancy_code_end=' . urlencode($line->numero_compte);
+				$urlzoom = DOL_URL_ROOT.'/accountancy/bookkeeping/listbyaccount.php?search_accountancy_code_start='.urlencode($line->numero_compte).'&search_accountancy_code_end='.urlencode($line->numero_compte);
 				if (GETPOSTISSET('date_startmonth')) {
-					$urlzoom .= '&search_date_startmonth=' . GETPOSTINT('date_startmonth') . '&search_date_startday=' . GETPOSTINT('date_startday') . '&search_date_startyear=' . GETPOSTINT('date_startyear');
+					$urlzoom .= '&search_date_startmonth='.GETPOSTINT('date_startmonth').'&search_date_startday='.GETPOSTINT('date_startday').'&search_date_startyear='.GETPOSTINT('date_startyear');
 				}
 				if (GETPOSTISSET('date_endmonth')) {
-					$urlzoom .= '&search_date_endmonth=' . GETPOSTINT('date_endmonth') . '&search_date_endday=' . GETPOSTINT('date_endday') . '&search_date_endyear=' . GETPOSTINT('date_endyear');
+					$urlzoom .= '&search_date_endmonth='.GETPOSTINT('date_endmonth').'&search_date_endday='.GETPOSTINT('date_endday').'&search_date_endyear='.GETPOSTINT('date_endyear');
 				}
 			}
 		}
@@ -643,9 +643,9 @@ 
 		print '<td class="right nowraponall amount">'.price(price2num($sous_total_debit, 'MT')).'</td>';
 		print '<td class="right nowraponall amount">'.price(price2num($sous_total_credit, 'MT')).'</td>';
 		if (getDolGlobalString('ACCOUNTANCY_SHOW_OPENING_BALANCE')) {
-			print '<td class="right nowraponall amount">' . price(price2num($sous_total_opening_balance + $sous_total_debit - $sous_total_credit, 'MT')) . '</td>';
+			print '<td class="right nowraponall amount">'.price(price2num($sous_total_opening_balance + $sous_total_debit - $sous_total_credit, 'MT')).'</td>';
 		} else {
-			print '<td class="right nowraponall amount">' . price(price2num($sous_total_debit - $sous_total_credit, 'MT')) . '</td>';
+			print '<td class="right nowraponall amount">'.price(price2num($sous_total_debit - $sous_total_credit, 'MT')).'</td>';
 		}
 		// Action column
 		if (!getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN')) {
@@ -666,9 +666,9 @@ 
 	print '<td class="right nowraponall amount">'.price(price2num($total_debit, 'MT')).'</td>';
 	print '<td class="right nowraponall amount">'.price(price2num($total_credit, 'MT')).'</td>';
 	if (getDolGlobalString('ACCOUNTANCY_SHOW_OPENING_BALANCE')) {
-		print '<td class="right nowraponall amount">' . price(price2num($total_opening_balance + $total_debit - $total_credit, 'MT')) . '</td>';
+		print '<td class="right nowraponall amount">'.price(price2num($total_opening_balance + $total_debit - $total_credit, 'MT')).'</td>';
 	} else {
-		print '<td class="right nowraponall amount">' . price(price2num($total_debit - $total_credit, 'MT')) . '</td>';
+		print '<td class="right nowraponall amount">'.price(price2num($total_debit - $total_credit, 'MT')).'</td>';
 	}
 	// Action column
 	if (!getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN')) {
@@ -682,7 +682,7 @@ 
 		if (getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN')) {
 			print "<td></td>\n";
 		}
-		print '<td class="right">' . $langs->trans("AccountingResult") . ':</td>';
+		print '<td class="right">'.$langs->trans("AccountingResult").':</td>';
 		if (getDolGlobalString('ACCOUNTANCY_SHOW_OPENING_BALANCE')) {
 			print '<td></td>';
 		}
@@ -695,8 +695,8 @@ 
 			$accountingResultCredit = price(price2num($accountingResult, 'MT'));
 			$accountingResultClassCSS = ' green';
 		}
-		print '<td class="right nowraponall amount' . $accountingResultClassCSS . '">' . $accountingResultDebit . '</td>';
-		print '<td class="right nowraponall amount' . $accountingResultClassCSS . '">' . $accountingResultCredit . '</td>';
+		print '<td class="right nowraponall amount'.$accountingResultClassCSS.'">'.$accountingResultDebit.'</td>';
+		print '<td class="right nowraponall amount'.$accountingResultClassCSS.'">'.$accountingResultCredit.'</td>';
 
 		print '<td></td>';
 		if (!getDolGlobalString('MAIN_CHECKBOX_LEFT_COLUMN')) {

htdocs/holiday/define_holiday.php

Spacing +3 added lines, -3 removed lines

@@ -47,7 +47,7 @@ 
 $limit = GETPOSTINT('limit') ? GETPOSTINT('limit') : $conf->liste_limit;
 $sortfield = GETPOST('sortfield', 'aZ09comma');
 $sortorder = GETPOST('sortorder', 'aZ09comma');
-$toselect   = GETPOST('toselect', 'array'); // Array of ids of elements selected into a list
+$toselect = GETPOST('toselect', 'array'); // Array of ids of elements selected into a list
 $confirm = GETPOST('confirm', 'alpha');
 
 $page = GETPOSTISSET('pageplusone') ? (GETPOSTINT('pageplusone') - 1) : GETPOSTINT("page");
@@ -265,12 +265,12 @@ 
 	foreach ($typeleaves as $key => $val) {
 		$labeltypes[$val['id']] = ($langs->trans($val['code']) != $val['code']) ? $langs->trans($val['code']) : $langs->trans($val['label']);
 	}
-	$formquestion [] = array( 'type' => 'other',
+	$formquestion [] = array('type' => 'other',
 		'name' => 'typeofholiday',
 		'label' => $langs->trans("Type"),
 		'value' => $form->selectarray('typeholiday', $labeltypes, GETPOST('typeholiday', 'alpha'), 1)
 	);
-	$formquestion [] = array( 'type' => 'other',
+	$formquestion [] = array('type' => 'other',
 		'name' => 'nbdaysholydays',
 		'label' => $langs->trans("NumberDayAddMass"),
 		'value' => '<input name="nbdaysholidays" class="maxwidth75" id="nbdaysholidays" value="'.GETPOSTINT('nbdaysholidays').'">'

dev/tools/phan/config_extended.php

Spacing +52 added lines, -52 removed lines

@@ -82,12 +82,12 @@ 
 	'bom' => 'Bom',
 	'bookcal' => 'BookCal',
 	'bookmark' => 'Bookmark',
-	'cashdesk' => null,  // TODO: fill in proper class
+	'cashdesk' => null, // TODO: fill in proper class
 	'category' => 'Categorie',
 	'clicktodial' => 'ClickToDial',
 	'collab' => 'Collab',
 	'comptabilite' => 'Comptabilite',
-	'contact' => null,  // TODO: fill in proper class
+	'contact' => null, // TODO: fill in proper class
 	'contract' => 'Contrat',
 	'cron' => 'Cron',
 	'datapolicy' => 'DataPolicy',
@@ -99,7 +99,7 @@ 
 	'don' => 'Don',
 	'dynamicprices' => 'DynamicPrices',
 	'ecm' => 'ECM',
-	'ecotax' => null,  // TODO: External module ?
+	'ecotax' => null, // TODO: External module ?
 	'emailcollector' => 'EmailCollector',
 	'eventorganization' => 'EventOrganization',
 	'expensereport' => 'ExpenseReport',
@@ -110,7 +110,7 @@ 
 	'fournisseur' => 'Fournisseur',
 	'ftp' => 'FTP',
 	'geoipmaxmind' => 'GeoIPMaxmind',
-	'google' => null,  // External ?
+	'google' => null, // External ?
 	'gravatar' => 'Gravatar',
 	'holiday' => 'Holiday',
 	'hrm' => 'HRM',
@@ -124,7 +124,7 @@ 
 	'ldap' => 'Ldap',
 	'loan' => 'Loan',
 	'mailing' => 'Mailing',
-	'mailman' => null,  // Same module as mailmanspip -> MailmanSpip ??
+	'mailman' => null, // Same module as mailmanspip -> MailmanSpip ??
 	'mailmanspip' => 'MailmanSpip',
 	'margin' => 'Margin',
 	'member' => 'Adherent',
@@ -137,7 +137,7 @@ 
 	'notification' => 'Notification',
 	'numberwords' => null, // Not provided by default, no module tests
 	'oauth' => 'Oauth',
-	'openstreetmap' => null,  // External module?
+	'openstreetmap' => null, // External module?
 	'opensurvey' => 'OpenSurvey',
 	'order' => 'Commande',
 	'partnership' => 'Partnership',
@@ -164,8 +164,8 @@ 
 	'stock' => 'Stock',
 	'stocktransfer' => 'StockTransfer',
 	'stripe' => 'Stripe',
-	'supplier_invoice' => null,  // Special case, uses invoice
-	'supplier_order' => null,  // Special case, uses invoice
+	'supplier_invoice' => null, // Special case, uses invoice
+	'supplier_order' => null, // Special case, uses invoice
 	'supplier_proposal' => 'SupplierProposal',
 	'syslog' => 'Syslog',
 	'takepos' => 'TakePos',
@@ -196,7 +196,7 @@ 
 	//	'processes' => 6,
 	'backward_compatibility_checks' => false,
 	'simplify_ast' => true,
-	'analyzed_file_extensions' => ['php','inc'],
+	'analyzed_file_extensions' => ['php', 'inc'],
 	'globals_type_map' => [
 		'conf' => '\Conf',
 		'db' => '\DoliDB',
@@ -226,7 +226,7 @@ 
 	// your application should be included in this list.
 	'directory_list' => [
 		'htdocs',
-		PHAN_DIR . '/stubs/',
+		PHAN_DIR.'/stubs/',
 	],
 
 	// A directory list that defines files that will be excluded
@@ -243,7 +243,7 @@ 
 	"exclude_analysis_directory_list" => [
 		'htdocs/includes/',
 		'htdocs/core/class/lessc.class.php', // External library
-		PHAN_DIR . '/stubs/',
+		PHAN_DIR.'/stubs/',
 	],
 	//'exclude_file_regex' => '@^vendor/.*/(tests?|Tests?)/@',
 	'exclude_file_regex' => '@^('  // @phpstan-ignore-line
@@ -254,7 +254,7 @@ 
 		.'|htdocs/includes/restler/.*'  // @phpstan-ignore-line
 		// Included as stub (did not seem properly analysed by phan without it)
 		.'|htdocs/includes/stripe/.*'  // @phpstan-ignore-line
-		.')@',  // @phpstan-ignore-line
+		.')@', // @phpstan-ignore-line
 
 	// A list of plugin files to execute.
 	// Plugins which are bundled with Phan can be added here by providing their name
@@ -275,7 +275,7 @@ 
 	'plugins' => [
 		__DIR__.'/plugins/NoVarDumpPlugin.php',
 		__DIR__.'/plugins/ParamMatchRegexPlugin.php',
-		__DIR__.'/plugins/GetPostFixerPlugin.php',   // Only detects without --automatic-fix
+		__DIR__.'/plugins/GetPostFixerPlugin.php', // Only detects without --automatic-fix
 		'DeprecateAliasPlugin',
 		//'EmptyMethodAndFunctionPlugin',
 		'InvalidVariableIssetPlugin',
@@ -327,24 +327,24 @@ 
 	// Add any issue types (such as 'PhanUndeclaredMethod')
 	// here to inhibit them from being reported
 	'suppress_issue_types' => [
-		'PhanPluginWhitespaceTab',		// Dolibarr used tabs
-		'PhanPluginCanUsePHP71Void',	// Dolibarr is maintaining 7.0 compatibility
-		'PhanPluginShortArray',			// Dolibarr uses array()
-		'PhanPluginShortArrayList',		// Dolibarr uses array()
+		'PhanPluginWhitespaceTab', // Dolibarr used tabs
+		'PhanPluginCanUsePHP71Void', // Dolibarr is maintaining 7.0 compatibility
+		'PhanPluginShortArray', // Dolibarr uses array()
+		'PhanPluginShortArrayList', // Dolibarr uses array()
 		// Fixers From PHPDocToRealTypesPlugin:
-		'PhanPluginCanUseParamType',			// Fixer - Report/Add types in the function definition (function abc(string $var) (adds string)
-		'PhanPluginCanUseReturnType',			// Fixer - Report/Add return types in the function definition (function abc(string $var) (adds string)
-		'PhanPluginCanUseNullableParamType',	// Fixer - Report/Add nullable parameter types in the function definition
-		'PhanPluginCanUseNullableReturnType',	// Fixer - Report/Add nullable return types in the function definition
-		'PhanPluginNonBoolBranch',			// Not essential - 31240+ occurrences
-		'PhanPluginNumericalComparison',	// Not essential - 19870+ occurrences
-		'PhanTypeMismatchArgument',			// Not essential - 12300+ occurrences
-		'PhanPluginNonBoolInLogicalArith',	// Not essential - 11040+ occurrences
-		'PhanPluginConstantVariableScalar',	// Not essential - 5180+ occurrences
+		'PhanPluginCanUseParamType', // Fixer - Report/Add types in the function definition (function abc(string $var) (adds string)
+		'PhanPluginCanUseReturnType', // Fixer - Report/Add return types in the function definition (function abc(string $var) (adds string)
+		'PhanPluginCanUseNullableParamType', // Fixer - Report/Add nullable parameter types in the function definition
+		'PhanPluginCanUseNullableReturnType', // Fixer - Report/Add nullable return types in the function definition
+		'PhanPluginNonBoolBranch', // Not essential - 31240+ occurrences
+		'PhanPluginNumericalComparison', // Not essential - 19870+ occurrences
+		'PhanTypeMismatchArgument', // Not essential - 12300+ occurrences
+		'PhanPluginNonBoolInLogicalArith', // Not essential - 11040+ occurrences
+		'PhanPluginConstantVariableScalar', // Not essential - 5180+ occurrences
 		'PhanPluginDuplicateAdjacentStatement',
-		'PhanPluginDuplicateConditionalTernaryDuplication',		// 2750+ occurrences
-		'PhanPluginDuplicateConditionalNullCoalescing',	// Not essential - 990+ occurrences
-		'PhanPluginRedundantAssignmentInGlobalScope',	// Not essential, a lot of false warning
+		'PhanPluginDuplicateConditionalTernaryDuplication', // 2750+ occurrences
+		'PhanPluginDuplicateConditionalNullCoalescing', // Not essential - 990+ occurrences
+		'PhanPluginRedundantAssignmentInGlobalScope', // Not essential, a lot of false warning
 	],
 	// You can put relative paths to internal stubs in this config option.
 	// Phan will continue using its detailed type annotations,
@@ -363,29 +363,29 @@ 
 	//'xdebug'	=> 'vendor/phan/phan/.phan/internal_stubs/xdebug.phan_php',
 	//'memcached'  => PHAN_DIR . '/your_internal_stubs_folder_name/memcached.phan_php',
 	//'PDO'  => PHAN_DIR . '/stubs/PDO.phan_php',
-	'brotli'  => PHAN_DIR . '/stubs/brotli.phan_php',
-	'curl'  => PHAN_DIR . '/stubs/curl.phan_php',
-	'calendar'  => PHAN_DIR . '/stubs/calendar.phan_php',
-	'fileinfo'  => PHAN_DIR . '/stubs/fileinfo.phan_php',
-	'ftp'  => PHAN_DIR . '/stubs/ftp.phan_php',
-	'gd'  => PHAN_DIR . '/stubs/gd.phan_php',
-	'geoip'  => PHAN_DIR . '/stubs/geoip.phan_php',
-	'imap'  => PHAN_DIR . '/stubs/imap.phan_php',
-	'intl'  => PHAN_DIR . '/stubs/intl.phan_php',
-	'ldap'  => PHAN_DIR . '/stubs/ldap.phan_php',
-	'mcrypt'  => PHAN_DIR . '/stubs/mcrypt.phan_php',
-	'memcache'  => PHAN_DIR . '/stubs/memcache.phan_php',
-	'mysqli'  => PHAN_DIR . '/stubs/mysqli.phan_php',
-	'pdo_cubrid'  => PHAN_DIR . '/stubs/pdo_cubrid.phan_php',
-	'pdo_mysql'  => PHAN_DIR . '/stubs/pdo_mysql.phan_php',
-	'pdo_pgsql'  => PHAN_DIR . '/stubs/pdo_pgsql.phan_php',
-	'pdo_sqlite'  => PHAN_DIR . '/stubs/pdo_sqlite.phan_php',
-	'pgsql'  => PHAN_DIR . '/stubs/pgsql.phan_php',
-	'session'  => PHAN_DIR . '/stubs/session.phan_php',
-	'simplexml'  => PHAN_DIR . '/stubs/SimpleXML.phan_php',
-	'soap'  => PHAN_DIR . '/stubs/soap.phan_php',
-	'sockets'  => PHAN_DIR . '/stubs/sockets.phan_php',
-	'zip'  => PHAN_DIR . '/stubs/zip.phan_php',
+	'brotli'  => PHAN_DIR.'/stubs/brotli.phan_php',
+	'curl'  => PHAN_DIR.'/stubs/curl.phan_php',
+	'calendar'  => PHAN_DIR.'/stubs/calendar.phan_php',
+	'fileinfo'  => PHAN_DIR.'/stubs/fileinfo.phan_php',
+	'ftp'  => PHAN_DIR.'/stubs/ftp.phan_php',
+	'gd'  => PHAN_DIR.'/stubs/gd.phan_php',
+	'geoip'  => PHAN_DIR.'/stubs/geoip.phan_php',
+	'imap'  => PHAN_DIR.'/stubs/imap.phan_php',
+	'intl'  => PHAN_DIR.'/stubs/intl.phan_php',
+	'ldap'  => PHAN_DIR.'/stubs/ldap.phan_php',
+	'mcrypt'  => PHAN_DIR.'/stubs/mcrypt.phan_php',
+	'memcache'  => PHAN_DIR.'/stubs/memcache.phan_php',
+	'mysqli'  => PHAN_DIR.'/stubs/mysqli.phan_php',
+	'pdo_cubrid'  => PHAN_DIR.'/stubs/pdo_cubrid.phan_php',
+	'pdo_mysql'  => PHAN_DIR.'/stubs/pdo_mysql.phan_php',
+	'pdo_pgsql'  => PHAN_DIR.'/stubs/pdo_pgsql.phan_php',
+	'pdo_sqlite'  => PHAN_DIR.'/stubs/pdo_sqlite.phan_php',
+	'pgsql'  => PHAN_DIR.'/stubs/pgsql.phan_php',
+	'session'  => PHAN_DIR.'/stubs/session.phan_php',
+	'simplexml'  => PHAN_DIR.'/stubs/SimpleXML.phan_php',
+	'soap'  => PHAN_DIR.'/stubs/soap.phan_php',
+	'sockets'  => PHAN_DIR.'/stubs/sockets.phan_php',
+	'zip'  => PHAN_DIR.'/stubs/zip.phan_php',
 	],
 
 	];

dev/tools/phan/config_fixer.php

Spacing +44 added lines, -44 removed lines

@@ -40,7 +40,7 @@ 
 	//	'processes' => 6,
 	'backward_compatibility_checks' => false,
 	'simplify_ast' => true,
-	'analyzed_file_extensions' => ['php','inc'],
+	'analyzed_file_extensions' => ['php', 'inc'],
 	'globals_type_map' => [
 		'conf' => '\Conf',
 		'db' => '\DoliDB',
@@ -70,7 +70,7 @@ 
 	// your application should be included in this list.
 	'directory_list' => [
 		'htdocs',
-		PHAN_DIR . '/stubs/',
+		PHAN_DIR.'/stubs/',
 	],
 
 	// A directory list that defines files that will be excluded
@@ -87,7 +87,7 @@ 
 	"exclude_analysis_directory_list" => [
 		'htdocs/includes/',
 		'htdocs/core/class/lessc.class.php', // External library
-		PHAN_DIR . '/stubs/',
+		PHAN_DIR.'/stubs/',
 	],
 	//'exclude_file_regex' => '@^vendor/.*/(tests?|Tests?)/@',
 	'exclude_file_regex' => '@^('  // @phpstan-ignore-line
@@ -100,7 +100,7 @@ 
 		.'|htdocs/includes/stripe/.*'  // @phpstan-ignore-line
 		//.'|htdocs/[^c][^o][^r][^e][^/].*'  // For testing @phpstan-ignore-line
 		//.'|htdocs/[^h].*' // For testing on restricted set @phpstan-ignore-line
-		.')@',  // @phpstan-ignore-line
+		.')@', // @phpstan-ignore-line
 
 	// A list of plugin files to execute.
 	// Plugins which are bundled with Phan can be added here by providing their name
@@ -172,26 +172,26 @@ 
 	// Add any issue types (such as 'PhanUndeclaredMethod')
 	// here to inhibit them from being reported
 	'suppress_issue_types' => [
-		'PhanPluginWhitespaceTab',		// Dolibarr used tabs
-		'PhanPluginCanUsePHP71Void',	// Dolibarr is maintaining 7.0 compatibility
-		'PhanPluginShortArray',			// Dolibarr uses array()
-		'PhanPluginShortArrayList',		// Dolibarr uses array()
+		'PhanPluginWhitespaceTab', // Dolibarr used tabs
+		'PhanPluginCanUsePHP71Void', // Dolibarr is maintaining 7.0 compatibility
+		'PhanPluginShortArray', // Dolibarr uses array()
+		'PhanPluginShortArrayList', // Dolibarr uses array()
 		// The following may require that --quick is not used
 		// Fixers From PHPDocToRealTypesPlugin:
-		'PhanPluginCanUseParamType',	// Fixer - Report/Add types in the function definition (function abc(string $var) (adds string)
-		'PhanPluginCanUseReturnType',	// Fixer - Report/Add return types in the function definition (function abc(string $var) (adds string)
-		'PhanPluginCanUseNullableParamType',	// Fixer - Report/Add nullable parameter types in the function definition
-		'PhanPluginCanUseNullableReturnType',	// Fixer - Report/Add nullable return types in the function definition
-
-		'PhanPluginNonBoolBranch',			// Not essential - 31240+ occurrences
-		'PhanPluginNumericalComparison',	// Not essential - 19870+ occurrences
-		'PhanTypeMismatchArgument',			// Not essential - 12300+ occurrences
-		'PhanPluginNonBoolInLogicalArith',	// Not essential - 11040+ occurrences
-		'PhanPluginConstantVariableScalar',	// Not essential - 5180+ occurrences
+		'PhanPluginCanUseParamType', // Fixer - Report/Add types in the function definition (function abc(string $var) (adds string)
+		'PhanPluginCanUseReturnType', // Fixer - Report/Add return types in the function definition (function abc(string $var) (adds string)
+		'PhanPluginCanUseNullableParamType', // Fixer - Report/Add nullable parameter types in the function definition
+		'PhanPluginCanUseNullableReturnType', // Fixer - Report/Add nullable return types in the function definition
+
+		'PhanPluginNonBoolBranch', // Not essential - 31240+ occurrences
+		'PhanPluginNumericalComparison', // Not essential - 19870+ occurrences
+		'PhanTypeMismatchArgument', // Not essential - 12300+ occurrences
+		'PhanPluginNonBoolInLogicalArith', // Not essential - 11040+ occurrences
+		'PhanPluginConstantVariableScalar', // Not essential - 5180+ occurrences
 		'PhanPluginDuplicateAdjacentStatement',
-		'PhanPluginDuplicateConditionalTernaryDuplication',		// 2750+ occurrences
-		'PhanPluginDuplicateConditionalNullCoalescing',	// Not essential - 990+ occurrences
-		'PhanPluginRedundantAssignmentInGlobalScope',	// Not essential, a lot of false warning
+		'PhanPluginDuplicateConditionalTernaryDuplication', // 2750+ occurrences
+		'PhanPluginDuplicateConditionalNullCoalescing', // Not essential - 990+ occurrences
+		'PhanPluginRedundantAssignmentInGlobalScope', // Not essential, a lot of false warning
 	],
 	// You can put relative paths to internal stubs in this config option.
 	// Phan will continue using its detailed type annotations,
@@ -210,29 +210,29 @@ 
 	//'xdebug'	=> 'vendor/phan/phan/.phan/internal_stubs/xdebug.phan_php',
 	//'memcached'  => PHAN_DIR . '/your_internal_stubs_folder_name/memcached.phan_php',
 	//'PDO'  => PHAN_DIR . '/stubs/PDO.phan_php',
-	'brotli'  => PHAN_DIR . '/stubs/brotli.phan_php',
-	'curl'  => PHAN_DIR . '/stubs/curl.phan_php',
-	'calendar'  => PHAN_DIR . '/stubs/calendar.phan_php',
-	'fileinfo'  => PHAN_DIR . '/stubs/fileinfo.phan_php',
-	'ftp'  => PHAN_DIR . '/stubs/ftp.phan_php',
-	'gd'  => PHAN_DIR . '/stubs/gd.phan_php',
-	'geoip'  => PHAN_DIR . '/stubs/geoip.phan_php',
-	'imap'  => PHAN_DIR . '/stubs/imap.phan_php',
-	'intl'  => PHAN_DIR . '/stubs/intl.phan_php',
-	'ldap'  => PHAN_DIR . '/stubs/ldap.phan_php',
-	'mcrypt'  => PHAN_DIR . '/stubs/mcrypt.phan_php',
-	'memcache'  => PHAN_DIR . '/stubs/memcache.phan_php',
-	'mysqli'  => PHAN_DIR . '/stubs/mysqli.phan_php',
-	'pdo_cubrid'  => PHAN_DIR . '/stubs/pdo_cubrid.phan_php',
-	'pdo_mysql'  => PHAN_DIR . '/stubs/pdo_mysql.phan_php',
-	'pdo_pgsql'  => PHAN_DIR . '/stubs/pdo_pgsql.phan_php',
-	'pdo_sqlite'  => PHAN_DIR . '/stubs/pdo_sqlite.phan_php',
-	'pgsql'  => PHAN_DIR . '/stubs/pgsql.phan_php',
-	'session'  => PHAN_DIR . '/stubs/session.phan_php',
-	'simplexml'  => PHAN_DIR . '/stubs/SimpleXML.phan_php',
-	'soap'  => PHAN_DIR . '/stubs/soap.phan_php',
-	'sockets'  => PHAN_DIR . '/stubs/sockets.phan_php',
-	'zip'  => PHAN_DIR . '/stubs/zip.phan_php',
+	'brotli'  => PHAN_DIR.'/stubs/brotli.phan_php',
+	'curl'  => PHAN_DIR.'/stubs/curl.phan_php',
+	'calendar'  => PHAN_DIR.'/stubs/calendar.phan_php',
+	'fileinfo'  => PHAN_DIR.'/stubs/fileinfo.phan_php',
+	'ftp'  => PHAN_DIR.'/stubs/ftp.phan_php',
+	'gd'  => PHAN_DIR.'/stubs/gd.phan_php',
+	'geoip'  => PHAN_DIR.'/stubs/geoip.phan_php',
+	'imap'  => PHAN_DIR.'/stubs/imap.phan_php',
+	'intl'  => PHAN_DIR.'/stubs/intl.phan_php',
+	'ldap'  => PHAN_DIR.'/stubs/ldap.phan_php',
+	'mcrypt'  => PHAN_DIR.'/stubs/mcrypt.phan_php',
+	'memcache'  => PHAN_DIR.'/stubs/memcache.phan_php',
+	'mysqli'  => PHAN_DIR.'/stubs/mysqli.phan_php',
+	'pdo_cubrid'  => PHAN_DIR.'/stubs/pdo_cubrid.phan_php',
+	'pdo_mysql'  => PHAN_DIR.'/stubs/pdo_mysql.phan_php',
+	'pdo_pgsql'  => PHAN_DIR.'/stubs/pdo_pgsql.phan_php',
+	'pdo_sqlite'  => PHAN_DIR.'/stubs/pdo_sqlite.phan_php',
+	'pgsql'  => PHAN_DIR.'/stubs/pgsql.phan_php',
+	'session'  => PHAN_DIR.'/stubs/session.phan_php',
+	'simplexml'  => PHAN_DIR.'/stubs/SimpleXML.phan_php',
+	'soap'  => PHAN_DIR.'/stubs/soap.phan_php',
+	'sockets'  => PHAN_DIR.'/stubs/sockets.phan_php',
+	'zip'  => PHAN_DIR.'/stubs/zip.phan_php',
 	],
 
 	];