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 |
Assigned to Chuck Hagenbuch
State ⇒ Resolved
name Accounts_Driver_ldap
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Accounts_Driver initializes wrong defaults
Queue ⇒ Horde Base
New Attachment: account.php.patch
State ⇒ Unconfirmed
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.