YetiForceCompany / YetiForceCRM

modules/Settings/DataAccess/models/Module.php

Braces +8 added lines, -6 removed lines

@@ -131,8 +131,9 @@ 
 
 	public static function getListBaseModuleField($baseModule)
 	{
-		if ($baseModule === 'All')
-			return $baseModule;
+		if ($baseModule === 'All') {
+					return $baseModule;
+		}
 		$baseModuleModel = Vtiger_Module_Model::getInstance($baseModule);
 		$list = $baseModuleModel->getFields();
 		$output = array();
@@ -276,10 +277,11 @@ 
 	public function getActionName($name, $typ)
 	{
 		$actionsName = explode(self::$separator, $name);
-		if ($typ)
-			return vtranslate('Action_' . $actionsName[1], 'DataAccess');
-		else
-			return vtranslate('Action_Desc_' . $actionsName[1], 'DataAccess');
+		if ($typ) {
+					return vtranslate('Action_' . $actionsName[1], 'DataAccess');
+		} else {
+					return vtranslate('Action_Desc_' . $actionsName[1], 'DataAccess');
+		}
 	}
 
 	public function listAccesDataDirector($module = false)

Doc Comments +6 added lines

@@ -177,6 +177,9 @@ 
 		}
 	}
 
+	/**
+	 * @param string $relId
+	 */
 	public function addConditions($conditions, $relId, $mendatory = true)
 	{
 		$conditionObj = json_decode($conditions);
@@ -309,6 +312,9 @@ 
 		return array_merge($mainFolderFiles, $moduleFolderFiles);
 	}
 
+	/**
+	 * @param string|boolean $prefix
+	 */
 	public function listFolderFiles($dir, $prefix)
 	{
 		$ffs = scandir($dir);

modules/Vtiger/actions/TransferOwnership.php

Braces +5 added lines, -4 removed lines

@@ -22,10 +22,11 @@
 		$modelClassName = Vtiger_Loader::getComponentClassName('Model', 'TransferOwnership', $module);
 		$transferModel = new $modelClassName();
 
-		if (empty($record))
-			$recordIds = $this->getBaseModuleRecordIds($request);
-		else
-			$recordIds = [$record];
+		if (empty($record)) {
+					$recordIds = $this->getBaseModuleRecordIds($request);
+		} else {
+					$recordIds = [$record];
+		}
 		if (!empty($recordIds)) {
 			$transferModel->transferRecordsOwnership($module, $transferOwnerId, $recordIds);
 		}

modules/Vtiger/models/DetailView.php

Spacing +1 added lines, -1 removed lines

@@ -144,7 +144,7 @@
 			$linkModelList['DETAILVIEW'][] = Vtiger_Link_Model::getInstanceFromValues($editViewLinks);
 		}
 
-		if (($recordModel->isEditable() && $recordModel->editFieldByModalPermission() ) || $recordModel->editFieldByModalPermission(true)) {
+		if (($recordModel->isEditable() && $recordModel->editFieldByModalPermission()) || $recordModel->editFieldByModalPermission(true)) {
 			$fieldByEditData = $recordModel->getFieldToEditByModal();
 			$basicActionLink = [
 				'linktype' => 'DETAILVIEW',

Braces +3 added lines, -2 removed lines

@@ -288,8 +288,9 @@
 	 */
 	public function getWidgets()
 	{
-		if (count($this->widgetsList) > 0)
-			return;
+		if (count($this->widgetsList) > 0) {
+					return;
+		}
 		$moduleModel = $this->getModule();
 		$Module = $this->getModuleName();
 		$Record = $this->getRecord()->getId();

Doc Comments +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
 
 	/**
 	 * Function to get the module
-	 * @return Vtiger_Module_Model
+	 * @return boolean
 	 */
 	public function getModule()
 	{

modules/Vtiger/models/EditRecordStructure.php

Braces +3 added lines, -2 removed lines

@@ -39,8 +39,9 @@
 							$fieldModel->set('fieldvalue', $recordModel->get($fieldName));
 						} else {
 							$defaultValue = $fieldModel->getDefaultFieldValue();
-							if (!empty($defaultValue) && !$recordId)
-								$fieldModel->set('fieldvalue', $defaultValue);
+							if (!empty($defaultValue) && !$recordId) {
+															$fieldModel->set('fieldvalue', $defaultValue);
+							}
 						}
 						$values[$blockLabel][$fieldName] = $fieldModel;
 					}

modules/Vtiger/models/Mobile.php

Braces +3 added lines, -2 removed lines

@@ -87,8 +87,9 @@
 		$users = [];
 		$keys = self::getAllMobileKeys('pushcall', $currentUser->getId());
 		foreach ($keys as $id => $key) {
-			if (in_array($currentUser->getId(), $key['privileges_users']))
-				$users[$key['userid']] = $key['fullusername'];
+			if (in_array($currentUser->getId(), $key['privileges_users'])) {
+							$users[$key['userid']] = $key['fullusername'];
+			}
 		}
 		return $users;
 	}

Doc Comments +3 added lines

@@ -48,6 +48,9 @@
 		return $return;
 	}
 
+	/**
+	 * @param boolean|string $service
+	 */
 	public function getAllMobileKeys($service, $userid = false)
 	{
 		$adb = PearDatabase::getInstance();

modules/Vtiger/models/Paging.php

Braces +6 added lines, -4 removed lines

@@ -199,10 +199,12 @@
 		$totalCount = $this->get('totalCount');
 		$startPaginFrom = $pageNumber - 2;
 
-		if ($pageNumber == $totalCount && 1 != $pageNumber)
-			$startPaginFrom = $pageNumber - 4;
-		if ($startPaginFrom <= 0 || 1 == $pageNumber)
-			$startPaginFrom = 1;
+		if ($pageNumber == $totalCount && 1 != $pageNumber) {
+					$startPaginFrom = $pageNumber - 4;
+		}
+		if ($startPaginFrom <= 0 || 1 == $pageNumber) {
+					$startPaginFrom = 1;
+		}
 
 		return $startPaginFrom;
 	}

modules/Vtiger/models/DashBoard.php

Braces +12 added lines, -8 removed lines

@@ -52,8 +52,9 @@ 
 		$currentUserPrivilegeModel = Users_Privileges_Model::getCurrentUserPrivilegesModel();
 		$moduleModel = $this->getModule();
 
-		if ($action == 'Header')
-			$action = 0;
+		if ($action == 'Header') {
+					$action = 0;
+		}
 		$sql = " SELECT 
 					vtiger_links.*,
 					mdw.userid,
@@ -86,23 +87,26 @@ 
 		while ($row = $db->fetch_array($result)) {
 			$row['linkid'] = $row['id'];
 			if ($row['linklabel'] == 'Mini List') {
-				if (!$row['isdeafult'])
-					$row['deleteFromList'] = true;
+				if (!$row['isdeafult']) {
+									$row['deleteFromList'] = true;
+				}
 				$minilistWidget = Vtiger_Widget_Model::getInstanceFromValues($row);
 				$minilistWidgetModel = new Vtiger_MiniList_Model();
 				$minilistWidgetModel->setWidgetModel($minilistWidget);
 				$minilistWidget->set('title', $minilistWidgetModel->getTitle());
 				$widgets[] = $minilistWidget;
 			} elseif ($row['linklabel'] == 'ChartFilter') {
-				if (!$row['isdeafult'])
-					$row['deleteFromList'] = true;
+				if (!$row['isdeafult']) {
+									$row['deleteFromList'] = true;
+				}
 				$charFilterWidget = Vtiger_Widget_Model::getInstanceFromValues($row);
 				$chartFilterWidgetModel = new Vtiger_ChartFilter_Model();
 				$chartFilterWidgetModel->setWidgetModel($charFilterWidget);
 				$charFilterWidget->set('title', $chartFilterWidgetModel->getTitle());
 				$widgets[] = $charFilterWidget;
-			} else
-				$widgets[] = Vtiger_Widget_Model::getInstanceFromValues($row);
+			} else {
+							$widgets[] = Vtiger_Widget_Model::getInstanceFromValues($row);
+			}
 		}
 
 		foreach ($widgets as $index => $widget) {

Doc Comments +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 	/**
 	 * Function to set the module instance
 	 * @param Vtiger_Module_Model $moduleInstance - module model
-	 * @return Vtiger_DetailView_Model
+	 * @return Vtiger_DashBoard_Model
 	 */
 	public function setModule($moduleInstance)
 	{

modules/Vtiger/models/Utility.php

Braces +3 added lines, -2 removed lines

@@ -23,8 +23,9 @@
 	{
 		$db = PearDatabase::getInstance();
 		if (!$module->isEntityModule()) {
-			if (!$module->isUtilityActionEnabled())
-				return false;
+			if (!$module->isUtilityActionEnabled()) {
+							return false;
+			}
 		}
 		$tabId = $module->getId();
 		$sql = 'SELECT 1 FROM vtiger_profile2utility WHERE tabid = ? && activityid = ? LIMIT 1';

modules/Vtiger/models/FindDuplicate.php

Braces +3 added lines, -2 removed lines

@@ -178,8 +178,9 @@
 		$fields = $request->get('fields');
 		$ignoreEmpty = $request->get('ignoreEmpty');
 		$ignoreEmptyValue = false;
-		if ($ignoreEmpty == 'on')
-			$ignoreEmptyValue = true;
+		if ($ignoreEmpty == 'on') {
+					$ignoreEmptyValue = true;
+		}
 
 		$fieldModels = $moduleModel->getFields();
 		if (is_array($fields)) {

Doc Comments +3 added lines

@@ -96,6 +96,9 @@
 		echo $viewer->view('Pagination.tpl', $moduleName, true);
 	}
 
+	/**
+	 * @param Vtiger_Module_Model $moduleModel
+	 */
 	public function transferListSearchParamsToFilterCondition($listSearchParams, $moduleModel)
 	{
 		return Vtiger_Util_Helper::transferListSearchParamsToFilterCondition($listSearchParams, $moduleModel);