YetiForceCompany / YetiForceCRM

vtlib/Vtiger/Version.php

Doc Comments +9 added lines

@@ -18,6 +18,7 @@ 
 
 	/**
 	 * Get current version of vtiger in use.
+	 * @return string|null
 	 */
 	static function current()
 	{
@@ -40,6 +41,9 @@ 
 		return version_compare($current_version, $with_version, $condition);
 	}
 
+	/**
+	 * @param string $endString
+	 */
 	static function endsWith($string, $endString)
 	{
 		$strLen = strlen($string);
@@ -49,6 +53,11 @@ 
 		return substr_compare($string, $endString, -$endStrLen) === 0;
 	}
 
+	/**
+	 * @param string $version
+	 *
+	 * @return string|null
+	 */
 	static function getUpperLimitVersion($version)
 	{
 		if (!self::endsWith($version, '.*'))

Braces +6 added lines, -4 removed lines

@@ -44,15 +44,17 @@
 	{
 		$strLen = strlen($string);
 		$endStrLen = strlen($endString);
-		if ($endStrLen > $strLen)
-			return false;
+		if ($endStrLen > $strLen) {
+					return false;
+		}
 		return substr_compare($string, $endString, -$endStrLen) === 0;
 	}
 
 	static function getUpperLimitVersion($version)
 	{
-		if (!self::endsWith($version, '.*'))
-			return $version;
+		if (!self::endsWith($version, '.*')) {
+					return $version;
+		}
 
 		$version = rtrim($version, '.*');
 		$lastVersionPartIndex = strrpos($version, '.');

vtlib/Vtiger/Webservice.php

Doc Comments +3 added lines

@@ -20,6 +20,7 @@ 
 	 * Helper function to log messages
 	 * @param String Message to log
 	 * @param Boolean true appends linebreak, false to avoid it
+	 * @param string $message
 	 * @access private
 	 */
 	static function log($message, $delim = true)
@@ -30,6 +31,7 @@ 
 	/**
 	 * Initialize webservice for the given module
 	 * @param Module Instance of the module.
+	 * @param Module $moduleInstance
 	 */
 	static function initialize($moduleInstance)
 	{
@@ -45,6 +47,7 @@ 
 	/**
 	 * Initialize webservice for the given module
 	 * @param Module Instance of the module.
+	 * @param Module $moduleInstance
 	 */
 	static function uninitialize($moduleInstance)
 	{

vtlib/Vtiger/Zip.php

Doc Comments +2 added lines

@@ -69,6 +69,7 @@ 
 
 	/**
 	 * Normalize the directory path separators.
+	 * @param string $path
 	 */
 	public function __normalizePath($path)
 	{
@@ -79,6 +80,7 @@ 
 
 	/**
 	 * Copy the directory on the disk into zip file.
+	 * @param string $zipdirname
 	 */
 	public function copyDirectoryFromDisk($dirname, $zipdirname = null, $excludeList = null, $basedirname = null)
 	{

Braces +18 added lines, -12 removed lines

@@ -50,8 +50,9 @@ 
 		if ($search_index === 0) {
 			$startindex = strlen($base_realpath) + 1;
 			// On windows $base_realpath ends with / and On Linux it will not have / at end!
-			if (strrpos($base_realpath, '/') == strlen($base_realpath) - 1)
-				$startindex -= 1;
+			if (strrpos($base_realpath, '/') == strlen($base_realpath) - 1) {
+							$startindex -= 1;
+			}
 			$relpath = substr($src_realpath, $startindex);
 		}
 		return $relpath;
@@ -62,8 +63,9 @@ 
 	 */
 	public function __fixDirSeparator($path)
 	{
-		if ($path != '' && (strripos($path, '/') != strlen($path) - 1))
-			$path .= '/';
+		if ($path != '' && (strripos($path, '/') != strlen($path) - 1)) {
+					$path .= '/';
+		}
 		return $path;
 	}
 
@@ -72,8 +74,9 @@ 
 	 */
 	public function __normalizePath($path)
 	{
-		if ($path && strpos($path, '\\') !== false)
-			$path = preg_replace("/\\\\/", "/", $path);
+		if ($path && strpos($path, '\\') !== false) {
+					$path = preg_replace("/\\\\/", "/", $path);
+		}
 		return $path;
 	}
 
@@ -83,18 +86,21 @@ 
 	public function copyDirectoryFromDisk($dirname, $zipdirname = null, $excludeList = null, $basedirname = null)
 	{
 		$dir = opendir($dirname);
-		if (strripos($dirname, '/') != strlen($dirname) - 1)
-			$dirname .= '/';
+		if (strripos($dirname, '/') != strlen($dirname) - 1) {
+					$dirname .= '/';
+		}
 
-		if ($basedirname == null)
-			$basedirname = realpath($dirname);
+		if ($basedirname == null) {
+					$basedirname = realpath($dirname);
+		}
 
 		while (false !== ($file = readdir($dir))) {
 			if ($file != '.' && $file != '..' &&
 				$file != '.svn' && $file != 'CVS') {
 				// Exclude the file/directory 
-				if (!empty($excludeList) && in_array("$dirname$file", $excludeList))
-					continue;
+				if (!empty($excludeList) && in_array("$dirname$file", $excludeList)) {
+									continue;
+				}
 
 				if (is_dir("$dirname$file")) {
 					$this->copyDirectoryFromDisk("$dirname$file", $zipdirname, $excludeList, $basedirname);

api.php

Spacing +10 added lines, -10 removed lines

@@ -11,38 +11,38 @@
 require_once 'config/api.php';
 $service = $_REQUEST['service'];
 
-if(isset($service))
+if (isset($service))
 {
-	if(!in_array($service,$enabledServices)){
-		die($service.' - Service is not active');
+	if (!in_array($service, $enabledServices)) {
+		die($service . ' - Service is not active');
 	}	
-	if($service == "customerportal")
+	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
 	{
-		echo "No Service Configured for ". strip_tags($service);
+		echo "No Service Configured for " . strip_tags($service);
 	}
 }
 else

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>";

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;

Doc Comments +3 added lines

@@ -31,6 +31,9 @@
 		file_put_contents(self::$usersFile, '<?php return ' . \vtlib\Functions::varExportMin($users) . ';');
 	}
 
+	/**
+	 * @param string $type
+	 */
 	public static function getUser($type)
 	{
 		if (self::$usersFileCache === false) {

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;
 		}

Doc Comments +32 added lines

@@ -127,6 +127,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $type
+	 */
 	public static function getDBConfig($type)
 	{
 		if (!self::$dbConfig) {
@@ -139,6 +142,14 @@ 
 		return false;
 	}
 
+	/**
+	 * @param string $dbtype
+	 * @param string $host
+	 * @param string $dbname
+	 * @param string $username
+	 * @param string $passwd
+	 * @param integer $port
+	 */
 	protected function loadDBConfig($dbtype, $host, $dbname, $username, $passwd, $port)
 	{
 		if ($host == '_SERVER_') {
@@ -232,6 +243,9 @@ 
 		$this->dieOnError = $value;
 	}
 
+	/**
+	 * @param boolean $value
+	 */
 	public function setAttribute($attribute, $value)
 	{
 		$this->database->setAttribute($attribute, $value);
@@ -296,6 +310,9 @@ 
 		return $result->columnCount();
 	}
 
+	/**
+	 * @return string
+	 */
 	public function fetch_array(&$result)
 	{
 		return $result->fetch(PDO::FETCH_ASSOC);
@@ -495,6 +512,9 @@ 
 		return $this->stmt->rowCount();
 	}
 
+	/**
+	 * @param integer $row
+	 */
 	public function query_result(&$result, $row, $col = 0)
 	{
 		return to_html($this->query_result_raw($result, $row, $col));
@@ -667,6 +687,9 @@ 
 		return $id;
 	}
 
+	/**
+	 * @param string $tableName
+	 */
 	public function checkExistTable($tableName, $cache = true)
 	{
 		$tablePresent = Vtiger_Cache::get('checkExistTable', $tableName);
@@ -695,6 +718,9 @@ 
 		return $lastInsertID;
 	}
 
+	/**
+	 * @param string $datetime
+	 */
 	public function formatDate($datetime, $strip_quotes = false)
 	{
 		/* remove single quotes to use the date as parameter for Prepared statement */
@@ -785,6 +811,9 @@ 
 		return ' ( ' . $l . ' ) ';
 	}
 
+	/**
+	 * @param PDOStatement $result
+	 */
 	public function getAffectedRowCount(&$result)
 	{
 		$rows = $result->rowCount();
@@ -854,6 +883,9 @@ 
 	protected $logSqlTimeID = false;
 	protected $logSqlTimeGroup = 1;
 
+	/**
+	 * @param double $endat
+	 */
 	public function logSqlTime($startat, $endat, $sql, $params = false)
 	{
 		if (!AppConfig::performance('SQL_LOG_INCLUDE_CALLER')) {

include/ChartUtils.php

Spacing +2 added lines, -2 removed lines

@@ -56,7 +56,7 @@ 
 				$x = "";
 				for ($j = 0; $j < count($val); $j++) {
 					if ($j != $n) {
-						$x .=" " . $val[$j];
+						$x .= " " . $val[$j];
 					} else {
 						$x .= "@#" . $val[$j];
 					}
@@ -240,7 +240,7 @@ 
 				$x = "";
 				for ($j = 0; $j < count($val); $j++) {
 					if ($j != $n) {
-						$x .=" " . $val[$j];
+						$x .= " " . $val[$j];
 					} else {
 						$x .= "@#" . $val[$j];
 					}

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') {

include/utils/EmailTemplate.php

Spacing +2 added lines, -2 removed lines

@@ -120,7 +120,7 @@ 
 				$moduleTableIndexList = $meta->getEntityTableIndexList();
 				foreach ($tableList as $index => $tableName) {
 					if ($tableName != $tableList[0]) {
-						$sql .=' INNER JOIN ' . $tableName . ' ON ' . $tableList[0] . '.' .
+						$sql .= ' INNER JOIN ' . $tableName . ' ON ' . $tableList[0] . '.' .
 							$moduleTableIndexList[$tableList[0]] . '=' . $tableName . '.' .
 							$moduleTableIndexList[$tableName];
 					}
@@ -128,7 +128,7 @@ 
 				//If module is Leads and if you are not selected any leads fields then query failure is happening.
 				//By default we are checking where condition on base table.
 				if ($module == 'Leads' && !in_array('vtiger_leaddetails', $tableList)) {
-					$sql .=' INNER JOIN vtiger_leaddetails ON vtiger_leaddetails.leadid = vtiger_crmentity.crmid';
+					$sql .= ' INNER JOIN vtiger_leaddetails ON vtiger_leaddetails.leadid = vtiger_crmentity.crmid';
 				}
 
 				$sql .= ' WHERE';

Braces +6 added lines, -4 removed lines

@@ -209,8 +209,9 @@ 
 						$needle = '$' . strtolower($this->module) . '-' . $parentFieldColumnMapping[$params['field']] . ':' . $column . '$';
 						$this->processedDescription = str_replace($needle, $values[array_search($column, $fieldColumnMapping)], $this->processedDescription);
 					}
-					if (!$params['owner'])
-						$this->processedmodules[$module] = true;
+					if (!$params['owner']) {
+											$this->processedmodules[$module] = true;
+					}
 				}
 			}
 		}
@@ -253,7 +254,8 @@ 
 		$rows = $adb->num_rows($res);
 		if ($rows > 0) {
 			return true;
-		} else
-			return false;
+		} else {
+					return false;
+		}
 	}
 }

include/utils/ConfigReader.php

Braces +6 added lines, -4 removed lines

@@ -37,8 +37,9 @@ 
 
 	public function setConfig($key, $value)
 	{
-		if (empty($key))
-			return;
+		if (empty($key)) {
+					return;
+		}
 		$this->properties[$key] = $value;
 		//not neccessary for php5.x versions
 		self::$propertiesCache[$this->name] = $this->properties;
@@ -46,8 +47,9 @@ 
 
 	public function getConfig($key)
 	{
-		if (empty($key))
-			return '';
+		if (empty($key)) {
+					return '';
+		}
 		return $this->properties[$key];
 	}
 }

Doc Comments +7 added lines

@@ -15,6 +15,10 @@ 
 	protected $name;
 	static $propertiesCache = [];
 
+	/**
+	 * @param string $path
+	 * @param string $name
+	 */
 	public function __construct($path, $name, $force = false)
 	{
 		$this->load($path, $name, $force);
@@ -41,6 +45,9 @@ 
 		self::$propertiesCache[$this->name] = $this->properties;
 	}
 
+	/**
+	 * @param string $key
+	 */
 	public function getConfig($key)
 	{
 		if (empty($key))