Dolibarr / dolibarr

htdocs/core/get_menudiv.php

Spacing +1 added lines, -1 removed lines

@@ -84,7 +84,7 @@
 $title = $langs->trans("Menu");
 
 // URL http://mydolibarr/core/get_menudiv.php?dol_use_jmobile=1 can be used for tests
-$head = '<!-- Menu -->'."\n";	// This is used by DoliDroid to know page is a menu page
+$head = '<!-- Menu -->'."\n"; // This is used by DoliDroid to know page is a menu page
 $arrayofjs = array();
 $arrayofcss = array();
 top_htmlhead($head, $title, 0, 0, $arrayofjs, $arrayofcss);

htdocs/core/search_page.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
 $title = $langs->trans("Search");
 
 // URL http://mydolibarr/core/search_page?dol_use_jmobile=1 can be used for tests
-$head = '<!-- Quick access -->'."\n";	// This is used by DoliDroid to know page is a search page
+$head = '<!-- Quick access -->'."\n"; // This is used by DoliDroid to know page is a search page
 $arrayofjs = array();
 $arrayofcss = array();
 top_htmlhead($head, $title, 0, 0, $arrayofjs, $arrayofcss);

htdocs/accountancy/closure/index.php

Spacing +4 added lines, -4 removed lines

@@ -87,9 +87,9 @@ 
 	// Specify as export : update field date_validated on selected month/year
 	$sql = " UPDATE ".MAIN_DB_PREFIX."accounting_bookkeeping";
 	$sql .= " SET date_validated = '".$db->idate($now)."'";
-	$sql .= " WHERE entity = " . ((int) $conf->entity);
-	$sql .= " AND doc_date >= '" . $db->idate($date_start) . "'";
-	$sql .= " AND doc_date <= '" . $db->idate($date_end) . "'";
+	$sql .= " WHERE entity = ".((int) $conf->entity);
+	$sql .= " AND doc_date >= '".$db->idate($date_start)."'";
+	$sql .= " AND doc_date <= '".$db->idate($date_end)."'";
 	$sql .= " AND date_validated IS NULL";
 
 	dol_syslog("/accountancy/closure/index.php action=validate_movement_confirm -> Set movements as validated", LOG_DEBUG);
@@ -122,7 +122,7 @@ 
 
 $title = $langs->trans('Closure');
 
-$help_url ='EN:Module_Double_Entry_Accounting';
+$help_url = 'EN:Module_Double_Entry_Accounting';
 
 llxHeader('', $title, $help_url);
 

htdocs/accountancy/bookkeeping/balance.php

Spacing +12 added lines, -12 removed lines

@@ -71,7 +71,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('balancelist'));  // Note that conf->hooks_modules contains array
+$hookmanager->initHooks(array('balancelist')); // Note that conf->hooks_modules contains array
 
 $formaccounting = new FormAccounting($db);
 $formother = new FormOther($db);
@@ -343,9 +343,9 @@ 
 	// without time and class limits (Class 6 and 7 accounts ???) and does not take into account the "a-nouveau" journal.
 	if (!empty($conf->global->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);
@@ -353,7 +353,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'];
 		}
 	}
 
@@ -383,9 +383,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.
@@ -417,7 +417,7 @@ 
 
 				// Show first line of a break
 				print '<tr class="trforbreak">';
-				print '<td colspan="'.($colspan+1).'" class="tdforbreak">'.$line->numero_compte.($root_account_description ? ' - '.$root_account_description : '').'</td>';
+				print '<td colspan="'.($colspan + 1).'" class="tdforbreak">'.$line->numero_compte.($root_account_description ? ' - '.$root_account_description : '').'</td>';
 				print '</tr>';
 
 				$displayed_account = $root_account_number;
@@ -472,9 +472,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 (!empty($conf->global->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>';
 		}
 		print "<td></td>\n";
 		print '</tr>';
@@ -487,9 +487,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 (!empty($conf->global->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>';
 	}
 	print "<td></td>\n";
 	print '</tr>';

htdocs/holiday/month_report.php

Spacing +1 added lines, -1 removed lines

@@ -149,7 +149,7 @@
 $sql .= " AND cp.statut = ".Holiday::STATUS_APPROVED;
 $sql .= " AND (";
 $sql .= " (date_format(cp.date_debut, '%Y-%m') = '".$db->escape($year_month)."' OR date_format(cp.date_fin, '%Y-%m') = '".$db->escape($year_month)."')";
-$sql .= " OR";	// For leave over several months
+$sql .= " OR"; // For leave over several months
 $sql .= " (date_format(cp.date_debut, '%Y-%m') < '".$db->escape($year_month)."' AND date_format(cp.date_fin, '%Y-%m') > '".$db->escape($year_month)."') ";
 $sql .= " )";
 if (!empty($search_ref)) {

htdocs/core/modules/bank/doc/pdf_sepamandate.modules.php

Spacing +3 added lines, -3 removed lines

@@ -244,7 +244,7 @@ 
 				if ($idbankfordirectdebit > 0) {
 					$tmpbankfordirectdebit = new Account($this->db);
 					$tmpbankfordirectdebit->fetch($idbankfordirectdebit);
-					$ics = $tmpbankfordirectdebit->ics;	// ICS for direct debit
+					$ics = $tmpbankfordirectdebit->ics; // ICS for direct debit
 				}
 				if (empty($ics) && !empty($conf->global->PRELEVEMENT_ICS)) {
 					$ics = $conf->global->PRELEVEMENT_ICS;
@@ -313,9 +313,9 @@ 
 
 				$address = '______________________________________________';
 				if ($thirdparty->id > 0) {
-					$tmpaddresswithoutcountry = $thirdparty->getFullAddress();	// we test on address without country
+					$tmpaddresswithoutcountry = $thirdparty->getFullAddress(); // we test on address without country
 					if ($tmpaddresswithoutcountry) {
-						$address = $thirdparty->getFullAddress(1);	// full address
+						$address = $thirdparty->getFullAddress(1); // full address
 					}
 				}
 				$posY = $pdf->GetY();

htdocs/core/modules/product/doc/pdf_standard.modules.php

Spacing +6 added lines, -6 removed lines

@@ -294,20 +294,20 @@
 					$nexY = $pdf->GetY();
 				}
 				if ($object->length) {
-					$texttoshow = $langs->trans("Length") . ' x ' . $langs->trans("Width") . ' x ' . $langs->trans("Height") . ': ' . ($object->length != '' ? $object->length : '?') . ' x ' . ($object->width != '' ? $object->width : '?') . ' x ' . ($object->height != '' ? $object->height : '?');
-					$texttoshow .= ' ' . measuringUnitString(0, "size", $object->length_units);
+					$texttoshow = $langs->trans("Length").' x '.$langs->trans("Width").' x '.$langs->trans("Height").': '.($object->length != '' ? $object->length : '?').' x '.($object->width != '' ? $object->width : '?').' x '.($object->height != '' ? $object->height : '?');
+					$texttoshow .= ' '.measuringUnitString(0, "size", $object->length_units);
 					$pdf->writeHTMLCell(190, 3, $this->marge_gauche, $nexY, $texttoshow, 0, 1);
 					$nexY = $pdf->GetY();
 				}
 				if ($object->surface) {
-					$texttoshow = $langs->trans("Surface") . ': ' . dol_htmlentitiesbr($object->surface);
-					$texttoshow .= ' ' . measuringUnitString(0, "surface", $object->surface_units);
+					$texttoshow = $langs->trans("Surface").': '.dol_htmlentitiesbr($object->surface);
+					$texttoshow .= ' '.measuringUnitString(0, "surface", $object->surface_units);
 					$pdf->writeHTMLCell(190, 3, $this->marge_gauche, $nexY, $texttoshow, 0, 1);
 					$nexY = $pdf->GetY();
 				}
 				if ($object->volume) {
-					$texttoshow = $langs->trans("Volume") . ': ' . dol_htmlentitiesbr($object->volume);
-					$texttoshow .= ' ' . measuringUnitString(0, "volume", $object->volume_units);
+					$texttoshow = $langs->trans("Volume").': '.dol_htmlentitiesbr($object->volume);
+					$texttoshow .= ' '.measuringUnitString(0, "volume", $object->volume_units);
 					$pdf->writeHTMLCell(190, 3, $this->marge_gauche, $nexY, $texttoshow, 0, 1);
 					$nexY = $pdf->GetY();
 				}

htdocs/core/modules/security/generate/modGeneratePassPerso.class.php

Spacing +3 added lines, -3 removed lines

@@ -192,7 +192,7 @@ 
 			return $pass;
 		}
 
-		return $this->getNewGeneratedPassword();	// warning, may generate infinite loop if conditions are not possible
+		return $this->getNewGeneratedPassword(); // warning, may generate infinite loop if conditions are not possible
 	}
 
 	/**
@@ -206,11 +206,11 @@ 
 	{
 		global $langs;
 
-		$this->initAll();	// For the case this method is called alone
+		$this->initAll(); // For the case this method is called alone
 
 		$password_a = preg_split('//u', $password, null, PREG_SPLIT_NO_EMPTY);
 		$maj = preg_split('//u', $this->Maj, null, PREG_SPLIT_NO_EMPTY);
-		$num = preg_split('//u', $this->Nb, null, PREG_SPLIT_NO_EMPTY);;
+		$num = preg_split('//u', $this->Nb, null, PREG_SPLIT_NO_EMPTY); ;
 		$spe = preg_split('//u', $this->Spe, null, PREG_SPLIT_NO_EMPTY);
 		/*
 		$password_a = str_split($password);

htdocs/eventorganization/lib/eventorganization_conferenceorbooth.lib.php

Spacing +9 added lines, -9 removed lines

@@ -37,8 +37,8 @@ 
 	$h = 0;
 	$head = array();
 
-	$withProjectUrl='';
-	if ($with_project>0) {
+	$withProjectUrl = '';
+	if ($with_project > 0) {
 		$withProjectUrl = "&withproject=1";
 	}
 
@@ -131,7 +131,7 @@ 
 	$head[$h][2] = 'conferenceorbooth';
 	// Enable caching of conf or booth count attendees
 	$nbAttendees = 0;
-	$nbConferenceOrBooth= 0;
+	$nbConferenceOrBooth = 0;
 	require_once DOL_DOCUMENT_ROOT.'/core/lib/memory.lib.php';
 	$cachekey = 'count_conferenceorbooth_project_'.$object->id;
 	$dataretrieved = dol_getcache($cachekey);
@@ -139,14 +139,14 @@ 
 		$nbAttendees = $dataretrieved;
 	} else {
 		require_once DOL_DOCUMENT_ROOT.'/eventorganization/class/conferenceorbooth.class.php';
-		$conforbooth=new ConferenceOrBooth($db);
+		$conforbooth = new ConferenceOrBooth($db);
 		$result = $conforbooth->fetchAll('', '', 0, 0, array('t.fk_project'=>$object->id));
-		if (!is_array($result) && $result<0) {
+		if (!is_array($result) && $result < 0) {
 			setEventMessages($conforbooth->error, $conforbooth->errors, 'errors');
 		} else {
 			$nbConferenceOrBooth = count($result);
 		}
-		dol_setcache($cachekey, $nbConferenceOrBooth, 120);	// If setting cache fails, this is not a problem, so we do not test result.
+		dol_setcache($cachekey, $nbConferenceOrBooth, 120); // If setting cache fails, this is not a problem, so we do not test result.
 	}
 	if ($nbConferenceOrBooth > 0) {
 		$head[$h][1] .= '<span class="badge marginleftonlyshort">'.$nbConferenceOrBooth.'</span>';
@@ -165,14 +165,14 @@ 
 		$nbAttendees = $dataretrieved;
 	} else {
 		require_once DOL_DOCUMENT_ROOT.'/eventorganization/class/conferenceorboothattendee.class.php';
-		$attendees=new ConferenceOrBoothAttendee($db);
+		$attendees = new ConferenceOrBoothAttendee($db);
 		$result = $attendees->fetchAll('', '', 0, 0, array('t.fk_project'=>$object->id));
-		if (!is_array($result) && $result<0) {
+		if (!is_array($result) && $result < 0) {
 			setEventMessages($attendees->error, $attendees->errors, 'errors');
 		} else {
 			$nbAttendees = count($result);
 		}
-		dol_setcache($cachekey, $nbAttendees, 120);	// If setting cache fails, this is not a problem, so we do not test result.
+		dol_setcache($cachekey, $nbAttendees, 120); // If setting cache fails, this is not a problem, so we do not test result.
 	}
 	if ($nbAttendees > 0) {
 		$head[$h][1] .= '<span class="badge marginleftonlyshort">'.$nbAttendees.'</span>';