YetiForceCompany / YetiForceCRM

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

modules/Settings/OSSPasswords/views/ConfigurePass.php

Braces +5 added lines, -4 removed lines

@@ -237,10 +237,11 @@
 		$viewer->assign('MODULENAME', $moduleName);
 		$viewer->assign('SAVE', 'Save');
 		$viewer->assign('CANCEL', 'Cancel');
-		if (\vtlib\Functions::userIsAdministrator($current_user))
-			$viewer->assign('ISADMIN', 1);
-		else
-			$viewer->assign('ISADMIN', 0);
+		if (\vtlib\Functions::userIsAdministrator($current_user)) {
+					$viewer->assign('ISADMIN', 1);
+		} else {
+					$viewer->assign('ISADMIN', 0);
+		}
 
 		// encryption variables
 		$viewer->assign('CONFIG', (!$config ? false : array('key' => $config['key'])));

modules/Vtiger/CRMEntity.php

Braces +11 added lines, -8 removed lines

@@ -52,8 +52,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";
 
@@ -180,10 +181,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;
 	}
@@ -196,8 +198,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/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',

Doc Comments +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 
 	/**
 	 * Function to get Module instance
-	 * @return Vtiger_Module_Model
+	 * @return boolean
 	 */
 	public function getModule()
 	{

Braces +3 added lines, -2 removed lines

@@ -306,8 +306,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();

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