Split Title and User ArrayFromResult classes to own files

Change-Id: I8744627330eb9c8f183a8d418d6eb168e6c5407b
This commit is contained in:
addshore 2014-01-26 18:49:18 +01:00 committed by Addshore
parent 9a8f477e86
commit e3c0ab20bf
5 changed files with 179 additions and 133 deletions

View file

@ -215,14 +215,14 @@ $wgAutoloadLocalClasses = array(
'TimestampException' => 'includes/TimestampException.php',
'Title' => 'includes/Title.php',
'TitleArray' => 'includes/TitleArray.php',
'TitleArrayFromResult' => 'includes/TitleArray.php',
'TitleArrayFromResult' => 'includes/TitleArrayFromResult.php',
'ThrottledError' => 'includes/Exception.php',
'UnlistedSpecialPage' => 'includes/specialpage/UnlistedSpecialPage.php',
'UploadSourceAdapter' => 'includes/Import.php',
'UppercaseCollation' => 'includes/Collation.php',
'User' => 'includes/User.php',
'UserArray' => 'includes/UserArray.php',
'UserArrayFromResult' => 'includes/UserArray.php',
'UserArrayFromResult' => 'includes/UserArrayFromResult.php',
'UserBlockedError' => 'includes/Exception.php',
'UserNotLoggedIn' => 'includes/Exception.php',
'UserCache' => 'includes/cache/UserCache.php',

View file

@ -1,6 +1,6 @@
<?php
/**
* Classes to walk into a list of Title objects.
* Class to walk into a list of Title objects.
*
* Note: this entire file is a byte-for-byte copy of UserArray.php with
* s/User/Title/. If anyone can figure out how to do this nicely with
@ -55,64 +55,3 @@ abstract class TitleArray implements Iterator {
return $array;
}
}
class TitleArrayFromResult extends TitleArray implements Countable {
/**
* @var ResultWrapper
*/
var $res;
var $key, $current;
function __construct( $res ) {
$this->res = $res;
$this->key = 0;
$this->setCurrent( $this->res->current() );
}
/**
* @param $row ResultWrapper
* @return void
*/
protected function setCurrent( $row ) {
if ( $row === false ) {
$this->current = false;
} else {
$this->current = Title::newFromRow( $row );
}
}
/**
* @return int
*/
public function count() {
return $this->res->numRows();
}
function current() {
return $this->current;
}
function key() {
return $this->key;
}
function next() {
$row = $this->res->next();
$this->setCurrent( $row );
$this->key++;
}
function rewind() {
$this->res->rewind();
$this->key = 0;
$this->setCurrent( $this->res->current() );
}
/**
* @return bool
*/
function valid() {
return $this->current !== false;
}
}

View file

@ -0,0 +1,86 @@
<?php
/**
* Class to walk into a list of Title objects.
*
* Note: this entire file is a byte-for-byte copy of UserArrayFromResult.php
* with s/User/Title/. If anyone can figure out how to do this nicely
* with inheritance or something, please do so.
*
* 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
* the Free Software Foundation; either version 2 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* 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.,
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
* http://www.gnu.org/copyleft/gpl.html
*
* @file
*/
class TitleArrayFromResult extends TitleArray implements Countable {
/**
* @var ResultWrapper
*/
var $res;
var $key, $current;
function __construct( $res ) {
$this->res = $res;
$this->key = 0;
$this->setCurrent( $this->res->current() );
}
/**
* @param $row ResultWrapper
* @return void
*/
protected function setCurrent( $row ) {
if ( $row === false ) {
$this->current = false;
} else {
$this->current = Title::newFromRow( $row );
}
}
/**
* @return int
*/
public function count() {
return $this->res->numRows();
}
function current() {
return $this->current;
}
function key() {
return $this->key;
}
function next() {
$row = $this->res->next();
$this->setCurrent( $row );
$this->key++;
}
function rewind() {
$this->res->rewind();
$this->key = 0;
$this->setCurrent( $this->res->current() );
}
/**
* @return bool
*/
function valid() {
return $this->current !== false;
}
}

View file

@ -1,6 +1,6 @@
<?php
/**
* Classes to walk into a list of User objects.
* Class to walk into a list of User objects.
*
* 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
@ -63,71 +63,4 @@ abstract class UserArray implements Iterator {
protected static function newFromResult_internal( $res ) {
return new UserArrayFromResult( $res );
}
}
class UserArrayFromResult extends UserArray implements Countable {
/**
* @var ResultWrapper
*/
var $res;
var $key, $current;
/**
* @param $res ResultWrapper
*/
function __construct( $res ) {
$this->res = $res;
$this->key = 0;
$this->setCurrent( $this->res->current() );
}
/**
* @param $row
* @return void
*/
protected function setCurrent( $row ) {
if ( $row === false ) {
$this->current = false;
} else {
$this->current = User::newFromRow( $row );
}
}
/**
* @return int
*/
public function count() {
return $this->res->numRows();
}
/**
* @return User
*/
function current() {
return $this->current;
}
function key() {
return $this->key;
}
function next() {
$row = $this->res->next();
$this->setCurrent( $row );
$this->key++;
}
function rewind() {
$this->res->rewind();
$this->key = 0;
$this->setCurrent( $this->res->current() );
}
/**
* @return bool
*/
function valid() {
return $this->current !== false;
}
}
}

View file

@ -0,0 +1,88 @@
<?php
/**
* Class to walk into a list of User objects.
*
* 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
* the Free Software Foundation; either version 2 of the License, or
* (at your option) any later version.
*
* This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU General Public License for more details.
*
* 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.,
* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
* http://www.gnu.org/copyleft/gpl.html
*
* @file
*/
class UserArrayFromResult extends UserArray implements Countable {
/**
* @var ResultWrapper
*/
var $res;
var $key, $current;
/**
* @param $res ResultWrapper
*/
function __construct( $res ) {
$this->res = $res;
$this->key = 0;
$this->setCurrent( $this->res->current() );
}
/**
* @param $row
* @return void
*/
protected function setCurrent( $row ) {
if ( $row === false ) {
$this->current = false;
} else {
$this->current = User::newFromRow( $row );
}
}
/**
* @return int
*/
public function count() {
return $this->res->numRows();
}
/**
* @return User
*/
function current() {
return $this->current;
}
function key() {
return $this->key;
}
function next() {
$row = $this->res->next();
$this->setCurrent( $row );
$this->key++;
}
function rewind() {
$this->res->rewind();
$this->key = 0;
$this->setCurrent( $this->res->current() );
}
/**
* @return bool
*/
function valid() {
return $this->current !== false;
}
}