YetiForceCompany / YetiForceCRM

modules/OSSProjectTemplates/OSSProjectTemplates.php

Braces +3 added lines, -2 removed lines

@@ -54,8 +54,9 @@
 
 	private function addWidgetToListView($moduleNames, $widgetName, $widgetType = 'LIST_VIEW_HEADER')
 	{
-		if (empty($moduleNames))
-			return;
+		if (empty($moduleNames)) {
+					return;
+		}
 
 		if (is_string($moduleNames)) {
 			$moduleNames = array($moduleNames);

Doc Comments +4 added lines

@@ -52,6 +52,10 @@
 				VALUES (?,?,?,?,?,?,?)", array($fieldid, $blockid, $sequence, 'Project Templates', '', 'LBL_PROJECT_TEMPLATES_DESCRIPTION', 'index.php?module=' . $moduleName . '&parent=Settings&view=Index'));
 	}
 
+	/**
+	 * @param string $moduleNames
+	 * @param string $widgetName
+	 */
 	private function addWidgetToListView($moduleNames, $widgetName, $widgetType = 'LIST_VIEW_HEADER')
 	{
 		if (empty($moduleNames))

modules/Ideas/Ideas.php

Braces +3 added lines, -2 removed lines

@@ -97,8 +97,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('Ideas'));
+				if (class_exists('ModComments')) {
+									ModComments::addWidgetTo(array('Ideas'));
+				}
 			}
 			$modcommentsModuleInstance = vtlib\Module::getInstance('ModTracker');
 			if ($modcommentsModuleInstance && file_exists('modules/ModTracker/ModTracker.php')) {

modules/SSalesProcesses/SSalesProcesses.php

Braces +3 added lines, -2 removed lines

@@ -87,8 +87,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('SSalesProcesses'));
+				if (class_exists('ModComments')) {
+									ModComments::addWidgetTo(array('SSalesProcesses'));
+				}
 			}
 			$modcommentsModuleInstance = vtlib\Module::getInstance('ModTracker');
 			if ($modcommentsModuleInstance && file_exists('modules/ModTracker/ModTracker.php')) {

Doc Comments +1 added lines

@@ -213,6 +213,7 @@
 	 * @param integer $id - ssalesprocessesid
 	 * @param array $parentSSalesProcesses - Array of all the parent sales
 	 * returns All the parent Sales of the given ssalesprocessesid in array format
+	 * @param integer[] $encounteredSalesProcesses
 	 */
 	public function getParentSales($id, &$parentSSalesProcesses, &$encounteredSalesProcesses, $depthBase = 0)
 	{

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

Braces +3 added lines, -2 removed lines

@@ -104,8 +104,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('OSSTimeControl'));
+				if (class_exists('ModComments')) {
+									ModComments::addWidgetTo(array('OSSTimeControl'));
+				}
 			}
 		} else if ($event_type == 'module.disabled') {
 			// TODO Handle actions when this module is disabled.

modules/OSSTimeControl/actions/GetTCInfo.php

Braces +10 added lines, -8 removed lines

@@ -37,17 +37,19 @@
 			$sourceData = $entity->column_fields;
 			if ($sourceModule == 'HelpDesk') {
 				$sourceData['contact_label'] = vtlib\Functions::getCRMRecordLabel($sourceData['contact_id']);
-				if (vtlib\Functions::getCRMRecordType($sourceData['parent_id']) != 'Accounts')
-					unset($sourceData['parent_id']);
-				else
-					$sourceData['account_label'] = vtlib\Functions::getCRMRecordLabel($sourceData['parent_id']);
+				if (vtlib\Functions::getCRMRecordType($sourceData['parent_id']) != 'Accounts') {
+									unset($sourceData['parent_id']);
+				} else {
+									$sourceData['account_label'] = vtlib\Functions::getCRMRecordLabel($sourceData['parent_id']);
+				}
 			} else if ($sourceModule == 'Project') {
 				$query = sprintf("select * from vtiger_account where accountid = %s", $sourceData['linktoaccountscontacts']);
 				$ifExist = $adb->query($query, true, "Błąd podczas pobierania danych z vtiger_crmentityrel");
-				if ($adb->num_rows($ifExist) > 0)
-					$sourceData['account_label'] = vtlib\Functions::getCRMRecordLabel($sourceData['linktoaccountscontacts']);
-				else
-					$sourceData['contact_label'] = vtlib\Functions::getCRMRecordLabel($sourceData['linktoaccountscontacts']);
+				if ($adb->num_rows($ifExist) > 0) {
+									$sourceData['account_label'] = vtlib\Functions::getCRMRecordLabel($sourceData['linktoaccountscontacts']);
+				} else {
+									$sourceData['contact_label'] = vtlib\Functions::getCRMRecordLabel($sourceData['linktoaccountscontacts']);
+				}
 			}
 		}
 

modules/ModComments/ModCommentsCore.php

Braces +21 added lines, -16 removed lines

@@ -96,10 +96,11 @@ 
 		$currentModule = vglobal('currentModule');
 
 		$sortorder = $this->default_sort_order;
-		if (!AppRequest::isEmpty('sorder'))
-			$sortorder = $this->db->sql_escape_string(AppRequest::get('sorder'));
-		else if ($_SESSION[$currentModule . '_Sort_Order'])
-			$sortorder = $_SESSION[$currentModule . '_Sort_Order'];
+		if (!AppRequest::isEmpty('sorder')) {
+					$sortorder = $this->db->sql_escape_string(AppRequest::get('sorder'));
+		} else if ($_SESSION[$currentModule . '_Sort_Order']) {
+					$sortorder = $_SESSION[$currentModule . '_Sort_Order'];
+		}
 
 		return $sortorder;
 	}
@@ -114,10 +115,11 @@ 
 		}
 
 		$orderby = $use_default_order_by;
-		if (!AppRequest::isEmpty('order_by'))
-			$orderby = $this->db->sql_escape_string(AppRequest::get('order_by'));
-		else if ($_SESSION[$currentModule . '_Order_By'])
-			$orderby = $_SESSION[$currentModule . '_Order_By'];
+		if (!AppRequest::isEmpty('order_by')) {
+					$orderby = $this->db->sql_escape_string(AppRequest::get('order_by'));
+		} else if ($_SESSION[$currentModule . '_Order_By']) {
+					$orderby = $_SESSION[$currentModule . '_Order_By'];
+		}
 		return $orderby;
 	}
 
@@ -146,8 +148,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";
 
@@ -285,10 +288,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');
@@ -317,8 +321,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/Settings/QuickCreateEditor/models/Module.php

Spacing +2 added lines, -2 removed lines

@@ -18,7 +18,7 @@ 
 		$db = PearDatabase::getInstance();
 
 		$query = 'UPDATE vtiger_field SET ';
-		$query .=' quickcreatesequence= CASE ';
+		$query .= ' quickcreatesequence= CASE ';
 		foreach ($blockFieldSequence as $newFieldSequence) {
 			$fieldId = $newFieldSequence['fieldid'];
 			$sequence = $newFieldSequence['sequence'];
@@ -27,7 +27,7 @@ 
 			$query .= ' WHEN fieldid=' . $fieldId . ' THEN ' . $sequence;
 		}
 
-		$query .=' END ';
+		$query .= ' END ';
 		$query .= sprintf(' WHERE fieldid IN (%s)', generateQuestionMarks($fieldIdList));
 		$db->pquery($query, array($fieldIdList));
 	}

modules/Settings/QuickCreateEditor/views/Index.php

Braces +6 added lines, -4 removed lines

@@ -37,13 +37,15 @@
 			//To get the first element
 			$firstElement = reset($menuModelsList);
 			$sourceModule = array($firstElement->get('name'));
-		} else
-			$sourceModule = array($sourceModule);
+		} else {
+					$sourceModule = array($sourceModule);
+		}
 
 		$quickCreateContents = array();
 
-		if (in_array('Calendar', $sourceModule))
-			$sourceModule = array('Calendar', 'Events');
+		if (in_array('Calendar', $sourceModule)) {
+					$sourceModule = array('Calendar', 'Events');
+		}
 
 		foreach ($sourceModule as $module) {
 			$recordModel = Vtiger_Record_Model::getCleanInstance($module);