CakePHP 4.2.9编辑操作未更新记录却提示保存成功问题

我一眼就揪出了问题的核心——你的编辑控制器逻辑里完全没有把用户提交的表单数据更新到用户实体中！所以你调用save()的时候，保存的还是从数据库里读出来的原始数据，自然不会有任何变更，但因为实体本身没有错误，save()操作返回成功，这就导致了“提示保存成功但数据没更新”的诡异现象。

问题代码的关键错误点 #

在你的edit方法的POST/PUT分支里，你重新获取了$user实体，但没有做任何数据更新操作，直接调用save()：

// 这里重新获取了旧数据，但没有合并新的表单数据
$user = $this->Users->get($id, ['contain' => ['Userdata']]);
if ($this->Users->save($user)) { ... }

相当于你只是把数据库里的数据读出来又写回去，没有任何变化，所以save成功但数据不变。

修正后的控制器代码 #

你需要用patchEntity()方法把表单提交的数据合并到已有的实体中，同时因为涉及关联模型Userdata的更新，还要在patch时指定关联：

public function edit($id = null) {
    $user = $this->Users->get($id, [
        'contain' => ['Userdata'],
    ]);
    if ($this->request->is(['post', 'put'])) {
        // 用表单数据更新实体，同时处理关联的Userdata
        $user = $this->Users->patchEntity($user, $this->request->getData(), [
            'associated' => ['Userdata']
        ]);
        if ($this->Users->save($user)) {
            $this->Flash->success(__('The user has been saved.'));
            return $this->redirect(['action' => 'index']);
        }
        $this->Flash->error(__('The user could not be saved. Please, try again.'));
    }
    $this->set(compact('user'));
}

补充说明 #

1. patchEntity()的作用：CakePHP的实体是带有状态追踪的对象，patchEntity()会把请求中的新数据覆盖到原实体上，并标记哪些字段发生了变更，这样save()时只会更新变更的字段。
2. 关联模型的处理：因为你的表单里包含了Userdata的字段（userdata.Email、userdata.UserName），所以必须在patchEntity()的选项中指定'associated' => ['Userdata']，否则关联模型的数据不会被更新。
3. 冗余代码移除：原来POST分支里重复调用$this->Users->get()是完全没必要的，直接使用之前获取的实体即可。

额外提醒（关于密码字段） #

你的表单里有PasswordHashed字段，如果你希望用户输入新密码时才更新哈希，建议在模型里设置密码的自动哈希逻辑，并在patch时处理空值（比如用户不输入密码时不覆盖原有哈希），不过这是额外的优化点，和当前的“数据不更新”问题无关。

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