YetiForceCompany / YetiForceCRM

app/Controller/Modal.php

Doc Comments +2 added lines, -2 removed lines

@@ -58,7 +58,7 @@ 
 	 *
 	 * @param \App\Request $request
 	 *
-	 * @return \Vtiger_JsScript_Model[]
+	 * @return Vtiger_JsScript_Model[]
 	 */
 	public function getModalScripts(\App\Request $request)
 	{
@@ -74,7 +74,7 @@ 
 	 *
 	 * @param \App\Request $request
 	 *
-	 * @return \Vtiger_CssScript_Model[]
+	 * @return Vtiger_CssScript_Model[]
 	 */
 	public function getModalCss(\App\Request $request)
 	{

modules/Products/models/Record.php

Doc Comments +4 added lines, -1 removed lines

@@ -221,6 +221,9 @@ 
 		return $status;
 	}
 
+	/**
+	 * @param integer $productId
+	 */
 	public function getPriceDetailsForProduct($productId, $unitPrice, $available = 'available', $itemType = 'Products')
 	{
 		\App\Log::trace('Entering into function getPriceDetailsForProduct(' . $productId . ')');
@@ -351,7 +354,7 @@ 
 	/**
 	 * The function decide about mandatory save record.
 	 *
-	 * @return type
+	 * @return boolean
 	 */
 	public function isMandatorySave()
 	{

modules/Vtiger/dashboards/ChartFilter.php

Spacing +1 added lines, -1 removed lines

@@ -22,7 +22,7 @@
 		$widget = Vtiger_Widget_Model::getInstanceWithWidgetId($widgetId, \App\User::getCurrentUserId());
 		$chartFilterWidgetModel = Vtiger_ChartFilter_Model::getInstance();
 		$chartFilterWidgetModel->setWidgetModel($widget);
-		$additionalFilterFields= $chartFilterWidgetModel->getAdditionalFiltersFields();
+		$additionalFilterFields = $chartFilterWidgetModel->getAdditionalFiltersFields();
 		$viewer->assign('WIDGET', $widget);
 		$viewer->assign('MODULE_NAME', $moduleName);
 		$viewer->assign('CHART_MODEL', $chartFilterWidgetModel);

modules/Vtiger/helpers/Util.php

Doc Comments +4 added lines, -1 removed lines

@@ -203,7 +203,7 @@ 
 	/**
 	 * Function to get the time value in user preferred hour format.
 	 *
-	 * @param <Time>               $time
+	 * @param string               $time
 	 * @param <Vtiger_Users_Model> $userObject
 	 *
 	 * @return string time with hour format
@@ -302,6 +302,9 @@ 
 	 * else returns empty string
 	 */
 
+	/**
+	 * @param string $dataType
+	 */
 	public function getDefaultMandatoryValue($dataType)
 	{
 		switch ($dataType) {

modules/Vtiger/models/ChartFilter.php

Doc Comments +7 added lines, -8 removed lines

@@ -272,7 +272,7 @@ 
 	 * @param int $linkId
 	 * @param int $userId
 	 *
-	 * @return \self
+	 * @return Vtiger_ChartFilter_Model
 	 */
 	public static function getInstance($linkId = 0, $userId = 0)
 	{
@@ -395,7 +395,7 @@ 
 	/**
 	 * Get filter id.
 	 *
-	 * @param string|int $dividingValue
+	 * @param integer $dividingValue
 	 *
 	 * @return int
 	 */
@@ -573,7 +573,7 @@ 
 	/**
 	 * Iterate through all rows collected from db.
 	 *
-	 * @param {callback} $callback
+	 * @param Closure $callback
 	 */
 	protected function iterateAllRows($callback)
 	{
@@ -652,7 +652,7 @@ 
 	/**
 	 * Set colors.
 	 *
-	 * @param {string} $from
+	 * @param string $from
 	 */
 	protected function setColorsFrom($from)
 	{
@@ -678,7 +678,7 @@ 
 	/**
 	 * Add query to get picklists id (for colors).
 	 *
-	 * @param $query
+	 * @param App\Db\Query $query
 	 * @param $queryGenerator
 	 *
 	 * @return $query
@@ -710,7 +710,7 @@ 
 	/**
 	 * Get query for specified filter.
 	 *
-	 * @param $filter
+	 * @param integer $filter
 	 *
 	 * @return \App\Db\Query
 	 */
@@ -866,7 +866,6 @@ 
 	/**
 	 * Get current rows.
 	 *
-	 * @param int        $index         row index
 	 * @param string|int $groupValue
 	 * @param string|int $dividingValue
 	 */
@@ -879,7 +878,7 @@ 
 	 * Get rows for dividing field chart.
 	 *
 	 * @param \App\QueryGenerator $query
-	 * @param string|int          $dividingValue
+	 * @param integer          $dividingValue
 	 *
 	 * @return array
 	 */

Spacing +7 added lines, -7 removed lines

@@ -594,7 +594,7 @@ 
 		$colors = \App\Fields\Picklist::getColors($fieldName);
 		$this->colorsFrom = 'picklist';
 		$this->colorsFromRow = 'picklist_id';
-		$this->iterateAllRows(function ($row, $groupValue, $dividingValue, $rowIndex) use ($colors) {
+		$this->iterateAllRows(function($row, $groupValue, $dividingValue, $rowIndex) use ($colors) {
 			if (isset($colors[$row['picklist_id']])) {
 				$this->colors[$row['picklist_id']] = $colors[$row['picklist_id']];
 			}
@@ -609,7 +609,7 @@ 
 		$this->colorsFrom = 'filters';
 		$this->colorsFromRow = 'color';
 		$colors = \App\Colors::getAllFilterColors();
-		$this->iterateAllRows(function (&$row, $groupValue, $dividingValue, $rowIndex) use ($colors) {
+		$this->iterateAllRows(function(&$row, $groupValue, $dividingValue, $rowIndex) use ($colors) {
 			$this->colors[$dividingValue] = $colors[$this->filterIds[$dividingValue]];
 		});
 	}
@@ -621,7 +621,7 @@ 
 	{
 		$this->colorsFrom = 'assigned_user_id';
 		$this->colorsFromRow = 'assigned_user_id';
-		$this->iterateAllRows(function ($row, $groupValue, $dividingValue, $rowIndex) {
+		$this->iterateAllRows(function($row, $groupValue, $dividingValue, $rowIndex) {
 			$this->colors[$row['assigned_user_id']] = \App\Fields\Owner::getColor($row['assigned_user_id']);
 		});
 	}
@@ -633,7 +633,7 @@ 
 	{
 		$this->colorsFrom = 'record_id';
 		$this->colorsFromRow = 'id';
-		$this->iterateAllRows(function ($row, $groupValue, $dividingValue, $rowIndex) {
+		$this->iterateAllRows(function($row, $groupValue, $dividingValue, $rowIndex) {
 			$this->colors[$row['id']] = \App\Colors::getRandomColor('from_id_' . $row['id']);
 		});
 	}
@@ -645,7 +645,7 @@ 
 	{
 		$this->colorsFrom = 'record_number';
 		$this->colorsFromRow = 'record_number';
-		$this->iterateAllRows(function ($row, $groupValue, $dividingValue, $rowIndex) {
+		$this->iterateAllRows(function($row, $groupValue, $dividingValue, $rowIndex) {
 			$this->colors[$groupValue] = \App\Colors::getRandomColor('generated_' . $groupValue);
 		});
 	}
@@ -938,7 +938,7 @@ 
 		$this->calculateAverage();
 		$this->normalizeData();
 		$this->setColorsFrom($this->findOutColorsFromRows());
-		$this->iterateAllRows(function ($row, $groupValue, $dividingValue, $rowIndex) {
+		$this->iterateAllRows(function($row, $groupValue, $dividingValue, $rowIndex) {
 			$this->setColorFromRow($row, $groupValue, $dividingValue);
 		});
 		return $this->data;
@@ -1183,7 +1183,7 @@ 
 	protected function generateSectorsData()
 	{
 		// in funnel chart there is only one dividingValue 0 so it will iterate only once like flat array
-		$this->iterateAllRows(function ($row, $groupValue, $dividingValue, $rowIndex) {
+		$this->iterateAllRows(function($row, $groupValue, $dividingValue, $rowIndex) {
 			$value = (float) $row[$this->valueName];
 			$sectorId = $this->getSectorForValue($value);
 			$sectorIndex = array_search($sectorId, $this->sectors);

modules/Vtiger/actions/Mass.php

Doc Comments +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
 	 *
 	 * @param \App\Request $request
 	 *
-	 * @return \App\QueryGenerator|bool
+	 * @return string[]
 	 */
 	public static function getQuery(\App\Request $request)
 	{

vtlib/Vtiger/Access.php

Doc Comments +1 added lines

@@ -117,6 +117,7 @@
 	 * @param string Tool (action name) like Import, Export
 	 * @param bool true to enable tool, false to disable
 	 * @param int (optional) profile id to use, false applies to all profile
+	 * @param boolean $flag
 	 */
 	public static function updateTool(ModuleBasic $moduleInstance, $toolAction, $flag, $profileid = false)
 	{

vtlib/Vtiger/ModuleBasic.php

Doc Comments +2 added lines, -1 removed lines

@@ -385,6 +385,7 @@ 
 	 * Disable tools for this module.
 	 *
 	 * @param mixed String or Array with value ['Import', 'Export']
+	 * @param string $tools
 	 */
 	public function disableTools($tools)
 	{
@@ -427,7 +428,7 @@ 
 	/**
 	 * Function to get the Module/Tab id.
 	 *
-	 * @return int
+	 * @return boolean
 	 */
 	public function getId()
 	{

app/AddressFinder.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()
 	{