From: Alexandre Emsenhuber <ialex@users.mediawiki.org>
Date: Sun, 30 Aug 2009 19:41:18 +0000 (+0000)
Subject: Use $wgScript rather than $this->getTitle()->getLocalURL() for the action parameter... 
X-Git-Tag: 1.31.0-rc.0~39998
X-Git-Url: https://git.cyclocoop.org/%7B%24www_url%7Dadmin/membres/message.php?a=commitdiff_plain;h=9f64844e05a2501feaec90f3aed86d08e04eda69;p=lhc%2Fweb%2Fwiklou.git

Use $wgScript rather than $this->getTitle()->getLocalURL() for the action parameter, since the title parameter is already present in the hidden fields and it doesn't work with ugly URLS
Reported by MaxSem on IRC.
---

diff --git a/includes/specials/SpecialListfiles.php b/includes/specials/SpecialListfiles.php
index dbfdbd5caa..ae19ea9ad7 100644
--- a/includes/specials/SpecialListfiles.php
+++ b/includes/specials/SpecialListfiles.php
@@ -158,10 +158,10 @@ class ImageListPager extends TablePager {
 	}
 
 	function getForm() {
-		global $wgRequest, $wgMiserMode;
+		global $wgRequest, $wgScript, $wgMiserMode;
 		$search = $wgRequest->getText( 'ilsearch' );
 
-		$s = Xml::openElement( 'form', array( 'method' => 'get', 'action' => $this->getTitle()->getLocalURL(), 'id' => 'mw-listfiles-form' ) ) .
+		$s = Xml::openElement( 'form', array( 'method' => 'get', 'action' => $wgScript, 'id' => 'mw-listfiles-form' ) ) .
 			Xml::openElement( 'fieldset' ) .
 			Xml::element( 'legend', null, wfMsg( 'listfiles' ) ) .
 			Xml::tags( 'label', null, wfMsgHtml( 'table_pager_limit', $this->getLimitSelect() ) );