Simplify the code by assigning the array after we've cleaned it up.

This allows shorter lines which means easier to understand code.
This commit is contained in:
Andrew Ruthven 2024-02-03 20:37:16 +13:00
parent 2a30a9646c
commit 9d40ae1aa7

View File

@ -524,17 +524,16 @@ function sync_LDAP_groups(){
foreach ($ldap_groups_tmp as $key => $ldap_group) {
$group_name = $ldap_group[$mapping['name']];
$ldap_groups_info[$group_mapping] = $ldap_group;
if ( isset($ldap_groups_info[$group_mapping][$member_field]) ) {
if ( is_array($ldap_groups_info[$group_mapping][$member_field]) ) {
unset( $ldap_groups_info[$group_mapping][$member_field]['count'] );
}
else {
$ldap_groups_info[$group_mapping][$member_field]
= array($ldap_groups_info[$group_mapping][$member_field]);
if ( isset($ldap_group[$member_field]) ) {
if ( is_array($ldap_group[$member_field]) ) {
unset( $ldap_group[$member_field]['count'] );
} else {
$ldap_group[$member_field] = array($ldap_group[$member_field]);
}
}
$ldap_groups_info[$group_name] = $ldap_group;
unset($ldap_groups_tmp[$key]);
}