YetiForceCompany / YetiForceCRM

modules/Settings/BruteForce/models/Module.php

Braces +17 added lines, -15 removed lines

@@ -58,22 +58,24 @@
 
 		$browser = $_SERVER['HTTP_USER_AGENT'];
 
-		if (strpos($browser, 'MSIE') !== false)
+		if (strpos($browser, 'MSIE') !== false) {
+					return 'Internet explorer';
+		} elseif (strpos($browser, 'Trident') !== false) {
+			//For Supporting IE 11
 			return 'Internet explorer';
-		elseif (strpos($browser, 'Trident') !== false) //For Supporting IE 11
-			return 'Internet explorer';
-		elseif (strpos($browser, 'Firefox') !== false)
-			return 'Mozilla Firefox';
-		elseif (strpos($browser, 'Chrome') !== false)
-			return 'Google Chrome';
-		elseif (strpos($browser, 'Opera Mini') !== false)
-			return "Opera Mini";
-		elseif (strpos($browser, 'Opera') !== false)
-			return "Opera";
-		elseif (strpos($browser, 'Safari') !== false)
-			return "Safari";
-		else
-			return 'unknow';
+		} elseif (strpos($browser, 'Firefox') !== false) {
+					return 'Mozilla Firefox';
+		} elseif (strpos($browser, 'Chrome') !== false) {
+					return 'Google Chrome';
+		} elseif (strpos($browser, 'Opera Mini') !== false) {
+					return "Opera Mini";
+		} elseif (strpos($browser, 'Opera') !== false) {
+					return "Opera";
+		} elseif (strpos($browser, 'Safari') !== false) {
+					return "Safari";
+		} else {
+					return 'unknow';
+		}
 	}
 
 	static public function checkBlocked()

modules/Settings/Vtiger/models/CompanyDetails.php

Braces +14 added lines, -10 removed lines

@@ -235,18 +235,22 @@
 		$lenght = strlen($field);
 		$field = str_replace(' ', '_', $field);
 		$field = strtolower($field);
-		if ('' == $field)
-			$result = 'not valid';
+		if ('' == $field) {
+					$result = 'not valid';
+		}
 
-		if (preg_match('/[^a-z_A-Z]+/', $field, $matches))
-			$result = 'not valid';
+		if (preg_match('/[^a-z_A-Z]+/', $field, $matches)) {
+					$result = 'not valid';
+		}
 
-		if ($lenght > 25)
-			$result = 'not valid';
+		if ($lenght > 25) {
+					$result = 'not valid';
+		}
 
-		if ($result == 'not valid')
-			return false;
-		else
-			return $field;
+		if ($result == 'not valid') {
+					return false;
+		} else {
+					return $field;
+		}
 	}
 }

modules/Settings/PublicHoliday/models/Module.php

Braces +21 added lines, -17 removed lines

@@ -76,10 +76,11 @@ 
 
 		$log->debug("Exiting Settings_PublicHoliday_Module_Model::delete() method ...");
 
-		if ($deleted == 1)
-			return true;
-		else
-			return false;
+		if ($deleted == 1) {
+					return true;
+		} else {
+					return false;
+		}
 	}
 
 	/**
@@ -103,10 +104,11 @@ 
 
 		$log->debug("Exiting Settings_PublicHoliday_Module_Model::save() method ...");
 
-		if ($saved == 1)
-			return true;
-		else
-			return false;
+		if ($saved == 1) {
+					return true;
+		} else {
+					return false;
+		}
 	}
 
 	/**
@@ -131,10 +133,11 @@ 
 
 		$log->debug("Exiting Settings_PublicHoliday_Module_Model::edit() method ...");
 
-		if ($saved == 1)
-			return true;
-		else
-			return false;
+		if ($saved == 1) {
+					return true;
+		} else {
+					return false;
+		}
 	}
 
 	/**
@@ -156,8 +159,9 @@ 
 
 		$log->debug("Exiting Settings_PublicHoliday_Module_Model::checkIfHoliday() method ...");
 
-		if ($num > 0)
-			return true;
+		if ($num > 0) {
+					return true;
+		}
 
 		return false;
 	}
@@ -181,9 +185,9 @@ 
 		$sql .= ' GROUP BY `holidaytype`';
 		$result = $db->pquery($sql, $params);
 		$numRows = $db->num_rows($result);
-		if (0 == $numRows)
-			$return = false;
-		else {
+		if (0 == $numRows) {
+					$return = false;
+		} else {
 			for ($i = 0; $i < $numRows; $i++) {
 				$count = $db->query_result($result, $i, 'count');
 				$type = $db->query_result($result, $i, 'holidaytype');

modules/Settings/BackUp/actions/Backup.php

Braces +5 added lines, -4 removed lines

@@ -52,10 +52,11 @@
 		$ftpPort = $request->get('ftpport');
 		$ftpPath = $request->get('ftppath');
 		$ftpActive = $request->get('ftpactive');
-		if ('true' == $ftpActive)
-			$ftpActive = true;
-		else
-			$ftpActive = false;
+		if ('true' == $ftpActive) {
+					$ftpActive = true;
+		} else {
+					$ftpActive = false;
+		}
 
 		if ('' != $ftpPort) {
 			$ftpConnect = @ftp_connect($ftpServerName, $ftpPort);

modules/OSSTimeControl/dashboards/TimeControl.php

Braces +40 added lines, -28 removed lines

@@ -16,8 +16,9 @@ 
 	{
 		$conditions = array();
 		$listSearchParams = array();
-		if ($assignedto != '')
-			array_push($conditions, array('assigned_user_id', 'e', $assignedto));
+		if ($assignedto != '') {
+					array_push($conditions, array('assigned_user_id', 'e', $assignedto));
+		}
 		if (!empty($date)) {
 			array_push($conditions, array('due_date', 'bw', $date . ',' . $date . ''));
 		}
@@ -40,8 +41,9 @@ 
 		$sql = "SELECT sum_time AS daytime, due_date, timecontrol_type FROM vtiger_osstimecontrol
 					INNER JOIN vtiger_crmentity ON vtiger_osstimecontrol.osstimecontrolid = vtiger_crmentity.crmid
 					WHERE vtiger_crmentity.setype = ? && vtiger_crmentity.smownerid = ? ";
-		if ($securityParameter != '')
-			$sql.= $securityParameter;
+		if ($securityParameter != '') {
+					$sql.= $securityParameter;
+		}
 		$sql .= "AND (vtiger_osstimecontrol.date_start >= ? && vtiger_osstimecontrol.due_date <= ?) && vtiger_osstimecontrol.deleted = 0 ORDER BY due_date ";
 		$result = $db->pquery($sql, $param);
 		$data = array();
@@ -62,20 +64,25 @@ 
 			if (!array_key_exists($row['timecontrol_type'], $timeTypes)) {
 				$timeTypes[$row['timecontrol_type']] = $counter++;
 			}
-			if (!in_array($row['due_date'], $workedDaysAmount) && 'PLL_WORKING_TIME' == $row['timecontrol_type'])
-				$workedDaysAmount[$row['due_date']] = 1;
+			if (!in_array($row['due_date'], $workedDaysAmount) && 'PLL_WORKING_TIME' == $row['timecontrol_type']) {
+							$workedDaysAmount[$row['due_date']] = 1;
+			}
 
-			if (!in_array($row['due_date'], $holidayDaysAmount) && 'PLL_HOLIDAY' == $row['timecontrol_type'])
-				$holidayDaysAmount[$row['due_date']] = 1;
+			if (!in_array($row['due_date'], $holidayDaysAmount) && 'PLL_HOLIDAY' == $row['timecontrol_type']) {
+							$holidayDaysAmount[$row['due_date']] = 1;
+			}
 
-			if ('PLL_WORKING_TIME' == $row['timecontrol_type'])
-				$sumWorkTime += $row['daytime'];
+			if ('PLL_WORKING_TIME' == $row['timecontrol_type']) {
+							$sumWorkTime += $row['daytime'];
+			}
 
-			if ('PLL_BREAK_TIME' == $row['timecontrol_type'])
-				$sumBreakTime += $row['daytime'];
+			if ('PLL_BREAK_TIME' == $row['timecontrol_type']) {
+							$sumBreakTime += $row['daytime'];
+			}
 
-			if (!in_array($row['due_date'], $days))
-				$days[] = $row['due_date'];
+			if (!in_array($row['due_date'], $days)) {
+							$days[] = $row['due_date'];
+			}
 		}
 
 		if ($numRows > 0) {
@@ -114,17 +121,19 @@ 
 			$response['workingDays'] = $allDaysAndWeekends['workingDays'];
 
 			if ($sumWorkTime > 0) {
-				if (0 == $workedDaysAmount)
-					$averageWorkingTime = $sumWorkTime;
-				else
-					$averageWorkingTime = $sumWorkTime / $workedDaysAmount;
+				if (0 == $workedDaysAmount) {
+									$averageWorkingTime = $sumWorkTime;
+				} else {
+									$averageWorkingTime = $sumWorkTime / $workedDaysAmount;
+				}
 			}
 
 			if ($sumBreakTime > 0) {
-				if (0 == $workedDaysAmount)
-					$averageBreakTime = $sumBreakTime;
-				else
-					$averageBreakTime = $sumBreakTime / $workedDaysAmount;
+				if (0 == $workedDaysAmount) {
+									$averageBreakTime = $sumBreakTime;
+				} else {
+									$averageBreakTime = $sumBreakTime / $workedDaysAmount;
+				}
 			}
 
 			$response['holiayDays'] = $holidayDaysAmount;
@@ -160,8 +169,9 @@ 
 		$time['start'] = vtlib\Functions::currentUserDisplayDate($time['start']);
 		$time['end'] = vtlib\Functions::currentUserDisplayDate($time['end']);
 
-		if ($user == NULL)
-			$user = $loggedUserId;
+		if ($user == NULL) {
+					$user = $loggedUserId;
+		}
 		$moduleModel = Vtiger_Module_Model::getInstance($moduleName);
 		$widget = Vtiger_Widget_Model::getInstance($linkId, $currentUser->getId());
 
@@ -237,12 +247,14 @@ 
 					}
 				}
 				foreach ($notWorkingDaysType as $key => $value) {
-					if ($whatDay == $value)
-						$isWorkDay = false;
+					if ($whatDay == $value) {
+											$isWorkDay = false;
+					}
 				}
 
-				if ($isWorkDay)
-					$workDays++;
+				if ($isWorkDay) {
+									$workDays++;
+				}
 
 				if ($whatDay > 5 && !$isWorkDay && $notWorkingDaysType) {
 					$weekends++;