Few braces and spaces
authorSam Reed <reedy@users.mediawiki.org>
Tue, 30 Nov 2010 19:06:28 +0000 (19:06 +0000)
committerSam Reed <reedy@users.mediawiki.org>
Tue, 30 Nov 2010 19:06:28 +0000 (19:06 +0000)
Fixing up documentation

includes/CategoryPage.php
includes/GlobalFunctions.php
includes/PageQueryPage.php
includes/User.php
includes/db/Database.php
includes/memcached-client.php

index 42e6ff4..682dda3 100644 (file)
@@ -23,11 +23,13 @@ class CategoryPage extends Article {
                $diff = $wgRequest->getVal( 'diff' );
                $diffOnly = $wgRequest->getBool( 'diffonly', $wgUser->getOption( 'diffonly' ) );
 
-               if ( isset( $diff ) && $diffOnly )
+               if ( isset( $diff ) && $diffOnly ) {
                        return parent::view();
+               }
 
-               if ( !wfRunHooks( 'CategoryPageView', array( &$this ) ) )
+               if ( !wfRunHooks( 'CategoryPageView', array( &$this ) ) ) {
                        return;
+               }
 
                if ( NS_CATEGORY == $this->mTitle->getNamespace() ) {
                        $this->openShowCategory();
index 43c8932..9dbfde1 100644 (file)
@@ -3206,7 +3206,7 @@ function wfFindFile( $title, $options = array() ) {
 /**
  * Get an object referring to a locally registered file.
  * Returns a valid placeholder object if the file does not exist.
- * @param $title Either a string or Title object
+ * @param $title Title|String
  * @return File, or null if passed an invalid Title
  */
 function wfLocalFile( $title ) {
index a2091e8..892ff25 100644 (file)
@@ -18,8 +18,9 @@ class PageQueryPage extends QueryPage {
                global $wgContLang;
                $title = Title::makeTitleSafe( $row->namespace, $row->title );
                $text = $row->title;
-               if ($title instanceof Title)
-                       $text = $wgContLang->convert( $title->getPrefixedText() ); 
-               return $skin->link( $title, htmlspecialchars($text), array(), array(), array('known', 'noclasses') );
+               if ( $title instanceof Title ) {
+                       $text = $wgContLang->convert( $title->getPrefixedText() );
+               }
+               return $skin->link( $title, htmlspecialchars( $text ), array(), array(), array('known', 'noclasses') );
        }
 }
index 7e77986..bd56fc1 100644 (file)
@@ -3207,7 +3207,7 @@ class User {
         * Return the set of defined explicit groups.
         * The implicit groups (by default *, 'user' and 'autoconfirmed')
         * are not included, as they are defined automatically, not in the database.
-        * @return \type{\arrayof{\string}} Array of internal group names
+        * @return Array of internal group names
         */
        static function getAllGroups() {
                global $wgGroupPermissions, $wgRevokePermissions;
@@ -3219,7 +3219,7 @@ class User {
 
        /**
         * Get a list of all available permissions.
-        * @return \type{\arrayof{\string}} Array of permission names
+        * @return Array of permission names
         */
        static function getAllRights() {
                if ( self::$mAllRights === false ) {
index 8f19635..3ff4145 100644 (file)
@@ -604,7 +604,7 @@ abstract class DatabaseBase implements DatabaseType {
         *     comment (you can use __METHOD__ or add some extra info)
         * @param  $tempIgnore Boolean:   Whether to avoid throwing an exception on errors...
         *     maybe best to catch the exception instead?
-        * @return true for a successful write query, ResultWrapper object for a successful read query,
+        * @return boolean|ResultWrapper true for a successful write query, ResultWrapper object for a successful read query,
         *     or false on failure if $tempIgnore set
         * @throws DBQueryError Thrown when the database returns an error of any kind
         */
index a23fd9c..53f0324 100644 (file)
@@ -240,7 +240,7 @@ class MWMemcached {
        /**
         * Memcache initializer
         *
-        * @param $args Associative array of settings
+        * @param $args Array Associative array of settings
         *
         * @return  mixed
         */