你可能会踩到的一个 Eloquent 小坑

2020-07-24 16:34:42 浏览数 (1)

今天在写一个模型版本控制的需求,目标就是模型有字段变化时创建版本记录,首先想到的肯定是用我之前写过的一个包:overtrue/laravel-versionable,原理很简单:

监听模型事件,当检测到属性值变化时,选择已快照或者差异化获取变化的内容,存储到一个版本记录表当中

Eloquent 模型事件

模型事件相信大家都非常熟练了,一句话介绍就是:当模型有各种变化时,Laravel 会触发对应的事件通知,目前支持的事件有:retrieved, creating, created, updating, updated, saving, saved, restoring, restored, replicating,deleting, deleted, forceDeleted

所以有了事件通知就可以做很多事情了,我的这个包做了一个功能,模型可以自己动态决定是否要为当前变更创建版本,只需要在模型里重载一个方法返回布尔值即可:

代码语言:javascript复制
public function shouldVersioning(){    return true;}

掉坑里了

我们的需求是只有当 document_id 有修改的时候才创建版本,心想那还不简单,Model 不是有一个 getChanges 方法吗:

代码语言:javascript复制
public function shouldVersioning(){    return array_key_exists('document_id', $this->getChanges());}

其实还有一个 isDirty 也可以达到目标,就感觉 getChanges 更表意一些,结果就掉坑里了:只有更新时才有版本记录,创建时没有。

当我翻到源码的时候眼泪掉下来:

getChanges 只在 Model::performUpdate() 这个方法时才会从 getDirty() 拷贝内容,而 getDirty() 的结果是只要赋值(fill)就能得到变化结果,所以,getChanges 只是针对 update 行为的结果有效。

都怪自己源码读的不够仔细,希望你不要掉这个坑里哦 ~ ?

0 人点赞