Allow admin to use any group pronouns

NEW
Unassigned

Status

()

Bugzilla
Query/Bug List
--
enhancement
6 years ago
4 years ago

People

(Reporter: Albert Ting, Unassigned)

Tracking

Details

Attachments

(1 attachment)

v1
583 bytes, patch
Frédéric Buclin
: review-
Details | Diff | Splinter Review
(Reporter)

Description

6 years ago
This may be a "wontfix", but posting a ticket just in case. 

I understand the security concerns of having a user use group pronouns that he is not connected with (bug 417048).  Yet I think an admin should not have such restrictions.  

In my specific case, I maintain a "disabled" group that is only attached to disabled accounts. Hence an admin won't be part of such a group. Yet the query below won't work for an admin.

   "Assignee" "is equal to" "%group.foobar%"

But it's not just my specific case, I can imagine several groups where the admin doesn't want to belong to (.ie groups used for whining), yet the admin may still want to do group pronoun queries.

Updated

6 years ago
Severity: normal → enhancement
(Reporter)

Comment 1

6 years ago
Created attachment 634511 [details] [diff] [review]
v1

Suggested patch
Attachment #634511 - Flags: review?(mkanat)

Comment 2

5 years ago
Technically, the code should make an exception for users with editusers privileges, not admins, as they are the ones who can access editusers.cgi with full power.

Comment 3

5 years ago
Comment on attachment 634511 [details] [diff] [review]
v1

This code seems to be against an older version of Bugzilla. ValidateGroupName() is no longer used in Search.pm.
Attachment #634511 - Flags: review?(mkanat) → review-
You need to log in before you can comment on or make changes to this bug.