From 718961733432d7df8f0c2891abc81f4186b09fe4 Mon Sep 17 00:00:00 2001
From: Andrew McMillan <debian@mcmillan.net.nz>
Date: Sun, 4 Nov 2007 21:41:20 +1300
Subject: [PATCH] in_array looks for values, but we need to look in the keys
 now.

---
 inc/drivers_ldap.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/inc/drivers_ldap.php b/inc/drivers_ldap.php
index d3218013..ba7be4cc 100644
--- a/inc/drivers_ldap.php
+++ b/inc/drivers_ldap.php
@@ -170,7 +170,7 @@ function sync_user_from_LDAP( &$usr, $mapping, $ldap_values ) {
   $validUserFields = get_fields('usr');
 
   foreach ( $c->authenticate_hook['config']['default_value'] as $field => $value ) {
-    if ( in_array($field, $validUserFields) ) {
+    if ( isset($validUserFields[$field]) ) {
       $usr->{$field} =  $value;
       dbg_error_log( "LDAP", "Setting usr->%s to %s from configured defaults", $field, $value );
     }
@@ -178,7 +178,7 @@ function sync_user_from_LDAP( &$usr, $mapping, $ldap_values ) {
 
   foreach ( $mapping as $field => $value ) {
     dbg_error_log( "LDAP", "Considering copying %s", $field );
-    if ( in_array($field, $validUserFields) ) {
+    if ( isset($validUserFields[$field]) ) {
       $usr->{$field} =  $ldap_values[$value];
       dbg_error_log( "LDAP", "Setting usr->%s to %s from LDAP field %s", $field, $ldap_values[$value], $value );
     }