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/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);

include/Loader.php

Braces +3 added lines, -2 removed lines

@@ -124,8 +124,9 @@
 			$secondFallBackModuleDir = $secondFallBackModuleClassPath = $actualModule;
 			if ($actualModule != 'Users') {
 				$baseModule = $moduleHierarchyParts[0];
-				if ($baseModule == 'Settings')
-					$baseModule = 'Settings:Vtiger';
+				if ($baseModule == 'Settings') {
+									$baseModule = 'Settings:Vtiger';
+				}
 				$firstFallBackDir = str_replace(':', '.', $baseModule);
 				$firstFallBackClassPath = str_replace(':', '_', $baseModule);
 			}

include/ChartUtils.php

Braces +55 added lines, -40 removed lines

@@ -32,14 +32,18 @@ 
 		} else {
 			$cache_file_name = $cachedFileName;
 		}
-		if (empty($width))
-			$width = '400';
-		if (empty($height))
-			$height = '300';
-		if ($target === false)
-			$target = [];
-		if (empty($color))
-			$color = 'black';
+		if (empty($width)) {
+					$width = '400';
+		}
+		if (empty($height)) {
+					$height = '300';
+		}
+		if ($target === false) {
+					$target = [];
+		}
+		if (empty($color)) {
+					$color = 'black';
+		}
 
 		$alts = [];
 		$temp = [];
@@ -48,8 +52,9 @@ 
 			$pos = substr_count($name, " ");
 			$alts[] = $name;
 			//If the daatx value of a string is greater, adding '\n' to it so that it'll cme inh 2nd line
-			if (strlen($name) >= 14)
-				$name = substr($name, 0, 44);
+			if (strlen($name) >= 14) {
+							$name = substr($name, 0, 44);
+			}
 			if ($pos >= 2) {
 				$val = explode(" ", $name);
 				$n = count($val) - 1;
@@ -102,8 +107,9 @@ 
 
 		for ($i = 0; $i < count($yaxisData); $i++) {
 			$x = 1 + $i;
-			if ($yaxisData[$i] >= $max)
-				$max = $yaxisData[$i];
+			if ($yaxisData[$i] >= $max) {
+							$max = $yaxisData[$i];
+			}
 			$dataset->addPoint(
 				$x, $yaxisData[$i], array(
 				'url' => $target[$i],
@@ -125,8 +131,9 @@ 
 		$bplot->setFillStyle($fill);
 
 		//You can change the width of the bars if you like
-		if (!empty($xaxisData))
-			$bplot->setBarWidth($barwidth / count($xaxisData), "%");
+		if (!empty($xaxisData)) {
+					$bplot->setBarWidth($barwidth / count($xaxisData), "%");
+		}
 		$bplot->setBackground(Image_Graph::factory('gradient', array(IMAGE_GRAPH_GRAD_HORIZONTAL, 'white', 'white')));
 		$xaxis = & $plotarea->getAxis(IMAGE_GRAPH_AXIS_X);
 		$yaxis = & $plotarea->getAxis(IMAGE_GRAPH_AXIS_Y);
@@ -150,8 +157,9 @@ 
 		$ticks = get_tickspacing(round(($max * 1.1) + 0.5));
 
 		// First make the labels look right
-		if ($charttype == 'horizontal')
-			$yaxis->setFontAngle('vertical');
+		if ($charttype == 'horizontal') {
+					$yaxis->setFontAngle('vertical');
+		}
 		$yaxis->setLabelInterval($ticks[0]);
 		$yaxis->setTickOptions(-5, 0);
 		$yaxis->setLabelInterval($ticks[1], 2);
@@ -165,8 +173,9 @@ 
 		$xaxis->setDataPreprocessor($array_data);
 		$xaxis->forceMinimum(0.5);
 		$xaxis->forceMaximum(0.5 + count($yaxisData));
-		if ($charttype == 'vertical')
-			$xaxis->setFontAngle('vertical');
+		if ($charttype == 'vertical') {
+					$xaxis->setFontAngle('vertical');
+		}
 		$xaxis->setLabelInterval(1);
 		$xaxis->setTickOptions(0, 0);
 		$xaxis->setLabelInterval(2, 2);
@@ -218,12 +227,15 @@ 
 			$cache_file_name = $cachedFileName;
 		}
 
-		if (empty($width))
-			$width = '500';
-		if (empty($height))
-			$height = '400';
-		if ($target === false)
-			$target = [];
+		if (empty($width)) {
+					$width = '500';
+		}
+		if (empty($height)) {
+					$height = '400';
+		}
+		if ($target === false) {
+					$target = [];
+		}
 
 		$alts = [];
 		$temp = [];
@@ -232,8 +244,9 @@ 
 			$pos = substr_count($name, " ");
 			$alts[] = $name;
 			//If the datax value of a string is greater, adding '\n' to it so that it'll come in 2nd line
-			if (strlen($name) >= 14)
-				$name = substr($name, 0, 34);
+			if (strlen($name) >= 14) {
+							$name = substr($name, 0, 34);
+			}
 			if ($pos >= 2) {
 				$val = explode(" ", $name);
 				$n = count($val) - 1;
@@ -297,8 +310,9 @@ 
 			$pcvalues[$yaxisData[$i]] = sprintf('%0.1f%%', 100 * $yaxisData[$i] / $sum);
 			$fills->addColor($colors[$i]);
 		}
-		if ($sum == 0)
-			return null;
+		if ($sum == 0) {
+					return null;
+		}
 		// create the pie chart and associate the filling colours
 		$gbplot = & $plotarea->addNew('pie', $dataset);
 		$plotarea->setPadding(array('top' => 0, 'bottom' => 0, 'left' => 0, 'right' => ($width / 20)));
@@ -361,8 +375,9 @@ 
 		$restrictedModules = false;
 		foreach ($reportModules as $mod) {
 			if (isPermitted($mod, 'index') != 'yes' || \includes\Modules::isModuleActive($mod) == false) {
-				if (!is_array($restrictedModules))
-					$restrictedModules = [];
+				if (!is_array($restrictedModules)) {
+									$restrictedModules = [];
+				}
 				$restrictedModules[] = $mod;
 			}
 		}
@@ -409,12 +424,12 @@ 
 					$groupFieldValue = $valueComp[1];
 				}
 				if ($dateField) {
-					if (!empty($groupFieldValue))
-						$groupByFields[] = CustomReportUtils::getXAxisDateFieldValue($groupFieldValue, $criteria);
-					else
-						$groupByFields[] = "Null";
-				}
-				else if (in_array($fieldname, array_keys($referenceFields))) {
+					if (!empty($groupFieldValue)) {
+											$groupByFields[] = CustomReportUtils::getXAxisDateFieldValue($groupFieldValue, $criteria);
+					} else {
+											$groupByFields[] = "Null";
+					}
+				} else if (in_array($fieldname, array_keys($referenceFields))) {
 					if (count($referenceFields[$fieldname]) > 1) {
 						$refenceModule = CustomReportUtils::getEntityTypeFromName($decodedGroupFieldValue, $referenceFields[$fieldname]);
 					} else {
@@ -443,10 +458,10 @@ 
 						$advanceSearchCondition = CustomReportUtils::getAdvanceSearchCondition($fieldDetails, $criteria, $groupFieldValue);
 						if ($module == 'Calendar') {
 							$link_val = "index.php?module=" . $module . "&query=true&action=ListView&" . $advanceSearchCondition;
-						} else
-							$link_val = "index.php?module=" . $module . "&query=true&action=index&" . $advanceSearchCondition;
-					}
-					else {
+						} else {
+													$link_val = "index.php?module=" . $module . "&query=true&action=index&" . $advanceSearchCondition;
+						}
+					} else {
 						$cvid = \vtlib\Deprecated::getIdOfCustomViewByNameAll($module);
 						$esc_search_str = urlencode($decodedGroupFieldValue);
 						if ($single == 'DT') {