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

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/8091
------------------------------------------------------------------------------
  Ticket             | 8091
  Created By         | thomas at gelf.net
  Summary            | Perms_UI is storing Auth::listUsers() "Display Name"
  Queue              | Horde Framework Packages
  Version            | HEAD
  Type               | Bug
  State              | Unconfirmed
  Priority           | 1. Low
  Milestone          |
  Patch              | 1
  Owners             |
+New Attachment     | Perms_UI-userkey.patch
------------------------------------------------------------------------------


thomas at gelf.net (2009-03-17 07:38) wrote:

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