YetiForceCompany / YetiForceCRM

modules/Settings/CurrencyUpdate/models/bankmodels/ECB.php

Spacing +1 added lines, -1 removed lines

@@ -128,7 +128,7 @@
 			if ($time["time"] == $dateParam) {
 				$num = count($time->Cube);
 				for ($i = 0; $i < $num; $i++) {
-					$currency = (string) $time->Cube[$i]['currency'];   // currency code
+					$currency = (string) $time->Cube[$i]['currency']; // currency code
 					foreach ($otherCurrencyCode as $key => $currId) {
 						if ($key == $currency && $currency != $mainCurrency) {
 							$exchange = $time->Cube[$i]['rate'];

Braces +5 added lines, -4 removed lines

@@ -83,10 +83,11 @@
 		$rateDay = strtotime($dateParam);
 		$datediff = $now - $rateDay;
 
-		if (floor($datediff / (60 * 60 * 24)) >= 90)
-			$sourceURL = $source[1];
-		else
-			$sourceURL = $source[0];
+		if (floor($datediff / (60 * 60 * 24)) >= 90) {
+					$sourceURL = $source[1];
+		} else {
+					$sourceURL = $source[0];
+		}
 
 		$XML = simplexml_load_file($sourceURL); // European Central Bank xml only contains business days! oh well....
 

modules/Settings/CurrencyUpdate/views/Index.php

Braces +4 added lines, -4 removed lines

@@ -26,11 +26,11 @@
 		$dateCur = '';
 		if ($date) {
 			// if its future date change it to present one
-			if (strtotime($date) > strtotime(date('Y-m-d')))
-				$date = date('Y-m-d');
+			if (strtotime($date) > strtotime(date('Y-m-d'))) {
+							$date = date('Y-m-d');
+			}
 			$dateCur = $date;
-		}
-		else {
+		} else {
 			$dateCur = date('Y-m-d');
 		}
 

modules/Reports/actions/Save.php

Braces +3 added lines, -2 removed lines

@@ -40,8 +40,9 @@
 		}
 
 		$reporttype = $request->get('reporttype');
-		if (empty($reporttype))
-			$reporttype = 'tabular';
+		if (empty($reporttype)) {
+					$reporttype = 'tabular';
+		}
 		$reportModel->set('reportname', $request->get('reportname'));
 		$reportModel->set('folderid', $request->get('folderid'));
 		$reportModel->set('description', $request->get('reports_description'));

modules/Reports/models/Chart.php

Braces +15 added lines, -10 removed lines

@@ -49,8 +49,9 @@ 
 	public function getChartType()
 	{
 		$type = $this->get('type');
-		if (empty($type))
-			$type = 'pieChart';
+		if (empty($type)) {
+					$type = 'pieChart';
+		}
 		return $type;
 	}
 
@@ -143,8 +144,9 @@ 
 
 	public function setQueryColumns($columns)
 	{
-		if ($columns && is_string($columns))
-			$columns = array($columns);
+		if ($columns && is_string($columns)) {
+					$columns = array($columns);
+		}
 
 		if (is_array($columns)) {
 			foreach ($columns as $column) {
@@ -187,14 +189,16 @@ 
 				}
 			}
 		}
-		if ($fieldModels)
-			$this->fieldModels = $fieldModels;
+		if ($fieldModels) {
+					$this->fieldModels = $fieldModels;
+		}
 	}
 
 	public function setGroupByColumns($columns)
 	{
-		if ($columns && is_string($columns))
-			$columns = array($columns);
+		if ($columns && is_string($columns)) {
+					$columns = array($columns);
+		}
 
 		if (is_array($columns)) {
 			foreach ($columns as $column) {
@@ -226,8 +230,9 @@ 
 				}
 			}
 		}
-		if ($fieldModels)
-			$this->groupByFieldModels = $fieldModels;
+		if ($fieldModels) {
+					$this->groupByFieldModels = $fieldModels;
+		}
 	}
 
 	public function getGroupbyColumnsByFieldModel()

modules/Leads/models/Record.php

Braces +6 added lines, -4 removed lines

@@ -170,12 +170,14 @@
 
 			for ($i = 0; $i < $numOfRows; $i++) {
 				$row = $db->query_result_rowdata($result, $i);
-				if (empty($row['leadfid']))
-					continue;
+				if (empty($row['leadfid'])) {
+									continue;
+				}
 
 				$leadFieldInstance = $leadFieldInstances[$row['leadfid']];
-				if (!$leadFieldInstance)
-					continue;
+				if (!$leadFieldInstance) {
+									continue;
+				}
 
 				$leadFieldName = $leadFieldInstance->getName();
 				$accountFieldInstance = $accountFieldInstances[$row['accountfid']];

Doc Comments +1 added lines

@@ -76,6 +76,7 @@
 	/**
 	 * Function returns field mapped to Leads field, used in Lead Convert for settings the field values
 	 * @param string $fieldName
+	 * @param string $moduleName
 	 * @return string
 	 */
 	public function getConvertLeadMappedField($fieldName, $moduleName)

modules/Products/views/Popup.php

Braces +6 added lines, -4 removed lines

@@ -47,8 +47,9 @@ 
 
 		$pagingModel = new Vtiger_Paging_Model();
 		$pagingModel->set('page', $pageNumber);
-		if (vglobal('popupAjax'))
-			$pagingModel->set('noLimit', true);
+		if (vglobal('popupAjax')) {
+					$pagingModel->set('noLimit', true);
+		}
 
 		$moduleModel = Vtiger_Module_Model::getInstance($moduleName);
 		$recordStructureInstance = Vtiger_RecordStructure_Model::getInstanceForModule($moduleModel);
@@ -102,8 +103,9 @@ 
 		// Limit the choice of products/services only to the ones related to currently selected Opportunity - second step.
 		if (Settings_SalesProcesses_Module_Model::checkRelatedToPotentialsLimit($sourceModule)) {
 			$salesProcessId = $request->get('salesprocessid');
-			if (empty($salesProcessId))
-				$salesProcessId = -1;
+			if (empty($salesProcessId)) {
+							$salesProcessId = -1;
+			}
 			$listViewModel->set('salesprocessid', $salesProcessId);
 			$viewer->assign('INVENTORY_LIMITED_FROM_POTENTIALS', true);
 		}

modules/ServiceContracts/ServiceContracts.php

Braces +17 added lines, -12 removed lines

@@ -124,8 +124,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";
 
@@ -299,10 +300,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;
 	}
@@ -315,8 +317,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";
 
@@ -406,8 +409,9 @@ 
 	 */
 	public function save_related_module($module, $crmid, $with_module, $with_crmids, $relatedName = false)
 	{
-		if (!is_array($with_crmids))
-			$with_crmids = Array($with_crmids);
+		if (!is_array($with_crmids)) {
+					$with_crmids = Array($with_crmids);
+		}
 		foreach ($with_crmids as $with_crmid) {
 			if ($with_module == 'HelpDesk') {
 				parent::save_related_module($module, $crmid, $with_module, $with_crmid);
@@ -423,8 +427,9 @@ 
 	public function updateHelpDeskRelatedTo($focusId, $entityIds)
 	{
 
-		if (!is_array($entityIds))
-			$entityIds = array($entityIds);
+		if (!is_array($entityIds)) {
+					$entityIds = array($entityIds);
+		}
 		$selectTicketsQuery = sprintf('SELECT ticketid FROM vtiger_troubletickets
 								WHERE (parent_id IS NULL || parent_id = 0)
 									AND ticketid IN (%s)', generateQuestionMarks($entityIds));

modules/OSSPasswords/actions/SaveAjax.php

Braces +3 added lines, -2 removed lines

@@ -43,8 +43,9 @@
 			$salutationType = $recordModel->getDisplayValue('salutationtype');
 			$firstNameDetails = $result['firstname'];
 			$firstNameDetails['display_value'] = $salutationType . " " . $firstNameDetails['display_value'];
-			if ($salutationType != '--None--')
-				$result['firstname'] = $firstNameDetails;
+			if ($salutationType != '--None--') {
+							$result['firstname'] = $firstNameDetails;
+			}
 		}
 
 		$result['_recordLabel'] = $recordModel->getName();

modules/CallHistory/models/Module.php

Braces +5 added lines, -4 removed lines

@@ -32,10 +32,11 @@
 	 */
 	public function isPermitted($actionName)
 	{
-		if ($actionName == 'EditView' || $actionName == 'Edit' || $actionName == 'CreateView')
-			return false;
-		else
-			return ($this->isActive() && Users_Privileges_Model::isPermitted($this->getName(), $actionName));
+		if ($actionName == 'EditView' || $actionName == 'Edit' || $actionName == 'CreateView') {
+					return false;
+		} else {
+					return ($this->isActive() && Users_Privileges_Model::isPermitted($this->getName(), $actionName));
+		}
 	}
 
 	/**