Merge pull request #15889 from Godmartinz/Fix-user-API-location-update

Fixed Users API `update` from clearing `location_id` unnecessarily

Author: snipe

app/Http/Controllers/Api/UsersController.php

@@ -480,10 +480,11 @@ public function update(SaveUserRequest $request, User $user): JsonResponse
                 $user->permissions = $permissions_array;
             }
 
-            // Update the location of any assets checked out to this user
-            Asset::where('assigned_type', User::class)
-                ->where('assigned_to', $user->id)->update(['location_id' => $request->input('location_id', null)]);
-
+            if($request->has('location_id')) {
+                // Update the location of any assets checked out to this user
+                Asset::where('assigned_type', User::class)
+                    ->where('assigned_to', $user->id)->update(['location_id' => $request->input('location_id', null)]);
+            }
             app('App\Http\Requests\ImageUploadRequest')->handleImages($user, 600, 'image', 'avatars', 'avatar');
 
             if ($user->save()) {