YetiForceCompany / YetiForceCRM

modules/Vtiger/views/TreePopup.php

Braces +3 added lines, -2 removed lines

@@ -36,8 +36,9 @@
 		} else {
 			vtlib\Functions::throwNewException(vtranslate('ERR_TREE_NOT_FOUND', $moduleName));
 		}
-		if (!$recordModel)
-			vtlib\Functions::throwNewException(vtranslate('ERR_TREE_NOT_FOUND', $moduleName));
+		if (!$recordModel) {
+					vtlib\Functions::throwNewException(vtranslate('ERR_TREE_NOT_FOUND', $moduleName));
+		}
 		if ($request->get('multiple')) {
 			$type = 'category';
 		}

modules/Vtiger/views/FindDuplicates.php

Braces +9 added lines, -6 removed lines

@@ -86,8 +86,9 @@ 
 
 		$ignoreEmpty = $request->get('ignoreEmpty');
 		$ignoreEmptyValue = false;
-		if ($ignoreEmpty == 'on' || $ignoreEmpty == 'true' || $ignoreEmpty == '1')
-			$ignoreEmptyValue = true;
+		if ($ignoreEmpty == 'on' || $ignoreEmpty == 'true' || $ignoreEmpty == '1') {
+					$ignoreEmptyValue = true;
+		}
 		$dataModelInstance->set('ignoreEmpty', $ignoreEmptyValue);
 
 		if (!$this->listViewEntries) {
@@ -109,8 +110,9 @@ 
 			}
 		}
 		//for calculating the page range
-		for ($i = 0; $i < $rowCount; $i++)
-			$dummyListEntries[] = $i;
+		for ($i = 0; $i < $rowCount; $i++) {
+					$dummyListEntries[] = $i;
+		}
 		$pagingModel->calculatePageRange($dummyListEntries);
 
 		$totalCount = $this->rows;
@@ -146,8 +148,9 @@ 
 
 		$ignoreEmpty = $request->get('ignoreEmpty');
 		$ignoreEmptyValue = false;
-		if ($ignoreEmpty == 'on' || $ignoreEmpty == 'true' || $ignoreEmpty == '1')
-			$ignoreEmptyValue = true;
+		if ($ignoreEmpty == 'on' || $ignoreEmpty == 'true' || $ignoreEmpty == '1') {
+					$ignoreEmptyValue = true;
+		}
 		$dataModelInstance->set('ignoreEmpty', $ignoreEmptyValue);
 
 		$dataModelInstance->set('fields', $duplicateSearchFields);

modules/HelpDesk/HelpDesk.php

Braces +14 added lines, -10 removed lines

@@ -258,10 +258,11 @@ 
 		$query .= getNonAdminAccessControlQuery('HelpDesk', $current_user);
 		$where_auto = " vtiger_crmentity.deleted = 0 ";
 
-		if ($where != '')
-			$query .= sprintf(' where (%s) && %s', $where, $where_auto);
-		else
-			$query .= sprintf(' where %s', $where_auto);
+		if ($where != '') {
+					$query .= sprintf(' where (%s) && %s', $where, $where_auto);
+		} else {
+					$query .= sprintf(' where %s', $where_auto);
+		}
 
 		$log->debug("Exiting create_export_query method ...");
 		return $query;
@@ -306,10 +307,12 @@ 
 			//Assigned to change log
 			if ($focus->column_fields['assigned_user_id'] != $old_owner_id) {
 				$ownerName = \includes\fields\Owner::getLabel($focus->column_fields['assigned_user_id']);
-				if ($assigntype == 'T')
-					$updatelog .= ' Transferred to group ' . $ownerName . '\.';
-				else
-					$updatelog .= ' Transferred to user ' . decode_html($ownerName) . '\.'; // Need to decode UTF characters which are migrated from versions < 5.0.4.
+				if ($assigntype == 'T') {
+									$updatelog .= ' Transferred to group ' . $ownerName . '\.';
+				} else {
+									$updatelog .= ' Transferred to user ' . decode_html($ownerName) . '\.';
+				}
+				// Need to decode UTF characters which are migrated from versions < 5.0.4.
 			}
 			//Status change log
 			if ($old_status != $focus->column_fields['ticketstatus'] && $focus->column_fields['ticketstatus'] != '') {
@@ -446,8 +449,9 @@ 
 	public function unlinkRelationship($id, $return_module, $return_id, $relatedName = false)
 	{
 		$log = LoggerManager::getInstance();
-		if (empty($return_module) || empty($return_id))
-			return;
+		if (empty($return_module) || empty($return_id)) {
+					return;
+		}
 
 		if ($return_module == 'Accounts') {
 			$sql = 'UPDATE vtiger_troubletickets SET parent_id=? WHERE ticketid=?';

modules/HelpDesk/dashboards/OpenTickets.php

Braces +3 added lines, -3 removed lines

@@ -66,9 +66,9 @@
 	public function getSearchParams($value)
 	{
 		$openTicketsStatus = Settings_SupportProcesses_Module_Model::getOpenTicketStatus();
-		if ($openTicketsStatus)
-			$openTicketsStatus = implode(',', $openTicketsStatus);
-		else {
+		if ($openTicketsStatus) {
+					$openTicketsStatus = implode(',', $openTicketsStatus);
+		} else {
 			$allTicketStatus = Settings_SupportProcesses_Module_Model::getAllTicketStatus();
 			$openTicketsStatus = implode(',', $allTicketStatus);
 		}

Doc Comments -1 removed lines

@@ -27,7 +27,6 @@
 
 	/**
 	 * Function returns Tickets grouped by Status
-	 * @param type $data
 	 * @return <Array>
 	 */
 	public function getTicketsByStatus($owner)

modules/HelpDesk/views/Detail.php

Braces +3 added lines, -2 removed lines

@@ -46,8 +46,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/ShowTimeHelpDesk.tpl', $moduleName);

modules/Calendar/iCalLastImport.php

Braces +3 added lines, -2 removed lines

@@ -37,8 +37,9 @@
 	{
 		$adb = PearDatabase::getInstance();
 
-		if (count($this->fieldData) == 0)
-			return;
+		if (count($this->fieldData) == 0) {
+					return;
+		}
 
 		if (!vtlib\Utils::CheckTable($this->tableName)) {
 			vtlib\Utils::CreateTable(

modules/Calendar/actions/SaveFollowupAjax.php

Braces +5 added lines, -4 removed lines

@@ -39,10 +39,11 @@
 		$recordModel->set('eventstatus', "Planned");
 
 		$activityType = $recordModel->get('activitytype');
-		if ($activityType == "Call")
-			$eventDuration = $request->get('defaultCallDuration');
-		else
-			$eventDuration = $request->get('defaultOtherEventDuration');
+		if ($activityType == "Call") {
+					$eventDuration = $request->get('defaultCallDuration');
+		} else {
+					$eventDuration = $request->get('defaultOtherEventDuration');
+		}
 
 		$followupStartTime = Vtiger_Time_UIType::getTimeValueWithSeconds($request->get('followup_time_start'));
 		$followupStartDateTime = Vtiger_Datetime_UIType::getDBDateTimeValue($request->get('followup_date_start') . " " . $followupStartTime);

modules/Calendar/actions/DragDropAjax.php

Braces +11 added lines, -8 removed lines

@@ -51,8 +51,9 @@ 
 			$resultDateTime = $this->changeDateTime($oldDateTime, $dayDelta, $minuteDelta);
 			$parts = explode(' ', $resultDateTime);
 			$record->set('due_date', $parts[0]);
-			if (activitytype != 'Task')
-				$record->set('time_end', $parts[1]);
+			if (activitytype != 'Task') {
+							$record->set('time_end', $parts[1]);
+			}
 
 			$startDateTime[] = $record->get('date_start');
 			$startDateTime[] = $record->get('time_start');
@@ -64,10 +65,11 @@ 
 			$endDateTime = implode(' ', $endDateTime);
 			$endDateTime = new DateTime($endDateTime);
 			//Checking if startDateTime is less than or equal to endDateTime
-			if ($startDateTime <= $endDateTime)
-				$record->save();
-			else
-				$result['error'] = true;
+			if ($startDateTime <= $endDateTime) {
+							$record->save();
+			} else {
+							$result['error'] = true;
+			}
 
 			$response->setResult($result);
 			$response->emit();
@@ -107,8 +109,9 @@ 
 			$resultDateTime = $this->changeDateTime($oldEndDateTime, $dayDelta, $minuteDelta);
 			$parts = explode(' ', $resultDateTime);
 			$record->set('due_date', $parts[0]);
-			if (activitytype != 'Task')
-				$record->set('time_end', $parts[1]);
+			if (activitytype != 'Task') {
+							$record->set('time_end', $parts[1]);
+			}
 			$record->save();
 
 			$response->setResult($result);

Doc Comments +3 added lines

@@ -82,6 +82,9 @@
 		$response->emit();
 	}
 
+	/**
+	 * @param string $datetime
+	 */
 	public function changeDateTime($datetime, $delta)
 	{
 		$date = new DateTime($datetime);

modules/Calendar/actions/ImportICS.php

Spacing +2 added lines, -2 removed lines

@@ -72,7 +72,7 @@ 
 					$module = $todoModule;
 				}
 
-				$totalCount[$module] ++;
+				$totalCount[$module]++;
 				$activityFieldsList = $activity->generateArray($icalActivities[$i]);
 
 				$recordModel = Vtiger_Record_Model::getCleanInstance($moduleName);
@@ -83,7 +83,7 @@ 
 				foreach ($requiredFields[$module] as $key) {
 					$value = $recordModel->get($key);
 					if (empty($value)) {
-						$skipCount[$module] ++;
+						$skipCount[$module]++;
 						$skipRecord = true;
 						break;
 					}