当神不再是我们的信仰,那么信仰自己吧,努力让自己变好,不辜负自己的信仰!

[开源]记一次 Laravel 源代码的提交

事情的起源是因为在调试一段模型事件时出现的,当时随便往数据库里插入了一段字符串 ID,而不是标准的 uuid
然后发现我的模型事件中查找不到当前 ID 的模型。
《[开源]记一次 Laravel 源代码的提交》
《[开源]记一次 Laravel 源代码的提交》
《[开源]记一次 Laravel 源代码的提交》
之后打印了一下变量,发现belongsToMany::sync方法返回的 ID 并不正确。
《[开源]记一次 Laravel 源代码的提交》
看了一下整数,很熟悉,觉得应该是整数类型大小溢出了。然后看了一下源码InteractsWithPivotTable::caseKey
《[开源]记一次 Laravel 源代码的提交》
基本确定就是整形数字溢出了,这里只是简单的根据变量类型进行强转,因为 PHP 是弱类型语言,所以当长字符是一个数字字符串时,将导致无法得到正确的结果。
但我在模型中已经处理了主键类型,这不应该是我想要的结果。
《[开源]记一次 Laravel 源代码的提交》


  • 然后想框架提了一个issues
  • Laravel 的组织成员tillkruss 告诉我可以提交一个 PR
  • 然后自己马上行动起来,说实话解决这个问题很简单,然后提交了代码,之后被驳回,原因是:没有写测试代码
  • 还好还好,不用从头开始。查看了一下其他数据库的测试代码,找了一个比较符合我的写了一个数据库的测试。提交。成功合并。[5.7] Improve the return value of the caseKey method
点赞

发表评论

电子邮件地址不会被公开。 必填项已用*标注