PHP7.4対応

[nanasess]

概要(Overview・Refs Issue)

• Resolve PHP7.4 対応 #4384
• Composer update #4396 を先にマージお願いします
• PHP7.4対応

方針(Policy)

• ユニットテスト、 E2Eテストを PHP7.4で実行するよう修正
• ユニットテストをPHP7.4で実行するよう修正
  • E2Eテストを PHP7.4 で実行すると大変成功率が低いため、 PHP7.3 で実行する
    • タイムアウトやタイミングの問題で失敗するようなので、本体の不具合ではないと思われる
    • E2Eテストがすべて通る場合もあることは確認済み(https://github.com/EC-CUBE/ec-cube/runs/355804720)

実装に関する補足(Appendix)

• AbstractRepository の eccubeConfig は、 Symfony Kernel 内部でアクセスするタイミングでは null になってしまう

テスト（Test)

ユニットテスト、 E2Eテストが通っていることを確認

相談（Discussion）

マイナーバージョン互換性保持のための制限事項チェックリスト

• 既存機能の仕様変更
• フックポイントの呼び出しタイミングの変更
• フックポイントのパラメータの削除・データ型の変更
• twigファイルに渡しているパラメータの削除・データ型の変更
• Serviceクラスの公開関数の、引数の削除・データ型の変更
• 入出力ファイル(CSVなど)のフォーマット変更

レビュワー確認項目

• 動作確認
• コードレビュー
• E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
• 互換性が保持されているか
• セキュリティ上の問題がないか

[nanasess]

E2Eテストの成功率が低めです。。。

[nanasess]

E2Eテストを PHP7.4 で実行すると大変成功率が低いので、 PHP7.3 で実行するようにしています。
PHP7.4 でもすべて通るタイミングがあるのは確認済み(https://github.com/EC-CUBE/ec-cube/runs/355804720)

[okazy]

ありがとうございます。マージしました。