YetiForceCompany / YetiForceCRM

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/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)

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);
@@ -66,6 +69,9 @@ 
 			$this->_lastTag = $tag;
 	}
 
+	/**
+	 * @param string $value
+	 */
 	protected function parseTransaction($value)
 	{
 		$transaction = array(
@@ -84,6 +90,9 @@ 
 		$this->operations[count($this->operations) - 1]['details'] = $transaction;
 	}
 
+	/**
+	 * @param string $value
+	 */
 	protected function parseOperation($value)
 	{
 		$this->operations[] = array(

modules/PaymentsIn/helpers/subclass/mt940_Pekao.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 == '61')
-					$this->parseTransaction($value);
-				else
-					$this->info .= $value;
+				if ($this->_lastTag == '61') {
+									$this->parseTransaction($value);
+				} else {
+									$this->info .= $value;
+				}
 				break;
 			default:
 				break;

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(
@@ -129,6 +135,9 @@ 
 		$this->operations[count($this->operations) - 1]['details'] = $transaction;
 	}
 
+	/**
+	 * @param string $value
+	 */
 	protected function parseOperation($value)
 	{
 		$this->operations[] = array(

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;

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(
@@ -129,6 +135,9 @@ 
 		$this->operations[count($this->operations) - 1]['details'] = $transaction;
 	}
 
+	/**
+	 * @param string $value
+	 */
 	protected function parseOperation($value)
 	{
 		$this->operations[] = array(

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

Doc Comments +9 added lines, -2 removed lines

@@ -54,6 +54,9 @@ 
 		return $this->webappurl;
 	}
 
+	/**
+	 * @return string
+	 */
 	public function getOutboundContext()
 	{
 		return $this->outboundcontext;
@@ -82,6 +85,7 @@ 
 	/**
 	 * Function to set server parameters
 	 * @param <array>  authdetails
+	 * @param PBXManager_Server_Model $serverModel
 	 */
 	public function setServerParameters($serverModel)
 	{
@@ -100,6 +104,10 @@ 
 		return self::$SETTINGS_REQUIRED_PARAMETERS;
 	}
 
+	/**
+	 * @param Vtiger_Request $details
+	 * @param string $type
+	 */
 	protected function prepareParameters($details, $type)
 	{
 		switch ($type) {
@@ -294,7 +302,6 @@ 
 
 	/**
 	 * Function to respond for outgoing calls
-	 * @param Vtiger_Request $details 
 	 */
 	public function respondToOutgoingCall($to)
 	{
@@ -324,7 +331,7 @@ 
 	/**
 	 * Function to make outbound call 
 	 * @param string $number (Customer)
-	 * @param string $recordid
+	 * @param string $record
 	 */
 	public function call($number, $record)
 	{