6.0.0-alpha14
7/2/25

[#13367] Minor PHP 5.6 error in Horde/Core/Auth/Ldap->updateUser & ->removeUser
Summary Minor PHP 5.6 error in Horde/Core/Auth/Ldap->updateUser & ->removeUser
Queue Horde Framework Packages
Queue Version Git master
Type Bug
State Resolved
Priority 1. Low
Owners slusarz (at) horde (dot) org
Requester adorman (at) ironicdesign (dot) com
Created 07/14/2014 (4006 days ago)
Due
Updated 07/15/2014 (4005 days ago)
Assigned
Resolved 07/15/2014 (4005 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch No

History
07/15/2014 12:17:19 AM Michael Slusarz Comment #3
Assigned to Michael Slusarz
State ⇒ Resolved
Reply to this comment
Fixed in next version of Horde_Core.
07/15/2014 12:16:51 AM Git Commit Comment #2 Reply to this comment
Changes have been made in Git (master):

commit 1eaddadcc751daaff36a2100d734d482951341b1
Author: Michael M Slusarz <slusarz@horde.org>
Date:   Mon Jul 14 18:16:37 2014 -0600

     Bug #13367: Fix function signatures

  framework/Core/lib/Horde/Core/Auth/Ldap.php |    8 ++++++--
  1 files changed, 6 insertions(+), 2 deletions(-)

http://github.com/horde/horde/commit/1eaddadcc751daaff36a2100d734d482951341b1
07/14/2014 02:47:56 PM adorman (at) ironicdesign (dot) com Comment #1
Priority ⇒ 1. Low
Patch ⇒ No
Milestone ⇒
Queue ⇒ Horde Framework Packages
Summary ⇒ Minor PHP 5.6 error in Horde/Core/Auth/Ldap->updateUser & ->removeUser
Type ⇒ Bug
State ⇒ Unconfirmed
Reply to this comment
I noticed the following PHP errors in debug mode.  I would be happy to 
submit a fix, except I am not sure which file has the "preferred" 
versions of updateUser & removeUser.

2014-07-11T14:27:11.292685-05:00 yorick HORDE: PHP ERROR: Declaration 
of Horde_Core_Auth_Ldap::removeUser() should be compatible with 
Horde_Auth_Ldap::removeUser($userId, $dn = NULL) [pid 9897 on line 17 
of "/usr/share/php/Horde/Core/Auth/Ldap.php"]

2014-07-11T14:27:11.292786-05:00 yorick HORDE: PHP ERROR: Declaration 
of Horde_Core_Auth_Ldap::updateUser() should be compatible with 
Horde_Auth_Ldap::updateUser($oldID, $newID, $credentials, $olddn = 
NULL, $newdn = NULL) [pid 9897 on line 17 of 
"/usr/share/php/Horde/Core/Auth/Ldap.php"]

Saved Queries