YetiForceCompany / YetiForceCRM

modules/Documents/models/Field.php

Braces +7 added lines, -6 removed lines

@@ -24,12 +24,13 @@
 			$downloadType = $recordInstance->get('filelocationtype');
 			if ($downloadType == 'I') {
 				$filesize = $value;
-				if ($filesize < 1024)
-					$value = $filesize . ' B';
-				elseif ($filesize > 1024 && $filesize < 1048576)
-					$value = round($filesize / 1024, 2) . ' KB';
-				else if ($filesize > 1048576)
-					$value = round($filesize / (1024 * 1024), 2) . ' MB';
+				if ($filesize < 1024) {
+									$value = $filesize . ' B';
+				} elseif ($filesize > 1024 && $filesize < 1048576) {
+									$value = round($filesize / 1024, 2) . ' KB';
+				} else if ($filesize > 1048576) {
+									$value = round($filesize / (1024 * 1024), 2) . ' MB';
+				}
 			} else {
 				$value = ' --';
 			}

modules/Users/models/Privileges.php

Braces +3 added lines, -2 removed lines

@@ -133,8 +133,9 @@
 	 */
 	public static function getInstanceById($userId, $module = null)
 	{
-		if (empty($userId))
-			return null;
+		if (empty($userId)) {
+					return null;
+		}
 
 		if (isset(self::$userPrivilegesModelCache[$userId])) {
 			return self::$userPrivilegesModelCache[$userId];

Doc Comments +6 added lines, -2 removed lines

@@ -94,7 +94,6 @@ 
 
 	/**
 	 * Function to check whether the user has access to the specified action/operation on a given module by tabid
-	 * @param <Number> $tabId
 	 * @param <String/Number> $action
 	 * @return boolean true/false
 	 */
@@ -112,7 +111,7 @@ 
 	/**
 	 * Static Function to get the instance of the User Privileges model from the given list of key-value array
 	 * @param <Array> $valueMap
-	 * @return Users_Privilege_Model object
+	 * @return Users_Privileges_Model object
 	 */
 	public static function getInstance($valueMap)
 	{
@@ -187,6 +186,9 @@ 
 
 	protected static $lockEditCache = [];
 
+	/**
+	 * @param string $moduleName
+	 */
 	public static function checkLockEdit($moduleName, Vtiger_Record_Model $recordModel)
 	{
 		$recordId = $recordModel->getId();
@@ -273,6 +275,8 @@ 
 
 	/**
 	 * Function to get set Shared Owner Recursively
+	 * @param integer $recordId
+	 * @param string $moduleName
 	 */
 	public static function getSharedRecordsRecursively($recordId, $moduleName)
 	{

modules/Users/views/DeleteUser.php

Braces +3 added lines, -2 removed lines

@@ -36,8 +36,9 @@
 		$viewer->assign('USERID', $userid);
 		$viewer->assign('DELETE_USER_NAME', $userRecordModel->getName());
 		$viewer->assign('USER_LIST', $usersList);
-		if ($request->get('mode') == 'permanent')
-			$viewer->assign('PERMANENT', true);
+		if ($request->get('mode') == 'permanent') {
+					$viewer->assign('PERMANENT', true);
+		}
 		$viewer->view('DeleteUser.tpl', $moduleName);
 	}
 }

modules/IStorages/data_access/checkType.php

Braces +5 added lines, -4 removed lines

@@ -32,8 +32,8 @@ 
 		} else {
 			$saveRecord = true;
 		}
-		if (!$saveRecord)
-			return [
+		if (!$saveRecord) {
+					return [
 				'save_record' => $saveRecord,
 				'type' => 0,
 				'info' => [
@@ -42,8 +42,9 @@ 
 					'type' => 'error'
 				]
 			];
-		else
-			return ['save_record' => true];
+		} else {
+					return ['save_record' => true];
+		}
 	}
 
 	public function getConfig($id, $module, $baseModule)

modules/PBXManager/models/Module.php

Braces +5 added lines, -4 removed lines

@@ -31,10 +31,11 @@
 	 */
 	public function isPermitted($actionName)
 	{
-		if ($actionName == 'EditView' || $actionName == 'CreateView')
-			return false;
-		else
-			return ($this->isActive() && Users_Privileges_Model::isPermitted($this->getName(), $actionName));
+		if ($actionName == 'EditView' || $actionName == 'CreateView') {
+					return false;
+		} else {
+					return ($this->isActive() && Users_Privileges_Model::isPermitted($this->getName(), $actionName));
+		}
 	}
 
 	/**

modules/Settings/SupportProcesses/models/Module.php

Braces +3 added lines, -2 removed lines

@@ -111,8 +111,9 @@
 			$getAllTicketStatus = self::getAllTicketStatus();
 			foreach ($getTicketStatusClosed as $key => $closedStatus) {
 				foreach ($getAllTicketStatus as $key => $status) {
-					if ($closedStatus == $status)
-						unset($getAllTicketStatus[$key]);
+					if ($closedStatus == $status) {
+											unset($getAllTicketStatus[$key]);
+					}
 				}
 			}
 			$result = $getAllTicketStatus;

vendor/yii/BaseYii.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -9,8 +9,8 @@
 use yii\base\InvalidConfigException;
 use yii\base\InvalidParamException;
 use yii\base\UnknownClassException;
-use yii\log\Logger;
 use yii\di\Container;
+use yii\log\Logger;
 
 /**
  * Gets the application start timestamp.

Doc Comments +2 added lines, -2 removed lines

@@ -512,7 +512,7 @@ 
 
 	/**
 	 * Configures an object with the initial property values.
-	 * @param object $object the object to be configured
+	 * @param base\Object $object the object to be configured
 	 * @param array $properties the property initial values given in terms of name-value pairs.
 	 * @return object the object itself
 	 */
@@ -530,7 +530,7 @@ 
 	 * This method is provided such that we can get the public member variables of an object.
 	 * It is different from "get_object_vars()" because the latter will return private
 	 * and protected variables if it is called within the object itself.
-	 * @param object $object the object to be handled
+	 * @param base\ArrayableTrait $object the object to be handled
 	 * @return array the public member variables of the object
 	 */
 	public static function getObjectVars($object)

vendor/yii/db/sqlite/Schema.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -8,9 +8,9 @@
 namespace yii\db\sqlite;
 
 use yii\base\NotSupportedException;
+use yii\db\ColumnSchema;
 use yii\db\Expression;
 use yii\db\TableSchema;
-use yii\db\ColumnSchema;
 use yii\db\Transaction;
 
 /**

include/runtime/Viewer.php

Braces +8 added lines, -6 removed lines

@@ -29,8 +29,9 @@ 
 	protected function log($message, $delimiter = '\n')
 	{
 		static $file = null;
-		if ($file === null)
-			$file = dirname(__FILE__) . '/../../cache/logs/viewer-debug.log';
+		if ($file === null) {
+					$file = dirname(__FILE__) . '/../../cache/logs/viewer-debug.log';
+		}
 		if (self::$debugViewer) {
 			file_put_contents($file, $message . $delimiter, FILE_APPEND);
 		}
@@ -177,10 +178,11 @@ 
 			foreach ($this->tpl_vars as $key => $smarty_variable) {
 				// Determine type of value being pased.
 				$valueType = 'literal';
-				if (is_object($smarty_variable->value))
-					$valueType = get_class($smarty_variable->value);
-				else if (is_array($smarty_variable->value))
-					$valueType = 'array';
+				if (is_object($smarty_variable->value)) {
+									$valueType = get_class($smarty_variable->value);
+				} else if (is_array($smarty_variable->value)) {
+									$valueType = 'array';
+				}
 				$this->log(sprintf("DATA: %s, TYPE: %s", $key, $valueType));
 			}
 		}

Doc Comments +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
 
 	/**
 	 * log message into the file if in debug mode.
-	 * @param type $message
+	 * @param string $message
 	 * @param type $delimiter 
 	 */
 	protected function log($message, $delimiter = '\n')
@@ -162,7 +162,7 @@ 
 	 * @param string $templateName
 	 * @param string $moduleName
 	 * @param boolean $fetch
-	 * @return html data
+	 * @return string|boolean data
 	 */
 	public function view($templateName, $moduleName = '', $fetch = false)
 	{