Tickets :: [#10554] Turba LDAP server support test isn't working

6.0.0-alpha14

6/24/25

Summary	Turba LDAP server support test isn't working
Queue	Turba
Queue Version	3.0.9
Type	Bug
State	Resolved
Priority	2. Medium
Owners	jan (at) horde (dot) org
Requester	Michael.Nitsche (at) nitsche (dot) de
Created	09/27/2011 (5019 days ago)
Due
Updated	10/03/2011 (5013 days ago)
Assigned	09/28/2011 (5018 days ago)
Resolved	10/03/2011 (5013 days ago)
Github Issue Link
Github Pull Request
Milestone	3.0.10
Patch	No

10/03/2011 05:40:54 PM	Jan Schneider	Assigned to Jan Schneider State ⇒ Resolved Milestone ⇒ 3.0.10

10/03/2011 05:40:37 PM	Git Commit	Comment #8	Reply to this comment
Changes have been made in Git for this ticket: [jan] Fix LDAP connection test (~~Bug #10554~~). 3 files changed, 4 insertions(+), 1 deletions(-) http://git.horde.org/horde-git/-/commit/f19e588709e7b8462df9316317a0019eb1f1ace4

09/28/2011 08:10:40 PM	Michael (dot) Nitsche (at) nitsche (dot) de	Comment #7 New Attachment: test.php	Reply to this comment
Fix is in line 78 of attached file.

09/28/2011 07:57:45 PM	Michael (dot) Nitsche (at) nitsche (dot) de	Comment #6	Reply to this comment
How about telling us what file you are talking about, to begin with? Turba LDAP server support test obviously is in turba/lib/test.php.

09/28/2011 07:45:08 PM	Jan Schneider	Comment #5	Reply to this comment
How about telling us what file you are talking about, to begin with?

09/28/2011 07:41:39 PM	Michael (dot) Nitsche (at) nitsche (dot) de	Comment #4	Reply to this comment
Adding comments is fixed. Your answer from the screenshot was not helpful though. I don't know what I could tell you more. I told you what the problem is, where it comes from and how to fix it. The function is called without the parameters, which have been collected before. So with no parameters there can be no result. In function appTest() _doConnectionTest() is called without parameters. Fix: - $ret .= $this->_doConnectionTest(); (remove this line) + $ret .= $this->_doConnectionTest($params); (replace with this line)

09/28/2011 01:38:31 PM	Jan Schneider	Comment #3	Reply to this comment
Adding comments is fixed. Your answer from the screenshot was not helpful though.

09/28/2011 09:43:14 AM	Michael (dot) Nitsche (at) nitsche (dot) de	New Attachment: Error.png

09/28/2011 09:16:13 AM	Jan Schneider	Comment #2 State ⇒ Feedback	Reply to this comment
Turba contains 14000 lines of PHP code. Can you be more specific?

09/27/2011 05:14:37 PM	Michael (dot) Nitsche (at) nitsche (dot) de	Comment #1 State ⇒ Unconfirmed Patch ⇒ No Milestone ⇒ Queue ⇒ Turba Summary ⇒ Turba LDAP server support test isn't working Type ⇒ Bug Priority ⇒ 2. Medium	Reply to this comment
In function appTest() _doConnectionTest() is called without parameters. Fix: - $ret .= $this->_doConnectionTest(); + $ret .= $this->_doConnectionTest($params);