YetiForceCompany / YetiForceCRM

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/LettersOut/LettersOut.php

Braces +14 added lines, -10 removed lines

@@ -114,8 +114,9 @@ 
 		$joinedTables = array();
 
 		// Select Custom Field Table Columns if present
-		if (!empty($this->customFieldTable))
-			$query .= ", " . $this->customFieldTable[0] . ".* ";
+		if (!empty($this->customFieldTable)) {
+					$query .= ", " . $this->customFieldTable[0] . ".* ";
+		}
 
 		$query .= " FROM $this->table_name";
 
@@ -250,10 +251,11 @@ 
 		$query .= $this->getNonAdminAccessControlQuery($thismodule, $current_user);
 		$where_auto = " vtiger_crmentity.deleted=0";
 
-		if ($where != '')
-			$query .= " WHERE ($where) && $where_auto";
-		else
-			$query .= " WHERE $where_auto";
+		if ($where != '') {
+					$query .= " WHERE ($where) && $where_auto";
+		} else {
+					$query .= " WHERE $where_auto";
+		}
 
 		return $query;
 	}
@@ -274,8 +276,9 @@ 
 		$select_clause = sprintf('SELECT %s.%s AS recordid, vtiger_users_last_import.deleted, %s', $this->table_name, $this->table_index, $table_cols);
 
 		// Select Custom Field Table Columns if present
-		if (isset($this->customFieldTable))
-			$query .= ", " . $this->customFieldTable[0] . ".* ";
+		if (isset($this->customFieldTable)) {
+					$query .= ", " . $this->customFieldTable[0] . ".* ";
+		}
 
 		$from_clause = " FROM $this->table_name";
 
@@ -328,8 +331,9 @@ 
 			$modcommentsModuleInstance = vtlib\Module::getInstance('ModComments');
 			if ($modcommentsModuleInstance && file_exists('modules/ModComments/ModComments.php')) {
 				include_once 'modules/ModComments/ModComments.php';
-				if (class_exists('ModComments'))
-					ModComments::addWidgetTo(array('LettersOut'));
+				if (class_exists('ModComments')) {
+									ModComments::addWidgetTo(array('LettersOut'));
+				}
 			}
 
 			$modcommentsModuleInstance = vtlib\Module::getInstance('ModTracker');

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/ChartSaveAjax.php

Braces +3 added lines, -2 removed lines

@@ -44,8 +44,9 @@
 		}
 
 		$dataFields = $request->get('datafields', 'count(*)');
-		if (is_string($dataFields))
-			$dataFields = array($dataFields);
+		if (is_string($dataFields)) {
+					$dataFields = array($dataFields);
+		}
 
 		$reportModel->set('reporttypedata', \includes\utils\Json::encode(array(
 				'type' => $request->get('charttype', 'pieChart'),

modules/Reports/views/ChartEdit.php

Braces +3 added lines, -2 removed lines

@@ -98,8 +98,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/OSSPasswords/OSSPasswords.php

Spacing +1 added lines, -1 removed lines

@@ -336,7 +336,7 @@
 		} else if ($eventType == 'module.preuninstall') {
 			// TODO Handle actions when this module is about to be deleted.
 			$this->log->debug('Before starting uninstall script...');
-			require_once( 'modules/Settings/' . $moduleName . '/views/uninstall.php' );
+			require_once('modules/Settings/' . $moduleName . '/views/uninstall.php');
 			$this->log->debug('After uninstall script.');
 
 			header('Location: index.php?module=Vtiger&parent=Settings&view=Index');

Braces +14 added lines, -10 removed lines

@@ -115,8 +115,9 @@ 
 		$query = "SELECT vtiger_crmentity.*, $this->table_name.*";
 
 		// Select Custom Field Table Columns if present
-		if (!empty($this->customFieldTable))
-			$query .= ", " . $this->customFieldTable[0] . ".* ";
+		if (!empty($this->customFieldTable)) {
+					$query .= ", " . $this->customFieldTable[0] . ".* ";
+		}
 
 		$query .= " FROM $this->table_name";
 
@@ -224,10 +225,11 @@ 
 
 		$where_auto = " vtiger_crmentity.deleted=0";
 
-		if ($where != '')
-			$query .= " WHERE ($where) && $where_auto";
-		else
-			$query .= " WHERE $where_auto";
+		if ($where != '') {
+					$query .= " WHERE ($where) && $where_auto";
+		} else {
+					$query .= " WHERE $where_auto";
+		}
 
 		require('user_privileges/user_privileges_' . $current_user->id . '.php');
 		require('user_privileges/sharing_privileges_' . $current_user->id . '.php');
@@ -245,8 +247,9 @@ 
 	 */
 	public function transform_export_value($key, $value)
 	{
-		if ($key == 'owner')
-			return \App\Fields\Owner::getLabel($value);
+		if ($key == 'owner') {
+					return \App\Fields\Owner::getLabel($value);
+		}
 		return parent::transform_export_value($key, $value);
 	}
 
@@ -258,8 +261,9 @@ 
 		$select_clause = sprintf('SELECT %s.%s AS recordid, vtiger_users_last_import.deleted, %s', $this->table_name, $this->table_index, $table_cols);
 
 		// Select Custom Field Table Columns if present
-		if (isset($this->customFieldTable))
-			$query .= ", " . $this->customFieldTable[0] . ".* ";
+		if (isset($this->customFieldTable)) {
+					$query .= ", " . $this->customFieldTable[0] . ".* ";
+		}
 
 		$from_clause = " FROM $this->table_name";
 

modules/PaymentsIn/PaymentsIn.php

Braces +3 added lines, -2 removed lines

@@ -105,8 +105,9 @@
 			$modcommentsModuleInstance = vtlib\Module::getInstance('ModComments');
 			if ($modcommentsModuleInstance && file_exists('modules/ModComments/ModComments.php')) {
 				include_once 'modules/ModComments/ModComments.php';
-				if (class_exists('ModComments'))
-					ModComments::addWidgetTo(array('Payments'));
+				if (class_exists('ModComments')) {
+									ModComments::addWidgetTo(array('Payments'));
+				}
 			}
 			$adb->pquery('UPDATE vtiger_tab SET customized=0 WHERE name=?', array($modulename));
 			$tabid = vtlib\Functions::getModuleId($modulename);

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)

Doc Comments +9 added lines

@@ -20,6 +20,9 @@ 
 			$this->parseLine($line);
 	}
 
+	/**
+	 * @param string $line
+	 */
 	protected function parseLine($line)
 	{
 		$tag = substr($line, 1, strpos($line, ':', 1) - 1);
@@ -65,6 +68,9 @@ 
 		$this->_lastTag = $tag;
 	}
 
+	/**
+	 * @param string $value
+	 */
 	protected function parseTransaction($value)
 	{
 		$transaction = array(
@@ -109,6 +115,9 @@ 
 		$this->operations[count($this->operations) - 1]['details'] = $transaction;
 	}
 
+	/**
+	 * @param string $value
+	 */
 	protected function parseOperation($value)
 	{
 		$this->operations[] = array(