Merge pull request #13322 from uberbrady/fix_ldap_location_3

Fix to ldap_location attribute
This commit is contained in:
snipe 2023-07-18 13:24:59 +01:00 committed by GitHub
commit 6f8c3ca167
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -212,9 +212,12 @@ class LdapSync extends Command
$item['manager'] = $results[$i][$ldap_result_manager][0] ?? '';
$item['location'] = $results[$i][$ldap_result_location][0] ?? '';
$location = Location::firstOrCreate([
'name' => $item['location'],
]);
// ONLY if you are using the "ldap_location" option *AND* you have an actual result
if ($ldap_result_location && $item['location']) {
$location = Location::firstOrCreate([
'name' => $item['location'],
]);
}
$department = Department::firstOrCreate([
'name' => $item['department'],
]);