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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

johnbarclay’s picture

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

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

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

Version: 7.x-1.x-dev » 7.x-2.0-beta5
Priority: Normal » Minor
Status: Closed (fixed) » Active
FileSize
298.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

Component: User interface » Code
FileSize
902 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

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

johnbarclay’s picture

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

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

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