Tweak/add some documentation as hints for some code analysis
authorSam Reed <reedy@users.mediawiki.org>
Tue, 28 Sep 2010 01:33:11 +0000 (01:33 +0000)
committerSam Reed <reedy@users.mediawiki.org>
Tue, 28 Sep 2010 01:33:11 +0000 (01:33 +0000)
includes/RecentChange.php
includes/api/ApiBase.php
includes/api/ApiMain.php

index 6663d58..970411e 100644 (file)
@@ -115,6 +115,10 @@ class RecentChange {
                $this->mExtra = $extra;
        }
 
+    /**
+     *
+     * @return Title
+     */
        public function &getTitle() {
                if( $this->mTitle === false ) {
                        $this->mTitle = Title::makeTitle( $this->mAttribs['rc_namespace'], $this->mAttribs['rc_title'] );
@@ -579,7 +583,7 @@ class RecentChange {
        /**
         * Get an attribute value
         *
-        * @param $name Attribute name
+        * @param $name String Attribute name
         * @return mixed
         */
        public function getAttribute( $name ) {
index 9ef45ef..78d44bc 100644 (file)
@@ -551,7 +551,7 @@ abstract class ApiBase {
         * Return true if we're to watch the page, false if not, null if no change.
         * @param $watchlist String Valid values: 'watch', 'unwatch', 'preferences', 'nochange'
         * @param $titleObj Title the page under consideration
-        * @param $userOption The user option to consider when $watchlist=preferences.
+        * @param $userOption String The user option to consider when $watchlist=preferences.
         *      If not set will magically default to either watchdefault or watchcreations
         * @returns mixed
         */
index e490ece..4d8d671 100644 (file)
@@ -530,7 +530,7 @@ class ApiMain extends ApiBase {
 
        /**
         * Set up the module for response
-        * @return Object the module that will handle this action
+        * @return ApiBase The module that will handle this action
         */
        protected function setupModule() {
                // Instantiate the module requested by the user