YetiForceCompany / YetiForceCRM

modules/Calendar/RepeatEvents.php

Doc Comments +2 added lines

@@ -28,6 +28,7 @@ 
 
 	/**
 	 * Increment the time by interval and return value in YYYY-MM-DD HH:MM format.
+	 * @param integer $basetiming
 	 */
 	static function nexttime($basetiming, $interval)
 	{
@@ -146,6 +147,7 @@ 
 
 	/**
 	 * Repeat Activity instance till given limit.
+	 * @param RecurringType|null $recurObj
 	 */
 	static function repeat($focus, $recurObj)
 	{

Braces +3 added lines, -2 removed lines

@@ -175,8 +175,9 @@
 			$endDate = date('Y-m-d', $endDateTime);
 
 			// Reset the new_focus and prepare for reuse
-			if (isset($new_focus->id))
-				unset($new_focus->id);
+			if (isset($new_focus->id)) {
+							unset($new_focus->id);
+			}
 			$new_focus->column_fields = array();
 
 			foreach ($base_focus->column_fields as $key => $value) {

modules/Emails/mail.php

Doc Comments +10 added lines

@@ -103,6 +103,7 @@ 
 /** 	Function to get the user Email id based on column name and column value
  * 	$name -- column name of the vtiger_users vtiger_table
  * 	$val  -- column value
+ * @param string $name
  */
 function getUserEmailId($name, $val)
 {
@@ -162,6 +163,8 @@ 
  * 	$attachment	-- whether we want to attach the currently selected file or all vtiger_files.
   [values = current,all] - optional
  * 	$emailid	-- id of the email object which will be used to get the vtiger_attachments - optional
+ * @param PHPMailer $mail
+ * @param string $to_email
  */
 function setMailerProperties($mail, $subject, $contents, $fromEmail, $fromName, $to_email, $attachment = '', $emailid = '', $module = '', $logo = '')
 {
@@ -297,6 +300,7 @@ 
  * 	$mail -- reference of the mail object
  * 	$filename -- filename which is going to added with the mail
  * 	$record -- id of the record - optional
+ * @param string $record
  */
 function addAttachment($mail, $filename, $record)
 {
@@ -313,6 +317,7 @@ 
 /**     Function to add all the vtiger_files as attachment with the mail object
  *     $mail -- reference of the mail object
  *     $record -- email id ie., record id which is used to get the all vtiger_attachments from database
+ * @param string $record
  */
 function addAllAttachments($mail, $record)
 {
@@ -347,6 +352,9 @@ 
  * 	$mail -- reference of the mail object
  * 	$cc_mod -- mode to set the address ie., cc or bcc
  * 	$cc_val -- addresss with comma seperated to set as CC or BCC in the mail
+ * @param PHPMailer $mail
+ * @param string $cc_mod
+ * @param string $cc_val
  */
 function setCCAddress($mail, $cc_mod, $cc_val)
 {
@@ -375,6 +383,7 @@ 
 
 /** 	Function to send the mail which will be called after set all the mail object values
  * 	$mail -- reference of the mail object
+ * @param PHPMailer $mail
  */
 function MailSend($mail)
 {
@@ -423,6 +432,7 @@ 
  * 	$mail_status -- status of the mail which is sent or not
  * 	$to -- the email address to whom we sent the mail and failes
  * 	return -- Mail error occured during the mail sending process
+ * @param PHPMailer $mail
  */
 function getMailError($mail, $mail_status, $to)
 {

Spacing +5 added lines, -5 removed lines

@@ -179,7 +179,7 @@ 
 	$mail->Body = decode_html($contents);
 	$mail->AltBody = strip_tags(preg_replace(array("/<p>/i", "/<br>/i", "/<br \/>/i"), array("\n", "\n", "\n"), $contents));
 
-	$mail->IsSMTP();  //set mailer to use SMTP
+	$mail->IsSMTP(); //set mailer to use SMTP
 
 	setMailServerProperties($mail);
 
@@ -230,7 +230,7 @@ 
 		addAllAttachments($mail, $emailid);
 	}
 
-	$mail->IsHTML(true);  // set email format to HTML
+	$mail->IsHTML(true); // set email format to HTML
 
 	return;
 }
@@ -278,7 +278,7 @@ 
 	if ($smtp_auth) {
 		$mail->SMTPAuth = true; // turn on SMTP authentication
 	}
-	$mail->Host = $server;  // specify main and backup server
+	$mail->Host = $server; // specify main and backup server
 	$mail->Username = $username; // SMTP username
 	$mail->Password = $password; // SMTP password
 	// To Support TLS
@@ -525,8 +525,8 @@ 
 
 function isUserInitiated()
 {
-	return ( AppRequest::get('module') == 'Emails' &&
-		( AppRequest::get('action') == 'mailsend' || AppRequest::get('action') == 'Save'));
+	return (AppRequest::get('module') == 'Emails' &&
+		(AppRequest::get('action') == 'mailsend' || AppRequest::get('action') == 'Save'));
 }
 
 /**

Braces +31 added lines, -24 removed lines

@@ -245,27 +245,30 @@ 
 	$log->debug('Inside the function setMailServerProperties');
 
 	$res = $adb->pquery('select * from vtiger_systems where server_type=?', array('email'));
-	if (AppRequest::has('server'))
-		$server = AppRequest::get('server');
-	else
-		$server = $adb->query_result_raw($res, 0, 'server');
-	if (AppRequest::has('server_username'))
-		$username = AppRequest::get('server_username');
-	else
-		$username = $adb->query_result_raw($res, 0, 'server_username');
-
-	if (AppRequest::has('server_password'))
-		$password = AppRequest::get('server_password');
-	else
-		$password = $adb->query_result_raw($res, 0, 'server_password');
+	if (AppRequest::has('server')) {
+			$server = AppRequest::get('server');
+	} else {
+			$server = $adb->query_result_raw($res, 0, 'server');
+	}
+	if (AppRequest::has('server_username')) {
+			$username = AppRequest::get('server_username');
+	} else {
+			$username = $adb->query_result_raw($res, 0, 'server_username');
+	}
+
+	if (AppRequest::has('server_password')) {
+			$password = AppRequest::get('server_password');
+	} else {
+			$password = $adb->query_result_raw($res, 0, 'server_password');
+	}
 
 	// Define default state
 	$smtp_auth = false;
 
 	// Prasad: First time read smtp_auth from the request
-	if (AppRequest::get('smtp_auth') == 'on')
-		$smtp_auth = true;
-	else if (AppRequest::get('module') == 'Settings' && (!AppRequest::has('smtp_auth'))) {
+	if (AppRequest::get('smtp_auth') == 'on') {
+			$smtp_auth = true;
+	} else if (AppRequest::get('module') == 'Settings' && (!AppRequest::has('smtp_auth'))) {
 		//added to avoid issue while editing the values in the outgoing mail server.
 		$smtp_auth = false;
 	} else {
@@ -353,10 +356,12 @@ 
 	$log = LoggerManager::getInstance();
 	$log->debug('Inside the functin setCCAddress');
 
-	if ($cc_mod == 'cc')
-		$method = 'AddCC';
-	if ($cc_mod == 'bcc')
-		$method = 'AddBCC';
+	if ($cc_mod == 'cc') {
+			$method = 'AddCC';
+	}
+	if ($cc_mod == 'bcc') {
+			$method = 'AddBCC';
+	}
 	if ($cc_val != '') {
 		$ccmail = explode(",", trim($cc_val, ","));
 		for ($i = 0; $i < count($ccmail); $i++) {
@@ -367,8 +372,9 @@ 
 				$cc_name = $name_addr_pair[0];
 				$addr = trim($name_addr_pair[1], ">");
 			}
-			if ($ccmail[$i] != '')
-				$mail->$method($addr, $cc_name);
+			if ($ccmail[$i] != '') {
+							$mail->$method($addr, $cc_name);
+			}
 		}
 	}
 }
@@ -541,8 +547,9 @@ 
 		$userGroups = new GetGroupUsers();
 		$userGroups->getAllUsersInGroup($groupId);
 
-		if (count($userGroups->group_users) == 0)
-			return [];
+		if (count($userGroups->group_users) == 0) {
+					return [];
+		}
 
 		$query = sprintf('SELECT 
 					email1 

modules/HelpDesk/dashboards/TicketsByStatus.php

Doc Comments -1 removed lines

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

Braces +14 added lines, -10 removed lines

@@ -19,8 +19,9 @@ 
 
 		$listSearchParams = [];
 		$conditions = array(array('ticketstatus', 'e', $value));
-		if (!empty($assignedto))
-			array_push($conditions, array('assigned_user_id', 'e', $assignedto));
+		if (!empty($assignedto)) {
+					array_push($conditions, array('assigned_user_id', 'e', $assignedto));
+		}
 		$listSearchParams[] = $conditions;
 		return '&viewname=All&search_params=' . json_encode($listSearchParams);
 	}
@@ -72,8 +73,9 @@ 
 				$priorities[$row['priority']] = $counter++;
 				$colors[$row['priority']] = $row['color'];
 			}
-			if (!in_array($row['statusvalue'], $status))
-				$status[] = $row['statusvalue'];
+			if (!in_array($row['statusvalue'], $status)) {
+							$status[] = $row['statusvalue'];
+			}
 		}
 		if (!empty($tickets)) {
 			$counter = 0;
@@ -117,13 +119,15 @@ 
 		$data = $request->get('data');
 		$createdTime = $request->get('createdtime');
 		$widget = Vtiger_Widget_Model::getInstance($linkId, $currentUser->getId());
-		if (!$request->has('owner'))
-			$owner = Settings_WidgetsManagement_Module_Model::getDefaultUserId($widget, $moduleName);
-		else
-			$owner = $request->get('owner');
+		if (!$request->has('owner')) {
+					$owner = Settings_WidgetsManagement_Module_Model::getDefaultUserId($widget, $moduleName);
+		} else {
+					$owner = $request->get('owner');
+		}
 		$ownerForwarded = $owner;
-		if ($owner == 'all')
-			$owner = '';
+		if ($owner == 'all') {
+					$owner = '';
+		}
 
 		//Date conversion from user to database format
 		if (!empty($createdTime)) {

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') {
 			

modules/OSSMail/MailAttachmentMIME.php

Doc Comments +1 added lines, -1 removed lines

@@ -6,7 +6,7 @@
 
 	/**
 	 * Detect the MIME type based on extension in the filename.
-	 * @return MIME type text.
+	 * @return string type text.
 	 */
 	static function detect($filename)
 	{

modules/PaymentsIn/helpers/mt940.php

Doc Comments +15 added lines

@@ -50,6 +50,9 @@ 
 		return $xml;
 	}
 
+	/**
+	 * @param integer $level
+	 */
 	protected function createXML($key, $value, $level)
 	{
 		$indent = '';
@@ -69,6 +72,9 @@ 
 		return $xml;
 	}
 
+	/**
+	 * @param string $line
+	 */
 	protected function parseLine($line)
 	{
 		$tag = substr($line, 1, strpos($line, ':', 1) - 1);
@@ -114,6 +120,9 @@ 
 		$this->_lastTag = $tag;
 	}
 
+	/**
+	 * @param string $value
+	 */
 	protected function parseOperation($value)
 	{
 		$this->operations[] = array(
@@ -124,6 +133,9 @@ 
 		);
 	}
 
+	/**
+	 * @param string $value
+	 */
 	protected function parseBalance($value)
 	{
 		return array(
@@ -134,6 +146,9 @@ 
 		);
 	}
 
+	/**
+	 * @param string $value
+	 */
 	protected function parseTransaction($value)
 	{
 		$transaction = array(

Braces +19 added lines, -14 removed lines

@@ -34,8 +34,9 @@ 
 	public function parse()
 	{
 		$tab = $this->prepareFile();
-		foreach ($tab as $line)
-			$this->parseLine($line);
+		foreach ($tab as $line) {
+					$this->parseLine($line);
+		}
 	}
 
 	public function getXML()
@@ -53,13 +54,15 @@ 
 	protected function createXML($key, $value, $level)
 	{
 		$indent = '';
-		for ($i = 0; $i <= $level; $i++)
-			$indent .= "\t";
+		for ($i = 0; $i <= $level; $i++) {
+					$indent .= "\t";
+		}
 		if (is_array($value)) {
 			$xml = "$indent<$key>\n";
 			foreach ($value as $subKey => $subVal) {
-				if (is_numeric($subKey))
-					$subKey = substr($key, 0, -1);
+				if (is_numeric($subKey)) {
+									$subKey = substr($key, 0, -1);
+				}
 				$xml .= $this->createXML($subKey, $subVal, $level + 1);
 			}
 			$xml .= "$indent</$key>\n";
@@ -85,10 +88,11 @@ 
 				break;
 			case 'NS':
 				$code = substr($value, 0, 2);
-				if ($code == '22')
-					$this->ownerName = substr($value, 2);
-				else if ($code == '23')
-					$this->accountName = substr($value, 2);
+				if ($code == '22') {
+									$this->ownerName = substr($value, 2);
+				} else if ($code == '23') {
+									$this->accountName = substr($value, 2);
+				}
 				break;
 			case '60F':
 				$this->openBalance = $this->parseBalance($value);
@@ -103,10 +107,11 @@ 
 				self::parseOperation($value);
 				break;
 			case '86':
-				if ($this->_lastTag == '61')
-					$this->parseTransaction($value);
-				else
-					$this->info .= $value;
+				if ($this->_lastTag == '61') {
+									$this->parseTransaction($value);
+				} else {
+									$this->info .= $value;
+				}
 				break;
 			default:
 				break;

modules/PaymentsIn/helpers/subclass/mt940_Milenium.php

Doc Comments +6 added lines

@@ -20,6 +20,9 @@ 
 			$this->parseLine($line);
 	}
 
+	/**
+	 * @param string $line
+	 */
 	protected function parseLine($line)
 	{
 		$tag = substr($line, 1, strpos($line, ':', 1) - 1);
@@ -65,6 +68,9 @@ 
 		$this->_lastTag = $tag;
 	}
 
+	/**
+	 * @param string $value
+	 */
 	protected function parseOperation($value)
 	{
 		$this->operations[] = array(

Braces +13 added lines, -10 removed lines

@@ -16,8 +16,9 @@ 
 	public function parse()
 	{
 		$tab = $this->prepareFile();
-		foreach ($tab as $line)
-			$this->parseLine($line);
+		foreach ($tab as $line) {
+					$this->parseLine($line);
+		}
 	}
 
 	protected function parseLine($line)
@@ -36,10 +37,11 @@ 
 				break;
 			case 'NS':
 				$code = substr($value, 0, 2);
-				if ($code == '22')
-					$this->ownerName = substr($value, 2);
-				else if ($code == '23')
-					$this->accountName = substr($value, 2);
+				if ($code == '22') {
+									$this->ownerName = substr($value, 2);
+				} else if ($code == '23') {
+									$this->accountName = substr($value, 2);
+				}
 				break;
 			case '60F':
 				$this->openBalance = $this->parseBalance($value);
@@ -54,10 +56,11 @@ 
 				self::parseOperation($value);
 				break;
 			case '86':
-				if ($this->_lastTag == '61')
-					$this->parseTransaction($value);
-				else
-					$this->info .= $value;
+				if ($this->_lastTag == '61') {
+									$this->parseTransaction($value);
+				} else {
+									$this->info .= $value;
+				}
 				break;
 			default:
 				break;