Thijs/wiki.techinc.nl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Removed "feed" parameter to query string added to feed links.

Browse source 
The "feed" parameter was set two times in the query string of feed links and thus was breaking the links.
Also removed a trailing whitespace pointed out by Krinkle in my last commit.

Change-Id: Ia210e6e998acf29cb0a0fad5b6f7f8d6062998d2
This commit is contained in:
Alexandre Emsenhuber 2012-04-04 22:51:51 +02:00
parent f94b0edb48
commit f51c4da96c
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
includes/specials/SpecialNewpages.php
View file

@ -143,7 +143,9 @@ class SpecialNewpages extends IncludableSpecialPage {
return $this->feed( $feedType );
}
$out->setFeedAppendQuery( wfArrayToCGI( $this->opts->getAllValues() ) );
$allValues = $this->opts->getAllValues();
unset( $allValues['feed'] );
$out->setFeedAppendQuery( wfArrayToCGI( $allValues ) );
}
$pager = new NewPagesPager( $this, $this->opts );
@ -370,7 +372,7 @@ class SpecialNewpages extends IncludableSpecialPage {
$oldTitleText = '';
$oldTitle = Title::makeTitle( $result->rc_namespace, $result->rc_title );
if ( !$title->equals( $oldTitle ) ) {
$oldTitleText = $this->msg( 'rc-old-title' )->params( $oldTitle->getPrefixedText() )->escaped();
$oldTitleText = $this->msg( 'rc-old-title' )->params( $oldTitle->getPrefixedText() )->escaped();
}
return "<li{$css}>{$time} {$dm}{$plink} {$hist} {$dm}{$length} {$dm}{$ulink} {$comment} {$tagDisplay} {$oldTitleText}</li>\n";