ec-cube icon indicating copy to clipboard operation
ec-cube copied to clipboard

Eccube/feature/fix php74 deprecated type declarations

Open ci-wataru-kashii opened this issue 4 months ago • 5 comments

コマンド実行時、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.

概要(Overview・Refs Issue)

PHP74_型宣言の追加.txt

方針(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 テスト確認(テストの追加・変更が必要かどうか)
  • [ ] 互換性が保持されているか
  • [ ] セキュリティ上の問題がないか
    • [ ] 権限を超えた操作が可能にならないか
    • [ ] 不要なファイルアップロードがないか
    • [ ] 外部へ公開されるファイルや機能の追加ではないか
    • [ ] テンプレートでのエスケープ漏れがないか

ci-wataru-kashii avatar Sep 30 '24 02:09 ci-wataru-kashii