Laravel修改用户邮箱提示‘所选邮箱无效’的问题排查

核心问题：验证规则错误 #

你在更新方法里用的exists:users,email规则完全搞错了用途：

• exists规则是强制要求提交的邮箱必须已经存在于users表中，但用户更新邮箱时，大概率是要改成一个新的、未被其他用户使用的邮箱，这时候这个规则就会直接触发错误。
• 创建时用unique:users,email是正确的，确保新用户邮箱未被注册；更新时需要的是允许用户保留原邮箱，或者改成未被其他用户占用的新邮箱，这时候应该用带排除条件的unique规则。

正确的邮箱验证规则 #

字符串写法 #

'email' => 'required|email|unique:users,email,' . $user->id,

这里的$user->id是当前要更新的用户ID，unique规则会排除这个ID对应的邮箱，确保只有当前用户能使用自己的邮箱，其他用户不能重复。

规则对象写法（更清晰，推荐） #

需要先引入Rule类：

use Illuminate\Validation\Rule;

然后修改验证规则：

'email' => [
    'required',
    'email',
    Rule::unique('users')->ignore($user->id),
]

其他代码问题修复 #

1. 事务逻辑错误
   你当前的finally块里会无条件执行DB::commit()，如果try块抛出异常，已经执行了DB::rollBack()，再commit会导致事务状态混乱。正确的写法是把DB::commit()放在try块的末尾，只有操作成功才提交：

DB::beginTransaction();
try {
    $updateData = [
        'name' => $request->name,
        'email' => $request->email,
        'avatar' => parse_url($request->avatar)['path'],
    ];
    // 仅当密码字段有值时才更新
    if ($request->filled('password')) {
        $updateData['password'] = Hash::make($request->password);
    }
    
    $user->update($updateData);
    $user->syncRoles($request->role);

    Alert::toast(__('posts.alert.delete.message.success'), 'success');
    DB::commit();
    return redirect()->route('users.index');
} catch (\Throwable $th) {
    DB::rollBack();
    Alert::toast(__('posts.alert.delete.message.error', ['error' => $th->getMessage()]), 'errors');
    return redirect()->back()->withInput($request->all())->withErrors($validator);
}

2. 密码更新逻辑问题
   原代码不管用户有没有填写新密码，都会执行Hash::make($request->password)，如果表单里password字段为空，会生成一个空字符串的哈希值，导致用户无法登录。上面的代码已经修复了这个问题，只在$request->filled('password')时才更新密码。

内容的提问来源于stack exchange，提问作者Wildan Maulana