From c0ea2e70fc5b6565682735a3de7f5ad52aca50cc Mon Sep 17 00:00:00 2001 From: Alexandre Emsenhuber Date: Wed, 4 Jul 2012 10:55:09 +0200 Subject: [PATCH] No need to wrap the result of DatabaseBase::select() in a ResultWrapper object. Change-Id: I2cceb363b4573a850a7a63c93d2ca9899e477a7f DatabaseBase::select() already returns a ResultWrapper object, so its useless to wrap it again. --- includes/Pager.php | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/includes/Pager.php b/includes/Pager.php index 9d1f9fcb7e..750636d0bb 100644 --- a/includes/Pager.php +++ b/includes/Pager.php @@ -306,8 +306,7 @@ abstract class IndexPager extends ContextSource implements Pager { */ public function reallyDoQuery( $offset, $limit, $descending ) { list( $tables, $fields, $conds, $fname, $options, $join_conds ) = $this->buildQueryInfo( $offset, $limit, $descending ); - $result = $this->mDb->select( $tables, $fields, $conds, $fname, $options, $join_conds ); - return new ResultWrapper( $this->mDb, $result ); + return $this->mDb->select( $tables, $fields, $conds, $fname, $options, $join_conds ); } /** -- 2.20.1