refactor: use mass assignment for user settings

This commit is contained in:
Roardom
2023-06-23 07:42:49 +00:00
parent d9ba25951a
commit 4d16bc08df
9 changed files with 47 additions and 43 deletions
+14 -18
View File
@@ -18,6 +18,7 @@ use App\Models\PrivateMessage;
use App\Models\User;
use App\Rules\EmailBlacklist;
use Illuminate\Http\Request;
use Illuminate\Validation\Rule;
class EmailController extends Controller
{
@@ -32,25 +33,20 @@ class EmailController extends Controller
abort_if($changedByStaff && ! $request->user()->group->is_owner && $request->user()->group->level < $user->group->level, 403);
if (config('email-blacklist.enabled')) {
$request->validate([
'email' => [
'required',
'string',
'email',
'max:70',
'unique:users',
new EmailBlacklist(),
],
]);
} else {
$request->validate([
'email' => 'required|string|email|max:70|unique:users',
]);
}
$request->validate([
'email' => [
'required',
'string',
'email',
'max:70',
'unique:users',
Rule::when(config('email-blacklist.enabled'), fn () => new EmailBlacklist()),
],
]);
$user->email = $request->email;
$user->save();
$user->update([
'email' => $request->email,
]);
if ($changedByStaff) {
PrivateMessage::create([