YetiForceCompany / YetiForceCRM

api.php

Braces +7 added lines, -14 removed lines

@@ -19,33 +19,26 @@
 	if($service == "customerportal")
 	{
 		include("api/customerportal.php");
-	}	
-	elseif($service == "mobile")	
+	} elseif($service == "mobile")	
 	{		
 		include("api/mobile.php");
-	}
-	elseif($service == "yetiportal")	
+	} elseif($service == "yetiportal")	
 	{		
 		include("api/yetiportal.php");
-	}
-	elseif($service == "firefox")
+	} elseif($service == "firefox")
 	{
 		include("api/firefoxtoolbar.php");
-	}
-	elseif($service == "wordplugin")
+	} elseif($service == "wordplugin")
 	{
 		include("api/wordplugin.php");
-	}
-	elseif($service == "thunderbird")
+	} elseif($service == "thunderbird")
 	{
 		include("api/thunderbirdplugin.php");
-	}
-	else
+	} else
 	{
 		echo "No Service Configured for ". strip_tags($service);
 	}
-}
-else
+} else
 {
 	echo "<h1>YetiForceCRM API Services</h1>";
 	echo "<li>YetiForceCRM Yeti Portal EndPoint URL -- Click <a href='api.php?service=yetiportal'>here</a></li>";

install/views/Index.php

Braces +3 added lines, -2 removed lines

@@ -87,8 +87,9 @@
 	public function process(Vtiger_Request $request)
 	{
 		$default_charset = AppConfig::main('default_charset');
-		if (empty($default_charset))
-			$default_charset = 'UTF-8';
+		if (empty($default_charset)) {
+					$default_charset = 'UTF-8';
+		}
 		$mode = $request->getMode();
 		if (!empty($mode) && $this->isMethodExposed($mode)) {
 			return $this->$mode($request);

include/recaptcha/recaptchalib.php

Braces +7 added lines, -4 removed lines

@@ -47,8 +47,9 @@ 
 function _recaptcha_qsencode($data)
 {
 	$req = "";
-	foreach ($data as $key => $value)
-		$req .= $key . '=' . urlencode(stripslashes($value)) . '&';
+	foreach ($data as $key => $value) {
+			$req .= $key . '=' . urlencode(stripslashes($value)) . '&';
+	}
 
 	// Cut the last '&'
 	$req = substr($req, 0, strlen($req) - 1);
@@ -83,8 +84,10 @@ 
 
 	fwrite($fs, $http_request);
 
-	while (!feof($fs))
-		$response .= fgets($fs, 1160); // One TCP-IP packet
+	while (!feof($fs)) {
+			$response .= fgets($fs, 1160);
+	}
+	// One TCP-IP packet
 	fclose($fs);
 	$response = explode("\r\n\r\n", $response, 2);
 

include/InventoryPDFController.php

Braces +30 added lines, -20 removed lines

@@ -83,8 +83,9 @@ 
 
 	public function Output($filename, $type)
 	{
-		if (is_null($this->focus))
-			return;
+		if (is_null($this->focus)) {
+					return;
+		}
 
 		$pdfgenerator = $this->getPDFGenerator();
 
@@ -256,24 +257,32 @@ 
 
 			$addressValues = [];
 			$addressValues[] = $resultrow['address'];
-			if (!empty($resultrow['city']))
-				$addressValues[] = "\n" . $resultrow['city'];
-			if (!empty($resultrow['state']))
-				$addressValues[] = "," . $resultrow['state'];
-			if (!empty($resultrow['code']))
-				$addressValues[] = $resultrow['code'];
-			if (!empty($resultrow['country']))
-				$addressValues[] = "\n" . $resultrow['country'];
+			if (!empty($resultrow['city'])) {
+							$addressValues[] = "\n" . $resultrow['city'];
+			}
+			if (!empty($resultrow['state'])) {
+							$addressValues[] = "," . $resultrow['state'];
+			}
+			if (!empty($resultrow['code'])) {
+							$addressValues[] = $resultrow['code'];
+			}
+			if (!empty($resultrow['country'])) {
+							$addressValues[] = "\n" . $resultrow['country'];
+			}
 
 			$additionalCompanyInfo = [];
-			if (!empty($resultrow['phone']))
-				$additionalCompanyInfo[] = "\n" . \includes\Language::translate("Phone: ", $this->moduleName) . $resultrow['phone'];
-			if (!empty($resultrow['fax']))
-				$additionalCompanyInfo[] = "\n" . \includes\Language::translate("Fax: ", $this->moduleName) . $resultrow['fax'];
-			if (!empty($resultrow['website']))
-				$additionalCompanyInfo[] = "\n" . \includes\Language::translate("Website: ", $this->moduleName) . $resultrow['website'];
-			if (!empty($resultrow['vatid']))
-				$additionalCompanyInfo[] = "\n" . \includes\Language::translate("VAT ID: ", $this->moduleName) . $resultrow['vatid'];
+			if (!empty($resultrow['phone'])) {
+							$additionalCompanyInfo[] = "\n" . \includes\Language::translate("Phone: ", $this->moduleName) . $resultrow['phone'];
+			}
+			if (!empty($resultrow['fax'])) {
+							$additionalCompanyInfo[] = "\n" . \includes\Language::translate("Fax: ", $this->moduleName) . $resultrow['fax'];
+			}
+			if (!empty($resultrow['website'])) {
+							$additionalCompanyInfo[] = "\n" . \includes\Language::translate("Website: ", $this->moduleName) . $resultrow['website'];
+			}
+			if (!empty($resultrow['vatid'])) {
+							$additionalCompanyInfo[] = "\n" . \includes\Language::translate("VAT ID: ", $this->moduleName) . $resultrow['vatid'];
+			}
 
 			$modelColumnLeft = array(
 				'logo' => "storage/Logo/" . $resultrow['logoname'],
@@ -430,8 +439,9 @@ 
 	{
 		$valueString = '';
 		foreach ($values as $value) {
-			if (empty($value))
-				continue;
+			if (empty($value)) {
+							continue;
+			}
 			$valueString .= $value . $delimeter;
 		}
 		return rtrim($valueString, $delimeter);

include/Modules.php

Braces +10 added lines, -8 removed lines

@@ -16,10 +16,11 @@ 
 	{
 		$entity = false;
 		if ($mixed) {
-			if (is_numeric($mixed))
-				$entity = isset(self::$moduleEntityCacheById[$mixed]) ? self::$moduleEntityCacheById[$mixed] : false;
-			else
-				$entity = isset(self::$moduleEntityCacheByName[$mixed]) ? self::$moduleEntityCacheByName[$mixed] : false;
+			if (is_numeric($mixed)) {
+							$entity = isset(self::$moduleEntityCacheById[$mixed]) ? self::$moduleEntityCacheById[$mixed] : false;
+			} else {
+							$entity = isset(self::$moduleEntityCacheByName[$mixed]) ? self::$moduleEntityCacheByName[$mixed] : false;
+			}
 		}
 		if (!$entity) {
 			$adb = \PearDatabase::getInstance();
@@ -31,10 +32,11 @@ 
 				self::$moduleEntityCacheById[$row['tabid']] = $row;
 			}
 			if ($mixed) {
-				if (is_numeric($mixed))
-					return self::$moduleEntityCacheById[$mixed];
-				else
-					return self::$moduleEntityCacheByName[$mixed];
+				if (is_numeric($mixed)) {
+									return self::$moduleEntityCacheById[$mixed];
+				} else {
+									return self::$moduleEntityCacheByName[$mixed];
+				}
 			}
 		}
 		return $entity;

include/Tracker.php

Braces +3 added lines, -2 removed lines

@@ -80,8 +80,9 @@
 				$fieldlists = explode(',', $fieldsname);
 				$fl = [];
 				foreach ($fieldlists as $w => $c) {
-					if (count($fl))
-						$fl[] = "' '";
+					if (count($fl)) {
+											$fl[] = "' '";
+					}
 					$fl[] = $c;
 				}
 				$fieldsname = $adb->concat($fl);

include/database/Postgres8.php

Braces +81 added lines, -56 removed lines

@@ -22,8 +22,9 @@ 
 	$groupClause = "";
 	$orderClause = "";
 
-	if ($debug)
-		$log->info("fixPostgresQuery: " . $query);
+	if ($debug) {
+			$log->info("fixPostgresQuery: " . $query);
+	}
 
 	// If we already have an order or group cluase separate ist for later use
 	if (stripos($queryRecord, 'GROUP BY') > 0) {
@@ -45,32 +46,36 @@ 
 	$privateGroupList = [];
 	$token = strtok($queryFields, ", ()	");
 	while ($token !== false) {
-		if (strpos($token, ".") !== false)
-			array_push($privateGroupList, $token);
+		if (strpos($token, ".") !== false) {
+					array_push($privateGroupList, $token);
+		}
 		$token = strtok(", ()	");
 	}
 	sort($privateGroupList);
 	$groupFields = "";
 	$last = "";
 	for ($i = 0; $i < count($privateGroupList); $i++) {
-		if ($last != $privateGroupList[$i])
-			if ($groupFields == "")
+		if ($last != $privateGroupList[$i]) {
+					if ($groupFields == "")
 				$groupFields = $privateGroupList[$i];
-			else
-				$groupFields .= "," . $privateGroupList[$i];
+		} else {
+							$groupFields .= "," . $privateGroupList[$i];
+			}
 		$last = $privateGroupList[$i];
 	}
 
 	// Rebuild the query
 	$query = sprintf("SELECT %s%s", $queryFields, $queryRecord);
-	if ($groupClause != "")
-		$groupClause = $groupClause . "," . $groupFields;
-	else
-		$groupClause = $groupFields;
+	if ($groupClause != "") {
+			$groupClause = $groupClause . "," . $groupFields;
+	} else {
+			$groupClause = $groupFields;
+	}
 	$query .= expandStar($groupClause, $log) . " " . $orderClause;
 
-	if ($debug)
-		$log->info("fixPostgresQuery result: " . $query);
+	if ($debug) {
+			$log->info("fixPostgresQuery result: " . $query);
+	}
 
 	return( $query);
 }
@@ -94,22 +99,25 @@ 
 				$subfield = substr($field, $pos + 1, strlen($field) - $pos);
 
 				//do we need to expand?
-				if ($subfield == "*")
-					$field = expandRecord($table, $log);
+				if ($subfield == "*") {
+									$field = expandRecord($table, $log);
+				}
 			}
 
 			//add the propably expanded field to the querylist
-			if ($expanded == "")
-				$expanded = $field;
-			else
-				$expanded .= "," . $field;
+			if ($expanded == "") {
+							$expanded = $field;
+			} else {
+							$expanded .= "," . $field;
+			}
 		}
 
 		//next field
 		$field = strtok(",");
 	}
-	if ($expanded != '')
-		$expanded = " GROUP BY " . trim($expanded, ",");
+	if ($expanded != '') {
+			$expanded = " GROUP BY " . trim($expanded, ",");
+	}
 	//return the expanded fieldlist
 	return( $expanded);
 }
@@ -122,68 +130,84 @@ 
 	$subfields = [];
 
 	//vtiger_products table
-	if ($table == "vtiger_products")
-		$subfields = array("productid", "productname", "productcode", "productcategory", "manufacturer", "qty_per_unit", "unit_price", "weight", "pack_size", "sales_start_date", "sales_end_date", "start_date", "expiry_date", "cost_factor", "commissionrate", "commissionmethod", "discontinued", "usageunit", "currency", "reorderlevel", "website", "taxclass", "mfr_part_no", "vendor_part_no", "serialno", "qtyinstock", "productsheet", "qtyindemand", "glacct", "vendor_id", "imagename");
+	if ($table == "vtiger_products") {
+			$subfields = array("productid", "productname", "productcode", "productcategory", "manufacturer", "qty_per_unit", "unit_price", "weight", "pack_size", "sales_start_date", "sales_end_date", "start_date", "expiry_date", "cost_factor", "commissionrate", "commissionmethod", "discontinued", "usageunit", "currency", "reorderlevel", "website", "taxclass", "mfr_part_no", "vendor_part_no", "serialno", "qtyinstock", "productsheet", "qtyindemand", "glacct", "vendor_id", "imagename");
+	}
 	
 	//vtiger_activity table
-	elseif ($table == "vtiger_activity")
-		$subfields = array("activityid", "subject", "semodule", "activitytype", "date_start", "due_date", "time_start", "time_end", "sendnotification", "duration_hours", "duration_minutes", "status", "eventstatus", "priority", "location", "notime", "visibility", "recurringtype");
+	elseif ($table == "vtiger_activity") {
+			$subfields = array("activityid", "subject", "semodule", "activitytype", "date_start", "due_date", "time_start", "time_end", "sendnotification", "duration_hours", "duration_minutes", "status", "eventstatus", "priority", "location", "notime", "visibility", "recurringtype");
+	}
 
 	//vtiger_notes table
-	elseif ($table == "vtiger_notes")
-		$subfields = array("notesid", "contact_id", "title", "filename", "notecontent");
+	elseif ($table == "vtiger_notes") {
+			$subfields = array("notesid", "contact_id", "title", "filename", "notecontent");
+	}
 
 	//vtiger_faq table
-	elseif ($table == "vtiger_faq")
-		$subfields = array("id", "product_id", "question", "answer", "category", "status");
+	elseif ($table == "vtiger_faq") {
+			$subfields = array("id", "product_id", "question", "answer", "category", "status");
+	}
 
 	//vtiger_profile2field
-	elseif ($table == "vtiger_profile2field")
-		$subfields = array("profileid", "tabid", "fieldid", "visible", "readonly");
+	elseif ($table == "vtiger_profile2field") {
+			$subfields = array("profileid", "tabid", "fieldid", "visible", "readonly");
+	}
 
 	//vtiger_field
-	elseif ($table == "vtiger_field")
-		$subfields = array("tabid", "fieldid", "columnname", "tablename", "generatedtype", "uitype", "fieldname", "fieldlabel", "readonly", "presence", "selected", "maximumlength", "sequence", "block", "displaytype", "typeofdata", "quickcreate", "quickcreatesequence", "info_type");
+	elseif ($table == "vtiger_field") {
+			$subfields = array("tabid", "fieldid", "columnname", "tablename", "generatedtype", "uitype", "fieldname", "fieldlabel", "readonly", "presence", "selected", "maximumlength", "sequence", "block", "displaytype", "typeofdata", "quickcreate", "quickcreatesequence", "info_type");
+	}
 
 	//vtiger_producttaxrel
-	elseif ($table == "vtiger_producttaxrel")
-		$subfields = array("productid", "taxid", "taxpercentage");
+	elseif ($table == "vtiger_producttaxrel") {
+			$subfields = array("productid", "taxid", "taxpercentage");
+	}
 
 	//vtiger_inventorytaxinfo
-	elseif ($table == "vtiger_inventorytaxinfo")
-		$subfields = array("taxid", "taxname", "taxlabel", "percentage", "deleted");
+	elseif ($table == "vtiger_inventorytaxinfo") {
+			$subfields = array("taxid", "taxname", "taxlabel", "percentage", "deleted");
+	}
 
 	//vtiger_role2picklist
-	elseif ($table == "vtiger_role2picklist")
-		$subfields = array("roleid", "picklistid", "sortid");
+	elseif ($table == "vtiger_role2picklist") {
+			$subfields = array("roleid", "picklistid", "sortid");
+	}
 
 	//vtiger_contactdetails
-	elseif ($table == "vtiger_contactdetails")
-		$subfields = array("lastname", "contactid", "accountid", "salutation", "firstname", "email", "phone", "mobile", "title", "department", "fax", "reportsto", "training", "usertype", "contacttype", "otheremail", "yahooid", "donotcall", "emailoptout", "imagename", "reference", "notify_owner");
+	elseif ($table == "vtiger_contactdetails") {
+			$subfields = array("lastname", "contactid", "accountid", "salutation", "firstname", "email", "phone", "mobile", "title", "department", "fax", "reportsto", "training", "usertype", "contacttype", "otheremail", "yahooid", "donotcall", "emailoptout", "imagename", "reference", "notify_owner");
+	}
 
 	//vtiger_crmentity
-	elseif ($table == "vtiger_crmentity")
-		$subfields = array("crmid", "smcreatorid", "smownerid", "modifiedby", "setype", "description", "createdtime", "modifiedtime", "viewedtime", "status", "version", "presence", "deleted");
+	elseif ($table == "vtiger_crmentity") {
+			$subfields = array("crmid", "smcreatorid", "smownerid", "modifiedby", "setype", "description", "createdtime", "modifiedtime", "viewedtime", "status", "version", "presence", "deleted");
+	}
 
 	//vtiger_seactivityrel
-	elseif ($table == "vtiger_seactivityrel")
-		$subfields = array("crmid", "activityid");
+	elseif ($table == "vtiger_seactivityrel") {
+			$subfields = array("crmid", "activityid");
+	}
 
 	//vtiger_cntactivityrel
-	elseif ($table == "vtiger_cntactivityrel")
-		$subfields = array("contactid", "activityid");
+	elseif ($table == "vtiger_cntactivityrel") {
+			$subfields = array("contactid", "activityid");
+	}
 
 	//vtiger_leaddetails
-	elseif ($table == "vtiger_leaddetails")
-		$subfields = array("leadid", "email", "interest", "firstname", "salutation", "lastname", "company", "annualrevenue", "industry", "campaign", "rating", "leadstatus", "leadsource", "converted", "designation", "space", "comments", "priority", "demorequest", "partnercontact", "productversion", "product", "maildate", "nextstepdate", "fundingsituation", "purpose", "evaluationstatus", "transferdate", "revenuetype", "noofemployees", "yahooid", "assignleadchk");
+	elseif ($table == "vtiger_leaddetails") {
+			$subfields = array("leadid", "email", "interest", "firstname", "salutation", "lastname", "company", "annualrevenue", "industry", "campaign", "rating", "leadstatus", "leadsource", "converted", "designation", "space", "comments", "priority", "demorequest", "partnercontact", "productversion", "product", "maildate", "nextstepdate", "fundingsituation", "purpose", "evaluationstatus", "transferdate", "revenuetype", "noofemployees", "yahooid", "assignleadchk");
+	}
 
 	//vtiger_pricebook
-	elseif ($table == "vtiger_pricebook")
-		$subfields = array("pricebookid", "bookname", "active", "description");
+	elseif ($table == "vtiger_pricebook") {
+			$subfields = array("pricebookid", "bookname", "active", "description");
+	}
 
 	//fields of the requested array still undefined
-	else
-		$log->info("function expandRecord: please add structural information for table '" . $table . "'");
+	else {
+			$log->info("function expandRecord: please add structural information for table '" . $table . "'");
+	}
 
 	//construct an entity string
 	for ($i = 0; $i < count($subfields); $i++) {
@@ -191,8 +215,9 @@ 
 	}
 
 	//remove the trailiung ,
-	if (strlen($result) > 0)
-		$result = substr($result, 0, strlen($result) - 1);
+	if (strlen($result) > 0) {
+			$result = substr($result, 0, strlen($result) - 1);
+	}
 
 	//return out new string
 	return( $result);

include/database/PearDatabase.php

Braces +33 added lines, -22 removed lines

@@ -464,8 +464,9 @@ 
 			$this->checkError('Missing table name');
 			return false;
 		}
-		if ($where != '')
-			$where = sprintf('WHERE %s', $where);
+		if ($where != '') {
+					$where = sprintf('WHERE %s', $where);
+		}
 		if (count($params) === 0) {
 			$this->query(sprintf('DELETE FROM %s %s', $table, $where));
 		} else {
@@ -558,10 +559,12 @@ 
 	//TODO DEPRECATED
 	public function flatten_array($input, $output = null)
 	{
-		if ($input == null)
-			return null;
-		if ($output == null)
-			$output = [];
+		if ($input == null) {
+					return null;
+		}
+		if ($output == null) {
+					$output = [];
+		}
 		foreach ($input as $value) {
 			if (is_array($value)) {
 				$output = $this->flatten_array($value, $output);
@@ -634,15 +637,17 @@ 
 	{
 		if (isset($result) && $rowNum < 0) {
 			$row = $this->getRow($result);
-			if ($encode && is_array($row))
-				return array_map('to_html', $row);
+			if ($encode && is_array($row)) {
+							return array_map('to_html', $row);
+			}
 			return $row;
 		}
 		if ($this->getRowCount($result) > $rowNum) {
 			$row = $this->raw_query_result_rowdata($result, $rowNum);
 		}
-		if ($encode && is_array($row))
-			return array_map('to_html', $row);
+		if ($encode && is_array($row)) {
+					return array_map('to_html', $row);
+		}
 		return $row;
 	}
 
@@ -787,8 +792,9 @@ 
 			$this->log('sqlExprDatalist: empty arrays not allowed', 'error');
 			$this->checkError('sqlExprDatalist: empty arrays not allowed');
 		}
-		foreach ($array as $key => $val)
-			$l .= ($l ? ',' : '') . $this->quote($val);
+		foreach ($array as $key => $val) {
+					$l .= ($l ? ',' : '') . $this->quote($val);
+		}
 		return ' ( ' . $l . ' ) ';
 	}
 
@@ -803,8 +809,9 @@ 
 	{
 		$result = $this->query($sql, $dieOnError, $msg);
 
-		if ($this->getRowCount($result) == 1)
-			return $result;
+		if ($this->getRowCount($result) == 1) {
+					return $result;
+		}
 		$this->log('Rows Returned:' . $this->getRowCount($result) . ' More than 1 row returned for ' . $sql, 'error');
 		$this->checkError('Rows Returned:' . $this->getRowCount($result) . ' More than 1 row returned for ' . $sql, $dieOnError);
 		return '';
@@ -816,8 +823,9 @@ 
 	{
 		$result = $this->pquery($sql, $params, $dieOnError, $msg);
 
-		if ($this->getRowCount($result) == 1)
-			return $result;
+		if ($this->getRowCount($result) == 1) {
+					return $result;
+		}
 		$this->log('Rows Returned:' . $this->getRowCount($result) . ' More than 1 row returned for ' . $sql, 'error');
 		$this->checkError('Rows Returned:' . $this->getRowCount($result) . ' More than 1 row returned for ' . $sql, $dieOnError);
 		return '';
@@ -881,8 +889,9 @@ 
 			if (isset($_SERVER['REQUEST_METHOD'])) {
 				$uri = $_SERVER['REQUEST_URI'];
 				$qmarkIndex = strpos($_SERVER['REQUEST_URI'], '?');
-				if ($qmarkIndex !== false)
-					$uri = substr($uri, 0, $qmarkIndex);
+				if ($qmarkIndex !== false) {
+									$uri = substr($uri, 0, $qmarkIndex);
+				}
 				$data = $uri . '?' . http_build_query($_SERVER['REQUEST_METHOD'] == 'GET' ? $_GET : $_POST);
 			}
 			$stmt = $db->database->prepare($logQuery);
@@ -917,10 +926,12 @@ 
 					}
 					$args = rtrim($args, ',');
 				}
-				if (!empty($callerfunc))
-					$callerfunc = " ($callerfunc)";
-				if (!empty($args))
-					$callerfunc .= "[$args] ";
+				if (!empty($callerfunc)) {
+									$callerfunc = " ($callerfunc)";
+				}
+				if (!empty($args)) {
+									$callerfunc .= "[$args] ";
+				}
 			}
 			$data[] = 'CALLER: (' . $callers[$calleridx]['line'] . ') ' . $callers[$calleridx]['file'] . $callerfunc;
 		}

include/RelatedListView.php

Braces +26 added lines, -18 removed lines

@@ -115,22 +115,26 @@ 
 
 	$limit_start_rec = ($start - 1) * $listMaxEntriesPerPage;
 
-	if ($adb->isPostgres())
-		$list_result = $adb->pquery($query .
+	if ($adb->isPostgres()) {
+			$list_result = $adb->pquery($query .
 			" OFFSET $limit_start_rec LIMIT $listMaxEntriesPerPage ", []);
-	else
-		$list_result = $adb->pquery($query .
+	} else {
+			$list_result = $adb->pquery($query .
 			" LIMIT $limit_start_rec, $listMaxEntriesPerPage ", []);
+	}
 
 	$header = [];
 	$header[] = $mod_strings['LBL_LIST_PRODUCT_NAME'];
-	if (getFieldVisibilityPermission('Products', $current_user->id, 'productcode') == '0')
-		$header[] = $mod_strings['LBL_PRODUCT_CODE'];
-	if (getFieldVisibilityPermission('Products', $current_user->id, 'unit_price') == '0')
-		$header[] = $mod_strings['LBL_PRODUCT_UNIT_PRICE'];
+	if (getFieldVisibilityPermission('Products', $current_user->id, 'productcode') == '0') {
+			$header[] = $mod_strings['LBL_PRODUCT_CODE'];
+	}
+	if (getFieldVisibilityPermission('Products', $current_user->id, 'unit_price') == '0') {
+			$header[] = $mod_strings['LBL_PRODUCT_UNIT_PRICE'];
+	}
 	$header[] = $mod_strings['LBL_PB_LIST_PRICE'];
-	if (isPermitted("PriceBooks", "EditView", "") == 'yes' || isPermitted("PriceBooks", "Delete", "") == 'yes')
-		$header[] = $mod_strings['LBL_ACTION'];
+	if (isPermitted("PriceBooks", "EditView", "") == 'yes' || isPermitted("PriceBooks", "Delete", "") == 'yes') {
+			$header[] = $mod_strings['LBL_ACTION'];
+	}
 
 	$currency_id = $focus->column_fields['currency_id'];
 	$numRows = $adb->num_rows($list_result);
@@ -146,10 +150,12 @@ 
 
 		$entries = [];
 		$entries[] = textlength_check($adb->query_result($list_result, $i, "productname"));
-		if (getFieldVisibilityPermission('Products', $current_user->id, 'productcode') == '0')
-			$entries[] = $adb->query_result($list_result, $i, "productcode");
-		if (getFieldVisibilityPermission('Products', $current_user->id, 'unit_price') == '0')
-			$entries[] = CurrencyField::convertToUserFormat($unit_price, null, true);
+		if (getFieldVisibilityPermission('Products', $current_user->id, 'productcode') == '0') {
+					$entries[] = $adb->query_result($list_result, $i, "productcode");
+		}
+		if (getFieldVisibilityPermission('Products', $current_user->id, 'unit_price') == '0') {
+					$entries[] = CurrencyField::convertToUserFormat($unit_price, null, true);
+		}
 
 		$entries[] = CurrencyField::convertToUserFormat($listprice, null, true);
 		$action = "";
@@ -159,12 +165,14 @@ 
 			$action .= '<img src="' . vtiger_imageurl('blank.gif', $theme) . '" border="0" />';
 		}
 		if (isPermitted("PriceBooks", "Delete", "") == 'yes' && isPermitted('Products', 'Delete', $entity_id) == 'yes') {
-			if ($action != "")
-				$action .= '&nbsp;|&nbsp;';
+			if ($action != "") {
+							$action .= '&nbsp;|&nbsp;';
+			}
 			$action .= '<img src="' . vtiger_imageurl('delete.gif', $theme) . '" onclick="if(confirm(\'' . \includes\Language::translate('ARE_YOU_SURE') . '\')) deletePriceBookProductRel(' . $entity_id . ',' . $pricebook_id . ');" alt="' . \includes\Language::translate('LBL_DELETE') . '" title="' . \includes\Language::translate('LBL_DELETE') . '" style="cursor:pointer;" border="0">';
 		}
-		if ($action != "")
-			$entries[] = $action;
+		if ($action != "") {
+					$entries[] = $action;
+		}
 		$entries_list[] = $entries;
 	}
 	$navigationOutput[] = getRecordRangeMessage($list_result, $limit_start_rec, $noofrows);