*/
class NewPagesPager extends ReverseChronologicalPager {
- // Stored opts
+ /**
+ * @var FormOptions
+ */
protected $opts;
/**
- * @var HTMLForm
+ * @var SpecialNewpages
*/
protected $mForm;
- function __construct( $form, FormOptions $opts ) {
+ /**
+ * @param SpecialNewpages $form
+ * @param FormOptions $opts
+ */
+ public function __construct( $form, FormOptions $opts ) {
parent::__construct( $form->getContext() );
$this->mForm = $form;
$this->opts = $opts;
$fields = array_merge( $rcQuery['fields'], [
'length' => 'page_len', 'rev_id' => 'page_latest', 'page_namespace', 'page_title'
] );
- $join_conds = [ 'page' => [ 'INNER JOIN', 'page_id=rc_cur_id' ] ] + $rcQuery['joins'];
+ $join_conds = [ 'page' => [ 'JOIN', 'page_id=rc_cur_id' ] ] + $rcQuery['joins'];
// Avoid PHP 7.1 warning from passing $this by reference
$pager = $this;
return $this->mForm->formatRow( $row );
}
- function getStartBody() {
+ protected function getStartBody() {
# Do a batch existence check on pages
$linkBatch = new LinkBatch();
foreach ( $this->mResult as $row ) {
return '<ul>';
}
- function getEndBody() {
+ protected function getEndBody() {
return '</ul>';
}
}