jenkins-bot has submitted this change and it was merged.

Change subject: add missing title-param to abusefilter-log form
......................................................................


add missing title-param to abusefilter-log form

without it, you get redirected to the main page (index.php) when
$wgUsePathInfo = false; as no title-parameter is given.

also don't rely on __tostring.

Change-Id: I6b97d61c58b196c95836036219201cff8e2e0657
---
M Views/AbuseFilterViewExamine.php
M Views/AbuseFilterViewHistory.php
M Views/AbuseFilterViewList.php
M Views/AbuseFilterViewRevert.php
M Views/AbuseFilterViewTestBatch.php
M special/SpecialAbuseLog.php
6 files changed, 7 insertions(+), 6 deletions(-)

Approvals:
  Hoo man: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/Views/AbuseFilterViewExamine.php b/Views/AbuseFilterViewExamine.php
index e191884..04c25be 100644
--- a/Views/AbuseFilterViewExamine.php
+++ b/Views/AbuseFilterViewExamine.php
@@ -40,7 +40,7 @@
 
                $selector .= Xml::buildForm( $selectFields, 
'abusefilter-examine-submit' );
                $selector .= Html::hidden( 'submit', 1 );
-               $selector .= Html::hidden( 'title', $this->getTitle( 'examine' 
)->getPrefixedText() );
+               $selector .= Html::hidden( 'title', $this->getTitle( 'examine' 
)->getPrefixedDBkey() );
                $selector = Xml::tags( 'form',
                        array(
                                'action' => $this->getTitle( 'examine' 
)->getLocalURL(),
diff --git a/Views/AbuseFilterViewHistory.php b/Views/AbuseFilterViewHistory.php
index 7662137..5cc39f5 100644
--- a/Views/AbuseFilterViewHistory.php
+++ b/Views/AbuseFilterViewHistory.php
@@ -53,7 +53,7 @@
                $fields['abusefilter-history-select-user'] = Xml::input( 
'user', 45, $user );
 
                $filterForm = Xml::buildForm( $fields, 
'abusefilter-history-select-submit' );
-               $filterForm .= "\n" . Html::hidden( 'title', $this->getTitle( 
"history/$filter" ) );
+               $filterForm .= "\n" . Html::hidden( 'title', $this->getTitle( 
"history/$filter" )->getPrefixedDBkey() );
                $filterForm = Xml::tags( 'form',
                        array(
                                'action' => $this->getTitle( "history/$filter" 
)->getLocalURL(),
diff --git a/Views/AbuseFilterViewList.php b/Views/AbuseFilterViewList.php
index 463444e..a80d798 100644
--- a/Views/AbuseFilterViewList.php
+++ b/Views/AbuseFilterViewList.php
@@ -112,7 +112,7 @@
                $fields['abusefilter-list-limit'] = $pager->getLimitSelect();
 
                $options = Xml::buildForm( $fields, 
'abusefilter-list-options-submit' );
-               $options .= Html::hidden( 'title', 
$this->getTitle()->getPrefixedText() );
+               $options .= Html::hidden( 'title', 
$this->getTitle()->getPrefixedDBkey() );
                $options = Xml::tags( 'form',
                        array(
                                'method' => 'get',
diff --git a/Views/AbuseFilterViewRevert.php b/Views/AbuseFilterViewRevert.php
index 773fcdc..941fa66 100644
--- a/Views/AbuseFilterViewRevert.php
+++ b/Views/AbuseFilterViewRevert.php
@@ -83,7 +83,7 @@
                        // Add a button down the bottom.
                        $confirmForm =
                                Html::hidden( 'editToken', $user->getEditToken( 
"abusefilter-revert-$filter" ) ) .
-                               Html::hidden( 'title', $this->getTitle( 
"revert/$filter" )->getPrefixedText() ) .
+                               Html::hidden( 'title', $this->getTitle( 
"revert/$filter" )->getPrefixedDBkey() ) .
                                Html::hidden( 'wpPeriodStart', 
$this->origPeriodStart ) .
                                Html::hidden( 'wpPeriodEnd', 
$this->origPeriodEnd ) .
                                Xml::inputLabel(
diff --git a/Views/AbuseFilterViewTestBatch.php 
b/Views/AbuseFilterViewTestBatch.php
index 871ad31..3ee8e58 100644
--- a/Views/AbuseFilterViewTestBatch.php
+++ b/Views/AbuseFilterViewTestBatch.php
@@ -60,7 +60,7 @@
 
                $output .= Xml::buildForm( $selectFields, 
'abusefilter-test-submit' );
 
-               $output .= Html::hidden( 'title', $this->getTitle( 'test' 
)->getPrefixedText() );
+               $output .= Html::hidden( 'title', $this->getTitle( 'test' 
)->getPrefixedDBkey() );
                $output = Xml::tags( 'form',
                        array(
                                'action' => $this->getTitle( 'test' 
)->getLocalURL(),
diff --git a/special/SpecialAbuseLog.php b/special/SpecialAbuseLog.php
index 9d2623f..3aab66a 100644
--- a/special/SpecialAbuseLog.php
+++ b/special/SpecialAbuseLog.php
@@ -126,7 +126,8 @@
 
                $output .= Xml::tags( 'form',
                        array( 'method' => 'get', 'action' => 
$this->getPageTitle()->getLocalURL() ),
-                       Xml::buildForm( $fields, 
'abusefilter-log-search-submit' )
+                       Xml::buildForm( $fields, 
'abusefilter-log-search-submit' ) .
+                       Html::hidden( 'title', 
$this->getPageTitle()->getPrefixedDBkey() )
                );
                $output = Xml::tags( 'fieldset', null, $output );
 

-- 
To view, visit https://gerrit.wikimedia.org/r/181793
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I6b97d61c58b196c95836036219201cff8e2e0657
Gerrit-PatchSet: 5
Gerrit-Project: mediawiki/extensions/AbuseFilter
Gerrit-Branch: master
Gerrit-Owner: Se4598 <[email protected]>
Gerrit-Reviewer: Hoo man <[email protected]>
Gerrit-Reviewer: Jackmcbarn <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: Se4598 <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to