jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/368553 )
Change subject: Completely disable Special:AbuseFilter/tools for non-maintaners
......................................................................
Completely disable Special:AbuseFilter/tools for non-maintaners
As described in the task, anyone can view the page but do nothing
with it. It doesn't make sense, it should either work or be disabled.
I chose the latter.
Bug: T124096
Change-Id: I7271cd5a0d00d5fdba28ce3974c50ea5dfd0ad9b
---
M includes/Views/AbuseFilterViewTools.php
1 file changed, 34 insertions(+), 33 deletions(-)
Approvals:
Legoktm: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/Views/AbuseFilterViewTools.php
b/includes/Views/AbuseFilterViewTools.php
index 488883c..c862561 100644
--- a/includes/Views/AbuseFilterViewTools.php
+++ b/includes/Views/AbuseFilterViewTools.php
@@ -5,6 +5,11 @@
$out = $this->getOutput();
$user = $this->getUser();
+ if ( !$user->isAllowed( 'abusefilter-modify' ) ) {
+ $out->addWikiMsg( 'abusefilter-mustbeeditor' );
+ return;
+ }
+
// Header
$out->addWikiMsg( 'abusefilter-tools-text' );
@@ -12,43 +17,39 @@
$eval = '';
$eval .= AbuseFilter::buildEditBox( '', 'wpTestExpr' );
- // Only let users with permission actually test it
- if ( $user->isAllowed( 'abusefilter-modify' ) ) {
- $eval .= Xml::tags( 'p', null,
- Xml::element( 'input',
- [
- 'type' => 'button',
- 'id' => 'mw-abusefilter-submitexpr',
- 'value' => $this->msg(
'abusefilter-tools-submitexpr' )->text() ]
- )
- );
- $eval .= Xml::element( 'p', [ 'id' =>
'mw-abusefilter-expr-result' ], ' ' );
- }
+ $eval .= Xml::tags( 'p', null,
+ Xml::element( 'input',
+ [
+ 'type' => 'button',
+ 'id' => 'mw-abusefilter-submitexpr',
+ 'value' => $this->msg(
'abusefilter-tools-submitexpr' )->text() ]
+ )
+ );
+ $eval .= Xml::element( 'p', [ 'id' =>
'mw-abusefilter-expr-result' ], ' ' );
+
$eval = Xml::fieldset( $this->msg( 'abusefilter-tools-expr'
)->text(), $eval );
$out->addHTML( $eval );
$out->addModules( 'ext.abuseFilter.tools' );
- if ( $user->isAllowed( 'abusefilter-modify' ) ) {
- // Hacky little box to re-enable autoconfirmed if it
got disabled
- $rac = '';
- $rac .= Xml::inputLabel(
- $this->msg(
'abusefilter-tools-reautoconfirm-user' )->text(),
- 'wpReAutoconfirmUser',
- 'reautoconfirm-user',
- 45
- );
- $rac .= ' ';
- $rac .= Xml::element(
- 'input',
- [
- 'type' => 'button',
- 'id' =>
'mw-abusefilter-reautoconfirmsubmit',
- 'value' => $this->msg(
'abusefilter-tools-reautoconfirm-submit' )->text()
- ]
- );
- $rac = Xml::fieldset( $this->msg(
'abusefilter-tools-reautoconfirm' )->text(), $rac );
- $out->addHTML( $rac );
- }
+ // Hacky little box to re-enable autoconfirmed if it got
disabled
+ $rac = '';
+ $rac .= Xml::inputLabel(
+ $this->msg( 'abusefilter-tools-reautoconfirm-user'
)->text(),
+ 'wpReAutoconfirmUser',
+ 'reautoconfirm-user',
+ 45
+ );
+ $rac .= ' ';
+ $rac .= Xml::element(
+ 'input',
+ [
+ 'type' => 'button',
+ 'id' => 'mw-abusefilter-reautoconfirmsubmit',
+ 'value' => $this->msg(
'abusefilter-tools-reautoconfirm-submit' )->text()
+ ]
+ );
+ $rac = Xml::fieldset( $this->msg(
'abusefilter-tools-reautoconfirm' )->text(), $rac );
+ $out->addHTML( $rac );
}
}
--
To view, visit https://gerrit.wikimedia.org/r/368553
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7271cd5a0d00d5fdba28ce3974c50ea5dfd0ad9b
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/AbuseFilter
Gerrit-Branch: master
Gerrit-Owner: Matěj Suchánek <matejsuchane...@gmail.com>
Gerrit-Reviewer: Bartosz Dziewoński <matma....@gmail.com>
Gerrit-Reviewer: Huji <huji.h...@gmail.com>
Gerrit-Reviewer: Jackmcbarn <jackmcb...@gmail.com>
Gerrit-Reviewer: Legoktm <lego...@member.fsf.org>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
