From: Alexandre Emsenhuber <ialex@users.mediawiki.org>
Date: Thu, 23 Jul 2009 20:18:34 +0000 (+0000)
Subject: * (bug 16009) Limit selection forms based on Pager now links to the correct page... 
X-Git-Tag: 1.31.0-rc.0~40758
X-Git-Url: http://git.cyclocoop.org/%40spipnet%40?a=commitdiff_plain;h=4eea7755ae855743975fed9d4ce390d106d49582;p=lhc%2Fweb%2Fwiklou.git

* (bug 16009) Limit selection forms based on Pager now links to the correct page when using long urls
---

diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index 78f95a03a6..24b342afa2 100644
--- a/RELEASE-NOTES
+++ b/RELEASE-NOTES
@@ -322,6 +322,8 @@ this. Was used when mwEmbed was going to be an extension.
   target page
 * (bug 16877) Moving a page over a redirect no longer leaves an orphan entry in
   the recentchanges table
+* (bug 16009) Limit selection forms based on Pager now links to the correct page
+  when using long urls 
 
 == API changes in 1.16 ==
 
diff --git a/includes/Pager.php b/includes/Pager.php
index 6a8e670e68..3f96251077 100644
--- a/includes/Pager.php
+++ b/includes/Pager.php
@@ -904,14 +904,15 @@ abstract class TablePager extends IndexPager {
 	 * Get a form containing a limit selection dropdown
 	 */
 	function getLimitForm() {
+		global $wgScript;
+
 		# Make the select with some explanatory text
-		$url = $this->getTitle()->escapeLocalURL();
 		$msgSubmit = wfMsgHtml( 'table_pager_limit_submit' );
 		return
-			"<form method=\"get\" action=\"$url\">" .
+			Xml::openElement( 'form', array( 'method' => 'get', 'action' => $wgScript ) ) . "\n" .		
 			wfMsgHtml( 'table_pager_limit', $this->getLimitSelect() ) .
 			"\n<input type=\"submit\" value=\"$msgSubmit\"/>\n" .
-			$this->getHiddenFields( array('limit','title') ) .
+			$this->getHiddenFields( array( 'limit' ) ) .
 			"</form>\n";
 	}