Completed
Pull Request — master (#14664)
by Herbert
25:04 queued 09:47
created
framework/db/Query.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -768,7 +768,7 @@  discard block
 block discarded – undo
768 768
 
769 769
     /**
770 770
      * Sets the GROUP BY part of the query.
771
-     * @param string|array|Expression $columns the columns to be grouped by.
771
+     * @param string $columns the columns to be grouped by.
772 772
      * Columns can be specified in either a string (e.g. "id, name") or an array (e.g. ['id', 'name']).
773 773
      * The method will automatically quote the column names unless a column contains some parenthesis
774 774
      * (which means the column contains a DB expression).
@@ -967,7 +967,7 @@  discard block
 block discarded – undo
967 967
 
968 968
     /**
969 969
      * Appends a SQL statement using UNION operator.
970
-     * @param string|Query $sql the SQL statement to be appended using UNION
970
+     * @param Query $sql the SQL statement to be appended using UNION
971 971
      * @param bool $all TRUE if using UNION ALL and FALSE if using UNION
972 972
      * @return $this the query object itself
973 973
      */
Please login to merge, or discard this patch.
framework/captcha/CaptchaAction.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -144,7 +144,7 @@
 block discarded – undo
144 144
     /**
145 145
      * Generates a hash code that can be used for client-side validation.
146 146
      * @param string $code the CAPTCHA code
147
-     * @return string a hash code generated from the CAPTCHA code
147
+     * @return integer a hash code generated from the CAPTCHA code
148 148
      */
149 149
     public function generateValidationHash($code)
150 150
     {
Please login to merge, or discard this patch.
framework/db/ActiveQuery.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -392,7 +392,7 @@
 block discarded – undo
392 392
      * @param bool|array $eagerLoading whether to eager load the relations specified in `$with`.
393 393
      * When this is a boolean, it applies to all relations specified in `$with`. Use an array
394 394
      * to explicitly list which relations in `$with` need to be eagerly loaded. Defaults to `true`.
395
-     * @param string|array $joinType the join type of the relations specified in `$with`.
395
+     * @param string $joinType the join type of the relations specified in `$with`.
396 396
      * When this is a string, it applies to all relations specified in `$with`. Use an array
397 397
      * in the format of `relationName => joinType` to specify different join types for different relations.
398 398
      * @return $this the query object itself
Please login to merge, or discard this patch.
framework/db/DataReader.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -114,7 +114,7 @@
 block discarded – undo
114 114
     /**
115 115
      * Returns a single column from the next row of a result set.
116 116
      * @param int $columnIndex zero-based column index
117
-     * @return mixed the column of the current row, false if no more rows available
117
+     * @return string the column of the current row, false if no more rows available
118 118
      */
119 119
     public function readColumn($columnIndex)
120 120
     {
Please login to merge, or discard this patch.
framework/web/Response.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -789,7 +789,7 @@
 block discarded – undo
789 789
      * });
790 790
      * ```
791 791
      *
792
-     * @param string|array $url the URL to be redirected to. This can be in one of the following formats:
792
+     * @param string $url the URL to be redirected to. This can be in one of the following formats:
793 793
      *
794 794
      * - a string representing a URL (e.g. "http://example.com")
795 795
      * - a string representing a URL alias (e.g. "@example.com")
Please login to merge, or discard this patch.
framework/web/User.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -175,7 +175,7 @@  discard block
 block discarded – undo
175 175
      * stored in session and reconstruct the corresponding identity object, if it has not done so before.
176 176
      * @param bool $autoRenew whether to automatically renew authentication status if it has not been done so before.
177 177
      * This is only useful when [[enableSession]] is true.
178
-     * @return IdentityInterface|null the identity object associated with the currently logged-in user.
178
+     * @return null|boolean the identity object associated with the currently logged-in user.
179 179
      * `null` is returned if the user is not logged in (not authenticated).
180 180
      * @see login()
181 181
      * @see logout()
@@ -259,7 +259,7 @@  discard block
 block discarded – undo
259 259
      * with the provided access token. If successful, it will call [[login()]] to log in the authenticated user.
260 260
      * If authentication fails or [[login()]] is unsuccessful, it will return null.
261 261
      * @param string $token the access token
262
-     * @param mixed $type the type of the token. The value of this parameter depends on the implementation.
262
+     * @param string $type the type of the token. The value of this parameter depends on the implementation.
263 263
      * For example, [[\yii\filters\auth\HttpBearerAuth]] will set this parameter to be `yii\filters\auth\HttpBearerAuth`.
264 264
      * @return IdentityInterface|null the identity associated with the given access token. Null is returned if
265 265
      * the access token is invalid or [[login()]] is unsuccessful.
Please login to merge, or discard this patch.
build/controllers/PhpDocController.php 1 patch
Doc Comments   +24 added lines, -2 removed lines patch added patch discarded remove patch
@@ -113,6 +113,9 @@  discard block
 block discarded – undo
113 113
         return array_merge(parent::options($actionID), ['updateFiles', 'skipFrameworkRequirements']);
114 114
     }
115 115
 
116
+    /**
117
+     * @param string|null $root
118
+     */
116 119
     protected function findFiles($root, $needsInclude = true)
117 120
     {
118 121
         $except = [];
@@ -233,6 +236,9 @@  discard block
 block discarded – undo
233 236
         return FileHelper::findFiles($root, $options);
234 237
     }
235 238
 
239
+    /**
240
+     * @param string $extensionPath
241
+     */
236 242
     private function setUpExtensionAliases($extensionPath)
237 243
     {
238 244
         foreach (scandir($extensionPath) as $extension) {
@@ -340,6 +346,9 @@  discard block
 block discarded – undo
340 346
         }
341 347
     }
342 348
 
349
+    /**
350
+     * @param string $line
351
+     */
343 352
     protected function fixParamTypes($line)
344 353
     {
345 354
         return preg_replace_callback('~@(param|return) ([\w\\|]+)~i', function ($matches) {
@@ -473,6 +482,10 @@  discard block
 block discarded – undo
473 482
         // TODO
474 483
     }
475 484
 
485
+    /**
486
+     * @param null|string $className
487
+     * @param string $propertyDoc
488
+     */
476 489
     protected function updateClassPropertyDocs($file, $className, $propertyDoc)
477 490
     {
478 491
         try {
@@ -553,7 +566,7 @@  discard block
 block discarded – undo
553 566
     /**
554 567
      * remove multi empty lines and trim trailing whitespace
555 568
      *
556
-     * @param $doc
569
+     * @param string $doc
557 570
      * @return string
558 571
      */
559 572
     protected function cleanDocComment($doc)
@@ -572,7 +585,7 @@  discard block
 block discarded – undo
572 585
 
573 586
     /**
574 587
      * Replace property annotations in doc comment
575
-     * @param $doc
588
+     * @param string $doc
576 589
      * @param $properties
577 590
      * @return string
578 591
      */
@@ -739,6 +752,9 @@  discard block
 block discarded – undo
739 752
         return [$className, $phpdoc];
740 753
     }
741 754
 
755
+    /**
756
+     * @param string $pattern
757
+     */
742 758
     protected function match($pattern, $subject, $split = false)
743 759
     {
744 760
         $sets = [];
@@ -762,6 +778,9 @@  discard block
 block discarded – undo
762 778
         return $sets;
763 779
     }
764 780
 
781
+    /**
782
+     * @param string $str
783
+     */
765 784
     protected function fixSentence($str)
766 785
     {
767 786
         // TODO fix word wrap
@@ -771,6 +790,9 @@  discard block
 block discarded – undo
771 790
         return strtoupper(substr($str, 0, 1)) . substr($str, 1) . ($str[strlen($str) - 1] != '.' ? '.' : '');
772 791
     }
773 792
 
793
+    /**
794
+     * @param string $param
795
+     */
774 796
     protected function getPropParam($prop, $param)
775 797
     {
776 798
         return isset($prop['property']) ? $prop['property'][$param] : (isset($prop['get']) ? $prop['get'][$param] : $prop['set'][$param]);
Please login to merge, or discard this patch.
build/controllers/ReleaseController.php 1 patch
Doc Comments   +20 added lines, -1 removed lines patch added patch discarded remove patch
@@ -366,7 +366,7 @@  discard block
 block discarded – undo
366 366
 
367 367
     /**
368 368
      * @param array $what list of items
369
-     * @param array $limit list of things to allow, or empty to allow any, can be `app`, `framework`, `extension`
369
+     * @param string[] $limit list of things to allow, or empty to allow any, can be `app`, `framework`, `extension`
370 370
      * @throws \yii\base\Exception
371 371
      */
372 372
     protected function validateWhat(array $what, $limit = [], $ensureGitClean = true)
@@ -545,6 +545,10 @@  discard block
 block discarded – undo
545 545
         $this->stdout("\n");
546 546
     }
547 547
 
548
+    /**
549
+     * @param string $name
550
+     * @param string $path
551
+     */
548 552
     protected function releaseApplication($name, $path, $version)
549 553
     {
550 554
         $this->stdout("\n");
@@ -647,6 +651,9 @@  discard block
 block discarded – undo
647 651
         Yii::setAlias('@app', $this->_oldAlias);
648 652
     }
649 653
 
654
+    /**
655
+     * @param string $name
656
+     */
650 657
     protected function packageApplication($name, $version, $packagePath)
651 658
     {
652 659
         FileHelper::createDirectory($packagePath);
@@ -808,6 +815,9 @@  discard block
 block discarded – undo
808 815
     }
809 816
 
810 817
 
818
+    /**
819
+     * @param string[] $what
820
+     */
811 821
     protected function closeChangelogs($what, $version)
812 822
     {
813 823
         $v = str_replace('\\-', '[\\- ]', preg_quote($version, '/'));
@@ -819,6 +829,9 @@  discard block
 block discarded – undo
819 829
         );
820 830
     }
821 831
 
832
+    /**
833
+     * @param string[] $what
834
+     */
822 835
     protected function openChangelogs($what, $version)
823 836
     {
824 837
         $headline = "\n$version under development\n";
@@ -975,6 +988,9 @@  discard block
 block discarded – undo
975 988
             $frameworkPath . '/BaseYii.php');
976 989
     }
977 990
 
991
+    /**
992
+     * @param string $pattern
993
+     */
978 994
     protected function sed($pattern, $replace, $files)
979 995
     {
980 996
         foreach ((array) $files as $file) {
@@ -1007,6 +1023,9 @@  discard block
 block discarded – undo
1007 1023
     const MINOR = 'minor';
1008 1024
     const PATCH = 'patch';
1009 1025
 
1026
+    /**
1027
+     * @param string $type
1028
+     */
1010 1029
     protected function getNextVersions(array $versions, $type)
1011 1030
     {
1012 1031
         foreach ($versions as $k => $v) {
Please login to merge, or discard this patch.
framework/helpers/BaseHtml.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -216,7 +216,7 @@
 block discarded – undo
216 216
 
217 217
     /**
218 218
      * Generates a link tag that refers to an external CSS file.
219
-     * @param array|string $url the URL of the external CSS file. This parameter will be processed by [[Url::to()]].
219
+     * @param string|boolean $url the URL of the external CSS file. This parameter will be processed by [[Url::to()]].
220 220
      * @param array $options the tag options in terms of name-value pairs. The following options are specially handled:
221 221
      *
222 222
      * - condition: specifies the conditional comments for IE, e.g., `lt IE 9`. When this is specified,
Please login to merge, or discard this patch.