From 595dad0f18c0ef042f2dacc1439e250b1f1b10e5 Mon Sep 17 00:00:00 2001 From: Godfrey M Date: Sat, 16 Mar 2024 19:51:40 -0700 Subject: [PATCH 1/4] updates all assets assigned to user with new location --- app/Console/Commands/LdapSync.php | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/app/Console/Commands/LdapSync.php b/app/Console/Commands/LdapSync.php index 05f60ee4e..5b7a59f32 100755 --- a/app/Console/Commands/LdapSync.php +++ b/app/Console/Commands/LdapSync.php @@ -2,6 +2,7 @@ namespace App\Console\Commands; +use App\Models\Asset; use App\Models\Department; use App\Models\Group; use Illuminate\Console\Command; @@ -390,6 +391,10 @@ class LdapSync extends Command $user->location_id = $location->id; } } + $assets = Asset::whereColumn('assigned_to', '=', $user->id)->get(); + foreach($assets as $asset){ + $asset->location = $user->location_id; + } $user->ldap_import = 1; From 05a8d3030b15a7da1f71604d2a775c0e3a29e9c1 Mon Sep 17 00:00:00 2001 From: Godfrey M Date: Sat, 16 Mar 2024 19:54:48 -0700 Subject: [PATCH 2/4] correct column name --- app/Console/Commands/LdapSync.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/Console/Commands/LdapSync.php b/app/Console/Commands/LdapSync.php index 5b7a59f32..a26c15747 100755 --- a/app/Console/Commands/LdapSync.php +++ b/app/Console/Commands/LdapSync.php @@ -393,7 +393,7 @@ class LdapSync extends Command } $assets = Asset::whereColumn('assigned_to', '=', $user->id)->get(); foreach($assets as $asset){ - $asset->location = $user->location_id; + $asset->location_id = $user->location_id; } $user->ldap_import = 1; From 866f0a9f561c7fc93335b55c2728e52a6448a5e0 Mon Sep 17 00:00:00 2001 From: Godfrey M Date: Tue, 19 Mar 2024 11:58:49 -0700 Subject: [PATCH 3/4] syncs assets location to users location --- app/Console/Commands/LdapSync.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/Console/Commands/LdapSync.php b/app/Console/Commands/LdapSync.php index a26c15747..34eb4642b 100755 --- a/app/Console/Commands/LdapSync.php +++ b/app/Console/Commands/LdapSync.php @@ -391,9 +391,11 @@ class LdapSync extends Command $user->location_id = $location->id; } } - $assets = Asset::whereColumn('assigned_to', '=', $user->id)->get(); + //updates assets location based on user's location + $assets = Asset::where('assigned_to', '=', $user->id)->get(); foreach($assets as $asset){ $asset->location_id = $user->location_id; + $asset->save(); } $user->ldap_import = 1; From c62a022252b88e03379a0c4f1baa4ee8434c6570 Mon Sep 17 00:00:00 2001 From: Godfrey M Date: Thu, 21 Mar 2024 14:34:47 -0700 Subject: [PATCH 4/4] replaced for loop with eloquent query update --- app/Console/Commands/LdapSync.php | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/app/Console/Commands/LdapSync.php b/app/Console/Commands/LdapSync.php index 34eb4642b..7fcbc8eb0 100755 --- a/app/Console/Commands/LdapSync.php +++ b/app/Console/Commands/LdapSync.php @@ -392,11 +392,7 @@ class LdapSync extends Command } } //updates assets location based on user's location - $assets = Asset::where('assigned_to', '=', $user->id)->get(); - foreach($assets as $asset){ - $asset->location_id = $user->location_id; - $asset->save(); - } + Asset::where('assigned_to', '=', $user->id)->update(['location_id' => $user->location_id]); $user->ldap_import = 1;