| Index: trunk/extensions/FeaturedFeeds/FeaturedFeeds.body.php |
| — | — | @@ -133,6 +133,7 @@ |
| 134 | 134 | * Purges cache on message edit |
| 135 | 135 | * |
| 136 | 136 | * @param Article $article |
| | 137 | + * @return bool |
| 137 | 138 | */ |
| 138 | 139 | public static function articleSaveComplete( $article ) { |
| 139 | 140 | global $wgFeaturedFeeds, $wgMemc, $wgLanguageCode; |
| — | — | @@ -172,7 +173,6 @@ |
| 173 | 174 | |
| 174 | 175 | $feeds = array(); |
| 175 | 176 | $requestedLang = Language::factory( $langCode ); |
| 176 | | - $parser = new Parser(); |
| 177 | 177 | foreach ( $feedDefs as $name => $opts ) { |
| 178 | 178 | $feed = new FeaturedFeedChannel( $name, $opts, $requestedLang ); |
| 179 | 179 | if ( !$feed->isOK() ) { |
| — | — | @@ -271,10 +271,17 @@ |
| 272 | 272 | } |
| 273 | 273 | } |
| 274 | 274 | |
| | 275 | + /** |
| | 276 | + * @param $key string |
| | 277 | + * @return Message |
| | 278 | + */ |
| 275 | 279 | private function msg( $key ) { |
| 276 | 280 | return wfMessage( $key )->inLanguage( $this->language ); |
| 277 | 281 | } |
| 278 | 282 | |
| | 283 | + /** |
| | 284 | + * @return bool |
| | 285 | + */ |
| 279 | 286 | public function isOK() { |
| 280 | 287 | $this->init(); |
| 281 | 288 | return $this->page !== false; |
| — | — | @@ -312,6 +319,9 @@ |
| 313 | 320 | $this->entryName = $this->msg( $this->options['entryName'] )->plain(); |
| 314 | 321 | } |
| 315 | 322 | |
| | 323 | + /** |
| | 324 | + * @return array |
| | 325 | + */ |
| 316 | 326 | public function getFeedItems() { |
| 317 | 327 | $this->init(); |
| 318 | 328 | if ( $this->items === false ) { |
| — | — | @@ -365,7 +375,7 @@ |
| 366 | 376 | /** |
| 367 | 377 | * Returns a URL to the feed |
| 368 | 378 | * |
| 369 | | - * @param type $format: Feed format, 'rss' or 'atom' |
| | 379 | + * @param $format string Feed format, 'rss' or 'atom' |
| 370 | 380 | * @return String |
| 371 | 381 | */ |
| 372 | 382 | public function getURL( $format ) { |