Dolibarr / dolibarr

htdocs/core/tpl/resource_view.tpl.php

Braces +3 added lines, -2 removed lines

@@ -51,8 +51,9 @@
 			print '</div>';
 		} else {
 			$class = '';
-			if ($linked_resource['rowid'] == GETPOST('lineid'))
-				$class = 'highlight';
+			if ($linked_resource['rowid'] == GETPOST('lineid')) {
+							$class = 'highlight';
+			}
 
 			print '<div class="tagtr oddeven'.($class ? ' '.$class : '').'">';
 

htdocs/core/tpl/massactions_pre.tpl.php

Braces +12 added lines, -6 removed lines

@@ -54,10 +54,12 @@ 
 			{
 				$listofselectedid[$toselectid] = $toselectid;
 				$thirdpartyid = ($objecttmp->fk_soc ? $objecttmp->fk_soc : $objecttmp->socid);
-				if ($objecttmp->element == 'societe')
-					$thirdpartyid = $objecttmp->id;
-				if ($objecttmp->element == 'expensereport')
-					$thirdpartyid = $objecttmp->fk_user_author;
+				if ($objecttmp->element == 'societe') {
+									$thirdpartyid = $objecttmp->id;
+				}
+				if ($objecttmp->element == 'expensereport') {
+									$thirdpartyid = $objecttmp->fk_user_author;
+				}
 				$listofselectedthirdparties[$thirdpartyid] = $thirdpartyid;
 				$listofselectedref[$thirdpartyid][$toselectid] = $objecttmp->ref;
 			}
@@ -82,16 +84,20 @@ 
 		$formmail->fromid = $user->id;
 	}
 	$formmail->trackid = $trackid;
-	if (!empty($conf->global->MAIN_EMAIL_ADD_TRACK_ID) && ($conf->global->MAIN_EMAIL_ADD_TRACK_ID & 2)) // If bit 2 is set
+	if (!empty($conf->global->MAIN_EMAIL_ADD_TRACK_ID) && ($conf->global->MAIN_EMAIL_ADD_TRACK_ID & 2)) {
+		// If bit 2 is set
 	{
 		include DOL_DOCUMENT_ROOT.'/core/lib/functions2.lib.php';
+	}
 		$formmail->frommail = dolAddEmailTrackId($formmail->frommail, $trackid);
 	}
 	$formmail->withfrom = 1;
 	$liste = $langs->trans("AllRecipientSelected", count($arrayofselected));
-	if (count($listofselectedthirdparties) == 1) // Only 1 different recipient selected, we can suggest contacts
+	if (count($listofselectedthirdparties) == 1) {
+		// Only 1 different recipient selected, we can suggest contacts
 	{
 		$liste = array();
+	}
 		$thirdpartyid = array_shift($listofselectedthirdparties);
 		if ($objecttmp->element == 'expensereport')
 		{

htdocs/core/tpl/notes.tpl.php

Braces +17 added lines, -6 removed lines

@@ -41,8 +41,11 @@ 
 	if (GETPOST('action', 'aZ09') == 'edit'.$note_public)
 	{
 		$value_public = dol_concatdesc($value_public, ($value_public ? "\n" : "")."-- ".$stringtoadd);
-		if (dol_textishtml($value_public)) $value_public .= "<br>\n";
-		else $value_public .= "\n";
+		if (dol_textishtml($value_public)) {
+			$value_public .= "<br>\n";
+		} else {
+			$value_public .= "\n";
+		}
 	}
 }
 if (!empty($conf->global->MAIN_AUTO_TIMESTAMP_IN_PRIVATE_NOTES))
@@ -51,8 +54,11 @@ 
 	if (GETPOST('action', 'aZ09') == 'edit'.$note_private)
 	{
 		$value_private = dol_concatdesc($value_private, ($value_private ? "\n" : "")."-- ".$stringtoadd);
-		if (dol_textishtml($value_private)) $value_private .= "<br>\n";
-		else $value_private .= "\n";
+		if (dol_textishtml($value_private)) {
+			$value_private .= "<br>\n";
+		} else {
+			$value_private .= "\n";
+		}
 	}
 }
 
@@ -82,8 +88,13 @@ 
 }
 //else dol_print_error('','Bad value '.$module.' for param module');
 
-if (!empty($conf->fckeditor->enabled) && !empty($conf->global->FCKEDITOR_ENABLE_SOCIETE)) $typeofdata = 'ckeditor:dolibarr_notes:100%:200::1:12:95%:0'; // Rem: This var is for all notes, not only thirdparties note.
-else $typeofdata = 'textarea:12:95%';
+if (!empty($conf->fckeditor->enabled) && !empty($conf->global->FCKEDITOR_ENABLE_SOCIETE)) {
+	$typeofdata = 'ckeditor:dolibarr_notes:100%:200::1:12:95%:0';
+}
+// Rem: This var is for all notes, not only thirdparties note.
+else {
+	$typeofdata = 'textarea:12:95%';
+}
 
 print '<!-- BEGIN PHP TEMPLATE NOTES -->'."\n";
 print '<div class="tagtable border table-border tableforfield centpercent">'."\n";

htdocs/core/tpl/admin_extrafields_view.tpl.php

Braces +6 added lines, -2 removed lines

@@ -102,7 +102,9 @@ 
 				print $langs->trans("All");
 			} else {
 				global $multicompanylabel_cache;
-				if (!is_array($multicompanylabel_cache)) $multicompanylabel_cache = array();
+				if (!is_array($multicompanylabel_cache)) {
+					$multicompanylabel_cache = array();
+				}
 				if (empty($multicompanylabel_cache[$extrafields->attributes[$elementtype]['entityid'][$key]])) {
 					global $mc;
 					$mc->getInfo($extrafields->attributes[$elementtype]['entityid'][$key]);
@@ -118,7 +120,9 @@ 
 	}
 } else {
 	$colspan = 13;
-	if (!empty($conf->multicompany->enabled))  $colspan++;
+	if (!empty($conf->multicompany->enabled)) {
+		$colspan++;
+	}
 
 	print '<tr class="oddeven">';
 	print '<td class="opacitymedium" colspan="'.$colspan.'">';

htdocs/core/class/fileupload.class.php

Braces +3 added lines, -1 removed lines

@@ -286,7 +286,9 @@
 
 			$res = vignette($file_path, $maxwidthmini, $maxheightmini, '_mini'); // We don't use ->addThumbs here because there is no object and we don't need all thumbs, only the "mini".
 
-			if (preg_match('/error/i', $res)) return false;
+			if (preg_match('/error/i', $res)) {
+				return false;
+			}
 			return true;
 		} else {
 			return false;

htdocs/core/datepicker.php

Braces +73 added lines, -21 removed lines

@@ -26,20 +26,39 @@ 
  *       \brief      File to manage popup date selector
  */
 
-if (!defined('NOREQUIREUSER'))   define('NOREQUIREUSER', '1'); // disabled
+if (!defined('NOREQUIREUSER')) {
+	define('NOREQUIREUSER', '1');
+}
+// disabled
 //if (! defined('NOREQUIREDB'))   define('NOREQUIREDB','1');	// Not disabled cause need to load personalized language
-if (!defined('NOREQUIRESOC'))    define('NOREQUIRESOC', '1');
+if (!defined('NOREQUIRESOC')) {
+	define('NOREQUIRESOC', '1');
+}
 //if (! defined('NOREQUIRETRAN')) define('NOREQUIRETRAN','1');	// Not disabled cause need to do translations
-if (!defined('NOCSRFCHECK'))     define('NOCSRFCHECK', 1);
-if (!defined('NOTOKENRENEWAL'))  define('NOTOKENRENEWAL', 1);
-if (!defined('NOLOGIN')) define('NOLOGIN', 1); // disabled
-if (!defined('NOREQUIREMENU'))   define('NOREQUIREMENU', 1);
-if (!defined('NOREQUIREHTML'))   define('NOREQUIREHTML', 1);
+if (!defined('NOCSRFCHECK')) {
+	define('NOCSRFCHECK', 1);
+}
+if (!defined('NOTOKENRENEWAL')) {
+	define('NOTOKENRENEWAL', 1);
+}
+if (!defined('NOLOGIN')) {
+	define('NOLOGIN', 1);
+}
+// disabled
+if (!defined('NOREQUIREMENU')) {
+	define('NOREQUIREMENU', 1);
+}
+if (!defined('NOREQUIREHTML')) {
+	define('NOREQUIREHTML', 1);
+}
 
 require_once '../main.inc.php';
 require_once DOL_DOCUMENT_ROOT.'/core/lib/date.lib.php';
 
-if (GETPOST('lang', 'aZ09')) $langs->setDefaultLang(GETPOST('lang', 'aZ09')); // If language was forced on URL by the main.inc.php
+if (GETPOST('lang', 'aZ09')) {
+	$langs->setDefaultLang(GETPOST('lang', 'aZ09'));
+}
+// If language was forced on URL by the main.inc.php
 
 // Load translation files required by the page
 $langs->loadLangs(array("main", "agenda"));
@@ -93,14 +112,22 @@ 
 
 $qualified = true;
 
-if (!isset($_GET["sd"])) $_GET["sd"] = "00000000";
+if (!isset($_GET["sd"])) {
+	$_GET["sd"] = "00000000";
+}
 
-if (!isset($_GET["m"]) || !isset($_GET["y"])) $qualified = false;
+if (!isset($_GET["m"]) || !isset($_GET["y"])) {
+	$qualified = false;
+}
 if (isset($_GET["m"]) && isset($_GET["y"]))
 {
-	if ($_GET["m"] < 1 || $_GET["m"] > 12) $qualified = false;
-	if ($_GET["y"] < 0 || $_GET["y"] > 9999) $qualified = false;
-}
+	if ($_GET["m"] < 1 || $_GET["m"] > 12) {
+		$qualified = false;
+	}
+	if ($_GET["y"] < 0 || $_GET["y"] > 9999) {
+		$qualified = false;
+	}
+	}
 
 // If parameters provided, we show calendar
 if ($qualified)
@@ -171,11 +198,25 @@ 
 		<td class="dpButtons"
 			onClick="loadMonth('<?php echo DOL_URL_ROOT.'/core/' ?>','<?php echo $month?>','<?php echo $year - 1?>','<?php echo $xyz ?>','<?php echo $langs->defaultlang ?>')">&lt;&lt;</td>
 		<td class="dpButtons"
-			onClick="loadMonth('<?php echo DOL_URL_ROOT.'/core/' ?>','<?php if ($month == 1) echo "12"; else echo $month - 1?>','<?php if ($month == 1) echo $year - 1; else echo $year?>','<?php echo $xyz ?>','<?php echo $langs->defaultlang ?>')">&lt;</td>
+			onClick="loadMonth('<?php echo DOL_URL_ROOT.'/core/' ?>','<?php if ($month == 1) {
+	echo "12";
+} else {
+	echo $month - 1?>','<?php if ($month == 1) echo $year - 1;
+}
+else {
+	echo $year?>','<?php echo $xyz ?>','<?php echo $langs->defaultlang ?>')">&lt;</td>
 		<td colspan="3" class="dpButtons"
 			onClick="loadMonth('<?php echo DOL_URL_ROOT.'/core/' ?>','<?php echo (int) dol_print_date($today, '%m')?>','<?php echo $todayArray["year"]?>','<?php echo $xyz ?>','<?php echo $langs->defaultlang ?>')"><?php echo '-' ?></td>
 		<td class="dpButtons"
-			onClick="loadMonth('<?php echo DOL_URL_ROOT.'/core/' ?>','<?php if ($month == 12) echo "1"; else echo $month + 1?>','<?php if ($month == 12) echo $year + 1; else echo $year; ?>','<?php echo $xyz ?>','<?php echo $langs->defaultlang ?>')">&gt;</td>
+			onClick="loadMonth('<?php echo DOL_URL_ROOT.'/core/' ?>','<?php if ($month == 12) echo "1";
+}
+else {
+	echo $month + 1?>','<?php if ($month == 12) echo $year + 1;
+}
+else {
+	echo $year;
+}
+?>','<?php echo $xyz ?>','<?php echo $langs->defaultlang ?>')">&gt;</td>
 		<td class="dpButtons"
 			onClick="loadMonth('<?php echo DOL_URL_ROOT.'/core/' ?>','<?php echo $month?>','<?php echo $year + 1?>','<?php echo $xyz ?>','<?php echo $langs->defaultlang ?>')">&gt;&gt;</td>
 	</tr>
@@ -198,9 +239,11 @@ 
 	while (!$stoploop)
 	{
 		//print_r($mydate);
-		if ($mydate < $firstdate)	// At first run
+		if ($mydate < $firstdate) {
+			// At first run
 		{
 			echo "<tr class=\"dpWeek\">";
+		}
 			//echo $conf->global->MAIN_START_WEEK.' '.$firstdate["wday"].' '.$startday;
 			$cols = 0;
 			for ($i = 0; $i < 7; $i++)
@@ -223,8 +266,11 @@ 
 		}
 
 		$dayclass = "dpReg";
-		if ($thedate == $selDate) $dayclass = "dpSelected";
-		elseif ($thedate == $today) $dayclass = "dpToday";
+		if ($thedate == $selDate) {
+			$dayclass = "dpSelected";
+		} elseif ($thedate == $today) {
+			$dayclass = "dpToday";
+		}
 
 		if ($langs->trans("FormatDateShortJavaInput") == "FormatDateShortJavaInput")
 		{
@@ -239,7 +285,9 @@ 
 		echo ">".sprintf("%02s", $mydate["mday"])."</td>";
 		$cols++;
 
-		if (($mydate["wday"] + 1) % 7 == $startday) echo "</TR>\n";
+		if (($mydate["wday"] + 1) % 7 == $startday) {
+			echo "</TR>\n";
+		}
 
 		//$thedate=strtotime("tomorrow",$thedate);
 		$day++;
@@ -249,13 +297,17 @@ 
 			$stoploop = 1;
 		} else {
 			$mydate = dol_getdate($thedate);
-			if ($firstdate["month"] != $mydate["month"]) $stoploop = 1;
+			if ($firstdate["month"] != $mydate["month"]) {
+				$stoploop = 1;
+			}
 		}
 	}
 
 	if ($cols < 7)
 	{
-		for ($i = 6; $i >= $cols; $i--) echo "<td>&nbsp;</td>";
+		for ($i = 6; $i >= $cols; $i--) {
+			echo "<td>&nbsp;</td>";
+		}
 		echo "</tr>\n";
 	}
 	?>

htdocs/core/triggers/interface_50_modTicket_TicketEmail.class.php

Braces +13 added lines, -4 removed lines

@@ -109,7 +109,10 @@ 
 	{
 		$ok = 0;
 
-		if (empty($conf->ticket->enabled)) return 0; // Module not active, we do nothing
+		if (empty($conf->ticket->enabled)) {
+			return 0;
+		}
+		// Module not active, we do nothing
 
 		switch ($action) {
 			case 'TICKET_ASSIGNED':
@@ -284,10 +287,16 @@ 
 							}
 
 							$qualified = true;
-							if (empty($enabled)) $qualified = false;
-							if (empty($perms)) $qualified = false;
+							if (empty($enabled)) {
+								$qualified = false;
+							}
+							if (empty($perms)) {
+								$qualified = false;
+							}
 
-							if ($qualified) $message_customer .= '<li>'.$langs->trans($key).' : '.$value.'</li>';
+							if ($qualified) {
+								$message_customer .= '<li>'.$langs->trans($key).' : '.$value.'</li>';
+							}
 						}
 
 						$message_customer .= '</ul>';

htdocs/zapier/hook_document.php

Braces +9 added lines, -3 removed lines

@@ -54,8 +54,12 @@ 
 $offset = $limit * $page;
 $pageprev = $page - 1;
 $pagenext = $page + 1;
-if (!$sortorder) $sortorder = "ASC";
-if (!$sortfield) $sortfield = "name";
+if (!$sortorder) {
+	$sortorder = "ASC";
+}
+if (!$sortfield) {
+	$sortfield = "name";
+}
 //if (! $sortfield) $sortfield="position_name";
 
 // Initialize technical objects
@@ -71,7 +75,9 @@ 
 include DOL_DOCUMENT_ROOT.'/core/actions_fetchobject.inc.php'; // Must be include, not include_once  // Must be include, not include_once. Include fetch and fetch_thirdparty but not fetch_optionals
 
 //if ($id > 0 || ! empty($ref)) $upload_dir = $conf->sellyoursaas->multidir_output[$object->entity] . "/myobject/" . dol_sanitizeFileName($object->id);
-if ($id > 0 || !empty($ref)) $upload_dir = $conf->sellyoursaas->multidir_output[$object->entity]."/myobject/".dol_sanitizeFileName($object->ref);
+if ($id > 0 || !empty($ref)) {
+	$upload_dir = $conf->sellyoursaas->multidir_output[$object->entity]."/myobject/".dol_sanitizeFileName($object->ref);
+}
 
 
 /*

htdocs/zapier/lib/zapier_hook.lib.php

Braces +12 added lines, -4 removed lines

@@ -43,11 +43,17 @@ 
 
 	if (isset($object->fields['note_public']) || isset($object->fields['note_private'])) {
 		$nbNote = 0;
-		if (!empty($object->note_private)) $nbNote++;
-		if (!empty($object->note_public)) $nbNote++;
+		if (!empty($object->note_private)) {
+			$nbNote++;
+		}
+		if (!empty($object->note_public)) {
+			$nbNote++;
+		}
 		$head[$h][0] = dol_buildpath('/mymodule/myobject_note.php', 1).'?id='.$object->id;
 		$head[$h][1] = $langs->trans('Notes');
-		if ($nbNote > 0) $head[$h][1] .= '<span class="badge marginleftonlyshort">'.$nbNote.'</span>';
+		if ($nbNote > 0) {
+			$head[$h][1] .= '<span class="badge marginleftonlyshort">'.$nbNote.'</span>';
+		}
 		$head[$h][2] = 'note';
 		$h++;
 	}
@@ -59,7 +65,9 @@ 
 	$nbLinks = Link::count($db, $object->element, $object->id);
 	$head[$h][0] = dol_buildpath("/mymodule/myobject_document.php", 1).'?id='.$object->id;
 	$head[$h][1] = $langs->trans('Documents');
-	if (($nbFiles + $nbLinks) > 0) $head[$h][1] .= '<span class="badge marginleftonlyshort">'.($nbFiles + $nbLinks).'</span>';
+	if (($nbFiles + $nbLinks) > 0) {
+		$head[$h][1] .= '<span class="badge marginleftonlyshort">'.($nbFiles + $nbLinks).'</span>';
+	}
 	$head[$h][2] = 'document';
 	$h++;