@@ -21,10 +21,6 @@ |
||
21 | 21 | |
22 | 22 | /** |
23 | 23 | * @param array $config Configuration options |
24 | - * @param string $config['type'] 'date', 'time', or 'datetime' |
|
25 | - * @param string $config['min'] Minimum date, time, or datetime |
|
26 | - * @param string $config['max'] Maximum date, time, or datetime |
|
27 | - * @param bool $config['clearable'] Whether to provide for blanking the value. |
|
28 | 24 | */ |
29 | 25 | public function __construct( array $config = [] ) { |
30 | 26 | // We need $this->type set before calling the parent constructor |
@@ -196,7 +196,7 @@ discard block |
||
196 | 196 | * Get the default action for this special page, if none is given via URL/POST data. |
197 | 197 | * Subclasses should override this (or override loadAuth() so this is never called). |
198 | 198 | * @param string $subPage Subpage of the special page. |
199 | - * @return string an AuthManager::ACTION_* constant. |
|
199 | + * @return string|null an AuthManager::ACTION_* constant. |
|
200 | 200 | */ |
201 | 201 | protected function getDefaultAction( $subPage ) { |
202 | 202 | throw new BadMethodCallException( 'Subclass did not implement getDefaultAction' ); |
@@ -732,7 +732,7 @@ discard block |
||
732 | 732 | * Get an array value, or a default if it does not exist. |
733 | 733 | * @param array $array |
734 | 734 | * @param string $fieldName |
735 | - * @param mixed $default |
|
735 | + * @param integer $default |
|
736 | 736 | * @return mixed |
737 | 737 | */ |
738 | 738 | protected static function getField( array $array, $fieldName, $default = null ) { |
@@ -85,6 +85,9 @@ discard block |
||
85 | 85 | unset( $this->db ); |
86 | 86 | } |
87 | 87 | |
88 | + /** |
|
89 | + * @param integer $row |
|
90 | + */ |
|
88 | 91 | public function seek( $row ) { |
89 | 92 | $this->cursor = min( $row, $this->nrows ); |
90 | 93 | } |
@@ -440,6 +443,9 @@ discard block |
||
440 | 443 | } |
441 | 444 | } |
442 | 445 | |
446 | + /** |
|
447 | + * @return string |
|
448 | + */ |
|
443 | 449 | function lastError() { |
444 | 450 | if ( $this->mConn === false ) { |
445 | 451 | $e = oci_error(); |
@@ -480,6 +486,9 @@ discard block |
||
480 | 486 | return false; |
481 | 487 | } |
482 | 488 | |
489 | + /** |
|
490 | + * @param string $table |
|
491 | + */ |
|
483 | 492 | function insert( $table, $a, $fname = __METHOD__, $options = [] ) { |
484 | 493 | if ( !count( $a ) ) { |
485 | 494 | return true; |
@@ -509,6 +518,9 @@ discard block |
||
509 | 518 | return $retVal; |
510 | 519 | } |
511 | 520 | |
521 | + /** |
|
522 | + * @param string $table |
|
523 | + */ |
|
512 | 524 | private function fieldBindStatement( $table, $col, &$val, $includeCol = false ) { |
513 | 525 | $col_info = $this->fieldInfoMulti( $table, $col ); |
514 | 526 | $col_type = $col_info != false ? $col_info->type() : 'CONSTANT'; |
@@ -751,6 +763,9 @@ discard block |
||
751 | 763 | return strtoupper( parent::tableName( $name, $format ) ); |
752 | 764 | } |
753 | 765 | |
766 | + /** |
|
767 | + * @param string $name |
|
768 | + */ |
|
754 | 769 | function tableNameInternal( $name ) { |
755 | 770 | $name = $this->tableName( $name ); |
756 | 771 | |
@@ -1173,6 +1188,9 @@ discard block |
||
1173 | 1188 | return true; |
1174 | 1189 | } |
1175 | 1190 | |
1191 | + /** |
|
1192 | + * @param string $db |
|
1193 | + */ |
|
1176 | 1194 | function selectDB( $db ) { |
1177 | 1195 | $this->mDBname = $db; |
1178 | 1196 | if ( $db == null || $db == $this->mUser ) { |
@@ -1208,6 +1226,9 @@ discard block |
||
1208 | 1226 | return "'" . $this->strencode( $s ) . "'"; |
1209 | 1227 | } |
1210 | 1228 | |
1229 | + /** |
|
1230 | + * @param string $s |
|
1231 | + */ |
|
1211 | 1232 | public function addIdentifierQuotes( $s ) { |
1212 | 1233 | if ( !$this->getFlag( DBO_DDLMODE ) ) { |
1213 | 1234 | $s = '/*Q*/' . $s; |
@@ -1311,6 +1332,10 @@ discard block |
||
1311 | 1332 | return [ $startOpts, $useIndex, $preLimitTail, $postLimitTail, $ignoreIndex ]; |
1312 | 1333 | } |
1313 | 1334 | |
1335 | + /** |
|
1336 | + * @param string $table |
|
1337 | + * @param string $conds |
|
1338 | + */ |
|
1314 | 1339 | public function delete( $table, $conds, $fname = __METHOD__ ) { |
1315 | 1340 | if ( is_array( $conds ) ) { |
1316 | 1341 | $conds = $this->wrapConditionsForWhere( $table, $conds ); |
@@ -1472,6 +1497,9 @@ discard block |
||
1472 | 1497 | return 'BITOR(' . $fieldLeft . ', ' . $fieldRight . ')'; |
1473 | 1498 | } |
1474 | 1499 | |
1500 | + /** |
|
1501 | + * @return string |
|
1502 | + */ |
|
1475 | 1503 | function getDBname() { |
1476 | 1504 | return $this->mDBname; |
1477 | 1505 | } |
@@ -182,7 +182,7 @@ |
||
182 | 182 | /** |
183 | 183 | * Get the Database object in use |
184 | 184 | * |
185 | - * @return IDatabase |
|
185 | + * @return Database|null |
|
186 | 186 | */ |
187 | 187 | public function getDatabase() { |
188 | 188 | return $this->mDb; |
@@ -186,6 +186,11 @@ |
||
186 | 186 | $this->handleResultFromHook( $srchres, $namespaces, $search, $limit, $offset ) ); |
187 | 187 | } |
188 | 188 | |
189 | + /** |
|
190 | + * @param string $search |
|
191 | + * @param integer $limit |
|
192 | + * @param integer $offset |
|
193 | + */ |
|
189 | 194 | private function handleResultFromHook( $srchres, $namespaces, $search, $limit, $offset ) { |
190 | 195 | if ( $offset === 0 ) { |
191 | 196 | // Only perform exact db match if offset === 0 |
@@ -136,14 +136,14 @@ discard block |
||
136 | 136 | * NOTE: This ONLY works for getTitleInfo() and isKnownEmpty(), NOT FOR ANYTHING ELSE. |
137 | 137 | * In particular, it doesn't work for getContent() or getScript() etc. |
138 | 138 | * |
139 | - * @return IDatabase|null |
|
139 | + * @return Database|null |
|
140 | 140 | */ |
141 | 141 | protected function getDB() { |
142 | 142 | return wfGetDB( DB_REPLICA ); |
143 | 143 | } |
144 | 144 | |
145 | 145 | /** |
146 | - * @param string $title |
|
146 | + * @param string $titleText |
|
147 | 147 | * @return null|string |
148 | 148 | */ |
149 | 149 | protected function getContent( $titleText ) { |
@@ -336,7 +336,7 @@ discard block |
||
336 | 336 | * @since 1.28 |
337 | 337 | * @param ResourceLoaderContext $context |
338 | 338 | * @param IDatabase $db |
339 | - * @param string[] $modules |
|
339 | + * @param string[] $moduleNames |
|
340 | 340 | */ |
341 | 341 | public static function preloadTitleInfo( |
342 | 342 | ResourceLoaderContext $context, IDatabase $db, array $moduleNames |
@@ -371,7 +371,7 @@ |
||
371 | 371 | * Do all updates and commit them. More or less a replacement |
372 | 372 | * for the original initStats, but without output. |
373 | 373 | * |
374 | - * @param IDatabase|bool $database |
|
374 | + * @param Database|null $database |
|
375 | 375 | * - boolean: Whether to use the master DB |
376 | 376 | * - IDatabase: Database connection to use |
377 | 377 | * @param array $options Array of options, may contain the following values |
@@ -107,7 +107,7 @@ discard block |
||
107 | 107 | /** |
108 | 108 | * Create a FormOptions object with options as specified by the user |
109 | 109 | * |
110 | - * @param array $parameters |
|
110 | + * @param string $parameters |
|
111 | 111 | * |
112 | 112 | * @return FormOptions |
113 | 113 | */ |
@@ -337,7 +337,7 @@ discard block |
||
337 | 337 | /** |
338 | 338 | * Return a IDatabase object for reading |
339 | 339 | * |
340 | - * @return IDatabase |
|
340 | + * @return Database|null |
|
341 | 341 | */ |
342 | 342 | protected function getDB() { |
343 | 343 | return wfGetDB( DB_REPLICA ); |
@@ -161,7 +161,7 @@ discard block |
||
161 | 161 | * Subclasses return an array of fields to order by here. Don't append |
162 | 162 | * DESC to the field names, that'll be done automatically if |
163 | 163 | * sortDescending() returns true. |
164 | - * @return array |
|
164 | + * @return string[] |
|
165 | 165 | * @since 1.18 |
166 | 166 | */ |
167 | 167 | function getOrderFields() { |
@@ -373,7 +373,7 @@ discard block |
||
373 | 373 | |
374 | 374 | /** |
375 | 375 | * Get a DB connection to be used for slow recache queries |
376 | - * @return IDatabase |
|
376 | + * @return Database|null |
|
377 | 377 | */ |
378 | 378 | function getRecacheDB() { |
379 | 379 | return wfGetDB( DB_REPLICA, [ $this->getName(), 'QueryPage::recache', 'vslow' ] ); |
@@ -434,7 +434,7 @@ discard block |
||
434 | 434 | /** |
435 | 435 | * Somewhat deprecated, you probably want to be using execute() |
436 | 436 | * @param int|bool $offset |
437 | - * @param int|bool $limit |
|
437 | + * @param integer $limit |
|
438 | 438 | * @return ResultWrapper |
439 | 439 | */ |
440 | 440 | public function doQuery( $offset = false, $limit = false ) { |
@@ -775,7 +775,7 @@ discard block |
||
775 | 775 | /** |
776 | 776 | * Override for custom handling. If the titles/links are ok, just do |
777 | 777 | * feedItemDesc() |
778 | - * @param object $row |
|
778 | + * @param null|stdClass $row |
|
779 | 779 | * @return FeedItem|null |
780 | 780 | */ |
781 | 781 | function feedResult( $row ) { |
@@ -807,6 +807,9 @@ discard block |
||
807 | 807 | return isset( $row->comment ) ? htmlspecialchars( $row->comment ) : ''; |
808 | 808 | } |
809 | 809 | |
810 | + /** |
|
811 | + * @return string |
|
812 | + */ |
|
810 | 813 | function feedItemAuthor( $row ) { |
811 | 814 | return isset( $row->user_text ) ? $row->user_text : ''; |
812 | 815 | } |