From d264769a1d64016ed49b5fb898cb0ef613c09579 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?C=C3=BCneyt=20=C5=9Eent=C3=BCrk?= Date: Tue, 25 Oct 2022 12:15:57 +0300 Subject: [PATCH] fixed user api enable / disable endpoint issue. --- app/Jobs/Auth/UpdateUser.php | 28 +++++++++++++++------------- 1 file changed, 15 insertions(+), 13 deletions(-) diff --git a/app/Jobs/Auth/UpdateUser.php b/app/Jobs/Auth/UpdateUser.php index 859691e41..3ebc4dc41 100644 --- a/app/Jobs/Auth/UpdateUser.php +++ b/app/Jobs/Auth/UpdateUser.php @@ -89,26 +89,28 @@ class UpdateUser extends Job implements ShouldUpdate } // Can't unassigned company, The company must be assigned at least one user. - $companies = (array) $this->request->get('companies', []); - $user_companies = $this->model->companies()->pluck('id')->toArray(); + if ($this->request->has('companies')) { + $companies = (array) $this->request->get('companies', []); + $user_companies = $this->model->companies()->pluck('id')->toArray(); - $company_diff = array_diff($user_companies, $companies); + $company_diff = array_diff($user_companies, $companies); - if ($company_diff) { - $errors = []; + if ($company_diff) { + $errors = []; - foreach ($company_diff as $company_id) { - $company = Company::withCount('users')->find($company_id); + foreach ($company_diff as $company_id) { + $company = Company::withCount('users')->find($company_id); - if ($company->users_count < 2) { - $errors[] = trans('auth.error.unassigned', ['company' => $company->name]); + if ($company->users_count < 2) { + $errors[] = trans('auth.error.unassigned', ['company' => $company->name]); + } } - } - if ($errors) { - $message = implode('\n', $errors); + if ($errors) { + $message = implode('\n', $errors); - throw new \Exception($message); + throw new \Exception($message); + } } } }