Added scoping for destroy
Signed-off-by: snipe <snipe@snipe.net>
This commit is contained in:
parent
ed0a441e4d
commit
710370ac24
1 changed files with 32 additions and 27 deletions
|
@ -513,7 +513,11 @@ class UsersController extends Controller
|
||||||
public function destroy($id)
|
public function destroy($id)
|
||||||
{
|
{
|
||||||
$this->authorize('delete', User::class);
|
$this->authorize('delete', User::class);
|
||||||
$user = User::findOrFail($id);
|
$user = User::with('assets', 'assets.model', 'consumables', 'accessories', 'licenses', 'userloc')->withTrashed();
|
||||||
|
$user = Company::scopeCompanyables($user)->find($id);
|
||||||
|
|
||||||
|
if ($user) {
|
||||||
|
|
||||||
$this->authorize('delete', $user);
|
$this->authorize('delete', $user);
|
||||||
|
|
||||||
if (($user->assets) && ($user->assets->count() > 0)) {
|
if (($user->assets) && ($user->assets->count() > 0)) {
|
||||||
|
@ -545,6 +549,7 @@ class UsersController extends Controller
|
||||||
|
|
||||||
return response()->json(Helper::formatStandardApiResponse('success', null, trans('admin/users/message.success.delete')));
|
return response()->json(Helper::formatStandardApiResponse('success', null, trans('admin/users/message.success.delete')));
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
|
||||||
return response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/users/message.error.delete')));
|
return response()->json(Helper::formatStandardApiResponse('error', null, trans('admin/users/message.error.delete')));
|
||||||
}
|
}
|
||||||
|
|
Loading…
Add table
Reference in a new issue