YetiForceCompany / YetiForceCRM

modules/Calendar/actions/DragDropAjax.php

Doc Comments +3 added lines

@@ -119,6 +119,9 @@
 	 * Function adds days and minutes to datetime string
 	 */
 
+	/**
+	 * @param string $datetime
+	 */
 	public function changeDateTime($datetime, $daysToAdd, $minutesToAdd)
 	{
 		$datetime = strtotime($datetime);

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

modules/Calendar/Date.php

Doc Comments +1 added lines, -5 removed lines

@@ -384,7 +384,7 @@ 
 
 	/**
 	 *
-	 * @return Date
+	 * @return string
 	 */
 	public function get_DB_formatted_date()
 	{
@@ -529,10 +529,6 @@ 
 	 * This should be used whereever possible
 	 *
 	 * @param integer       $index - number between 0 to 42
-	 * @param string        $day   - date
-	 * @param string        $month - month
-	 * @param string        $year  - year
-	 * return vt_DateTime obj  $datetimevalue
 	 */
 	public function getThisMonthsDayByIndex($index)
 	{

Braces +37 added lines, -26 removed lines

@@ -87,12 +87,15 @@ 
 	 */
 	public function getTodayDatetimebyIndex($index, $day = '', $month = '', $year = '')
 	{
-		if ($day === '')
-			$day = $this->day;
-		if ($month === '')
-			$month = $this->month;
-		if ($year === '')
-			$year = $this->year;
+		if ($day === '') {
+					$day = $this->day;
+		}
+		if ($month === '') {
+					$month = $this->month;
+		}
+		if ($year === '') {
+					$year = $this->year;
+		}
 		$day_array = array();
 
 		if ($index < 0 || $index > 23) {
@@ -140,12 +143,15 @@ 
 	 */
 	public function getThismonthDaysbyIndex($index, $day = '', $month = '', $year = '')
 	{
-		if ($day == '')
-			$day = $index + 1;
-		if ($month == '')
-			$month = $this->month;
-		if ($year == '')
-			$year = $this->year;
+		if ($day == '') {
+					$day = $index + 1;
+		}
+		if ($month == '') {
+					$month = $this->month;
+		}
+		if ($year == '') {
+					$year = $this->year;
+		}
 		$month_array = array();
 		$month_array['day'] = $day;
 		$month_array['month'] = $month;
@@ -356,8 +362,9 @@ 
 		} else {
 			throw new \Exception\AppException('year was not set');
 		}
-		if (empty($hour) && $hour !== 0)
-			$hour = 0;
+		if (empty($hour) && $hour !== 0) {
+					$hour = 0;
+		}
 		$this->ts = mktime($hour, $minute, $second, $month, $day, $year);
 		$this->setDateTime($this->ts);
 	}
@@ -399,10 +406,12 @@ 
 	{
 		$hour = $this->z_hour;
 		$min = $this->minute;
-		if (empty($hour))
-			$hour = '00';
-		if (empty($min))
-			$min = '00';
+		if (empty($hour)) {
+					$hour = '00';
+		}
+		if (empty($min)) {
+					$min = '00';
+		}
 		return $hour . ':' . $min;
 	}
 
@@ -413,10 +422,11 @@ 
 	 */
 	public function get_changed_day($mode)
 	{
-		if ($mode == 'increment')
-			$day = $this->day + 1;
-		else
-			$day = $this->day - 1;
+		if ($mode == 'increment') {
+					$day = $this->day + 1;
+		} else {
+					$day = $this->day - 1;
+		}
 		$date_data = array('day' => $day,
 			'month' => $this->month,
 			'year' => $this->year
@@ -432,10 +442,11 @@ 
 	public function get_first_day_of_changed_week($mode)
 	{
 		$first_day = $this->getThisweekDaysbyIndex(1);
-		if ($mode == 'increment')
-			$day = $first_day->day + 7;
-		else
-			$day = $first_day->day - 7;
+		if ($mode == 'increment') {
+					$day = $first_day->day + 7;
+		} else {
+					$day = $first_day->day - 7;
+		}
 		$date_data = array('day' => $day,
 			'month' => $first_day->month,
 			'year' => $first_day->year

modules/Import/helpers/Utils.php

Doc Comments +3 added lines

@@ -233,6 +233,9 @@
 		return false;
 	}
 
+	/**
+	 * @param Vtiger_Request $request
+	 */
 	public static function validateFileUpload($request)
 	{
 		$current_user = Users_Record_Model::getCurrentUserModel();

modules/Import/views/Main.php

Doc Comments +9 added lines

@@ -27,6 +27,9 @@ 
 		$this->user = $user;
 	}
 
+	/**
+	 * @param Vtiger_Request $request
+	 */
 	public static function import($request, $user)
 	{
 		$importController = new Import_Main_View($request, $user);
@@ -100,6 +103,9 @@ 
 		}
 	}
 
+	/**
+	 * @param boolean $continueImport
+	 */
 	public static function showCurrentStatus($importInfo, $importStatusCount, $continueImport)
 	{
 		$moduleName = $importInfo['module'];
@@ -210,6 +216,9 @@ 
 		Import_Queue_Action::add($this->request, $this->user);
 	}
 
+	/**
+	 * @param Vtiger_Request $request
+	 */
 	public static function deleteMap($request)
 	{
 		$moduleName = $request->getModule();

modules/ModComments/ModComments.php

Doc Comments +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
 
 	/**
 	 * Add widget to other module.
-	 * @param unknown_type $moduleNames
+	 * @param string[] $moduleNames
 	 * @return unknown_type
 	 */
 	static function addWidgetTo($moduleNames, $widgetType = 'DETAILVIEWWIDGET', $widgetName = 'DetailViewBlockCommentWidget')

Braces +12 added lines, -8 removed lines

@@ -64,11 +64,13 @@ 
 	 */
 	static function addWidgetTo($moduleNames, $widgetType = 'DETAILVIEWWIDGET', $widgetName = 'DetailViewBlockCommentWidget')
 	{
-		if (empty($moduleNames))
-			return;
+		if (empty($moduleNames)) {
+					return;
+		}
 
-		if (is_string($moduleNames))
-			$moduleNames = array($moduleNames);
+		if (is_string($moduleNames)) {
+					$moduleNames = array($moduleNames);
+		}
 
 		$commentWidgetModules = array();
 		foreach ($moduleNames as $moduleName) {
@@ -95,11 +97,13 @@ 
 	 */
 	static function removeWidgetFrom($moduleNames, $widgetType = 'DETAILVIEWWIDGET', $widgetName = 'DetailViewBlockCommentWidget')
 	{
-		if (empty($moduleNames))
-			return;
+		if (empty($moduleNames)) {
+					return;
+		}
 
-		if (is_string($moduleNames))
-			$moduleNames = array($moduleNames);
+		if (is_string($moduleNames)) {
+					$moduleNames = array($moduleNames);
+		}
 
 		$commentWidgetModules = array();
 		foreach ($moduleNames as $moduleName) {

modules/OSSEmployees/OSSEmployees.php

Doc Comments +7 added lines

@@ -149,6 +149,10 @@ 
 		return $hierarchy;
 	}
 
+	/**
+	 * @param integer $id
+	 * @param integer[] $encountered_accounts
+	 */
 	public function __getParentEmployees($id, &$parent_accounts, &$encountered_accounts)
 	{
 		$adb = PearDatabase::getInstance();
@@ -200,6 +204,9 @@ 
 		return $parent_accounts;
 	}
 
+	/**
+	 * @param integer $id
+	 */
 	public function __getChildEmployees($id, &$child_accounts, $depth)
 	{
 		$adb = PearDatabase::getInstance();

Braces +3 added lines, -2 removed lines

@@ -257,8 +257,9 @@
 			$modcommentsModuleInstance = vtlib\Module::getInstance('ModComments');
 			if ($modcommentsModuleInstance && file_exists('modules/ModComments/ModComments.php')) {
 				include_once 'modules/ModComments/ModComments.php';
-				if (class_exists('ModComments'))
-					ModComments::addWidgetTo(array('OSSEmployees'));
+				if (class_exists('ModComments')) {
+									ModComments::addWidgetTo(array('OSSEmployees'));
+				}
 			}
 		} else if ($event_type == 'module.disabled') {
 			

include/Loader.php

Braces +3 added lines, -2 removed lines

@@ -124,8 +124,9 @@
 			$secondFallBackModuleDir = $secondFallBackModuleClassPath = $actualModule;
 			if ($actualModule != 'Users') {
 				$baseModule = $moduleHierarchyParts[0];
-				if ($baseModule == 'Settings')
-					$baseModule = 'Settings:Vtiger';
+				if ($baseModule == 'Settings') {
+									$baseModule = 'Settings:Vtiger';
+				}
 				$firstFallBackDir = str_replace(':', '.', $baseModule);
 				$firstFallBackClassPath = str_replace(':', '_', $baseModule);
 			}

include/utils/EmailTemplate.php

Spacing +2 added lines, -2 removed lines

@@ -120,7 +120,7 @@ 
 				$moduleTableIndexList = $meta->getEntityTableIndexList();
 				foreach ($tableList as $index => $tableName) {
 					if ($tableName != $tableList[0]) {
-						$sql .=' INNER JOIN ' . $tableName . ' ON ' . $tableList[0] . '.' .
+						$sql .= ' INNER JOIN ' . $tableName . ' ON ' . $tableList[0] . '.' .
 							$moduleTableIndexList[$tableList[0]] . '=' . $tableName . '.' .
 							$moduleTableIndexList[$tableName];
 					}
@@ -128,7 +128,7 @@ 
 				//If module is Leads and if you are not selected any leads fields then query failure is happening.
 				//By default we are checking where condition on base table.
 				if ($module == 'Leads' && !in_array('vtiger_leaddetails', $tableList)) {
-					$sql .=' INNER JOIN vtiger_leaddetails ON vtiger_leaddetails.leadid = vtiger_crmentity.crmid';
+					$sql .= ' INNER JOIN vtiger_leaddetails ON vtiger_leaddetails.leadid = vtiger_crmentity.crmid';
 				}
 
 				$sql .= ' WHERE';

Braces +6 added lines, -4 removed lines

@@ -209,8 +209,9 @@ 
 						$needle = '$' . strtolower($this->module) . '-' . $parentFieldColumnMapping[$params['field']] . ':' . $column . '$';
 						$this->processedDescription = str_replace($needle, $values[array_search($column, $fieldColumnMapping)], $this->processedDescription);
 					}
-					if (!$params['owner'])
-						$this->processedmodules[$module] = true;
+					if (!$params['owner']) {
+											$this->processedmodules[$module] = true;
+					}
 				}
 			}
 		}
@@ -253,7 +254,8 @@ 
 		$rows = $adb->num_rows($res);
 		if ($rows > 0) {
 			return true;
-		} else
-			return false;
+		} else {
+					return false;
+		}
 	}
 }

include/utils/ConfigReader.php

Braces +6 added lines, -4 removed lines

@@ -37,8 +37,9 @@ 
 
 	public function setConfig($key, $value)
 	{
-		if (empty($key))
-			return;
+		if (empty($key)) {
+					return;
+		}
 		$this->properties[$key] = $value;
 		//not neccessary for php5.x versions
 		self::$propertiesCache[$this->name] = $this->properties;
@@ -46,8 +47,9 @@ 
 
 	public function getConfig($key)
 	{
-		if (empty($key))
-			return '';
+		if (empty($key)) {
+					return '';
+		}
 		return $this->properties[$key];
 	}
 }

Doc Comments +7 added lines

@@ -15,6 +15,10 @@ 
 	protected $name;
 	static $propertiesCache = [];
 
+	/**
+	 * @param string $path
+	 * @param string $name
+	 */
 	public function __construct($path, $name, $force = false)
 	{
 		$this->load($path, $name, $force);
@@ -41,6 +45,9 @@ 
 		self::$propertiesCache[$this->name] = $this->properties;
 	}
 
+	/**
+	 * @param string $key
+	 */
 	public function getConfig($key)
 	{
 		if (empty($key))