YetiForceCompany / YetiForceCRM

modules/WSAPP/SyncServer.php

Braces +53 added lines, -40 removed lines

@@ -31,8 +31,9 @@ 
 	{
 		$db = PearDatabase::getInstance();
 		$appidresult = $db->pquery("SELECT appid FROM vtiger_wsapp WHERE appkey=?", array($key));
-		if ($db->num_rows($appidresult))
-			return $db->query_result($appidresult, 0, 'appid');
+		if ($db->num_rows($appidresult)) {
+					return $db->query_result($appidresult, 0, 'appid');
+		}
 		return false;
 	}
 
@@ -42,8 +43,9 @@ 
 	 */
 	public function idmap_get_clientmap($appid, $serverids)
 	{
-		if (!is_array($serverids))
-			$serverids = array($serverids);
+		if (!is_array($serverids)) {
+					$serverids = array($serverids);
+		}
 		$db = PearDatabase::getInstance();
 		;
 		$result = $db->pquery(sprintf(
@@ -65,8 +67,9 @@ 
 	 */
 	public function idmap_get_clientservermap($appid, $clientids)
 	{
-		if (!is_array($clientids))
-			$clientids = array($clientids);
+		if (!is_array($clientids)) {
+					$clientids = array($clientids);
+		}
 
 		$db = PearDatabase::getInstance();
 
@@ -83,8 +86,9 @@ 
 
 	public function idmap_storeRecordsInQueue($syncServerId, $recordDetails, $flag, $appid)
 	{
-		if (!is_array($recordDetails))
-			$recordDetails = array($recordDetails);
+		if (!is_array($recordDetails)) {
+					$recordDetails = array($recordDetails);
+		}
 		$db = PearDatabase::getInstance();
 		$params = array();
 		$params[] = $syncServerId;
@@ -99,8 +103,9 @@ 
 		$db = PearDatabase::getInstance();
 		$checkQuery = "SELECT syncserverid FROM vtiger_wsapp_queuerecords WHERE syncserverid=?";
 		$result = $db->pquery($checkQuery, array($syncServerId));
-		if ($db->num_rows($result) > 0)
-			return true;
+		if ($db->num_rows($result) > 0) {
+					return true;
+		}
 		return false;
 	}
 
@@ -148,19 +153,20 @@ 
 	public function idmap_put($appid, $serverid, $clientid, $clientModifiedTime, $serverModifiedTime, $serverAppId, $mode = "save")
 	{
 		$db = PearDatabase::getInstance();
-		if ($mode == $this->create)
-			$this->idmap_create($appid, $serverid, $clientid, $clientModifiedTime, $serverModifiedTime, $serverAppId);
-		else if ($mode == $this->update)
-			$this->idmap_update($appid, $serverid, $clientid, $clientModifiedTime, $serverModifiedTime, $serverAppId);
-		else if ($mode == $this->save) {
+		if ($mode == $this->create) {
+					$this->idmap_create($appid, $serverid, $clientid, $clientModifiedTime, $serverModifiedTime, $serverAppId);
+		} else if ($mode == $this->update) {
+					$this->idmap_update($appid, $serverid, $clientid, $clientModifiedTime, $serverModifiedTime, $serverAppId);
+		} else if ($mode == $this->save) {
 			$result = $db->pquery("SELECT * FROM vtiger_wsapp_recordmapping WHERE appid=? and serverid=? and clientid=?", array($appid, $serverid, $clientid));
-			if ($db->num_rows($result) <= 0)
-				$this->idmap_create($appid, $serverid, $clientid, $clientModifiedTime, $serverModifiedTime, $serverAppId);
-			else
-				$this->idmap_update($appid, $serverid, $clientid, $clientModifiedTime, $serverModifiedTime, $serverAppId);
+			if ($db->num_rows($result) <= 0) {
+							$this->idmap_create($appid, $serverid, $clientid, $clientModifiedTime, $serverModifiedTime, $serverAppId);
+			} else {
+							$this->idmap_update($appid, $serverid, $clientid, $clientModifiedTime, $serverModifiedTime, $serverAppId);
+			}
+		} else if ($mode == $this->delete) {
+					$this->idmap_delete($appid, $serverid, $clientid, $serverAppId);
 		}
-		else if ($mode == $this->delete)
-			$this->idmap_delete($appid, $serverid, $clientid, $serverAppId);
 	}
 
 	/**
@@ -229,11 +235,13 @@ 
 		if (empty($type)) {
 			throw new WebServiceException('WSAPP06', "No sync type specified");
 		}
-		if (is_array($name))
-			throw new WebServiceException("WSAPP07", "type is in the wrong format");
+		if (is_array($name)) {
+					throw new WebServiceException("WSAPP07", "type is in the wrong format");
+		}
 		$type = strtolower($type);
-		if (!in_array($type, $this->syncTypes))
-			throw new WebServiceException('WSAPP05', "Wrong sync type specified");
+		if (!in_array($type, $this->syncTypes)) {
+					throw new WebServiceException('WSAPP05', "Wrong sync type specified");
+		}
 		$db = PearDatabase::getInstance();
 		$uid = uniqid();
 		$db->pquery("INSERT INTO vtiger_wsapp (name, appkey,type) VALUES(?,?,?)", array($name, $uid, $type));
@@ -267,10 +275,11 @@ 
 			throw new WebServiceException('WSAPP04', "Access restricted to app");
 		}
 
-		if (!is_array($element))
-			$records = array($element);
-		else
-			$records = $element;
+		if (!is_array($element)) {
+					$records = array($element);
+		} else {
+					$records = $element;
+		}
 
 		//hardcoded since the destination handler will be vtigerCRM
 		$serverKey = wsapp_getAppKey("vtigerCRM");
@@ -293,13 +302,15 @@ 
 			$clientRecordId = $record['id'];
 
 			// Missing client record id?
-			if (empty($clientRecordId))
-				continue;
+			if (empty($clientRecordId)) {
+							continue;
+			}
 
 			$lookupRecordId = false;
 			$lookupResult = $db->pquery("SELECT serverid,clientmodifiedtime FROM vtiger_wsapp_recordmapping WHERE appid=? && clientid=?", array($appid, $clientRecordId));
-			if ($db->num_rows($lookupResult))
-				$lookupRecordId = $db->query_result($lookupResult, 0, 'serverid');
+			if ($db->num_rows($lookupResult)) {
+							$lookupRecordId = $db->query_result($lookupResult, 0, 'serverid');
+			}
 			if (empty($lookupRecordId) && $record['mode'] != "delete") {
 				$createRecords[$clientRecordId] = $record['values'];
 				$createRecords[$clientRecordId]['module'] = $record['module'];
@@ -309,8 +320,9 @@ 
 					$deleteRecords[$clientRecordId] = $lookupRecordId;
 				} else if (!(empty($lookupRecordId))) {
 					$clientLastModifiedTime = $db->query_result($lookupResult, 0, 'clientmodifiedtime');
-					if ($clientLastModifiedTime >= $record['values']['modifiedtime'])
-						continue;
+					if ($clientLastModifiedTime >= $record['values']['modifiedtime']) {
+											continue;
+					}
 					$record['values']['id'] = $lookupRecordId;
 					$updateRecords[$clientRecordId] = $record['values'];
 					$updateRecords[$clientRecordId]['module'] = $record['module'];
@@ -446,8 +458,9 @@ 
 	 */
 	public function map($key, $element, $user)
 	{
-		if (empty($element))
-			return;
+		if (empty($element)) {
+					return;
+		}
 		$db = PearDatabase::getInstance();
 		$appid = $this->appid_with_key($key);
 		$createDetails = $element["create"];
@@ -501,9 +514,9 @@ 
 
 	public function convertToQueueRecordFormat($record, $flag)
 	{
-		if ($flag != $this->delete)
-			return $record;
-		else {
+		if ($flag != $this->delete) {
+					return $record;
+		} else {
 			$recordFormat = array();
 			$recordFormat['id'] = $record['id'];
 			return $recordFormat;

modules/WSAPP/WSAPP.php

Braces +3 added lines, -2 removed lines

@@ -124,8 +124,9 @@
 		$appTypehandler['handlerpath'] = "modules/WSAPP/Handlers/vtigerCRMHandler.php";
 		$handlerDetails[] = $appTypehandler;
 
-		foreach ($handlerDetails as $appHandlerDetails)
-			$adb->pquery("INSERT INTO vtiger_wsapp_handlerdetails VALUES(?,?,?)", array($appHandlerDetails['type'], $appHandlerDetails['handlerclass'], $appHandlerDetails['handlerpath']));
+		foreach ($handlerDetails as $appHandlerDetails) {
+					$adb->pquery("INSERT INTO vtiger_wsapp_handlerdetails VALUES(?,?,?)", array($appHandlerDetails['type'], $appHandlerDetails['handlerclass'], $appHandlerDetails['handlerpath']));
+		}
 	}
 
 	public function registerVtigerCRMApp()

modules/Assets/views/EditFieldByModal.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
 		}
 		$showFields = array_keys($recordModel->getModule()->getQuickCreateFields());
 		$configureFields = AppConfig::module($moduleName, 'SHOW_FIELD_IN_MODAL');
-		if($configureFields){
+		if ($configureFields) {
 			$showFields = array_merge($showFields, $configureFields);
 		}
 

modules/Reports/views/Edit.php

Braces +3 added lines, -2 removed lines

@@ -97,8 +97,9 @@
 			$viewer->assign('MODE', 'edit');
 		} else {
 			$firstModuleName = reset($modulesList);
-			if ($firstModuleName)
-				$reportModel->setPrimaryModule($firstModuleName);
+			if ($firstModuleName) {
+							$reportModel->setPrimaryModule($firstModuleName);
+			}
 			$viewer->assign('MODE', '');
 		}
 

modules/Reports/ReportUtils.php

Braces +8 added lines, -6 removed lines

@@ -15,8 +15,9 @@ 
 function getFieldByReportLabel($module, $label)
 {
 	$cacheLabel = VTCacheUtils::getReportFieldByLabel($module, $label);
-	if ($cacheLabel)
-		return $cacheLabel;
+	if ($cacheLabel) {
+			return $cacheLabel;
+	}
 
 	// this is required so the internal cache is populated or reused.
 	getColumnFields($module);
@@ -27,10 +28,11 @@ 
 	if ($module == 'Calendar') {
 		$cachedEventsFields = VTCacheUtils::lookupFieldInfo_Module('Events');
 		if ($cachedEventsFields) {
-			if (empty($cachedModuleFields))
-				$cachedModuleFields = $cachedEventsFields;
-			else
-				$cachedModuleFields = array_merge($cachedModuleFields, $cachedEventsFields);
+			if (empty($cachedModuleFields)) {
+							$cachedModuleFields = $cachedEventsFields;
+			} else {
+							$cachedModuleFields = array_merge($cachedModuleFields, $cachedEventsFields);
+			}
 		}
 		if ($label == 'Start_Date_and_Time') {
 			$label = 'Start_Date_&_Time';

modules/Project/views/Detail.php

Braces +3 added lines, -2 removed lines

@@ -26,8 +26,9 @@
 
 		$viewer = $this->getViewer($request);
 		$moduleModel = Vtiger_Module_Model::getInstance('OSSTimeControl');
-		if ($moduleModel)
-			$data = $moduleModel->getTimeUsers($recordId, $moduleName);
+		if ($moduleModel) {
+					$data = $moduleModel->getTimeUsers($recordId, $moduleName);
+		}
 		$viewer->assign('MODULE_NAME', $moduleName);
 		$viewer->assign('DATA', $data);
 		$viewer->view('charts/ShowTimeProjectUsers.tpl', $moduleName);

modules/PaymentsIn/helpers/mt940.php

Braces +19 added lines, -14 removed lines

@@ -34,8 +34,9 @@ 
 	public function parse()
 	{
 		$tab = $this->prepareFile();
-		foreach ($tab as $line)
-			$this->parseLine($line);
+		foreach ($tab as $line) {
+					$this->parseLine($line);
+		}
 	}
 
 	public function getXML()
@@ -53,13 +54,15 @@ 
 	protected function createXML($key, $value, $level)
 	{
 		$indent = '';
-		for ($i = 0; $i <= $level; $i++)
-			$indent .= "\t";
+		for ($i = 0; $i <= $level; $i++) {
+					$indent .= "\t";
+		}
 		if (is_array($value)) {
 			$xml = "$indent<$key>\n";
 			foreach ($value as $subKey => $subVal) {
-				if (is_numeric($subKey))
-					$subKey = substr($key, 0, -1);
+				if (is_numeric($subKey)) {
+									$subKey = substr($key, 0, -1);
+				}
 				$xml .= $this->createXML($subKey, $subVal, $level + 1);
 			}
 			$xml .= "$indent</$key>\n";
@@ -85,10 +88,11 @@ 
 				break;
 			case 'NS':
 				$code = substr($value, 0, 2);
-				if ($code == '22')
-					$this->ownerName = substr($value, 2);
-				else if ($code == '23')
-					$this->accountName = substr($value, 2);
+				if ($code == '22') {
+									$this->ownerName = substr($value, 2);
+				} else if ($code == '23') {
+									$this->accountName = substr($value, 2);
+				}
 				break;
 			case '60F':
 				$this->openBalance = $this->parseBalance($value);
@@ -103,10 +107,11 @@ 
 				self::parseOperation($value);
 				break;
 			case '86':
-				if ($this->_lastTag == '61')
-					$this->parseTransaction($value);
-				else
-					$this->info .= $value;
+				if ($this->_lastTag == '61') {
+									$this->parseTransaction($value);
+				} else {
+									$this->info .= $value;
+				}
 				break;
 			default:
 				break;

modules/PaymentsIn/helpers/subclass/mt940_BGK.php

Braces +11 added lines, -8 removed lines

@@ -16,8 +16,9 @@ 
 	public function parse()
 	{
 		$tab = $this->prepareFile();
-		foreach ($tab as $line)
-			$this->parseLine($line);
+		foreach ($tab as $line) {
+					$this->parseLine($line);
+		}
 	}
 
 	protected function parseLine($line)
@@ -36,10 +37,11 @@ 
 				break;
 			case 'NS':
 				$code = substr($value, 0, 2);
-				if ($code == '22')
-					$this->ownerName = substr($value, 2);
-				else if ($code == '23')
-					$this->accountName = substr($value, 2);
+				if ($code == '22') {
+									$this->ownerName = substr($value, 2);
+				} else if ($code == '23') {
+									$this->accountName = substr($value, 2);
+				}
 				break;
 			case '60F':
 				$this->openBalance = $this->parseBalance($value);
@@ -62,8 +64,9 @@ 
 			default:
 				break;
 		}
-		if ($tag)
-			$this->_lastTag = $tag;
+		if ($tag) {
+					$this->_lastTag = $tag;
+		}
 	}
 
 	protected function parseTransaction($value)

modules/PaymentsIn/helpers/subclass/mt940_SGB.php

Braces +16 added lines, -12 removed lines

@@ -18,8 +18,9 @@ 
 	public function parse()
 	{
 		$tab = $this->prepareFile();
-		foreach ($tab as $line)
-			$this->parseLine($line);
+		foreach ($tab as $line) {
+					$this->parseLine($line);
+		}
 	}
 
 	protected function parseLine($line)
@@ -38,10 +39,11 @@ 
 				break;
 			case 'NS':
 				$code = substr($value, 0, 2);
-				if ($code == '22')
-					$this->ownerName = substr($value, 2);
-				else if ($code == '23')
-					$this->accountName = substr($value, 2);
+				if ($code == '22') {
+									$this->ownerName = substr($value, 2);
+				} else if ($code == '23') {
+									$this->accountName = substr($value, 2);
+				}
 				break;
 			case '60F':
 				$this->openBalance = $this->parseBalance($value);
@@ -56,16 +58,18 @@ 
 				self::parseOperation($value);
 				break;
 			case '86':
-				if ($this->_lastTag == '61')
-					$this->parseTransaction($value);
-				else
-					$this->info .= $value;
+				if ($this->_lastTag == '61') {
+									$this->parseTransaction($value);
+				} else {
+									$this->info .= $value;
+				}
 				break;
 			default:
 				break;
 		}
-		if ($tag)
-			$this->_lastTag = $tag;
+		if ($tag) {
+					$this->_lastTag = $tag;
+		}
 	}
 
 	protected function parseTransaction($value)