[Bug 19362] New user permission abusefilter-viewprivate

2009-09-18 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=19362


Andrew Garrett  changed:

   What|Removed |Added

 Status|REOPENED|RESOLVED
 Resolution||FIXED




--- Comment #10 from Andrew Garrett   2009-09-18 
13:01:53 UTC ---
(In reply to comment #9)
> I've just removed my abuse-filter-editor permission and find I can't view
> private filters now.

That's because it hasn't been given to administrators on any wikis yet.

If you want to give it to administrators on a particular wiki, please file a
site request bug saying so.


-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 19362] New user permission abusefilter-viewprivate

2009-09-18 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=19362


xenocidic  changed:

   What|Removed |Added

 CC||xenow...@gmail.com
 Status|RESOLVED|REOPENED
 Resolution|FIXED   |




--- Comment #9 from xenocidic   2009-09-18 13:00:55 UTC ---
I've just removed my abuse-filter-editor permission and find I can't view
private filters now.


-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 19362] New user permission abusefilter-viewprivate

2009-07-03 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=19362


Andrew Garrett  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Comment #8 from Andrew Garrett   2009-07-03 
14:17:29 UTC ---
Committed with adjustments in r52743.


-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 19362] New user permission abusefilter-viewprivate

2009-06-30 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=19362


Cenarium  changed:

   What|Removed |Added

 CC||cenarium.sy...@gmail.com




--- Comment #7 from Cenarium   2009-06-30 21:29:47 
UTC ---
Per the discussion at [[Wikipedia talk:Abuse filter#Adding
abuse-filter-view-private into sysop package]], we would like this permission
into the sysop package, but no separate user group. Thanks.


-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 19362] New user permission abusefilter-viewprivate

2009-06-30 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=19362


Haza-w  changed:

   What|Removed |Added

Attachment #6255 is|0   |1
   obsolete||




--- Comment #6 from Haza-w   2009-06-30 21:08:59 UTC ---
Created an attachment (id=6288)
 --> (https://bugzilla.wikimedia.org/attachment.cgi?id=6288)
abusefilter-view-private user right

Created canView function and changed the static variable declaration slightly.

I'm assuming this works... perhaps someone could test it? 


-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 19362] New user permission abusefilter-viewprivate

2009-06-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=19362





--- Comment #5 from Andrew Garrett   2009-06-23 
12:45:30 UTC ---
(In reply to comment #4)
> Apologies for making such a mess of the attachment... Bugzilla and I are still
> not quite acquainted!
> 
> This is a second version of the patch – I suddenly realised that I hadn't
> added any language variables for the new user right.
> 

Since we're both online at the same time, it might make sense for you to drop
into #mediawiki and I'll do a live review with you.


-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 19362] New user permission abusefilter-viewprivate

2009-06-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=19362





--- Comment #4 from Haza-w   2009-06-23 12:44:12 UTC ---
Created an attachment (id=6255)
 --> (https://bugzilla.wikimedia.org/attachment.cgi?id=6255)
abusefilter-view-private user right, added English language variables

Apologies for making such a mess of the attachment... Bugzilla and I are still
not quite acquainted!

This is a second version of the patch – I suddenly realised that I hadn't
added any language variables for the new user right.


-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 19362] New user permission abusefilter-viewprivate

2009-06-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=19362





--- Comment #3 from Andrew Garrett   2009-06-23 
12:41:03 UTC ---
-   if (isset($row->af_hidden) && $row->af_hidden &&
!$this->canEdit()) {
+   if (isset($row->af_hidden) && $row->af_hidden &&
!($this->canEdit() || $wgUser->isAllowed( 'abusefilter view-private' ))) {

Would prefer that this was broken out to a canView() method, instead of hacked
in here.


-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 19362] New user permission abusefilter-viewprivate

2009-06-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=19362


Haza-w  changed:

   What|Removed |Added

Attachment #6254 is|0   |1
   obsolete||




--- Comment #2 from Haza-w   2009-06-23 12:39:05 UTC ---
(From update of attachment 6254)
>Index: AbuseFilter/AbuseFilter.php
>===
>--- AbuseFilter/AbuseFilter.php(revision 52305)
>+++ AbuseFilter/AbuseFilter.php(working copy)
>@@ -78,6 +78,7 @@
> $wgAvailableRights[] = 'abusefilter-private';
> $wgAvailableRights[] = 'abusefilter-modify-restricted';
> $wgAvailableRights[] = 'abusefilter-revert';
>+$wgAvailableRights[] = 'abusefilter-view-private';
> 
> $wgLogTypes[] = 'abusefilter';
> $wgLogNames['abusefilter']  = 'abusefilter-log-name';
>Index: AbuseFilter/Views/AbuseFilterViewEdit.php
>===
>--- AbuseFilter/Views/AbuseFilterViewEdit.php  (revision 52305)
>+++ AbuseFilter/Views/AbuseFilterViewEdit.php  (working copy)
>@@ -220,7 +220,7 @@
>   $wgOut->setSubtitle( wfMsg( 'abusefilter-edit-subtitle', 
> $filter, $history_id ) );
> 
>   // Hide hidden filters.
>-  if (isset($row->af_hidden) && $row->af_hidden && 
>!$this->canEdit()) {
>+  if (isset($row->af_hidden) && $row->af_hidden && 
>!($this->canEdit() || $wgUser->isAllowed( 'abusefilter-view-private' ))) {
>   return wfMsg( 'abusefilter-edit-denied' );
>   }
> 


-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l


[Bug 19362] New user permission abusefilter-viewprivate

2009-06-23 Thread bugzilla-daemon
https://bugzilla.wikimedia.org/show_bug.cgi?id=19362





--- Comment #1 from Haza-w   2009-06-23 12:38:09 UTC ---
(From update of attachment 6254)
Index: AbuseFilter/AbuseFilter.php
===
--- AbuseFilter/AbuseFilter.php (revision 52305)
+++ AbuseFilter/AbuseFilter.php (working copy)
@@ -78,6 +78,7 @@
 $wgAvailableRights[] = 'abusefilter-private';
 $wgAvailableRights[] = 'abusefilter-modify-restricted';
 $wgAvailableRights[] = 'abusefilter-revert';
+$wgAvailableRights[] = 'abusefilter-view-private';

 $wgLogTypes[] = 'abusefilter';
 $wgLogNames['abusefilter']  = 'abusefilter-log-name';
Index: AbuseFilter/Views/AbuseFilterViewEdit.php
===
--- AbuseFilter/Views/AbuseFilterViewEdit.php   (revision 52305)
+++ AbuseFilter/Views/AbuseFilterViewEdit.php   (working copy)
@@ -220,7 +220,7 @@
$wgOut->setSubtitle( wfMsg( 'abusefilter-edit-subtitle',
$filter, $history_id ) );

// Hide hidden filters.
-   if (isset($row->af_hidden) && $row->af_hidden &&
!$this->canEdit()) {
+   if (isset($row->af_hidden) && $row->af_hidden &&
!($this->canEdit() || $wgUser->isAllowed( 'abusefilter-view-private' ))) {
return wfMsg( 'abusefilter-edit-denied' );
}

Index: AbuseFilter/AbuseFilter.i18n.php
===
--- AbuseFilter/AbuseFilter.i18n.php(revision 52305)
+++ AbuseFilter/AbuseFilter.i18n.php(working copy)
@@ -61,6 +61,7 @@
'right-abusefilter-private' => 'View private data in the abuse log',
'right-abusefilter-modify-restricted' => 'Modify abuse filters with
restricted actions',
'right-abusefilter-revert' => 'Revert all changes by a given abuse
filter',
+   'right-abusefilter-view-private' => 'View abuse filters marked as
private',

'action-abusefilter-modify' => 'modify abuse filters',
'action-abusefilter-view' => 'view abuse filters',
@@ -69,6 +70,7 @@
'action-abusefilter-private' => 'view private data in the abuse log',
'action-abusefilter-modify-restricted' => 'modify abuse filters with
restricted actions',
'action-abusefilter-revert' => 'revert all changes by a given abuse
filter',
+   'action-abuserfilter-view-private' => 'view abuse filters marked as
private',

// Abuse Log
'abusefilter-log' => 'Abuse filter log',


-- 
Configure bugmail: https://bugzilla.wikimedia.org/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Wikibugs-l mailing list
Wikibugs-l@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l