Laravel PR 流し読み(2021/07/19)

なるようになるブログのrailsのコミットログを流しよみしてるのよいなと思ったので僕も laravel の PR を雑に読んでいこうかなーと思った。

y-yagi.hatenablog.com

 

今日マージされたものを見ていく感じでやっていこうと思う(全部カバーできないかもしれない)。

 

まずはこれ。

[8.x] ErrorException: Undefined array key "exception" by kekstoaster · Pull Request #38059 · laravel/framework · GitHub

 

やっていることは単純に isset で存在確認を行うようにしたようです。

下記のリンクで最新の PHP でだけ Warning になることが確認できたのでまあそういう修正。

https://3v4l.org/hkUPE

 

次はこれ。

[8.x] Allow eloquent model type in be/actingAs by michael-rubel · Pull Request #38055 · laravel/framework · GitHub

 

actingAs と beメソッドの PHPDoc がおかしいというものだったようです。

※ 今書いてる途中で Revert された。

コメントを見る限り注釈付きのジェネリックで修正できそう的なことらしい。

larastan の例を上げてくれているがよくわからんかった。知っている人いたらコメントほしい。

 

次はこれ。

[8.x] Fix unvalidated array keys without implicit attributes by bonroyage · Pull Request #38052 · laravel/framework · GitHub

 

Ver 8.45で追加された機能の修正のようです。

バリデーション 8.x Laravel

zenn.dev

 

次はこれ。

[8.x] Display unexpected validation errors when asserting status by jessarcher · Pull Request #38046 · laravel/framework · GitHub

 

内容を読んだ感じだとこれの拡張

[8.x] Track exceptions and display them on failed status checks for dx by taylorotwell · Pull Request #38025 · laravel/framework · GitHub

 

テスト記載しててよくわからんエラーがでたときにお世話になってた $this->withoutExceptionHandling(); を記載しなくてもわかりやすくなる見たい。

 

次はこれ。

[8.x] Fix PHPDocs for JSON resources by Kolyunya · Pull Request #38044 · laravel/framework · GitHub

 

PHPDocs の修正。