Don't use getCheck, because the opts array is reused to generate the query for the...

getCheck checks for presence, not for true/false, so that was why rss feeds behaved as if
deletedOnly was set.

Follow up to r65651
Fixes bug #27546

diff --git a/includes/specials/SpecialContributions.php b/includes/specials/SpecialContributions.php
index 10dbb94..cfe2e01 100644 (file)
--- a/includes/specials/SpecialContributions.php
+++ b/includes/specials/SpecialContributions.php
@@ -58,7 +58,7 @@ class SpecialContributions extends SpecialPage {
                        $this->opts['contribs'] = 'newbie';
                }
 
-               $this->opts['deletedOnly'] = $wgRequest->getCheck( 'deletedOnly' );
+               $this->opts['deletedOnly'] = $wgRequest->getBool( 'deletedOnly' );
 
                if( !strlen( $target ) ) {
                        $wgOut->addHTML( $this->getForm() );
@@ -67,7 +67,7 @@ class SpecialContributions extends SpecialPage {
 
                $this->opts['limit'] = $wgRequest->getInt( 'limit', $wgUser->getOption('rclimit') );
                $this->opts['target'] = $target;
-               $this->opts['topOnly'] = $wgRequest->getCheck( 'topOnly' );
+               $this->opts['topOnly'] = $wgRequest->getBool( 'topOnly' );
 
                $nt = Title::makeTitleSafe( NS_USER, $target );
                if( !$nt ) {