Occasionally I get the debug window appearing - but this is no longer needed. I seem to remember turning it on - but don't know how to turn it off.

I've attached a screenshot to make it clear what is shown.

CommentFileSizeAuthor
#6 commentout_debug_line-1710032-6.patch902 bytesmd2
#5 login-ldap-debug.jpg298.2 KBddalquist
debug.png59.24 KBbailey86

Comments

johnbarclay’s picture

johnbarclay commented

its in the first tab. the overall ldap settings where encryption etc are. You can also just turn off the ldap_help module which drives it. There is no need for it to be on unless you are using it. Glad the debug was helpful to you.

bailey86’s picture

bailey86 commented

Thanks for that - I seem to remember I turned it of. One of the beta testers reported the debug message - but since I can't replicate it I'll wait to see if it happens again.

johnbarclay’s picture

johnbarclay commented
Title: Turn off debug window » LDAP Authorization: remove debug statment
Version: 7.x-1.0-beta11 » 7.x-1.x-dev
Assigned: Unassigned » johnbarclay
Category: support » bug
Status: Active » Fixed

I ran across this and fixed it. I throw debug statements in to debug the simpletests and missed these 2 in the commit.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

ddalquist’s picture

ddalquist commented
Version: 7.x-1.x-dev » 7.x-2.0-beta5
Priority: Normal » Minor
Status: Closed (fixed) » Active
StatusFileSize
new login-ldap-debug.jpg298.2 KB

My test users also receive a debug statement on login only, despite my first unchecking "Enabled Detailed LDAP Watchdog logging..." and then unchecking the "LDAP Help" module. All cache were cleared afterward w/Drush; however, the debug statement remains.
(It is beta....Thank you for all your work on this module.)

md2’s picture

md2 commented
Component: User interface » Code
StatusFileSize
new commentout_debug_line-1710032-6.patch902 bytes

The following patch should fix the issue. It is caused by a debug line being left in on line 556 of ldap_authorization/ldap_authorization_og/LdapAuthorizationConsumerOG.class.php

~Mark

katannshaw’s picture

katannshaw commented

I agree that this is a great module. Thanks for the fix md2. #6 solved this issue for me.

johnbarclay’s picture

johnbarclay commented
Version: 7.x-2.0-beta5 » 7.x-2.x-dev
Status: Active » Reviewed & tested by the community
geerlingguy’s picture

geerlingguy commented
Status: Reviewed & tested by the community » Closed (fixed)

This has already been fixed in dev (see: #2024599: Ldap Authorization OG: Fix to debug logging).