Tickets :: Comment on [#8091] Perms_UI is storing Auth::listUsers() "Display Name"

* Your Email Address
* Spam protection	Enter the letters below: .__ .__.. ..__..___. \| \[__]\|\/\|\| \| \| \|__/\| \|\| \|\|__\ \|
Comment	> Auth::listUsers() may return something like array('johndoe' => 'John > Doo', 'foo' => 'Foo Master', ...), Perms_UI is completely ignoring > key values as given by listUsers. This leads to problems (at least in > my special case) once you are going to use hasPermission() combined > with user-level permissions. > > > > The attached patch changes Perms_UI to: > > - store the key provided by listUsers() instead of the "Display Name" > > - show the "Display Name" for permissions granted to existing users > > - show "<username>" for unknown users (with quotes, helps to quickly > identify deleted users etc) > > > > Please have a quick crosscheck with other Authentication modules, > just to be sure listUsers() is doing what I'm supposing it to do! > > > > Best regards, > > Thomas Gelf > >
Attachment
Watch this ticket