6.0.0-beta1
8/8/25

[#5621] Accounts_Driver initializes wrong defaults
Summary Accounts_Driver initializes wrong defaults
Queue Horde Base
Queue Version HEAD
Type Bug
State Resolved
Priority 1. Low
Owners chuck (at) horde (dot) org
Requester bsr1070 (at) rit (dot) edu
Created 08/10/2007 (6573 days ago)
Due
Updated 08/12/2007 (6571 days ago)
Assigned
Resolved 08/12/2007 (6571 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
08/12/2007 05:17:18 AM Chuck Hagenbuch Comment #3
Assigned to Chuck Hagenbuch
State ⇒ Resolved
Reply to this comment
Committed, thanks!
08/10/2007 09:51:29 PM bsr1070 (at) rit (dot) edu Comment #2 Reply to this comment
I didn't clarify... this is in the LDAP driver for Accounts.  Class 
name Accounts_Driver_ldap
08/10/2007 07:33:50 PM bsr1070 (at) rit (dot) edu Comment #1
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Accounts_Driver initializes wrong defaults
Queue ⇒ Horde Base
New Attachment: account.php.patch Download
State ⇒ Unconfirmed
Reply to this comment
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.

Saved Queries