Legoktm has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/122279

Change subject: Add user_rights variable
......................................................................

Add user_rights variable

Allows for more broad filters to check for a given right, rather
than having to check against multiple groups, which becomes useful
with global groups.

Bug: 60191
Change-Id: I95b5477d6d868d4b83bcd98e779e6d535aa755b3
---
M AbuseFilter.class.php
M AbuseFilterVariableHolder.php
M i18n/en.json
M i18n/qqq.json
4 files changed, 13 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/AbuseFilter 
refs/changes/79/122279/1

diff --git a/AbuseFilter.class.php b/AbuseFilter.class.php
index 3186ac4..2dff93d 100644
--- a/AbuseFilter.class.php
+++ b/AbuseFilter.class.php
@@ -105,6 +105,7 @@
                        'user_age' => 'user-age',
                        'user_name' => 'user-name',
                        'user_groups' => 'user-groups',
+                       'user_rights' => 'user-rights',
                        'user_blocked' => 'user-blocked',
                        'user_emailconfirm' => 'user-emailconfirm',
                        'old_wikitext' => 'old-text',
@@ -222,6 +223,12 @@
                );
 
                $vars->setLazyLoadVar(
+                       'user_rights',
+                       'simple-user-accessor',
+                       array( 'user' => $user, 'method' => 'getRights' )
+               );
+
+               $vars->setLazyLoadVar(
                        'user_blocked',
                        'simple-user-accessor',
                        array( 'user' => $user, 'method' => 'isBlocked' )
diff --git a/AbuseFilterVariableHolder.php b/AbuseFilterVariableHolder.php
index 9329494..fcc2c00 100644
--- a/AbuseFilterVariableHolder.php
+++ b/AbuseFilterVariableHolder.php
@@ -175,6 +175,7 @@
                        'simple-user-accessor',
                        'user-age',
                        'user-groups',
+                       'user-rights',
                        'revision-text-by-id',
                        'revision-text-by-timestamp'
                );
diff --git a/i18n/en.json b/i18n/en.json
index 567e128..9fd271c 100644
--- a/i18n/en.json
+++ b/i18n/en.json
@@ -1,7 +1,8 @@
 {
     "@metadata": {
         "authors": [
-            "Andrew Garrett"
+            "Andrew Garrett",
+            "Kunal Mehta"
         ]
     },
     "abusefilter-desc": "Applies automatic heuristics to edits",
@@ -259,6 +260,7 @@
     "abusefilter-edit-builder-vars-user-age": "Age of user account",
     "abusefilter-edit-builder-vars-user-name": "Name of user account",
     "abusefilter-edit-builder-vars-user-groups": "Groups (including implicit) 
user is in",
+    "abusefilter-edit-builder-vars-user-rights": "Rights that a user has",
     "abusefilter-edit-builder-vars-user-blocked": "Whether user is blocked",
     "abusefilter-edit-builder-vars-user-emailconfirm": "Time email address was 
confirmed",
     "abusefilter-edit-builder-vars-recent-contributors": "Last ten users to 
contribute to the page",
diff --git a/i18n/qqq.json b/i18n/qqq.json
index f846cda..f3baad4 100644
--- a/i18n/qqq.json
+++ b/i18n/qqq.json
@@ -11,6 +11,7 @@
             "Huji",
             "Iwan Novirion",
             "Jon Harald Søby",
+            "Kunal Mehta",
             "Krenair",
             "Krinkle",
             "Kwj2772",
@@ -282,6 +283,7 @@
     "abusefilter-edit-builder-vars-movedto-text": "Paraphrased: Name of the 
destination of the page that is to be moved. Abuse filter syntax option in a 
dropdown from the group {{msg-mw|abusefilter-edit-builder-group-vars}}.",
     "abusefilter-edit-builder-vars-movedto-prefixedtext": "Paraphrased: Full 
name of the destination of the page that is to be moved. Abuse filter syntax 
option in a dropdown from the group 
{{msg-mw|abusefilter-edit-builder-group-vars}}.",
     "abusefilter-edit-builder-vars-user-groups": "See also:\n* 
{{msg-mw|Abusefilter-edit-builder-vars-global-user-groups}}",
+    "abusefilter-edit-builder-vars-user-rights": "Paraphrased: List of rights 
that the user has. Abuse filter syntax option in a dropdown from the group 
{{msg-mw|abusefilter-edit-builder-group-vars}}.",
     "abusefilter-edit-builder-vars-user-blocked": "Paraphrased: Boolean value 
on whether the user is blocked. Abuse filter syntax option in a dropdown from 
the group {{msg-mw|abusefilter-edit-builder-group-vars}}.",
     "abusefilter-edit-builder-vars-new-pst": "Paraphrased: The output wikitext 
after pre-save transform is applied to new_wikitext. Abuse filter syntax option 
in a dropdown from the group {{msg-mw|abusefilter-edit-builder-group-vars}}.",
     "abusefilter-edit-builder-vars-diff-pst": "Paraphrased: Edit diff of 
new_pst against old_wikitext. Abuse filter syntax option in a dropdown from the 
group {{msg-mw|abusefilter-edit-builder-group-vars}}.",

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I95b5477d6d868d4b83bcd98e779e6d535aa755b3
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/AbuseFilter
Gerrit-Branch: master
Gerrit-Owner: Legoktm <[email protected]>

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

Reply via email to