| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
单测也一块搬过来 |
Sorry, something went wrong.
Sorry, something went wrong.
代码审查结果功能概述此PR为验证器的数据库相关规则(exists 和 unique)添加了支持传递 ModelName::class 格式的table参数功能。当传递模型类名时,系统会自动解析模型的连接名和表名,避免在多数据库连接场景下手动拼接连接名称。 代码分析核心实现在 DatabaseRule.php 中新增了 resolveTableName() 方法:
优点
潜在问题
建议改进
测试覆盖测试用例较为完整,涵盖了:
总体评价这是一个实用的功能增强,能够简化多数据库连接场景下的验证规则使用。代码整体结构合理,测试覆盖充分,建议在考虑上述改进点后合并。 |
Sorry, something went wrong.
|
已按照建议调整了代码,也补充了文档 |
Sorry, something went wrong.
|
@limingxinleo 这个PR也处理好了,麻烦帮忙看看,谢谢 |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
目前的规则默认会使用default的数据库连接,当使用多个数据库连接时需要手动拼接连接名称。
传递ModelName::class作为table的值后,会自动解析模型的连接名和表明。
该功能从Laravel同步。