/**
* List for revision table items for a single page
*/
-abstract class Rev_List {
+abstract class RevisionListBase {
/**
* @var Title
*/
var $title;
/**
- * @var RequestContext
+ * @var IContextSource
*/
var $context;
var $ids, $res, $current;
- function __construct( RequestContext $context, Title $title, array $ids ) {
+ /**
+ * Construct a revision list for a given title
+ * @param $context IContextSource
+ * @param $title Title
+ */
+ function __construct( IContextSource $context, Title $title ) {
$this->context = $context;
$this->title = $title;
+ }
+
+ /**
+ * Select items only where the ID is any of the specified values
+ * @param $ids Array
+ */
+ function filterByIds( array $ids ) {
$this->ids = $ids;
}
/**
* Abstract base class for revision items
*/
-abstract class Rev_Item {
- /** The parent Rev_List */
+abstract class RevisionItemBase {
+ /** The parent RevisionListBase */
var $list;
/** The DB result row */
var $row;
/**
- * @param $list Rev_List
+ * @param $list RevisionListBase
* @param $row DB result row
*/
public function __construct( $list, $row ) {
abstract public function getHTML();
}
-class RevisionList extends Rev_List {
+class RevisionList extends RevisionListBase {
public function getType() {
return 'revision';
}
* @return mixed
*/
public function doQuery( $db ) {
- $ids = array_map( 'intval', $this->ids );
- return $db->select( array('revision','page'), '*',
- array(
- 'rev_page' => $this->title->getArticleID(),
- 'rev_id' => array_map( 'intval', $this->ids ),
- 'rev_page = page_id'
- ),
+ $conds = array(
+ 'rev_page' => $this->title->getArticleID(),
+ 'rev_page = page_id'
+ );
+ if ( $this->ids !== null ) {
+ $conds['rev_id'] = array_map( 'intval', $this->ids );
+ }
+ return $db->select(
+ array( 'revision', 'page' ),
+ '*',
+ $conds,
__METHOD__,
array( 'ORDER BY' => 'rev_id DESC' )
);
/**
* Item class for a live revision table row
*/
-class RevisionItem extends Rev_Item {
+class RevisionItem extends RevisionItemBase {
var $revision, $context;
public function __construct( $list, $row ) {