* (bug 16009) Limit selection forms based on Pager now links to the correct page...

diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index 78f95a0..24b342a 100644 (file)
--- 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 6a8e670..3f96251 100644 (file)
--- 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";
        }