| Etonkovidova | |
| Jan 30 2016, 12:59 AM |
| F3304030: Screen Shot 2016-02-02 at 12.49.50 PM.png | |
| Feb 2 2016, 9:12 PM |
| F3304035: Screen Shot 2016-02-02 at 12.51.28 PM.png | |
| Feb 2 2016, 9:12 PM |
| F3304056: Screen Shot 2016-02-02 at 1.06.49 PM.png | |
| Feb 2 2016, 9:12 PM |
| F3292036: Screen Shot 2016-01-29 at 5.39.27 PM.png | |
| Jan 30 2016, 1:43 AM |
| F3292046: Screen Shot 2016-01-29 at 5.33.40 PM.png | |
| Jan 30 2016, 1:43 AM |
| F3291957: Screen Shot 2016-01-29 at 5.07.46 PM.png | |
| Jan 30 2016, 1:14 AM |
| F3291925: Screen Shot 2016-01-29 at 4.43.38 PM.png | |
| Jan 30 2016, 12:59 AM |
Presently, in betalabs,
According to T121661: Clean up lego message in user-rights notification
#2: Groups removed but not added
SINGLE "Your user rights were changed: You are no longer a member of the Administrators group."
PLURAL "Your user rights were changed: You are no longer a member of the Administrators, Checkusers and Oversighters groups.”
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | SBisson | T121661 Clean up lego message in user-rights notification | |||
| Resolved | SBisson | T125279 Removed group rights notification messages do not display group names |
For testing: Check the following
This one indicates that there are problems with adding just one group rights.
I can't reproduce any of those issues locally. Wondering if the issue can be related to the localization of group names.
@Etonkovidova Could you check if a group that appears blank on removal would also appear blank when you add it back? Or could you play with different languages? (?uselang)
The screenshots from my last comment covered that case - re-add group rights after removal. I re-checked it again - if only one group is added - it's displayed. If more than one - the first group is not displayed, second(and other) are fully displayed.
Oh I missed that. So there is a specific group that appears blank in both add and remove cases, right? What is that group please?
Change 267703 had a related patch set uploaded (by Sbisson):
Make sure to use array_values for group names in user-rights notifications
Change 267703 merged by jenkins-bot:
Make sure to use array_values for group names in user-rights notifications
Checked the fix in betalabs.
ET8 was removed from three groups:
ET13 was added to two groups:
The language in the messages in the screenshots is not what's in the spreadsheet, which is as follows:
Your user rights were changed. You have been added to: Oversighters.
Your user rights were changed. You are no longer a member of: Oversighters, Administrators.
Your user rights were changed. You have been added to: Oversighters. You are no longer a member of: X, Y, Z.
Specifically, in the interest of brevity we got rid of the talk of "groups" and the "ands" and the "the"s, etc. (and remember to put a period after the first phrase then start a new sentence).
Unless there was a discussion I'm unaware of (and to avoid reopening discussion if there wasn't), let's please go with the spreadsheet language.
The screenshots above are outdated and are not representative of the current state of those notifications. They were taken after the first draft on T121661 but before the final work and for the purpose of illustrating a bug with some group names not showing up.