[Tickets #5621] Accounts_Driver initializes wrong defaults

[bugs at bugs.horde.org]

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

Ticket URL: http://bugs.horde.org/ticket/?id=5621
-----------------------------------------------------------------------
 Ticket             | 5621
 Created By         | bsr1070 at rit.edu
 Summary            | Accounts_Driver initializes wrong defaults
 Queue              | Horde Base
 Version            | HEAD
 Type               | Bug
 State              | Unconfirmed
 Priority           | 1. Low
 Owners             | 
+New Attachment     | account.php.patch
-----------------------------------------------------------------------


bsr1070 at rit.edu (2007-08-10 12:33) wrote:

The _getUsername() method performs a boolean check on a 'strip' parameter. 
Although set in the configuration, the constructor provides defaults if no
parameters are supplied and 'strip' is not one of them.  This will cause a
non-fatal PHP notice to be output if the parameter is not supplied by the
caller, unlike the rest.

The solution is to add the 'strip' parameter to the default parameter
array in the constructor.

I have attached a tiny patch that should correct the problem.  I've tested
it on a live Horde install without any issues arising.  Note that the
problem exists in HEAD and the patch is taken against HEAD.