ec-cube
ec-cube copied to clipboard
Eccube/feature/fix php74 deprecated type declarations
コマンド実行時、Webアプリケーションからのアクセス時に下記エラーを補足したため、修正しました。
Method "FQDN" might add "some_type" as a native return type declaration in the future. Do the same ... "ECCUBEのFQDN" now to avoid errors or add an explicit @return annotation to suppress this message.
- "FQDN" ... 継承元の型宣言
- "some_type' ... PHPの組み込み型「null , bool , int , float (浮動小数点数) , string , array , object , callable , resource」
- "ECCUBEのFQDN" ...
Eccube\
からはじまるFQDN
概要(Overview・Refs Issue)
方針(Policy)
EC-CUBE4.3 のシステム要件には、PHP8.1〜8.3 とのことですので、問題ない認識です。
https://www.php.net/manual/ja/language.types.declarations.php より引用
関数のパラメータや戻り値、 クラスのプロパティ (PHP 7.4.0 以降)、クラス定数 (PHP 8.3.0 以降) に対して型を宣言することができます。
- 親クラスの型宣言に合わせています。
- mixed には、
null
も含まれるということで?mixed
はつけていません。
実装に関する補足(Appendix)
テスト(Test)
相談(Discussion)
マイナーバージョン互換性保持のための制限事項チェックリスト
- [ ] 既存機能の仕様変更はありません
- [ ] フックポイントの呼び出しタイミングの変更はありません
- [ ] フックポイントのパラメータの削除・データ型の変更はありません
- [ ] twigファイルに渡しているパラメータの削除・データ型の変更はありません
- [ ] Serviceクラスの公開関数の、引数の削除・データ型の変更はありません
- [ ] 入出力ファイル(CSVなど)のフォーマット変更はありません
レビュワー確認項目
- [ ] 動作確認
- [ ] コードレビュー
- [ ] E2E/Unit テスト確認(テストの追加・変更が必要かどうか)
- [ ] 互換性が保持されているか
- [ ] セキュリティ上の問題がないか
- [ ] 権限を超えた操作が可能にならないか
- [ ] 不要なファイルアップロードがないか
- [ ] 外部へ公開されるファイルや機能の追加ではないか
- [ ] テンプレートでのエスケープ漏れがないか