From d2a93643bc2caea57fe10027180123d3d35aa942 Mon Sep 17 00:00:00 2001 From: Andrew Ruthven Date: Wed, 1 May 2024 00:20:07 +1200 Subject: [PATCH] Don't be afraid of whitespace --- inc/drivers_ldap.php | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/inc/drivers_ldap.php b/inc/drivers_ldap.php index 37bed5ca..5be41e3c 100644 --- a/inc/drivers_ldap.php +++ b/inc/drivers_ldap.php @@ -485,10 +485,12 @@ function LDAP_check($username, $password ){ if( $ldap_timestamp <= $db_timestamp ) { return $principal; // no need to update } + // we will need to update the user record + dbg_error_log( "LDAP", "user %s has been modified in LDAP, we need to update our DB", $username ); } else { - dbg_error_log( "LDAP", "user %s doesn't exist in local DB, we need to create it",$username ); + dbg_error_log( "LDAP", "user %s doesn't exist in local DB, we need to create it", $username ); } $principal->setUsername($username); @@ -595,7 +597,7 @@ function sync_LDAP_groups(){ // Groups where nothing was done $groups_nothing_done[] = null; - if ( sizeof ( $groups_to_create ) ){ + if ( sizeof ( $groups_to_create ) ) { $validUserFields = awl_get_fields('usr'); foreach ( $groups_to_create as $k => $group ){ @@ -662,11 +664,11 @@ function sync_LDAP_groups(){ $groups_to_sync_members[] = $group; } - $c->messages[] = sprintf( i18n('- creating groups : %s'), join(', ',$groups_to_create) ); + $c->messages[] = sprintf( i18n('- creating groups : %s'), join(', ', $groups_to_create) ); } if ( sizeof ( $groups_to_sync_members ) ){ - $c->messages[] = sprintf(i18n('- updating groups : %s'),join(', ',$groups_to_sync_members)); + $c->messages[] = sprintf(i18n('- updating groups : %s'), join(', ', $groups_to_sync_members)); // Support the old name of user_mapping_field.. $user_mapping = NULL;