2

起源

  • 办法总比困难多,所以发现问题比解决问题更难得到。
  • 事情的起源是因为在调试一段模型事件时出现的,当时随便往数据库里插入了一段字符串 ID,而不是标准的 uuid。然后发现我的模型事件中查找不到当前 ID 的模型。
  • 当然,我们不可能总是能遇到问题,也有可能遇到了问题是自己的错误,这时候可以去Laravel问题库看一下自己有什么能解决的问题

进展

  • 先排除了自己本身代码中的错误,然后一步一步调试



  • 打印了一下返回变量,发现belongsToMany::sync方法返回的 ID 并不正确。

  • 进一步查看,这个整数大小,很熟悉,觉得应该是整数类型大小溢出了。然后看了一下源码InteractsWithPivotTable::caseKey

  • 基本确定就是整形数字溢出了,这里只是简单的根据变量类型进行强转,因为 PHP 是弱类型语言,所以当长字符是一个数字字符串时,将导致无法得到正确的结果。
  • 但我在模型中已经设置了主键类型为字符串,这不应该是我想要的结果。

  • 然后我向框架提了一个issues
  • Laravel 的组织成员tillkruss 告诉我可以提交一个 PR
  • 然后自己马上行动起来,说实话解决这个问题很简单,因为只要获取主键类型,然后进行强转即可
  • 马上提交了代码,之后被驳回,原来是忘记了写测试代码
  • 查看了一下其他数据库的测试代码,找了一个比较符合我的写了一个数据库的测试。提交。成功合并。
  • 具体修改的代码,可以查看这里 5.7 Improve the return value of the caseKey method

结束

  • 第一次为大型项目提交PR,确实感觉到不能这么随意。
  • 写代码的时候请明确解释这段代码的原因,你会发现框架当中很多很多的注释
  • 千万不要忘记写测试文件
  • 当你提交PR的时候,请详细描述你本次做了什么改动,增加了什么新功能。
  • 尽情的表达你的想法,社区有很多人会帮助你的

最后欢迎加入Laravel学习交流群,群号:584453488


seth-shi
1.8k 声望231 粉丝

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