Some small doc tweaks to reduce Doxygen warnings, namely:
authorNick Jenkins <nickj@users.mediawiki.org>
Wed, 18 Apr 2007 09:50:10 +0000 (09:50 +0000)
committerNick Jenkins <nickj@users.mediawiki.org>
Wed, 18 Apr 2007 09:50:10 +0000 (09:50 +0000)
* @link. You might think @link would surely mean "here comes a web URL" ... but @link is a valid command
  in Doxygen, which means an entirely different kind of link (an internal link to somewhere, so that you can separate
  documentation and implementation). The result is a mess, and the best solution I can see is to use "@see" instead of "@link".
* Warning: argument `nourl' of command @param is not found in the argument list of Linker::makeMediaLinkObj($title,$text='')
* Moving few class descriptions to right above classes, and/or formatting into Javadoc style.
* "@addtogroup Special Pages" --> "@addtogroup SpecialPage" so that all special pages have the same @addtogroup tag.
* @fixme --> @todo (must have missed these before)
* "@param $specialPage @see" remove the "@" in the "@see" to stop warning.
* @throws wants type, then a brief description, to stop warning.

This last one is for PHPdocumentor only, but it fixes something for PHPDocumentor, and should be neutral for Doxygen:
* WARNING in includes/api/ApiFormatYaml_spyc.php on line 860: docblock template never terminated with /**#@-*/

26 files changed:
includes/BagOStuff.php
includes/Categoryfinder.php
includes/DatabaseOracle.php
includes/DatabasePostgres.php
includes/Exif.php
includes/Export.php
includes/GlobalFunctions.php
includes/IP.php
includes/Image.php
includes/Linker.php
includes/Sanitizer.php
includes/SpecialAllpages.php
includes/SpecialBooksources.php
includes/SpecialConfirmemail.php
includes/SpecialContributions.php
includes/SpecialRandomredirect.php
includes/SpecialRecentchanges.php
includes/SpecialResetpass.php
includes/SpecialUncategorizedimages.php
includes/SpecialUndelete.php
includes/SpecialUnusedtemplates.php
includes/SpecialUserlogin.php
includes/SpecialUserrights.php
includes/api/ApiFormatJson_json.php
includes/api/ApiFormatYaml_spyc.php
includes/api/ApiMain.php

index 7b7aaab..cc23936 100644 (file)
  * the PHP memcached client.
  *
  * backends for local hash array and SQL table included:
- * $bag = new HashBagOStuff();
- * $bag = new MysqlBagOStuff($tablename); # connect to db first
+ * <code>
+ *   $bag = new HashBagOStuff();
+ *   $bag = new MysqlBagOStuff($tablename); # connect to db first
+ * </code>
  *
  */
 class BagOStuff {
index 7a9d427..7faae93 100644 (file)
@@ -7,8 +7,8 @@
  * articles are in one or all of a given subset of categories.
  *
  * Example use :
- *
- *     # Determines wether the article with the page_id 12345 is in both
+ * <code>
+ *     # Determines whether the article with the page_id 12345 is in both
  *     # "Category 1" and "Category 2" or their subcategories, respectively
  *
  *     $cf = new Categoryfinder ;
@@ -19,7 +19,7 @@
  *     ) ;
  *     $a = $cf->run() ;
  *     print implode ( "," , $a ) ;
- *
+ * </code>
  *
  */
 class Categoryfinder {
@@ -35,7 +35,7 @@ class Categoryfinder {
 
        /**
         * Constructor (currently empty).
-       */
+        */
        function __construct() {
        }
 
@@ -62,8 +62,8 @@ class Categoryfinder {
        /**
         * Iterates through the parent tree starting with the seed values,
         * then checks the articles if they match the conditions
-        @return array of page_ids (those given to seed() that match the conditions)
-       */
+        @return array of page_ids (those given to seed() that match the conditions)
+        */
        function run () {
                $this->dbr = wfGetDB( DB_SLAVE );
                while ( count ( $this->next ) > 0 ) {
@@ -84,10 +84,10 @@ class Categoryfinder {
 
        /**
         * This functions recurses through the parent representation, trying to match the conditions
-        @param $id The article/category to check
-        @param $conds The array of categories to match
-        @return bool Does this match the conditions?
-       */
+        @param $id The article/category to check
+        @param $conds The array of categories to match
+        @return bool Does this match the conditions?
+        */
        function check ( $id , &$conds ) {
                # Shortcut (runtime paranoia): No contitions=all matched
                if ( count ( $conds ) == 0 ) return true ;
@@ -131,7 +131,7 @@ class Categoryfinder {
 
        /**
         * Scans a "parent layer" of the articles/categories in $this->next
-       */
+        */
        function scan_next_layer () {
                $fname = "Categoryfinder::scan_next_layer" ;
 
index 1c8b9b8..14ea7c2 100644 (file)
@@ -666,7 +666,7 @@ echo "error!\n";
        }
 
        public function setTimeout( $timeout ) {
-               /// @fixme no-op
+               // @todo no-op
        }
 
        function ping() {
index 6af0c02..e58ff0d 100644 (file)
@@ -1101,7 +1101,7 @@ END;
        }
 
        public function setTimeout( $timeout ) {
-               /// @fixme no-op
+               // @todo no-op
        }
 
        function ping() {
index ab2e209..a368908 100644 (file)
@@ -21,7 +21,7 @@
  * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
  * http://www.gnu.org/copyleft/gpl.html
  *
- * @link http://exif.org/Exif2-2.PDF The Exif 2.2 specification
+ * @see http://exif.org/Exif2-2.PDF The Exif 2.2 specification
  */
 
 /**
index f685a6a..1582d0d 100644 (file)
 # 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 # http://www.gnu.org/copyleft/gpl.html
 
+
 /**
  *
  * @addtogroup SpecialPage
  */
-
 class WikiExporter {
        var $list_authors = false ; # Return distinct author list (when not returning full history)
        var $author_list = "" ;
index e93bfdb..291224b 100644 (file)
@@ -1167,7 +1167,7 @@ function wfHttpError( $code, $label, $desc ) {
  * Note that some PHP configuration options may add output buffer
  * layers which cannot be removed; these are left in place.
  *
- * @parameter bool $resetGzipEncoding
+ * @param bool $resetGzipEncoding
  */
 function wfResetOutputBuffers( $resetGzipEncoding=true ) {
        if( $resetGzipEncoding ) {
@@ -1401,7 +1401,7 @@ define('TS_ISO_8601', 4);
 /**
  * An Exif timestamp (YYYY:MM:DD HH:MM:SS)
  *
- * @link http://exif.org/Exif2-2.PDF The Exif 2.2 spec, see page 28 for the
+ * @see http://exif.org/Exif2-2.PDF The Exif 2.2 spec, see page 28 for the
  *       DateTime tag and page 36 for the DateTimeOriginal and
  *       DateTimeDigitized tags.
  */
index fff9616..51d92ac 100644 (file)
@@ -280,7 +280,7 @@ class IP {
         * Split out an IP block as an array of 4 bytes and a mask,
         * return false if it can't be determined
         *
-        * @parameter $ip string A quad dotted/octet IP address
+        * @param $ip string A quad dotted/octet IP address
         * @return array
         */
        public static function toArray( $ipblock ) {
index 14dcf3d..6a5e730 100644 (file)
@@ -2204,7 +2204,7 @@ class Image
                                }
 
                                $dbw->insert( $table, $fields, __METHOD__ );
-                               /// @fixme this delete is not totally safe, potentially
+                               // @todo this delete is not totally safe, potentially
                                $dbw->delete( 'filearchive',
                                        array( 'fa_id' => $row->fa_id ),
                                        __METHOD__ );
index 96c7174..b216763 100644 (file)
@@ -681,8 +681,6 @@ class Linker {
         *
         * @param $title Title object.
         * @param $text  String: pre-sanitized HTML
-        * @param $nourl Boolean: Mask absolute URLs, so the parser doesn't
-        *                       linkify them (it is currently not context-aware)
         * @return string HTML
         *
         * @public
@@ -872,7 +870,7 @@ class Linker {
         * Since you can't set a default parameter for a reference, I've turned it
         * temporarily to a value pass. Should be adjusted further. --brion
         *
-        * $param string $comment
+        * @param string $comment
         * @param mixed $title Title object (to generate link to the section in autocomment) or null
         * @param bool $local Whether section links should refer to local page
         */
index dc12ec7..96f89dc 100644 (file)
@@ -711,10 +711,10 @@ class Sanitizer {
         * Given a value escape it so that it can be used in an id attribute and
         * return it, this does not validate the value however (see first link)
         *
-        * @link http://www.w3.org/TR/html401/types.html#type-name Valid characters
+        * @see http://www.w3.org/TR/html401/types.html#type-name Valid characters
         *                                                          in the id and
         *                                                          name attributes
-        * @link http://www.w3.org/TR/html401/struct/links.html#h-12.2.3 Anchors with the id attribute
+        * @see http://www.w3.org/TR/html401/struct/links.html#h-12.2.3 Anchors with the id attribute
         *
         * @static
         *
@@ -738,7 +738,7 @@ class Sanitizer {
         *
         * @todo For extra validity, input should be validated UTF-8.
         *
-        * @link http://www.w3.org/TR/CSS21/syndata.html Valid characters/format
+        * @see http://www.w3.org/TR/CSS21/syndata.html Valid characters/format
         *
         * @param string $class
         * @return string
index 398a596..9b89145 100644 (file)
@@ -6,7 +6,7 @@
 /**
  * Entry point : initialise variables and call subfunctions.
  * @param $par String: becomes "FOO" when called like Special:Allpages/FOO (default NULL)
- * @param $specialPage @see SpecialPage object.
+ * @param $specialPage See the SpecialPage object.
  */
 function wfSpecialAllpages( $par=NULL, $specialPage ) {
        global $wgRequest, $wgOut, $wgContLang;
index 58f25b4..d3136ea 100644 (file)
@@ -4,7 +4,7 @@
  * Special page outputs information on sourcing a book with a particular ISBN
  * The parser creates links to this page when dealing with ISBNs in wikitext
  *
- * @addtogroup Special pages
+ * @addtogroup SpecialPage
  * @author Rob Church <robchur@gmail.com>
  * @todo Validate ISBNs using the standard check-digit method
  */
index d20d9c6..e1feb8a 100644 (file)
@@ -4,7 +4,7 @@
  * Special page allows users to request email confirmation message, and handles
  * processing of the confirmation code when the link in the email is followed
  *
- * @addtogroup Special pages
+ * @addtogroup SpecialPage
  * @author Rob Church <robchur@gmail.com>
  */
  
index 248e23a..a41607c 100644 (file)
@@ -374,7 +374,7 @@ function wfSpecialContributions( $par = null ) {
 
 /**
  * Generates the subheading with links
- * @param $nt @see Title object for the target
+ * @param $nt see the Title object for the target
  */
 function contributionsSub( $nt ) {
        global $wgSysopUserBans, $wgUser;
index 60e9d65..903702a 100644 (file)
@@ -3,7 +3,7 @@
 /**
  * Special page to direct the user to a random redirect page (minus the second redirect)
  *
- * @addtogroup Special pages
+ * @addtogroup SpecialPage
  * @author Rob Church <robchur@gmail.com>
  * @license GNU General Public Licence 2.0 or later
  */
index de8579a..84444e6 100644 (file)
@@ -469,7 +469,7 @@ function rcDayLimitLinks( $days, $limit, $page='Recentchanges', $more='', $doall
 
 /**
  * Makes change an option link which carries all the other options
- * @param $title @see Title
+ * @param $title see Title
  * @param $override
  * @param $options
  */
index b83375f..e16a2fb 100644 (file)
@@ -134,7 +134,7 @@ class PasswordResetForm extends SpecialPage {
        }
        
        /**
-        * @throws PasswordError
+        * @throws PasswordError when cannot set the new password because requirements not met.
         */
        function attemptReset( $newpass, $retype ) {
                $user = User::newFromName( $this->mName );
index 7c91677..22e3466 100644 (file)
@@ -3,7 +3,7 @@
 /**
  * Special page lists images which haven't been categorised
  *
- * @addtogroup Special pages
+ * @addtogroup SpecialPage
  * @author Rob Church <robchur@gmail.com>
  */
 
index 36d3669..5437d2a 100644 (file)
@@ -4,7 +4,7 @@
  * Special page allowing users with the appropriate permissions to view
  * and restore deleted content
  *
- * @addtogroup Special pages
+ * @addtogroup SpecialPage
  */
 
 /**
index e31c10f..033066d 100644 (file)
@@ -1,17 +1,11 @@
 <?php
 
 /**
- * @addtogroup Special pages
- *
  * @author Rob Church <robchur@gmail.com>
  * @copyright © 2006 Rob Church
  * @license http://www.gnu.org/copyleft/gpl.html GNU General Public License 2.0 or later
- */
-
-/**
  * @addtogroup SpecialPage
  */
-
 class UnusedtemplatesPage extends QueryPage {
 
        function getName() { return( 'Unusedtemplates' ); }
index 879886c..a5f2521 100644 (file)
@@ -40,7 +40,7 @@ class LoginForm {
 
        /**
         * Constructor
-        * @param webrequest $request A webrequest object passed by reference
+        * @param WebRequest $request A WebRequest object passed by reference
         */
        function LoginForm( &$request ) {
                global $wgLang, $wgAllowRealName, $wgEnableEmail;
index 30c7945..7f00b1f 100644 (file)
@@ -3,7 +3,7 @@
 /**
  * Special page to allow managing user group membership
  *
- * @addtogroup Special pages
+ * @addtogroup SpecialPage
  * @todo This code is disgusting and needs a total rewrite
  */
 
index 91154e2..c15cdd8 100644 (file)
@@ -45,7 +45,6 @@
 * USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH
 * DAMAGE.
 *
-* @category
 * @addtogroup     Services_JSON
 * @author      Michal Migurski <mike-json@teczno.com>
 * @author      Matt Knapp <mdknapp[at]gmail[dot]com>
@@ -53,7 +52,7 @@
 * @copyright   2005 Michal Migurski
 * @version     CVS: $Id: JSON.php,v 1.30 2006/03/08 16:10:20 migurski Exp $
 * @license     http://www.opensource.org/licenses/bsd-license.php
-* @link        http://pear.php.net/pepr/pepr-proposal-show.php?id=198
+* @see         http://pear.php.net/pepr/pepr-proposal-show.php?id=198
 */
 
 /**
index 579fa5c..91155aa 100644 (file)
@@ -3,7 +3,7 @@
    * Spyc -- A Simple PHP YAML Class
    * @version 0.2.3 -- 2006-02-04
    * @author Chris Wanstrath <chris@ozmm.org>
-   * @link http://spyc.sourceforge.net/
+   * @see http://spyc.sourceforge.net/
    * @copyright Copyright 2005-2006 Chris Wanstrath
    * @license http://www.opensource.org/licenses/mit-license.php MIT License
    * @addtogroup Spyc
@@ -20,7 +20,7 @@
      */ 
     var $parent;
     var $id;
-    /**#@+*/
+    /**#@-*/
     /** 
      * @access public
      * @var mixed
     var $_isInline;
     var $_dumpIndent;
     var $_dumpWordWrap;
-    /**#@+*/
+    /**#@-*/
 
     /**** Private Methods ****/
     
       return $ret; 
     }
   }
-?>
\ No newline at end of file
+?>
index 43d1da4..214cfc7 100644 (file)
@@ -346,8 +346,8 @@ class ApiMain extends ApiBase {
 }
 
 /**
-* @desc This exception will be thrown when dieUsage is called to stop module execution.
-*/
+ * This exception will be thrown when dieUsage is called to stop module execution.
+ */
 class UsageException extends Exception {
 
        private $mCodestr;