And some more parameter documentation!!

diff --git a/includes/Article.php b/includes/Article.php
index 915ee7a..a5110f4 100644 (file)
--- a/includes/Article.php
+++ b/includes/Article.php
@@ -864,7 +864,7 @@ class Article {
        /**
         * Get a list of users who have edited this article, not including the user who made
         * the most recent revision, which you can get from $article->getUser() if you want it
-        * @return UserArray
+        * @return UserArrayFromResult
         */
        public function getContributors() {
                # @todo FIXME: This is expensive; cache this info somewhere.

diff --git a/includes/Feed.php b/includes/Feed.php
index e22ed93..528b5f5 100644 (file)
--- a/includes/Feed.php
+++ b/includes/Feed.php
@@ -36,9 +36,8 @@
  * @ingroup Feed
  */
 class FeedItem {
-       /**#@+
-        * @var string
-        * @private
+       /**
+        * @var Title
         */
        var $Title = 'Wiki';
        var $Description = '';
@@ -47,12 +46,11 @@ class FeedItem {
        var $Author = '';
        var $UniqueId = '';
        var $RSSIsPermalink;
-       /**#@-*/
 
        /**
         * Constructor
         *
-        * @param $Title String: Item's title
+        * @param $Title String|Title Item's title
         * @param $Description String
         * @param $Url String: URL uniquely designating the item.
         * @param $Date String: Item's date

diff --git a/includes/Metadata.php b/includes/Metadata.php
index 463b497..62d1043 100644 (file)
--- a/includes/Metadata.php
+++ b/includes/Metadata.php
@@ -101,6 +101,10 @@ abstract class RdfMetaData {
                }
        }
 
+       /**
+        * @param $name string
+        * @param $title Title
+        */
        protected function page( $name, $title ) {
                $this->url( $name, $title->getFullUrl() );
        }

diff --git a/includes/Pager.php b/includes/Pager.php
index fb657f8..c10dd95 100644 (file)
--- a/includes/Pager.php
+++ b/includes/Pager.php
@@ -93,6 +93,8 @@ abstract class IndexPager implements Pager {
 
        /**
         * Result object for the query. Warning: seek before use.
+        *
+        * @var ResultWrapper
         */
        public $mResult;
 

diff --git a/includes/actions/CreditsAction.php b/includes/actions/CreditsAction.php
index 26bb70b..9ce7963 100644 (file)
--- a/includes/actions/CreditsAction.php
+++ b/includes/actions/CreditsAction.php
@@ -119,7 +119,7 @@ class CreditsAction extends FormlessAction {
 
                # Sift for real versus user names
                foreach ( $contributors as $user ) {
-                       $cnt--;
+                       $cnt--; 
                        if ( $user->isLoggedIn() ) {
                                $link = self::link( $user );
                                if ( !in_array( 'realname', $wgHiddenPrefs ) && $user->getRealName() ) {

diff --git a/includes/cache/HTMLFileCache.php b/includes/cache/HTMLFileCache.php
index 31d92ff..3b3ce96 100644 (file)
--- a/includes/cache/HTMLFileCache.php
+++ b/includes/cache/HTMLFileCache.php
@@ -119,6 +119,8 @@ class HTMLFileCache {
        /**
         * Check if up to date cache file exists
         * @param $timestamp string
+        *
+        * @return bool
         */
        public function isFileCacheGood( $timestamp = '' ) {
                global $wgCacheEpoch;

diff --git a/includes/media/JpegOrTiff.php b/includes/media/JpegOrTiff.php
index 767cb19..d483500 100644 (file)
--- a/includes/media/JpegOrTiff.php
+++ b/includes/media/JpegOrTiff.php
@@ -95,6 +95,10 @@ class JpegOrTiffHandler extends BitmapHandler {
                return self::METADATA_GOOD;
        }
 
+       /**
+        * @param $image File
+        * @return array|bool
+        */
        function formatMetadata( $image ) {
                $metadata = $image->getMetadata();
                if ( !$metadata ||
@@ -116,8 +120,8 @@ class JpegOrTiffHandler extends BitmapHandler {
                return $this->formatMetadataHelper( $exif );
        }
 
-        function getMetadataType( $image ) {
-                return 'exif';
-        }
+       function getMetadataType( $image ) {
+               return 'exif';
+       }
 }
 

diff --git a/includes/specials/SpecialFilepath.php b/includes/specials/SpecialFilepath.php
index 744940f..bcf6965 100644 (file)
--- a/includes/specials/SpecialFilepath.php
+++ b/includes/specials/SpecialFilepath.php
@@ -70,6 +70,9 @@ class SpecialFilepath extends SpecialPage {
                }
        }
 
+       /**
+        * @param $title Title
+        */
        function showForm( $title ) {
                global $wgOut, $wgScript;
 

diff --git a/includes/specials/SpecialMIMEsearch.php b/includes/specials/SpecialMIMEsearch.php
index a64b287..aefe7bf 100644 (file)
--- a/includes/specials/SpecialMIMEsearch.php
+++ b/includes/specials/SpecialMIMEsearch.php
@@ -107,6 +107,10 @@ class MIMEsearchPage extends QueryPage {
                return "($download) $plink . . $dimensions . . $bytes . . $user . . $time";
        }
 
+       /**
+        * @param $type string
+        * @return bool
+        */
        protected static function isValidType( $type ) {
                // From maintenance/tables.sql => img_major_mime
                $types = array(

diff --git a/includes/specials/SpecialMergeHistory.php b/includes/specials/SpecialMergeHistory.php
index 99d3394..916230c 100644 (file)
--- a/includes/specials/SpecialMergeHistory.php
+++ b/includes/specials/SpecialMergeHistory.php
@@ -29,6 +29,10 @@
  */
 class SpecialMergeHistory extends SpecialPage {
        var $mAction, $mTarget, $mDest, $mTimestamp, $mTargetID, $mDestID, $mComment;
+
+       /**
+        * @var Title
+        */
        var $mTargetObj, $mDestObj;
 
        public function __construct() {

diff --git a/includes/specials/SpecialTags.php b/includes/specials/SpecialTags.php
index d3f4e7d..add54c1 100644 (file)
--- a/includes/specials/SpecialTags.php
+++ b/includes/specials/SpecialTags.php
@@ -63,8 +63,8 @@ class SpecialTags extends SpecialPage {
        }
 
        function doTagRow( $tag, $hitcount ) {
-               static $sk=null, $doneTags=array();
-               if (!$sk) {
+               static $sk = null, $doneTags = array();
+               if ( !$sk ) {
                        global $wgUser;
                        $sk = $wgUser->getSkin();
                }