Edgewall Software

Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#9656 closed defect (fixed)

[PATCH] Strip spaces in usernames when defining permissions — at Version 5

Reported by: erny@… 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.

Change History (6)

comment:1 by Remy Blank, 14 years ago

Keywords: bitesized added
Milestone: → next-minor-0.12.x
Owner: set to Remy Blank

Sounds simple enough. PatchWelcome.

by Mikael Relbe, 14 years ago

Attachment: t9656_strip-user-group-name-10198.patch​ added

Strips user and group names in perm. admin panel, apply on trunk r10198

in reply to:  1 comment:2 by Mikael Relbe, 14 years ago

Cc: mikael@… 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

Replying to rblank:

Sounds simple enough. PatchWelcome.

You're welcome ;)

comment:3 by Remy Blank, 14 years ago

Resolution: → fixed
Status: new → closed

Applied in [10210]. Thanks!

comment:4 by Remy Blank, 14 years ago

Owner: changed from Remy Blank to Mikael Relbe

comment:5 by Christian Boos, 14 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? ;-)

Note: See TracTickets for help on using tickets.