forge-joniko/ldc-supportweb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge branch 'fix-sites-update' into 'develop'

Browse source 
Used sync instead of detach/attach for sites and users update

See merge request ldc-web/projets/support-web!56
This commit is contained in:
Vincent BRIDIER 2023-08-14 14:16:56 +00:00
commit 8dac597561
2 changed files with 2 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
app/Http/Controllers/Bo/SiteController.php
View file

@ -129,7 +129,7 @@ class SiteController extends Controller
{
return DB::transaction(function () use ($request, $site) {
$site->fill($request->validated());
$site->users()->attach(collect($request->validated('users'))->pluck('id'));
$site->users()->sync(collect($request->validated('users'))->pluck('id'));
if ($site->saveOrFail()) {
return redirect()
->route('bo.sites.edit', $site)

5
app/Http/Controllers/Bo/UserController.php
View file

@ -120,10 +120,7 @@ class UserController extends Controller
{
return DB::transaction(function () use ($request, $user) {
$user->fill($request->safe()->except(['sites']));
$user->sites()->detach();
if (collect($request->validated('sites'))->pluck('id')) {
$user->sites()->attach(collect($request->validated('sites'))->pluck('id'));
}
$user->sites()->sync(collect($request->validated('sites'))->pluck('id'));
if ($user->saveOrFail()) {
return redirect()
->route('bo.users.edit', $user)