sprintf()の引数が足りず正しくエラーが出力されなかったので修正 #4454
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
概要(Overview・Refs Issue)
新しくテーブルを追加する際には
src/Eccube/Resource/doctrine/import_csv/ja/definition.yml
へテーブル名を追加する必要がある。追加せずEntityだけ追加すると「定義されていないよ」とエラーが出て欲しいが、エラー文言の生成で利用している
sprintf()
の引数の個数が間違っていてエラー出力ができないようになっていた。方針(Policy)
引数を減らし、エラー文言にべた書きした。
実装に関する補足(Appendix)
本来
$file
を出力させるべきではあるが、エラーメッセージのためだけに無名関数に渡すのは冗長な気がしたのでべたがきとしたテスト(Test)
definition.yml
を変更せず、Entityで新しいテーブルを追加すると発生する。手元の環境でエラーが出力されることを確認済み。
相談(Discussion)
マイナーバージョン互換性保持のための制限事項チェックリスト
レビュワー確認項目