YetiForceCompany / YetiForceCRM

app/Map/Address.php

Doc Comments +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
 	/**
 	 * Get default provider.
 	 *
-	 * @return string[]
+	 * @return string|null
 	 */
 	public static function getDefaultProvider()
 	{

modules/Vtiger/models/Widget.php

Doc Comments +8 added lines, -2 removed lines

@@ -45,7 +45,6 @@ 
 	/**
 	 * Function to get the position of the widget.
 	 *
-	 * @param int    $defaultPosition
 	 * @param string $coordinate
 	 *
 	 * @throws \App\Exceptions\AppException
@@ -118,6 +117,9 @@ 
 		return $self;
 	}
 
+	/**
+	 * @param integer $linkId
+	 */
 	public static function getInstance($linkId, $userId)
 	{
 		$row = (new \App\Db\Query())->from('vtiger_module_dashboard_widgets')
@@ -131,6 +133,10 @@ 
 		return $self;
 	}
 
+	/**
+	 * @param null|integer $linkId
+	 * @param integer|null $widgetId
+	 */
 	public static function updateWidgetPosition($position, $linkId, $widgetId, $userId)
 	{
 		if (!$linkId && !$widgetId) {
@@ -175,7 +181,7 @@ 
 	 * @param int $widgetId
 	 * @param int $userId
 	 *
-	 * @return \self
+	 * @return Vtiger_Widget_Model
 	 */
 	public static function getInstanceWithWidgetId($widgetId, $userId)
 	{

app/QueryGenerator.php

Doc Comments +15 added lines, -16 removed lines

@@ -138,7 +138,7 @@ 
 	/**
 	 * Get module model.
 	 *
-	 * @return string
+	 * @return \Vtiger_Module_Model
 	 */
 	public function getModuleModel()
 	{
@@ -180,7 +180,7 @@ 
 	 *
 	 * @param string[] $fields
 	 *
-	 * @return \self
+	 * @return QueryGenerator
 	 */
 	public function setFields($fields)
 	{
@@ -194,7 +194,7 @@ 
 	 *
 	 * @param int $offset
 	 *
-	 * @return \self
+	 * @return QueryGenerator
 	 */
 	public function setOffset($offset)
 	{
@@ -208,7 +208,7 @@ 
 	 *
 	 * @param int $limit
 	 *
-	 * @return \self
+	 * @return QueryGenerator
 	 */
 	public function setLimit($limit)
 	{
@@ -240,7 +240,7 @@ 
 	 *
 	 * @param string|string[] $fields
 	 *
-	 * @return \self
+	 * @return QueryGenerator
 	 */
 	public function setField($fields)
 	{
@@ -269,7 +269,7 @@ 
 	 *
 	 * @param type $columns
 	 *
-	 * @return \self
+	 * @return QueryGenerator
 	 */
 	public function setCustomColumn($columns)
 	{
@@ -290,9 +290,8 @@ 
 	/**
 	 * Set concat column.
 	 *
-	 * @param type $columns
 	 *
-	 * @return \self
+	 * @return QueryGenerator
 	 */
 	public function setConcatColumn($fieldName, $concat)
 	{
@@ -363,7 +362,7 @@ 
 	 *
 	 * @param string[] $field
 	 *
-	 * @return \self
+	 * @return QueryGenerator
 	 */
 	public function addRelatedField($field)
 	{
@@ -431,7 +430,7 @@ 
 	 * @param string $fieldName
 	 * @param string $order     ASC/DESC
 	 *
-	 * @return \self
+	 * @return QueryGenerator
 	 */
 	public function setOrder($fieldName, $order = false)
 	{
@@ -446,7 +445,7 @@ 
 	 *
 	 * @param string $fieldName
 	 *
-	 * @return \self
+	 * @return QueryGenerator
 	 */
 	public function setGroup($fieldName)
 	{
@@ -459,9 +458,9 @@ 
 	/**
 	 * Set custom group.
 	 *
-	 * @param string|array $groups
+	 * @param \yii\db\Expression[] $groups
 	 *
-	 * @return \self
+	 * @return QueryGenerator
 	 */
 	public function setCustomGroup($groups)
 	{
@@ -483,7 +482,7 @@ 
 	 * Function sets the field for which the duplicated values will be searched.
 	 *
 	 * @param string   $fieldName
-	 * @param int|bool $ignoreEmptyValue
+	 * @param boolean $ignoreEmptyValue
 	 */
 	public function setSearchFieldsForDuplicates($fieldName, $ignoreEmptyValue = true)
 	{
@@ -716,7 +715,7 @@ 
 	/**
 	 * Parsing advanced filters conditions.
 	 *
-	 * @return bool
+	 * @return false|null
 	 */
 	public function parseAdvFilter($advFilterList = false)
 	{
@@ -1220,7 +1219,7 @@ 
 	 * Set base search condition (search_key,search_value in url).
 	 *
 	 * @param string $fieldName
-	 * @param mixed  $value
+	 * @param mixed  $values
 	 * @param string $operator
 	 */
 	public function addBaseSearchConditions($fieldName, $values, $operator = 'e')

modules/Products/models/Record.php

Doc Comments +1 added lines, -1 removed lines

@@ -168,7 +168,7 @@
 	/**
 	 * The function decide about mandatory save record.
 	 *
-	 * @return type
+	 * @return boolean
 	 */
 	public function isMandatorySave()
 	{

modules/Chat/views/Modal.php

Spacing +1 added lines, -2 removed lines

@@ -85,8 +85,7 @@
 	private function isSoundNotification(): bool
 	{
 		return isset($_COOKIE['chat-isSoundNotification']) ?
-			filter_var($_COOKIE['chat-isSoundNotification'], FILTER_VALIDATE_BOOLEAN) :
-			\AppConfig::module('Chat', 'DEFAULT_SOUND_NOTIFICATION');
+			filter_var($_COOKIE['chat-isSoundNotification'], FILTER_VALIDATE_BOOLEAN) : \AppConfig::module('Chat', 'DEFAULT_SOUND_NOTIFICATION');
 	}
 
 	/**

config/modules/AddressFinder.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 	'REMAPPING_OPENCAGE' => null,
 	// Function to remapping fields in countries for OpenCage. It should be function
 	'REMAPPING_OPENCAGE_FOR_COUNTRY' => [
-		'Australia' => function ($row) {
+		'Australia' => function($row) {
 			return [
 				'addresslevel1' => [$row['components']['country'] ?? '', $row['components']['ISO_3166-1_alpha-2'] ?? ''],
 				'addresslevel2' => $row['components']['state'] ?? '',

modules/Users/models/Privileges.php

Doc Comments +7 added lines, -3 removed lines

@@ -103,8 +103,7 @@ 
 	/**
 	 * 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
+	 * @param string $action
 	 *
 	 * @return bool true/false
 	 */
@@ -180,6 +179,9 @@ 
 
 	protected static $lockEditCache = [];
 
+	/**
+	 * @param string $moduleName
+	 */
 	public static function checkLockEdit($moduleName, Vtiger_Record_Model $recordModel)
 	{
 		$recordId = $recordModel->getId();
@@ -224,7 +226,7 @@ 
 	/**
 	 * Clear user cache.
 	 *
-	 * @param int|bool $userId
+	 * @param integer $userId
 	 */
 	public static function clearCache($userId = false)
 	{
@@ -275,6 +277,8 @@ 
 
 	/**
 	 * Function to get set Shared Owner Recursively.
+	 * @param integer $recordId
+	 * @param string $moduleName
 	 */
 	public static function getSharedRecordsRecursively($recordId, $moduleName)
 	{

vtlib/Vtiger/Functions.php

Doc Comments +3 added lines, -1 removed lines

@@ -158,7 +158,6 @@ 
 	/**
 	 * Function get module field infos.
 	 *
-	 * @param int|string $mixed
 	 * @param bool       $returnByColumn
 	 *
 	 * @return mixed[]
@@ -615,6 +614,9 @@ 
 	 * @return <Date> - last working y
 	 */
 
+	/**
+	 * @param string $date
+	 */
 	public static function getLastWorkingDay($date)
 	{
 		if (empty($date)) {

modules/Settings/CurrencyUpdate/models/Module.php

Doc Comments +12 added lines

@@ -47,6 +47,9 @@ 
 	 * @return boolean - true if fetched new exchange rates, false otherwise
 	 */
 
+	/**
+	 * @param string $dateCur
+	 */
 	public function fetchCurrencyRates($dateCur, $cron = false)
 	{
 		if (!\App\RequestUtil::isNetConnection()) {
@@ -123,6 +126,9 @@ 
 	 * @param <Float> $exchange - exchange rate
 	 */
 
+	/**
+	 * @param string|false $id
+	 */
 	public function updateCurrencyRate($id, $exchange)
 	{
 		\App\Db::getInstance()->createCommand()
@@ -172,6 +178,9 @@ 
 	 * @return <Array> - array containing currency rates
 	 */
 
+	/**
+	 * @param string $dateCur
+	 */
 	public function getRatesHistory($bankId, $dateCur, \App\Request $request)
 	{
 		$query = (new App\Db\Query())->select(['exchange', 'currency_name', 'currency_code', 'currency_symbol', 'fetch_date', 'exchange_date'])
@@ -366,6 +375,9 @@ 
 	 * @return boolean - true on success or false
 	 */
 
+	/**
+	 * @param integer $bankId
+	 */
 	public function setActiveBankById($bankId)
 	{
 		$db = \App\Db::getInstance();