Passed
Push — developer ( c9d396...8e0a17 )
by Błażej
90:59 queued 76:12
created
include/database/PearDatabase.php 2 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -740,7 +740,7 @@
 block discarded – undo
740 740
 	public function generateQuestionMarks($items)
741 741
 	{
742 742
 		// array_map will call the function specified in the first parameter for every element of the list in second parameter
743
-		return implode(',', array_map(function ($a) {
743
+		return implode(',', array_map(function($a) {
744 744
 			return '?';
745 745
 		}, is_array($items) ? $items : explode(',', $items)));
746 746
 	}
Please login to merge, or discard this patch.
Doc Comments   +27 added lines, -1 removed lines patch added patch discarded remove patch
@@ -112,6 +112,14 @@  discard block
 block discarded – undo
112 112
 		}
113 113
 	}
114 114
 
115
+	/**
116
+	 * @param string $dbtype
117
+	 * @param string $host
118
+	 * @param string $dbname
119
+	 * @param string $username
120
+	 * @param string $passwd
121
+	 * @param integer $port
122
+	 */
115 123
 	protected function loadDBConfig($dbtype, $host, $dbname, $username, $passwd, $port)
116 124
 	{
117 125
 		if ($host == '_SERVER_') {
@@ -140,6 +148,9 @@  discard block
 block discarded – undo
140 148
 		return $msg;
141 149
 	}
142 150
 
151
+	/**
152
+	 * @param string $message
153
+	 */
143 154
 	public function checkError($message, $dieOnError = false, $query = false, $params = false)
144 155
 	{
145 156
 		if ($this->hasActiveTransaction) {
@@ -418,7 +429,7 @@  discard block
 block discarded – undo
418 429
 	 *
419 430
 	 * @param string $table  Table name
420 431
 	 * @param string $where  Conditions
421
-	 * @param array  $params Query data
432
+	 * @param string[]  $params Query data
422 433
 	 *
423 434
 	 * @return int Number of deleted records
424 435
 	 */
@@ -467,6 +478,9 @@  discard block
 block discarded – undo
467 478
 		return $this->stmt->rowCount();
468 479
 	}
469 480
 
481
+	/**
482
+	 * @param integer $row
483
+	 */
470 484
 	public function queryResult(&$result, $row, $col = 0)
471 485
 	{
472 486
 		return $this->queryResultRaw($result, $row, $col);
@@ -621,6 +635,9 @@  discard block
 block discarded – undo
621 635
 		}
622 636
 	}
623 637
 
638
+	/**
639
+	 * @param string $seqname
640
+	 */
624 641
 	public function getUniqueID($seqname)
625 642
 	{
626 643
 		$tableName = $seqname . '_seq';
@@ -637,6 +654,9 @@  discard block
 block discarded – undo
637 654
 		return $id;
638 655
 	}
639 656
 
657
+	/**
658
+	 * @param string $tableName
659
+	 */
640 660
 	public function checkExistTable($tableName, $cache = true)
641 661
 	{
642 662
 		$tablePresent = Vtiger_Cache::get('checkExistTable', $tableName);
@@ -665,6 +685,9 @@  discard block
 block discarded – undo
665 685
 		return $this->database->lastInsertId();
666 686
 	}
667 687
 
688
+	/**
689
+	 * @param string $datetime
690
+	 */
668 691
 	public function formatDate($datetime, $strip_quotes = false)
669 692
 	{
670 693
 		// remove single quotes to use the date as parameter for Prepared statement
@@ -809,6 +832,9 @@  discard block
 block discarded – undo
809 832
 	protected $logSqlTimeID = false;
810 833
 	protected $logSqlTimeGroup = 1;
811 834
 
835
+	/**
836
+	 * @param double $endat
837
+	 */
812 838
 	public function logSqlTime($startat, $endat, $sql, $params = false)
813 839
 	{
814 840
 		if (!AppConfig::performance('SQL_LOG_INCLUDE_CALLER')) {
Please login to merge, or discard this patch.
api/webservice/Portal/BaseModule/Record.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@
 block discarded – undo
61 61
 	 *
62 62
 	 * @throws \Api\Core\Exception
63 63
 	 *
64
-	 * @return bool
64
+	 * @return boolean|null
65 65
 	 */
66 66
 	public function checkPermission()
67 67
 	{
Please login to merge, or discard this patch.
install/models/Utils.php 1 patch
Doc Comments   +1 added lines, -10 removed lines patch added patch discarded remove patch
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
 	 *
67 67
 	 * @param type $dbType
68 68
 	 *
69
-	 * @return type
69
+	 * @return boolean
70 70
 	 */
71 71
 	public static function isMySQL($dbType)
72 72
 	{
@@ -76,15 +76,6 @@  discard block
 block discarded – undo
76 76
 	/**
77 77
 	 * Function checks the database connection.
78 78
 	 *
79
-	 * @param string $db_type
80
-	 * @param string $db_hostname
81
-	 * @param string $db_username
82
-	 * @param string $db_password
83
-	 * @param string $db_name
84
-	 * @param string $create_db
85
-	 * @param string $create_utf8_db
86
-	 * @param string $root_user
87
-	 * @param string $root_password
88 79
 	 *
89 80
 	 * @return <Array>
90 81
 	 */
Please login to merge, or discard this patch.
install/views/Index.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -288,7 +288,7 @@
 block discarded – undo
288 288
 	 *
289 289
 	 * @param \App\Request $request - request model
290 290
 	 *
291
-	 * @return Vtiger_CssScript_Model[]
291
+	 * @return App\Controller\Vtiger_CssScript_Model[]
292 292
 	 */
293 293
 	public function getHeaderCss(\App\Request $request)
294 294
 	{
Please login to merge, or discard this patch.
modules/Accounts/models/ListView.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -16,7 +16,7 @@
 block discarded – undo
16 16
 	 *
17 17
 	 * @param array $linkParams
18 18
 	 *
19
-	 * @return array - Associative array of Link type to List of  Vtiger_Link_Model instances for Mass Actions
19
+	 * @return Vtiger_Link_Model[] - Associative array of Link type to List of  Vtiger_Link_Model instances for Mass Actions
20 20
 	 */
21 21
 	public function getListViewMassActions($linkParams)
22 22
 	{
Please login to merge, or discard this patch.
modules/API/models/CalDAV.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -893,7 +893,7 @@
 block discarded – undo
893 893
 	 * Get vtime zone.
894 894
 	 *
895 895
 	 * @param Sabre\VObject\Component $vcalendar
896
-	 * @param type                    $tzid
896
+	 * @param string                    $tzid
897 897
 	 * @param int                     $from
898 898
 	 * @param int                     $to
899 899
 	 *
Please login to merge, or discard this patch.
modules/API/models/CardDAV.php 1 patch
Doc Comments   +6 added lines, -1 removed lines patch added patch discarded remove patch
@@ -41,6 +41,9 @@  discard block
 block discarded – undo
41 41
 		\App\Log::trace(__METHOD__ . ' | End');
42 42
 	}
43 43
 
44
+	/**
45
+	 * @param string $moduleName
46
+	 */
44 47
 	public function syncCrmRecord($moduleName)
45 48
 	{
46 49
 		$create = $deletes = $updates = 0;
@@ -238,6 +241,9 @@  discard block
 block discarded – undo
238 241
 		\App\Log::trace(__METHOD__ . ' | End');
239 242
 	}
240 243
 
244
+	/**
245
+	 * @param string $moduleName
246
+	 */
241 247
 	public function createRecord($moduleName, $card)
242 248
 	{
243 249
 		\App\Log::trace(__METHOD__ . ' | Start Card ID' . $card['id']);
@@ -441,7 +447,6 @@  discard block
 block discarded – undo
441 447
 	/**
442 448
 	 * Adds a change record to the addressbookchanges table.
443 449
 	 *
444
-	 * @param mixed  $addressBookId
445 450
 	 * @param string $objectUri
446 451
 	 * @param int    $operation     1 = add, 2 = modify, 3 = delete
447 452
 	 */
Please login to merge, or discard this patch.
modules/Calendar/iCalLastImport.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -90,7 +90,7 @@
 block discarded – undo
90 90
 	 * @param string $moduleName
91 91
 	 * @param int    $userId
92 92
 	 *
93
-	 * @return int|bool
93
+	 * @return integer|null
94 94
 	 */
95 95
 	public function undo($moduleName, $userId)
96 96
 	{
Please login to merge, or discard this patch.
modules/Calendar/models/CalendarFilters.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@
 block discarded – undo
22 22
 	/**
23 23
 	 * Constructor.
24 24
 	 *
25
-	 * @return bool
25
+	 * @return false|null
26 26
 	 */
27 27
 	public function __construct()
28 28
 	{
Please login to merge, or discard this patch.