前往小程序,Get更优阅读体验!
立即前往
首页
学习
活动
专区
工具
TVP
发布
社区首页 >专栏 >你可能会踩到的一个 Eloquent 小坑

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

作者头像
overtrue
发布2020-07-24 16:34:42
4900
发布2020-07-24 16:34:42
举报
文章被收录于专栏:假装我会写代码

今天在写一个模型版本控制的需求,目标就是模型有字段变化时创建版本记录,首先想到的肯定是用我之前写过的一个包: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 行为的结果有效。

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

本文参与 腾讯云自媒体同步曝光计划,分享自微信公众号。
原始发表:2020-05-26,如有侵权请联系 cloudcommunity@tencent.com 删除

本文分享自 假装我会写代码 微信公众号,前往查看

如有侵权,请联系 cloudcommunity@tencent.com 删除。

本文参与 腾讯云自媒体同步曝光计划  ,欢迎热爱写作的你一起参与!

评论
登录后参与评论
0 条评论
热度
最新
推荐阅读
领券
问题归档专栏文章快讯文章归档关键词归档开发者手册归档开发者手册 Section 归档