]> scripts.mit.edu Git - autoinstalls/mediawiki.git/blobdiff - includes/api/ApiQueryBase.php
MediaWiki 1.17.4
[autoinstalls/mediawiki.git] / includes / api / ApiQueryBase.php
index 896dd00c065289f63b2343f06cccff0d6ab8bf16..477b740a017f88b5f7691685272c526a45615f68 100644 (file)
@@ -1,11 +1,10 @@
 <?php
-
-/*
- * Created on Sep 7, 2006
- *
+/**
  * API for MediaWiki 1.8+
  *
- * Copyright (C) 2006 Yuri Astrakhan <Firstname><Lastname>@gmail.com
+ * Created on Sep 7, 2006
+ *
+ * Copyright © 2006 Yuri Astrakhan <Firstname><Lastname>@gmail.com
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  *
  * You should have received a copy of the GNU General Public License along
  * with this program; if not, write to the Free Software Foundation, Inc.,
- * 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA.
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
  * http://www.gnu.org/copyleft/gpl.html
+ *
+ * @file
  */
 
-if (!defined('MEDIAWIKI')) {
+if ( !defined( 'MEDIAWIKI' ) ) {
        // Eclipse helper - will be ignored in production
-       require_once ('ApiBase.php');
+       require_once( 'ApiBase.php' );
 }
 
 /**
  * This is a base class for all Query modules.
- * It provides some common functionality such as constructing various SQL queries.
+ * It provides some common functionality such as constructing various SQL
+ * queries.
  *
  * @ingroup API
  */
@@ -38,84 +40,103 @@ abstract class ApiQueryBase extends ApiBase {
 
        private $mQueryModule, $mDb, $tables, $where, $fields, $options, $join_conds;
 
-       public function __construct($query, $moduleName, $paramPrefix = '') {
-               parent :: __construct($query->getMain(), $moduleName, $paramPrefix);
+       public function __construct( ApiBase $query, $moduleName, $paramPrefix = '' ) {
+               parent::__construct( $query->getMain(), $moduleName, $paramPrefix );
                $this->mQueryModule = $query;
                $this->mDb = null;
                $this->resetQueryParams();
        }
 
+       /**
+        * Get the cache mode for the data generated by this module. Override
+        * this in the module subclass. For possible return values and other
+        * details about cache modes, see ApiMain::setCacheMode()
+        *
+        * Public caching will only be allowed if *all* the modules that supply
+        * data for a given request return a cache mode of public.
+        */
+       public function getCacheMode( $params ) {
+               return 'private';
+       }
+
        /**
         * Blank the internal arrays with query parameters
         */
        protected function resetQueryParams() {
-               $this->tables = array ();
-               $this->where = array ();
-               $this->fields = array ();
-               $this->options = array ();
-               $this->join_conds = array ();
+               $this->tables = array();
+               $this->where = array();
+               $this->fields = array();
+               $this->options = array();
+               $this->join_conds = array();
        }
 
        /**
         * Add a set of tables to the internal array
-        * @param mixed $tables Table name or array of table names
-        * @param mixed $alias Table alias, or null for no alias. Cannot be used with multiple tables
-        */
-       protected function addTables($tables, $alias = null) {
-               if (is_array($tables)) {
-                       if (!is_null($alias))
-                               ApiBase :: dieDebug(__METHOD__, 'Multiple table aliases not supported');
-                       $this->tables = array_merge($this->tables, $tables);
+        * @param $tables mixed Table name or array of table names
+        * @param $alias mixed Table alias, or null for no alias. Cannot be
+        *  used with multiple tables
+        */
+       protected function addTables( $tables, $alias = null ) {
+               if ( is_array( $tables ) ) {
+                       if ( !is_null( $alias ) ) {
+                               ApiBase::dieDebug( __METHOD__, 'Multiple table aliases not supported' );
+                       }
+                       $this->tables = array_merge( $this->tables, $tables );
                } else {
-                       if (!is_null($alias))
-                               $tables = $this->getAliasedName($tables, $alias);
+                       if ( !is_null( $alias ) ) {
+                               $tables = $this->getAliasedName( $tables, $alias );
+                       }
                        $this->tables[] = $tables;
                }
        }
-       
+
        /**
         * Get the SQL for a table name with alias
-        * @param string $table Table name
-        * @param string $alias Alias
+        * @param $table string Table name
+        * @param $alias string Alias
         * @return string SQL
         */
-       protected function getAliasedName($table, $alias) {
-               return $this->getDB()->tableName($table) . ' ' . $alias;
+       protected function getAliasedName( $table, $alias ) {
+               return $this->getDB()->tableName( $table ) . ' ' . $alias;
        }
-       
+
        /**
         * Add a set of JOIN conditions to the internal array
         *
-        * JOIN conditions are formatted as array( tablename => array(jointype, conditions)
-        * e.g. array('page' => array('LEFT JOIN', 'page_id=rev_page'))
-        * @param array $join_conds JOIN conditions
+        * JOIN conditions are formatted as array( tablename => array(jointype,
+        * conditions) e.g. array('page' => array('LEFT JOIN',
+        * 'page_id=rev_page')) . conditions may be a string or an
+        * addWhere()-style array
+        * @param $join_conds array JOIN conditions
         */
-       protected function addJoinConds($join_conds) {
-               if(!is_array($join_conds))
-                       ApiBase::dieDebug(__METHOD__, 'Join conditions have to be arrays');
-               $this->join_conds = array_merge($this->join_conds, $join_conds);
+       protected function addJoinConds( $join_conds ) {
+               if ( !is_array( $join_conds ) ) {
+                       ApiBase::dieDebug( __METHOD__, 'Join conditions have to be arrays' );
+               }
+               $this->join_conds = array_merge( $this->join_conds, $join_conds );
        }
 
        /**
         * Add a set of fields to select to the internal array
-        * @param mixed $value Field name or array of field names
+        * @param $value mixed Field name or array of field names
         */
-       protected function addFields($value) {
-               if (is_array($value))
-                       $this->fields = array_merge($this->fields, $value);
-               else
+       protected function addFields( $value ) {
+               if ( is_array( $value ) ) {
+                       $this->fields = array_merge( $this->fields, $value );
+               } else {
                        $this->fields[] = $value;
+               }
        }
 
        /**
         * Same as addFields(), but add the fields only if a condition is met
-        * @param mixed $value See addFields()
-        * @param bool $condition If false, do nothing
+        * @param $value mixed See addFields()
+        * @param $condition bool If false, do nothing
         * @return bool $condition
         */
-       protected function addFieldsIf($value, $condition) {
-               if ($condition) {
-                       $this->addFields($value);
+       protected function addFieldsIf( $value, $condition ) {
+               if ( $condition ) {
+                       $this->addFields( $value );
                        return true;
                }
                return false;
@@ -130,28 +151,29 @@ abstract class ApiQueryBase extends ApiBase {
         *
         * For example, array('foo=bar', 'baz' => 3, 'bla' => 'foo') translates
         * to "foo=bar AND baz='3' AND bla='foo'"
-        * @param mixed $value String or array
+        * @param $value mixed String or array
         */
-       protected function addWhere($value) {
-               if (is_array($value)) {
+       protected function addWhere( $value ) {
+               if ( is_array( $value ) ) {
                        // Sanity check: don't insert empty arrays,
                        // Database::makeList() chokes on them
-                       if ( count( $value ) )
-                               $this->where = array_merge($this->where, $value);
-               }
-               else
+                       if ( count( $value ) ) {
+                               $this->where = array_merge( $this->where, $value );
+                       }
+               } else {
                        $this->where[] = $value;
+               }
        }
 
        /**
         * Same as addWhere(), but add the WHERE clauses only if a condition is met
-        * @param mixed $value See addWhere()
-        * @param bool $condition If false, do nothing
+        * @param $value mixed See addWhere()
+        * @param $condition bool If false, do nothing
         * @return bool $condition
         */
-       protected function addWhereIf($value, $condition) {
-               if ($condition) {
-                       $this->addWhere($value);
+       protected function addWhereIf( $value, $condition ) {
+               if ( $condition ) {
+                       $this->addWhere( $value );
                        return true;
                }
                return false;
@@ -159,67 +181,88 @@ abstract class ApiQueryBase extends ApiBase {
 
        /**
         * Equivalent to addWhere(array($field => $value))
-        * @param string $field Field name
-        * @param string $value Value; ignored if null or empty array;
+        * @param $field string Field name
+        * @param $value string Value; ignored if null or empty array;
         */
-       protected function addWhereFld($field, $value) {
-               // Use count() to its full documented capabilities to simultaneously 
+       protected function addWhereFld( $field, $value ) {
+               // Use count() to its full documented capabilities to simultaneously
                // test for null, empty array or empty countable object
-               if ( count( $value ) )
+               if ( count( $value ) ) {
                        $this->where[$field] = $value;
+               }
        }
 
        /**
         * Add a WHERE clause corresponding to a range, and an ORDER BY
         * clause to sort in the right direction
-        * @param string $field Field name
-        * @param string $dir If 'newer', sort in ascending order, otherwise sort in descending order
-        * @param string $start Value to start the list at. If $dir == 'newer' this is the lower boundary, otherwise it's the upper boundary
-        * @param string $end Value to end the list at. If $dir == 'newer' this is the upper boundary, otherwise it's the lower boundary
-        */
-       protected function addWhereRange($field, $dir, $start, $end) {
-               $isDirNewer = ($dir === 'newer');
-               $after = ($isDirNewer ? '>=' : '<=');
-               $before = ($isDirNewer ? '<=' : '>=');
+        * @param $field string Field name
+        * @param $dir string If 'newer', sort in ascending order, otherwise
+        *  sort in descending order
+        * @param $start string Value to start the list at. If $dir == 'newer'
+        *  this is the lower boundary, otherwise it's the upper boundary
+        * @param $end string Value to end the list at. If $dir == 'newer' this
+        *  is the upper boundary, otherwise it's the lower boundary
+        * @param $sort bool If false, don't add an ORDER BY clause
+        */
+       protected function addWhereRange( $field, $dir, $start, $end, $sort = true ) {
+               $isDirNewer = ( $dir === 'newer' );
+               $after = ( $isDirNewer ? '>=' : '<=' );
+               $before = ( $isDirNewer ? '<=' : '>=' );
                $db = $this->getDB();
 
-               if (!is_null($start))
-                       $this->addWhere($field . $after . $db->addQuotes($start));
+               if ( !is_null( $start ) ) {
+                       $this->addWhere( $field . $after . $db->addQuotes( $start ) );
+               }
 
-               if (!is_null($end))
-                       $this->addWhere($field . $before . $db->addQuotes($end));
+               if ( !is_null( $end ) ) {
+                       $this->addWhere( $field . $before . $db->addQuotes( $end ) );
+               }
 
-               $order = $field . ($isDirNewer ? '' : ' DESC');
-               if (!isset($this->options['ORDER BY']))
-                       $this->addOption('ORDER BY', $order);
-               else
-                       $this->addOption('ORDER BY', $this->options['ORDER BY'] . ', ' . $order);
+               if ( $sort ) {
+                       $order = $field . ( $isDirNewer ? '' : ' DESC' );
+                       if ( !isset( $this->options['ORDER BY'] ) ) {
+                               $this->addOption( 'ORDER BY', $order );
+                       } else {
+                               $this->addOption( 'ORDER BY', $this->options['ORDER BY'] . ', ' . $order );
+                       }
+               }
        }
 
        /**
-        * Add an option such as LIMIT or USE INDEX
-        * @param string $name Option name
-        * @param string $value Option value
+        * Add an option such as LIMIT or USE INDEX. If an option was set
+        * before, the old value will be overwritten
+        * @param $name string Option name
+        * @param $value string Option value
         */
-       protected function addOption($name, $value = null) {
-               if (is_null($value))
+       protected function addOption( $name, $value = null ) {
+               if ( is_null( $value ) ) {
                        $this->options[] = $name;
-               else
+               } else {
                        $this->options[$name] = $value;
+               }
        }
 
        /**
         * Execute a SELECT query based on the values in the internal arrays
-        * @param string $method Function the query should be attributed to. You should usually use __METHOD__ here
+        * @param $method string Function the query should be attributed to.
+        *  You should usually use __METHOD__ here
+        * @param $extraQuery array Query data to add but not store in the object
+        *  Format is array( 'tables' => ..., 'fields' => ..., 'where' => ..., 'options' => ..., 'join_conds' => ... )
         * @return ResultWrapper
         */
-       protected function select($method) {
+       protected function select( $method, $extraQuery = array() ) {
+
+               $tables = array_merge( $this->tables, isset( $extraQuery['tables'] ) ? (array)$extraQuery['tables'] : array() );
+               $fields = array_merge( $this->fields, isset( $extraQuery['fields'] ) ? (array)$extraQuery['fields'] : array() );
+               $where = array_merge( $this->where, isset( $extraQuery['where'] ) ? (array)$extraQuery['where'] : array() );
+               $options = array_merge( $this->options, isset( $extraQuery['options'] ) ? (array)$extraQuery['options'] : array() );
+               $join_conds = array_merge( $this->join_conds, isset( $extraQuery['join_conds'] ) ? (array)$extraQuery['join_conds'] : array() );
 
                // getDB has its own profileDBIn/Out calls
                $db = $this->getDB();
 
                $this->profileDBIn();
-               $res = $db->select($this->tables, $this->fields, $this->where, $method, $this->options, $this->join_conds);
+               $res = $db->select( $tables, $fields, $where, $method, $options, $join_conds );
                $this->profileDBOut();
 
                return $res;
@@ -233,32 +276,34 @@ abstract class ApiQueryBase extends ApiBase {
        protected function checkRowCount() {
                $db = $this->getDB();
                $this->profileDBIn();
-               $rowcount = $db->estimateRowCount($this->tables, $this->fields, $this->where, __METHOD__, $this->options);
+               $rowcount = $db->estimateRowCount( $this->tables, $this->fields, $this->where, __METHOD__, $this->options );
                $this->profileDBOut();
 
                global $wgAPIMaxDBRows;
-               if($rowcount > $wgAPIMaxDBRows)
+               if ( $rowcount > $wgAPIMaxDBRows ) {
                        return false;
+               }
                return true;
        }
 
        /**
-        * Add information (title and namespace) about a Title object to a result array
-        * @param array $arr Result array à la ApiResult
-        * @param Title $title Title object
-        * @param string $prefix Module prefix
+        * Add information (title and namespace) about a Title object to a
+        * result array
+        * @param $arr array Result array à la ApiResult
+        * @param $title Title
+        * @param $prefix string Module prefix
         */
-       public static function addTitleInfo(&$arr, $title, $prefix='') {
-               $arr[$prefix . 'ns'] = intval($title->getNamespace());
+       public static function addTitleInfo( &$arr, $title, $prefix = '' ) {
+               $arr[$prefix . 'ns'] = intval( $title->getNamespace() );
                $arr[$prefix . 'title'] = $title->getPrefixedText();
        }
 
        /**
         * Override this method to request extra fields from the pageSet
         * using $pageSet->requestField('fieldName')
-        * @param ApiPageSet $pageSet
+        * @param $pageSet ApiPageSet
         */
-       public function requestExtraData($pageSet) {
+       public function requestExtraData( $pageSet ) {
        }
 
        /**
@@ -271,51 +316,76 @@ abstract class ApiQueryBase extends ApiBase {
 
        /**
         * Add a sub-element under the page element with the given page ID
-        * @param int $pageId Page ID
-        * @param array $data Data array à la ApiResult 
+        * @param $pageId int Page ID
+        * @param $data array Data array à la ApiResult
+        * @return bool Whether the element fit in the result
         */
-       protected function addPageSubItems($pageId, $data) {
+       protected function addPageSubItems( $pageId, $data ) {
                $result = $this->getResult();
-               $result->setIndexedTagName($data, $this->getModulePrefix());
-               $result->addValue(array ('query', 'pages', intval($pageId)),
+               $result->setIndexedTagName( $data, $this->getModulePrefix() );
+               return $result->addValue( array( 'query', 'pages', intval( $pageId ) ),
                        $this->getModuleName(),
-                       $data);
+                       $data );
        }
 
        /**
-        * Set a query-continue value
-        * @param $paramName Parameter name
-        * @param $paramValue Parameter value
+        * Same as addPageSubItems(), but one element of $data at a time
+        * @param $pageId int Page ID
+        * @param $item array Data array à la ApiResult
+        * @param $elemname string XML element name. If null, getModuleName()
+        *  is used
+        * @return bool Whether the element fit in the result
         */
-       protected function setContinueEnumParameter($paramName, $paramValue) {
+       protected function addPageSubItem( $pageId, $item, $elemname = null ) {
+               if ( is_null( $elemname ) ) {
+                       $elemname = $this->getModulePrefix();
+               }
+               $result = $this->getResult();
+               $fit = $result->addValue( array( 'query', 'pages', $pageId,
+                                        $this->getModuleName() ), null, $item );
+               if ( !$fit ) {
+                       return false;
+               }
+               $result->setIndexedTagName_internal( array( 'query', 'pages', $pageId,
+                               $this->getModuleName() ), $elemname );
+               return true;
+       }
 
-               $paramName = $this->encodeParamName($paramName);
+       /**
+        * Set a query-continue value
+        * @param $paramName string Parameter name
+        * @param $paramValue string Parameter value
+        */
+       protected function setContinueEnumParameter( $paramName, $paramValue ) {
+               $paramName = $this->encodeParamName( $paramName );
                $msg = array( $paramName => $paramValue );
-               $this->getResult()->addValue('query-continue', $this->getModuleName(), $msg);
+               $this->getResult()->disableSizeCheck();
+               $this->getResult()->addValue( 'query-continue', $this->getModuleName(), $msg );
+               $this->getResult()->enableSizeCheck();
        }
 
        /**
-        * Get the Query database connection (readonly)
+        * Get the Query database connection (read-only)
         * @return Database
         */
        protected function getDB() {
-               if (is_null($this->mDb))
-                       $this->mDb = $this->getQuery()->getDB();
+               if ( is_null( $this->mDb ) ) {
+                       $apiQuery = $this->getQuery();
+                       $this->mDb = $apiQuery->getDB();
+               }
                return $this->mDb;
        }
 
        /**
         * Selects the query database connection with the given name.
-        * If no such connection has been requested before, it will be created.
-        * Subsequent calls with the same $name will return the same connection
-        * as the first, regardless of $db or $groups new values.
-        * @param string $name Name to assign to the database connection
-        * @param int $db One of the DB_* constants
-        * @param array $groups Query groups
-        * @return Database 
+        * See ApiQuery::getNamedDB() for more information
+        * @param $name string Name to assign to the database connection
+        * @param $db int One of the DB_* constants
+        * @param $groups array Query groups
+        * @return Database
         */
-       public function selectNamedDB($name, $db, $groups) {
-               $this->mDb = $this->getQuery()->getNamedDB($name, $db, $groups);
+       public function selectNamedDB( $name, $db, $groups ) {
+               $this->mDb = $this->getQuery()->getNamedDB( $name, $db, $groups );
        }
 
        /**
@@ -328,51 +398,92 @@ abstract class ApiQueryBase extends ApiBase {
 
        /**
         * Convert a title to a DB key
-        * @param string $title Page title with spaces
+        * @param $title string Page title with spaces
         * @return string Page title with underscores
         */
-       public function titleToKey($title) {
-               # Don't throw an error if we got an empty string
-               if(trim($title) == '')
+       public function titleToKey( $title ) {
+               // Don't throw an error if we got an empty string
+               if ( trim( $title ) == '' ) {
                        return '';
-               $t = Title::newFromText($title);
-               if(!$t)
-                       $this->dieUsageMsg(array('invalidtitle', $title));
+               }
+               $t = Title::newFromText( $title );
+               if ( !$t ) {
+                       $this->dieUsageMsg( array( 'invalidtitle', $title ) );
+               }
                return $t->getPrefixedDbKey();
        }
 
        /**
         * The inverse of titleToKey()
-        * @param string $key Page title with underscores
+        * @param $key string Page title with underscores
         * @return string Page title with spaces
         */
-       public function keyToTitle($key) {
-               # Don't throw an error if we got an empty string
-               if(trim($key) == '')
+       public function keyToTitle( $key ) {
+               // Don't throw an error if we got an empty string
+               if ( trim( $key ) == '' ) {
                        return '';
-               $t = Title::newFromDbKey($key);
-               # This really shouldn't happen but we gotta check anyway
-               if(!$t)
-                       $this->dieUsageMsg(array('invalidtitle', $key));
+               }
+               $t = Title::newFromDbKey( $key );
+               // This really shouldn't happen but we gotta check anyway
+               if ( !$t ) {
+                       $this->dieUsageMsg( array( 'invalidtitle', $key ) );
+               }
                return $t->getPrefixedText();
        }
-       
+
        /**
         * An alternative to titleToKey() that doesn't trim trailing spaces
-        * @param string $titlePart Title part with spaces
+        * @param $titlePart string Title part with spaces
         * @return string Title part with underscores
         */
-       public function titlePartToKey($titlePart) {
-               return substr($this->titleToKey($titlePart . 'x'), 0, -1);
+       public function titlePartToKey( $titlePart ) {
+               return substr( $this->titleToKey( $titlePart . 'x' ), 0, - 1 );
        }
-       
+
        /**
         * An alternative to keyToTitle() that doesn't trim trailing spaces
-        * @param string $keyPart Key part with spaces
+        * @param $keyPart string Key part with spaces
         * @return string Key part with underscores
         */
-       public function keyPartToTitle($keyPart) {
-               return substr($this->keyToTitle($keyPart . 'x'), 0, -1);
+       public function keyPartToTitle( $keyPart ) {
+               return substr( $this->keyToTitle( $keyPart . 'x' ), 0, - 1 );
+       }
+
+       /**
+        * Filters hidden users (where the user doesn't have the right to view them)
+        * Also adds relevant block information
+        *
+        * @param bool $showBlockInfo
+        * @return void
+        */
+       public function showHiddenUsersAddBlockInfo( $showBlockInfo ) {
+               global $wgUser;
+               $userCanViewHiddenUsers = $wgUser->isAllowed( 'hideuser' );
+
+               if ( $showBlockInfo || !$userCanViewHiddenUsers ) {
+                       $this->addTables( 'ipblocks' );
+                       $this->addJoinConds( array(
+                               'ipblocks' => array( 'LEFT JOIN', 'ipb_user=user_id' ),
+                       ) );
+
+                       $this->addFields( 'ipb_deleted' );
+
+                       if ( $showBlockInfo ) {
+                               $this->addFields( array( 'ipb_reason', 'ipb_by_text', 'ipb_expiry' ) );
+                       }
+
+                       // Don't show hidden names
+                       if ( !$userCanViewHiddenUsers ) {
+                               $this->addWhere( 'ipb_deleted = 0 OR ipb_deleted IS NULL' );
+                       }
+               }
+       }
+
+       public function getPossibleErrors() {
+               return array_merge( parent::getPossibleErrors(), array(
+                       array( 'invalidtitle', 'title' ),
+                       array( 'invalidtitle', 'key' ),
+               ) );
        }
 
        /**
@@ -380,7 +491,7 @@ abstract class ApiQueryBase extends ApiBase {
         * @return string
         */
        public static function getBaseVersion() {
-               return __CLASS__ . ': $Id: ApiQueryBase.php 44461 2008-12-11 19:11:11Z ialex $';
+               return __CLASS__ . ': $Id$';
        }
 }
 
@@ -391,8 +502,8 @@ abstract class ApiQueryGeneratorBase extends ApiQueryBase {
 
        private $mIsGenerator;
 
-       public function __construct($query, $moduleName, $paramPrefix = '') {
-               parent :: __construct($query, $moduleName, $paramPrefix);
+       public function __construct( $query, $moduleName, $paramPrefix = '' ) {
+               parent::__construct( $query, $moduleName, $paramPrefix );
                $this->mIsGenerator = false;
        }
 
@@ -406,17 +517,21 @@ abstract class ApiQueryGeneratorBase extends ApiQueryBase {
 
        /**
         * Overrides base class to prepend 'g' to every generator parameter
+        * @param $paramName string Parameter name
+        * @return string Prefixed parameter name
         */
-       public function encodeParamName($paramName) {
-               if ($this->mIsGenerator)
-                       return 'g' . parent :: encodeParamName($paramName);
-               else
-                       return parent :: encodeParamName($paramName);
+       public function encodeParamName( $paramName ) {
+               if ( $this->mIsGenerator ) {
+                       return 'g' . parent::encodeParamName( $paramName );
+               } else {
+                       return parent::encodeParamName( $paramName );
+               }
        }
 
        /**
         * Execute this module as a generator
-        * @param $resultPageSet PageSet: All output should be appended to this object
+        * @param $resultPageSet ApiPageSet: All output should be appended to
+        *  this object
         */
-       public abstract function executeGenerator($resultPageSet);
+       public abstract function executeGenerator( $resultPageSet );
 }