@@ -39,6 +39,9 @@ |
||
39 | 39 | return LogPage::DELETED_ACTION; |
40 | 40 | } |
41 | 41 | |
42 | + /** |
|
43 | + * @param integer $target |
|
44 | + */ |
|
42 | 45 | public static function suggestTarget( $target, array $ids ) { |
43 | 46 | $result = wfGetDB( DB_SLAVE )->select( 'logging', |
44 | 47 | 'log_type', |
@@ -26,6 +26,9 @@ |
||
26 | 26 | /** @var Revision */ |
27 | 27 | public $revision; |
28 | 28 | |
29 | + /** |
|
30 | + * @param RevDelRevisionList $list |
|
31 | + */ |
|
29 | 32 | public function __construct( $list, $row ) { |
30 | 33 | parent::__construct( $list, $row ); |
31 | 34 | $this->revision = new Revision( $row ); |
@@ -57,7 +57,7 @@ discard block |
||
57 | 57 | /** |
58 | 58 | * Get the internal type name of this list. Equal to the table name. |
59 | 59 | * Override this function. |
60 | - * @return null |
|
60 | + * @return string |
|
61 | 61 | */ |
62 | 62 | public function getType() { |
63 | 63 | return null; |
@@ -216,7 +216,7 @@ discard block |
||
216 | 216 | |
217 | 217 | /** |
218 | 218 | * Get the timestamp in MW 14-char form |
219 | - * @return mixed |
|
219 | + * @return string|false |
|
220 | 220 | */ |
221 | 221 | public function getTimestamp() { |
222 | 222 | $field = $this->getTimestampField(); |
@@ -120,6 +120,7 @@ |
||
120 | 120 | * Returns an array where the element of $array at index $key becomes |
121 | 121 | * the first element. |
122 | 122 | * @param int $key key to pull to the front |
123 | + * @param string[] $array |
|
123 | 124 | * @return array $array with the item at $key pulled to the front |
124 | 125 | */ |
125 | 126 | private function pullFront( $key, $array ) { |
@@ -440,7 +440,7 @@ |
||
440 | 440 | * Basic wikitext removal |
441 | 441 | * @protected |
442 | 442 | * @param string $text |
443 | - * @return mixed |
|
443 | + * @return string |
|
444 | 444 | */ |
445 | 445 | function removeWiki( $text ) { |
446 | 446 | $text = preg_replace( "/\\{\\{([^|]+?)\\}\\}/", "", $text ); |
@@ -173,6 +173,10 @@ discard block |
||
173 | 173 | return $this->searchInternal( $term, false ); |
174 | 174 | } |
175 | 175 | |
176 | + /** |
|
177 | + * @param string $term |
|
178 | + * @param boolean $fulltext |
|
179 | + */ |
|
176 | 180 | protected function searchInternal( $term, $fulltext ) { |
177 | 181 | // This seems out of place, why is this called with empty term? |
178 | 182 | if ( trim( $term ) === '' ) { |
@@ -377,7 +381,7 @@ discard block |
||
377 | 381 | * Converts some characters for MySQL's indexing to grok it correctly, |
378 | 382 | * and pads short words to overcome limitations. |
379 | 383 | * @param string $string |
380 | - * @return mixed|string |
|
384 | + * @return string |
|
381 | 385 | */ |
382 | 386 | function normalizeText( $string ) { |
383 | 387 | global $wgContLang; |
@@ -165,6 +165,10 @@ discard block |
||
165 | 165 | return $this->searchInternal( $term, false ); |
166 | 166 | } |
167 | 167 | |
168 | + /** |
|
169 | + * @param string $term |
|
170 | + * @param boolean $fulltext |
|
171 | + */ |
|
168 | 172 | protected function searchInternal( $term, $fulltext ) { |
169 | 173 | global $wgContLang; |
170 | 174 | |
@@ -250,6 +254,9 @@ discard block |
||
250 | 254 | "WHERE page_id=$searchindex.rowid AND $match"; |
251 | 255 | } |
252 | 256 | |
257 | + /** |
|
258 | + * @param string $filteredTerm |
|
259 | + */ |
|
253 | 260 | function getCountQuery( $filteredTerm, $fulltext ) { |
254 | 261 | $match = $this->parseQuery( $filteredTerm, $fulltext ); |
255 | 262 | $page = $this->db->tableName( 'page' ); |
@@ -8,6 +8,9 @@ |
||
8 | 8 | protected $terms; |
9 | 9 | protected $totalHits; |
10 | 10 | |
11 | + /** |
|
12 | + * @param integer $total |
|
13 | + */ |
|
11 | 14 | function __construct( $resultSet, $terms, $total = null ) { |
12 | 15 | $this->resultSet = $resultSet; |
13 | 16 | $this->terms = $terms; |
@@ -191,7 +191,7 @@ |
||
191 | 191 | |
192 | 192 | /** |
193 | 193 | * @param DOMElement $element |
194 | - * @param $name |
|
194 | + * @param string $name |
|
195 | 195 | * @param string|null|bool $default |
196 | 196 | * |
197 | 197 | * @return null|string |