lunax/snipe_it
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity Actions

Merge pull request #15911 from Fiala06/patch-1

Browse source 
Fixed duplicate entries preventing LDAP sync from continuing
This commit is contained in:
snipe 2025-03-04 12:49:54 +00:00 • committed by GitHub
commit 6f847294ed
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 5 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
app/Console/Commands/LdapSync.php
View file

@ -434,8 +434,12 @@ class LdapSync extends Command
$item['note'] = $item['createorupdate']; $item['note'] = $item['createorupdate'];
$item['status'] = 'success'; $item['status'] = 'success';
if ($item['createorupdate'] === 'created' && $ldap_default_group) { if ($item['createorupdate'] === 'created' && $ldap_default_group) {
// Check if the relationship already exists
if (!$user->groups()->where('group_id', $ldap_default_group)->exists()) {
$user->groups()->attach($ldap_default_group); $user->groups()->attach($ldap_default_group);
} }
}
//updates assets location based on user's location //updates assets location based on user's location
if ($user->wasChanged('location_id')) { if ($user->wasChanged('location_id')) {
foreach ($user->assets as $asset) { foreach ($user->assets as $asset) {