Dolibarr / dolibarr

htdocs/core/boxes/box_birthdays.php

Spacing +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
 			$sql .= " AND u.statut = ".User::STATUS_ENABLED;
 			$sql .= dolSqlDateFilter('u.dateemployment', 0, $tmparray['mon'], 0);
 			$sql .= " AND u.dateemployment < '".$this->db->idate(dol_get_first_day($tmparray['year']))."'";
-			$sql .= " ORDER BY daya ASC";	// We want to have date of the month sorted by the day without taking into consideration the year
+			$sql .= " ORDER BY daya ASC"; // We want to have date of the month sorted by the day without taking into consideration the year
 			$sql .= $this->db->plimit($max, 0);
 
 			dol_syslog(get_class($this)."::loadBox", LOG_DEBUG);

htdocs/core/login/functions_dolibarr.php

Spacing +2 added lines, -2 removed lines

@@ -160,7 +160,7 @@ 
 					}
 				} else {
 					dol_syslog("functions_dolibarr::check_user_password_dolibarr Authentication KO user not found for '".$usertotest."'", LOG_NOTICE);
-					sleep(1);	// Anti brut force protection. Must be same delay when password is not valid
+					sleep(1); // Anti brut force protection. Must be same delay when password is not valid
 
 					// Load translation files required by the page
 					$langs->loadLangs(array('main', 'errors'));
@@ -174,7 +174,7 @@ 
 			}
 		} else {
 			dol_syslog("functions_dolibarr::check_user_password_dolibarr Authentication KO Too many attempts", LOG_NOTICE);
-			sleep(1);	// Anti brut force protection. Must be same delay when password is not valid
+			sleep(1); // Anti brut force protection. Must be same delay when password is not valid
 			// Load translation files required by the page
 			$langs->loadLangs(array('main', 'errors'));
 			$_SESSION["dol_loginmesg"] = $langs->transnoentitiesnoconv("ErrorTooManyAttempts");

htdocs/core/login/functions_googleoauth.php

Spacing +3 added lines, -3 removed lines

@@ -75,12 +75,12 @@
 			//global $dolibarr_main_url_root;
 			//$urlwithouturlroot = preg_replace('/'.preg_quote(DOL_URL_ROOT, '/').'$/i', '', trim($dolibarr_main_url_root));
 			//$urlwithroot = $urlwithouturlroot.DOL_URL_ROOT; // This is to use external domain name found into config file
-			$urlwithroot=DOL_MAIN_URL_ROOT;					// This is to use same domain name than current
+			$urlwithroot = DOL_MAIN_URL_ROOT; // This is to use same domain name than current
 
 			//$shortscope = 'userinfo_email,userinfo_profile';
-			$shortscope = 'openid,email,profile';	// For openid connect
+			$shortscope = 'openid,email,profile'; // For openid connect
 
-			$oauthstateanticsrf = bin2hex(random_bytes(128/8));
+			$oauthstateanticsrf = bin2hex(random_bytes(128 / 8));
 			$_SESSION['oauthstateanticsrf'] = $shortscope.'-'.$oauthstateanticsrf;
 
 			$url = $urlwithroot.'/core/modules/oauth/google_oauthcallback.php?shortscope='.urlencode($shortscope).'&state='.urlencode('forlogin-'.$shortscope.'-'.$oauthstateanticsrf).'&username='.urlencode($usertotest);

htdocs/asset/document.php

Spacing +6 added lines, -6 removed lines

@@ -117,7 +117,7 @@ 
 
 	// Object card
 	// ------------------------------------------------------------
-	$linkback = '<a href="' . dol_buildpath('/asset/asset_list.php', 1) . '?restore_lastsearch_values=1' . (!empty($socid) ? '&socid=' . $socid : '') . '">' . $langs->trans("BackToList") . '</a>';
+	$linkback = '<a href="'.dol_buildpath('/asset/asset_list.php', 1).'?restore_lastsearch_values=1'.(!empty($socid) ? '&socid='.$socid : '').'">'.$langs->trans("BackToList").'</a>';
 
 	$morehtmlref = '<div class="refidno">';
 	$morehtmlref .= '</div>';
@@ -130,10 +130,10 @@ 
 	print '<table class="border centpercent tableforfield">';
 
 	// Number of files
-	print '<tr><td class="titlefield">' . $langs->trans("NbOfAttachedFiles") . '</td><td colspan="3">' . count($filearray) . '</td></tr>';
+	print '<tr><td class="titlefield">'.$langs->trans("NbOfAttachedFiles").'</td><td colspan="3">'.count($filearray).'</td></tr>';
 
 	// Total size
-	print '<tr><td>' . $langs->trans("TotalSizeOfAttachedFiles") . '</td><td colspan="3">' . $totalsize . ' ' . $langs->trans("bytes") . '</td></tr>';
+	print '<tr><td>'.$langs->trans("TotalSizeOfAttachedFiles").'</td><td colspan="3">'.$totalsize.' '.$langs->trans("bytes").'</td></tr>';
 
 	print '</table>';
 
@@ -146,12 +146,12 @@ 
 	//  $permissiontoadd = 1;
 	$permtoedit = $user->hasRight('asset', 'write');
 	//  $permtoedit = 1;
-	$param = '&id=' . $object->id;
+	$param = '&id='.$object->id;
 
 	//$relativepathwithnofile='asset/' . dol_sanitizeFileName($object->id).'/';
-	$relativepathwithnofile = dol_sanitizeFileName($object->ref) . '/';
+	$relativepathwithnofile = dol_sanitizeFileName($object->ref).'/';
 
-	include DOL_DOCUMENT_ROOT . '/core/tpl/document_actions_post_headers.tpl.php';
+	include DOL_DOCUMENT_ROOT.'/core/tpl/document_actions_post_headers.tpl.php';
 } else {
 	accessforbidden('', 0, 1);
 }

htdocs/don/payment/card.php

Spacing +1 added lines, -1 removed lines

@@ -216,7 +216,7 @@
 		if (!$disable_delete) {
 			print dolGetButtonAction($langs->trans('Delete'), '', 'delete', $_SERVER["PHP_SELF"].'?id='.$object->id.'&action=delete&token='.newToken(), '', 1);
 		} else {
-			print dolGetButtonAction($langs->trans("CantRemovePaymentWithOneInvoicePaid"), $langs->trans('Delete'), '', $_SERVER["PHP_SELF"].'?id='.$object->id.'#', '', 1, [ 'attr' => ['classOverride' => 'butActionRefused']]);
+			print dolGetButtonAction($langs->trans("CantRemovePaymentWithOneInvoicePaid"), $langs->trans('Delete'), '', $_SERVER["PHP_SELF"].'?id='.$object->id.'#', '', 1, ['attr' => ['classOverride' => 'butActionRefused']]);
 		}
 	}
 }

htdocs/accountancy/tpl/export_journal.tpl.php

Spacing +1 added lines, -1 removed lines

@@ -72,5 +72,5 @@
 }
 
 if (empty($downloadMode)) {
-	header('Content-Disposition: attachment;filename=' . $completefilename);
+	header('Content-Disposition: attachment;filename='.$completefilename);
 }

htdocs/accountancy/admin/card.php

Spacing +14 added lines, -14 removed lines

@@ -76,9 +76,9 @@ 
 			setEventMessages($langs->trans("ErrorFieldRequired", $langs->transnoentities("Label")), null, 'errors');
 			$action = 'create';
 		} else {
-			$sql = "SELECT pcg_version FROM " . MAIN_DB_PREFIX . "accounting_system WHERE rowid = ".((int) getDolGlobalInt('CHARTOFACCOUNTS'));
+			$sql = "SELECT pcg_version FROM ".MAIN_DB_PREFIX."accounting_system WHERE rowid = ".((int) getDolGlobalInt('CHARTOFACCOUNTS'));
 
-			dol_syslog('accountancy/admin/card.php:: $sql=' . $sql);
+			dol_syslog('accountancy/admin/card.php:: $sql='.$sql);
 			$result = $db->query($sql);
 			$obj = $db->fetch_object($result);
 
@@ -123,7 +123,7 @@ 
 			if (!$error) {
 				setEventMessages("RecordCreatedSuccessfully", null, 'mesgs');
 				$urltogo = $backtopage ? $backtopage : DOL_URL_ROOT.'/accountancy/admin/account.php';
-				header("Location: " . $urltogo);
+				header("Location: ".$urltogo);
 				exit;
 			}
 		}
@@ -141,7 +141,7 @@ 
 
 			$sql = "SELECT pcg_version FROM ".MAIN_DB_PREFIX."accounting_system WHERE rowid=".((int) getDolGlobalInt('CHARTOFACCOUNTS'));
 
-			dol_syslog('accountancy/admin/card.php:: $sql=' . $sql);
+			dol_syslog('accountancy/admin/card.php:: $sql='.$sql);
 			$result2 = $db->query($sql);
 			$obj = $db->fetch_object($result2);
 
@@ -171,8 +171,8 @@ 
 			$result = $object->update($user);
 
 			if ($result > 0) {
-				$urltogo = $backtopage ? $backtopage : ($_SERVER["PHP_SELF"] . "?id=" . $id);
-				header("Location: " . $urltogo);
+				$urltogo = $backtopage ? $backtopage : ($_SERVER["PHP_SELF"]."?id=".$id);
+				header("Location: ".$urltogo);
 				exit();
 			} elseif ($result == -2) {
 				setEventMessages($langs->trans("ErrorAccountNumberAlreadyExists", $object->account_number), null, 'errors');
@@ -264,14 +264,14 @@ 
 	print '<input type="text" name="pcg_type" list="pcg_type_datalist" value="'.dol_escape_htmltag(GETPOSTISSET('pcg_type') ? GETPOST('pcg_type', 'alpha') : $object->pcg_type).'">';
 	// autosuggest from existing account types if found
 	print '<datalist id="pcg_type_datalist">';
-	$sql = "SELECT DISTINCT pcg_type FROM " . MAIN_DB_PREFIX . "accounting_account";
-	$sql .= " WHERE fk_pcg_version = '" . $db->escape($accountsystem->ref) . "'";
-	$sql .= ' AND entity in ('.getEntity('accounting_account', 0).')';		// Always limit to current entity. No sharing in accountancy.
+	$sql = "SELECT DISTINCT pcg_type FROM ".MAIN_DB_PREFIX."accounting_account";
+	$sql .= " WHERE fk_pcg_version = '".$db->escape($accountsystem->ref)."'";
+	$sql .= ' AND entity in ('.getEntity('accounting_account', 0).')'; // Always limit to current entity. No sharing in accountancy.
 	$sql .= ' LIMIT 50000'; // just as a sanity check
 	$resql = $db->query($sql);
 	if ($resql) {
 		while ($obj = $db->fetch_object($resql)) {
-			print '<option value="' . dol_escape_htmltag($obj->pcg_type) . '">';
+			print '<option value="'.dol_escape_htmltag($obj->pcg_type).'">';
 		}
 	}
 	print '</datalist>';
@@ -341,14 +341,14 @@ 
 			print '<input type="text" name="pcg_type" list="pcg_type_datalist" value="'.dol_escape_htmltag(GETPOSTISSET('pcg_type') ? GETPOST('pcg_type', 'alpha') : $object->pcg_type).'">';
 			// autosuggest from existing account types if found
 			print '<datalist id="pcg_type_datalist">';
-			$sql = 'SELECT DISTINCT pcg_type FROM ' . MAIN_DB_PREFIX . 'accounting_account';
-			$sql .= " WHERE fk_pcg_version = '" . $db->escape($accountsystem->ref) . "'";
-			$sql .= ' AND entity in ('.getEntity('accounting_account', 0).')';		// Always limit to current entity. No sharing in accountancy.
+			$sql = 'SELECT DISTINCT pcg_type FROM '.MAIN_DB_PREFIX.'accounting_account';
+			$sql .= " WHERE fk_pcg_version = '".$db->escape($accountsystem->ref)."'";
+			$sql .= ' AND entity in ('.getEntity('accounting_account', 0).')'; // Always limit to current entity. No sharing in accountancy.
 			$sql .= ' LIMIT 50000'; // just as a sanity check
 			$resql = $db->query($sql);
 			if ($resql) {
 				while ($obj = $db->fetch_object($resql)) {
-					print '<option value="' . dol_escape_htmltag($obj->pcg_type) . '">';
+					print '<option value="'.dol_escape_htmltag($obj->pcg_type).'">';
 				}
 			}
 			print '</datalist>';

htdocs/compta/tva/card.php

Spacing +5 added lines, -5 removed lines

@@ -252,7 +252,7 @@ 
 				}
 			}
 			if (empty($error)) {
-				header("Location: card.php?id=" . $object->id);
+				header("Location: card.php?id=".$object->id);
 				exit;
 			}
 		}
@@ -307,7 +307,7 @@ 
 		} else {
 			$result = $object->fetch($id);
 
-			$object->amount	= $amount;
+			$object->amount = $amount;
 
 			$result = $object->update($user);
 			if ($result <= 0) {
@@ -469,7 +469,7 @@ 
 
 	// Auto create payment
 	print '<tr><td><label for="auto_create_paiement">'.$langs->trans('AutomaticCreationPayment').'</label></td>';
-	print '<td><input id="auto_create_paiement" name="auto_create_paiement" type="checkbox" ' . (empty($auto_create_payment) ? '' : 'checked="checked"') . ' value="1"></td></tr>'."\n";
+	print '<td><input id="auto_create_paiement" name="auto_create_paiement" type="checkbox" '.(empty($auto_create_payment) ? '' : 'checked="checked"').' value="1"></td></tr>'."\n";
 
 	print '<tr class="hide_if_no_auto_create_payment">';
 	print '<td class="fieldrequired">'.$langs->trans("DatePayment").'</td><td>';
@@ -609,9 +609,9 @@ 
 	print '</td></tr>';
 
 	if ($action == 'edit') {
-		print '<tr><td class="fieldrequired">' . $langs->trans("Amount") . '</td><td><input name="amount" size="10" value="' . price($object->amount) . '"></td></tr>';
+		print '<tr><td class="fieldrequired">'.$langs->trans("Amount").'</td><td><input name="amount" size="10" value="'.price($object->amount).'"></td></tr>';
 	} else {
-		print '<tr><td>' . $langs->trans("Amount") . '</td><td>' . price($object->amount) . '</td></tr>';
+		print '<tr><td>'.$langs->trans("Amount").'</td><td>'.price($object->amount).'</td></tr>';
 	}
 
 	// Mode of payment

htdocs/compta/bank/class/paymentvarious.class.php

Spacing +2 added lines, -2 removed lines

@@ -306,7 +306,7 @@ 
 				$this->type_payment         = $obj->fk_typepayment;
 				$this->num_payment          = $obj->num_payment;
 				$this->label                = $obj->label;
-				$this->note                 = $obj->note_private;	// For backward compatibility
+				$this->note                 = $obj->note_private; // For backward compatibility
 				$this->note_private         = $obj->note_private;
 				$this->subledger_account    = $obj->subledger_account;
 				$this->accountancy_code     = $obj->accountancy_code;
@@ -486,7 +486,7 @@ 
 		$sql .= ", ".($this->fk_project > 0 ? ((int) $this->fk_project) : 0);
 		$sql .= ", ".((int) $user->id);
 		$sql .= ", '".$this->db->idate($now)."'";
-		$sql .= ", NULL";	// Filled later
+		$sql .= ", NULL"; // Filled later
 		$sql .= ", ".((int) $conf->entity);
 		$sql .= ")";