"${var}" / "${expr}" 形式の、文字列への値の埋め込み 対応

[ci-wataru-kashii]

前回、PHP8.2 以降の非推奨エラーが起きた際に、プルリクエストを送ったものです。

#6244

PHP8.2、PHP8.3 を対象に bin/console cache:clear --no-warmup を実行したところ
PHP8.2より非推奨になったエラーを検出したため、修正しました。

前回は、 bin/console cache:clear --no-warmup
今回は、 bin/console cache:warmup です。よろしくお願いします。

概要(Overview・Refs Issue)

方針(Policy)

実装に関する補足(Appendix)

テスト（Test)

相談（Discussion）

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

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

レビュワー確認項目

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

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Project coverage is 82.69%. Comparing base (16f7157) to head (1dfd3c1).
Report is 5 commits behind head on 4.3.

Files with missing lines	Patch %	Lines
.../ORM/Mapping/Driver/ReloadSafeAnnotationDriver.php	0.00%	2 Missing ⚠️
...Eccube/Controller/Admin/Store/PluginController.php	0.00%	1 Missing ⚠️
src/Eccube/Doctrine/ORM/Query/Extract.php	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##              4.3    #6286   +/-   ##
=======================================
  Coverage   82.69%   82.69%           
=======================================
  Files         480      480           
  Lines       26465    26465           
=======================================
  Hits        21885    21885           
  Misses       4580     4580           

Flag	Coverage Δ
E2E	82.69% <0.00%> (ø)
Unit	82.69% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[nanasess]

LGTM

[dotani1111]

@ci-wataru-kashii
ありがとうございます。
現在テストが失敗してしまうようなので、対応まで少々お待ちください。
#6289

[ji-eunsoo]

@ci-wataru-kashii
ありがとうございます。
テストが失敗している件を修正したので、最新の修正を取り込んでPushしていただけますでしょうか。

[ci-wataru-kashii]

@ji-eunsoo ご連絡おそくなりすみません、最新の修正を取り込んでPushさせていただきました。
引き続きよろしくおねがいします。

[ji-eunsoo]

@ci-wataru-kashii
ありがとうございます！マージいたします。