YetiForceCompany / YetiForceCRM

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

libraries/SabreDAV/VObject/Property/Boolean.php

Indentation +62 added lines, -62 removed lines

@@ -3,7 +3,7 @@ 
 namespace Sabre\VObject\Property;
 
 use
-    Sabre\VObject\Property;
+	Sabre\VObject\Property;
 
 /**
  * Boolean property.
@@ -19,66 +19,66 @@ 
  */
 class Boolean extends Property {
 
-    /**
-     * Sets a raw value coming from a mimedir (iCalendar/vCard) file.
-     *
-     * This has been 'unfolded', so only 1 line will be passed. Unescaping is
-     * not yet done, but parameters are not included.
-     *
-     * @param string $val
-     *
-     * @return void
-     */
-    function setRawMimeDirValue($val) {
-
-        $val = strtoupper($val) === 'TRUE' ? true : false;
-        $this->setValue($val);
-
-    }
-
-    /**
-     * Returns a raw mime-dir representation of the value.
-     *
-     * @return string
-     */
-    function getRawMimeDirValue() {
-
-        return $this->value ? 'TRUE' : 'FALSE';
-
-    }
-
-    /**
-     * Returns the type of value.
-     *
-     * This corresponds to the VALUE= parameter. Every property also has a
-     * 'default' valueType.
-     *
-     * @return string
-     */
-    function getValueType() {
-
-        return 'BOOLEAN';
-
-    }
-
-    /**
-     * Hydrate data from a XML subtree, as it would appear in a xCard or xCal
-     * object.
-     *
-     * @param array $value
-     *
-     * @return void
-     */
-    function setXmlValue(array $value) {
-
-        $value = array_map(
-            function($value) {
-                return 'true' === $value;
-            },
-            $value
-        );
-        parent::setXmlValue($value);
-
-    }
+	/**
+	 * Sets a raw value coming from a mimedir (iCalendar/vCard) file.
+	 *
+	 * This has been 'unfolded', so only 1 line will be passed. Unescaping is
+	 * not yet done, but parameters are not included.
+	 *
+	 * @param string $val
+	 *
+	 * @return void
+	 */
+	function setRawMimeDirValue($val) {
+
+		$val = strtoupper($val) === 'TRUE' ? true : false;
+		$this->setValue($val);
+
+	}
+
+	/**
+	 * Returns a raw mime-dir representation of the value.
+	 *
+	 * @return string
+	 */
+	function getRawMimeDirValue() {
+
+		return $this->value ? 'TRUE' : 'FALSE';
+
+	}
+
+	/**
+	 * Returns the type of value.
+	 *
+	 * This corresponds to the VALUE= parameter. Every property also has a
+	 * 'default' valueType.
+	 *
+	 * @return string
+	 */
+	function getValueType() {
+
+		return 'BOOLEAN';
+
+	}
+
+	/**
+	 * Hydrate data from a XML subtree, as it would appear in a xCard or xCal
+	 * object.
+	 *
+	 * @param array $value
+	 *
+	 * @return void
+	 */
+	function setXmlValue(array $value) {
+
+		$value = array_map(
+			function($value) {
+				return 'true' === $value;
+			},
+			$value
+		);
+		parent::setXmlValue($value);
+
+	}
 
 }

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