YetiForceCompany / YetiForceCRM

modules/PaymentsIn/helpers/subclass/mt940_BPH.php

Braces +13 added lines, -10 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);
@@ -54,10 +56,11 @@ 
 				self::parseOperation($value);
 				break;
 			case '86':
-				if ($this->_lastTag == 'NS')
-					$this->parseTransaction($value);
-				else
-					$this->info .= $value;
+				if ($this->_lastTag == 'NS') {
+									$this->parseTransaction($value);
+				} else {
+									$this->info .= $value;
+				}
 				break;
 			default:
 				break;

modules/PaymentsIn/helpers/subclass/mt940_ing.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,10 +58,11 @@ 
 				self::parseOperation($value);
 				break;
 			case '86':
-				if ($this->_lastTag == '86')
-					$this->parseTransaction($value);
-				else
-					$this->info .= $value;
+				if ($this->_lastTag == '86') {
+									$this->parseTransaction($value);
+				} else {
+									$this->info .= $value;
+				}
 				$this->lastLine = $value;
 				break;
 			default:
@@ -79,8 +82,9 @@ 
 			'contAccount' => '',
 			'currancy' => ''
 		);
-		if ($transaction['code'] == '940')
-			return;
+		if ($transaction['code'] == '940') {
+					return;
+		}
 		$delimiter = substr($value, 3, 1);
 		$transaction['currancy'] = substr($this->lastLine, 9, 3);
 		$transaction['currancyAmount'] = substr($this->lastLine, 12, 14);

modules/PaymentsIn/views/step1.php

Braces +3 added lines, -2 removed lines

@@ -37,8 +37,9 @@
 		$j = array();
 		foreach ($recordParse->operations as $transfers) {
 			foreach ($transfers as $key => $value) {
-				if ($key == 'indicator' && $value == 'C')
-					$paymentsIn[] = $transfers;
+				if ($key == 'indicator' && $value == 'C') {
+									$paymentsIn[] = $transfers;
+				}
 				if ($key == 'third_letter_currency_code') {
 					$j[] = $i;
 				}

modules/SMSNotifier/providers/ClickATell.php

Braces +3 added lines, -2 removed lines

@@ -125,8 +125,9 @@
 		$i = 0;
 		foreach ($responseLines as $responseLine) {
 			$responseLine = trim($responseLine);
-			if (empty($responseLine))
-				continue;
+			if (empty($responseLine)) {
+							continue;
+			}
 
 			$result = array('error' => false, 'statusmessage' => '');
 			if (preg_match("/ERR:(.*)/", trim($responseLine), $matches)) {

modules/SMSNotifier/providers/MyProvider.php

Braces +3 added lines, -2 removed lines

@@ -124,8 +124,9 @@
 		$results = array();
 		foreach ($responseLines as $responseLine) {
 			$responseLine = trim($responseLine);
-			if (empty($responseLine))
-				continue;
+			if (empty($responseLine)) {
+							continue;
+			}
 
 			$result = array('error' => false, 'statusmessage' => '');
 			if (preg_match("/ERR:(.*)/", trim($responseLine), $matches)) {

modules/SMSNotifier/SMSNotifierBase.php

Braces +21 added lines, -16 removed lines

@@ -98,10 +98,11 @@ 
 		$currentModule = vglobal('currentModule');
 
 		$sortorder = $this->default_sort_order;
-		if (!AppRequest::isEmpty('sorder'))
-			$sortorder = AppRequest::get('sorder');
-		else if ($_SESSION[$currentModule . '_Sort_Order'])
-			$sortorder = $_SESSION[$currentModule . '_Sort_Order'];
+		if (!AppRequest::isEmpty('sorder')) {
+					$sortorder = AppRequest::get('sorder');
+		} else if ($_SESSION[$currentModule . '_Sort_Order']) {
+					$sortorder = $_SESSION[$currentModule . '_Sort_Order'];
+		}
 
 		return $sortorder;
 	}
@@ -109,10 +110,11 @@ 
 	public function getOrderBy()
 	{
 		$orderby = $this->default_order_by;
-		if (!AppRequest::isEmpty('order_by'))
-			$sortorder = AppRequest::get('order_by');
-		else if ($_SESSION[$currentModule . '_Order_By'])
-			$orderby = $_SESSION[$currentModule . '_Order_By'];
+		if (!AppRequest::isEmpty('order_by')) {
+					$sortorder = AppRequest::get('order_by');
+		} else if ($_SESSION[$currentModule . '_Order_By']) {
+					$orderby = $_SESSION[$currentModule . '_Order_By'];
+		}
 		return $orderby;
 	}
 
@@ -138,8 +140,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";
 
@@ -270,10 +273,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');
@@ -302,8 +306,9 @@ 
 		$select_clause = "SELECT %s.%s AS recordid, vtiger_users_last_import.deleted,%s";
 		$select_clause = sprintf($select_clause, $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/PBXManager/connectors/PBXManager.php

Braces +8 added lines, -6 removed lines

@@ -239,10 +239,11 @@ 
 		$recordModel = PBXManager_Record_Model::getCleanInstance();
 		$recordModel->saveRecordWithArrray($params);
 
-		if ($direction == self::INCOMING_TYPE)
-			$this->respondToIncomingCall($details);
-		else
-			$this->respondToOutgoingCall($params['CustomerNumber']);
+		if ($direction == self::INCOMING_TYPE) {
+					$this->respondToIncomingCall($details);
+		} else {
+					$this->respondToOutgoingCall($params['CustomerNumber']);
+		}
 	}
 
 	/**
@@ -312,8 +313,9 @@ 
 		} else {
 			$response .= '<Number>SIP/';
 			$response .= $to;
-			if ($numberLength > 5)
-				$response .= '@' . $this->getOutboundTrunk();
+			if ($numberLength > 5) {
+							$response .= '@' . $this->getOutboundTrunk();
+			}
 			$response .= '</Number>';
 		}
 

modules/Accounts/data_access/unique_account.php

Braces +9 added lines, -7 removed lines

@@ -28,23 +28,25 @@
 			$recordModel = Vtiger_Record_Model::getInstanceById($ID, $moduleName);
 			$vatId = $recordModel->get('vat_id');
 		} else {
-			if (array_key_exists('vat_id', $recordForm))
-				$vatId = $recordForm['vat_id'];
+			if (array_key_exists('vat_id', $recordForm)) {
+							$vatId = $recordForm['vat_id'];
+			}
 		}
 		if ($ID != 0 && $ID != '' && !array_key_exists('accountname', $recordForm)) {
 			$recordModel = Vtiger_Record_Model::getInstanceById($ID, $moduleName);
 			$accountName = $recordModel->get('accountname');
 		} else {
-			if (array_key_exists('accountname', $recordForm))
-				$accountName = $recordForm['accountname'];
+			if (array_key_exists('accountname', $recordForm)) {
+							$accountName = $recordForm['accountname'];
+			}
 		}
 
 		$moduleModel = Vtiger_Module_Model::getInstance($moduleName);
 		$hierarchyField = Vtiger_Field_Model::getInstance('account_id', $moduleModel);
 		if ($hierarchyField->isActiveField()) {
-			if (array_key_exists('account_id', $recordForm))
-				$hierarchyValue = $recordForm['account_id'];
-			elseif ($ID != 0 && $ID != '' && !array_key_exists('account_id', $recordForm)) {
+			if (array_key_exists('account_id', $recordForm)) {
+							$hierarchyValue = $recordForm['account_id'];
+			} elseif ($ID != 0 && $ID != '' && !array_key_exists('account_id', $recordForm)) {
 				$recordModel = Vtiger_Record_Model::getInstanceById($ID, $moduleName);
 				$hierarchyValue = $recordModel->get('account_id');
 			}

modules/Users/Users.php

Spacing +1 added lines, -1 removed lines

@@ -296,7 +296,7 @@
 			$buffer = fread($handle, filesize($validate));
 			if (substr_count($buffer, $authkey) < $i)
 				return -1;
-		}else {
+		} else {
 			return -1;
 		}
 	}

Braces +32 added lines, -25 removed lines

@@ -118,10 +118,11 @@ 
 	{
 
 		\App\Log::trace("Entering getSortOrder() method ...");
-		if (AppRequest::has('sorder'))
-			$sorder = $this->db->sql_escape_string(AppRequest::get('sorder'));
-		else
-			$sorder = (($_SESSION['USERS_SORT_ORDER'] != '') ? ($_SESSION['USERS_SORT_ORDER']) : ($this->default_sort_order));
+		if (AppRequest::has('sorder')) {
+					$sorder = $this->db->sql_escape_string(AppRequest::get('sorder'));
+		} else {
+					$sorder = (($_SESSION['USERS_SORT_ORDER'] != '') ? ($_SESSION['USERS_SORT_ORDER']) : ($this->default_sort_order));
+		}
 		\App\Log::trace("Exiting getSortOrder method ...");
 		return $sorder;
 	}
@@ -140,10 +141,11 @@ 
 			$use_default_order_by = $this->default_order_by;
 		}
 
-		if (AppRequest::has('order_by'))
-			$order_by = $this->db->sql_escape_string(AppRequest::get('order_by'));
-		else
-			$order_by = (($_SESSION['USERS_ORDER_BY'] != '') ? ($_SESSION['USERS_ORDER_BY']) : ($use_default_order_by));
+		if (AppRequest::has('order_by')) {
+					$order_by = $this->db->sql_escape_string(AppRequest::get('order_by'));
+		} else {
+					$order_by = (($_SESSION['USERS_ORDER_BY'] != '') ? ($_SESSION['USERS_ORDER_BY']) : ($use_default_order_by));
+		}
 		\App\Log::trace("Exiting getOrderBy method ...");
 		return $order_by;
 	}
@@ -157,10 +159,11 @@ 
 	public function setPreference($name, $value)
 	{
 		if (!isset($this->user_preferences)) {
-			if (isset($_SESSION["USER_PREFERENCES"]))
-				$this->user_preferences = $_SESSION["USER_PREFERENCES"];
-			else
-				$this->user_preferences = [];
+			if (isset($_SESSION["USER_PREFERENCES"])) {
+							$this->user_preferences = $_SESSION["USER_PREFERENCES"];
+			} else {
+							$this->user_preferences = [];
+			}
 		}
 		if (!array_key_exists($name, $this->user_preferences) || $this->user_preferences[$name] != $value) {
 			\App\Log::trace("Saving To Preferences:" . $name . "=" . $value);
@@ -284,9 +287,10 @@ 
 		$authkey = base64_decode($authkey);
 		if (file_exists($validate) && $handle = fopen($validate, 'rb', true)) {
 			$buffer = fread($handle, filesize($validate));
-			if (substr_count($buffer, $authkey) < $i)
-				return -1;
-		}else {
+			if (substr_count($buffer, $authkey) < $i) {
+							return -1;
+			}
+		} else {
 			return -1;
 		}
 	}
@@ -378,8 +382,9 @@ 
 		}
 		\App\Log::trace("Starting user load for $usr_name");
 
-		if (!isset($this->column_fields["user_name"]) || $this->column_fields["user_name"] == "" || !isset($user_password) || $user_password == "")
-			return null;
+		if (!isset($this->column_fields["user_name"]) || $this->column_fields["user_name"] == "" || !isset($user_password) || $user_password == "") {
+					return null;
+		}
 
 		$authCheck = false;
 		$authCheck = $this->doLogin($user_password);
@@ -407,8 +412,9 @@ 
 		$this->loadPreferencesFromDB($row['user_preferences']);
 
 
-		if ($row['status'] != "Inactive")
-			$this->authenticated = true;
+		if ($row['status'] != "Inactive") {
+					$this->authenticated = true;
+		}
 
 		unset($_SESSION['loginattempts']);
 		return $this;
@@ -940,8 +946,9 @@ 
 		$db = App\Db::getInstance();
 		//to get the owner id
 		$ownerid = $this->column_fields['assigned_user_id'];
-		if (!isset($ownerid) || $ownerid == '')
-			$ownerid = $currentUserId;
+		if (!isset($ownerid) || $ownerid == '') {
+					$ownerid = $currentUserId;
+		}
 		$fileInstance = \App\Fields\File::loadFromRequest($fileDetails);
 		if (!$fileInstance->validate('image')) {
 			\App\Log::trace('Skip the save attachment process.');
@@ -1202,10 +1209,10 @@ 
 					$result = $adb->pquery($qry, [$id, $this->homeorder_array[$i]]);
 				}
 			}
-			if ($save_array != "")
-				$homeorder = implode(',', $save_array);
-		}
-		else {
+			if ($save_array != "") {
+							$homeorder = implode(',', $save_array);
+			}
+		} else {
 			$this->insertUserdetails('postinstall');
 		}
 		\App\Log::trace("Exiting from function saveHomeOrder($id)");