#9656 closed defect (fixed)
[PATCH] Strip spaces in usernames when defining permissions
| Reported by: | Owned by: | Mikael Relbe | |
|---|---|---|---|
| Priority: | normal | Milestone: | 0.12.2 |
| Component: | admin/web | Version: | 0.12-stable |
| Severity: | normal | Keywords: | permissions, bitesized |
| Cc: | mikael@… | Branch: | |
| Release Notes: |
admin: in permissions panel, strip whitespace around user and group names |
||
| API Changes: | |||
| Internal Changes: | |||
Description
When adding user to groups in /admin/general/perm spaces at the end of usernames are not stripped. The effect is that the indented user has not the correct permissions assigned.
Attachments (1)
Change History (9)
follow-up: 2 comment:1 by , 15 years ago
| Keywords: | bitesized added |
|---|---|
| Milestone: | → next-minor-0.12.x |
| Owner: | set to |
by , 15 years ago
| Attachment: | t9656_strip-user-group-name-10198.patch added |
|---|
Strips user and group names in perm. admin panel, apply on trunk r10198
comment:2 by , 15 years ago
| Cc: | added |
|---|---|
| Milestone: | next-minor-0.12.x → 0.12.2 |
| Summary: | Strip spaces in usernames when defining permissions → [PATCH] Strip spaces in usernames when defining permissions |
comment:4 by , 15 years ago
| Owner: | changed from to |
|---|
comment:5 by , 15 years ago
| Release Notes: | modified (diff) |
|---|---|
| Type: | enhancement → defect |
Requalifying as bug fix, as it was borderline anyway and we said no enhancements on 0.12.2, yesterday, right? ;-)
follow-ups: 7 8 comment:6 by , 15 years ago
Replying to erny@…:
The effect is that the indented user has not the correct permissions assigned.
Perhaps because the indented user was not the intended one? ;-)
comment:7 by , 15 years ago
Replying to cboos:
Perhaps because the indented user was not the intended one? ;-)
That caused me an IRL laugh, thanks!
I also realize that I made a mistake in comment:2: I addressed the ticket to 0.12.2, which corresponds to branches/0.12-stable, but the patch itself was for trunk… Pure luck this time, I'll try to behave in the future :)
Sounds simple enough. PatchWelcome.