新着商品ブロックにて、商品を動的に表示する

[matsuoshi]

概要(Overview・Refs Issue)

デフォルトでTOPページに表示される「新着商品」ブロックにて、動的に新着商品を表示するようにしました。

(現状は、商品がHTMLでハードコードされており、新着商品の更新にはブロック内のHTMLの編集が必要)

関連issue

#4858
#4623
#4504

方針(Policy)

作成日の新しいものから順に、最大5件の商品を自動表示。

実装に関する補足(Appendix)

データの取得方法は、既存の Newsブロックと同様にtwig内からリポジトリにアクセスしています。

テスト（Test)

ブロックが表示されることを確認するWebテストを追加しました。

相談（Discussion）

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

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

レビュワー確認項目

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

[k-yamamura]

サイトによってはカラー違いで同一商品が並ぶというような事も起きるため、
既存のブロックはそのまま残しておいて手動で修正できるようにしておき、
新たなブロックとして定義する方が良いのでは無いでしょうか。

[matsuoshi]

@k-yamamura コメントありがとうございます！
今回のブロックは既存ブロックをベースにループを足した形ですので、手動で運用したい場合でも修正負荷はさほど高くないと考えています。

手動で運用したい場合はいずれにせよブロックの修正は必須になりますので、手動更新したいユーザ／しないユーザ別にブロックを2つ用意するのは冗長かとも思ったのですが、いかがでしょうか。

[xuelian311]

実装方針として、今のブラックを修正するより、新しく動的ブロックを追加するのはいかがでしょうか？

[Yangsin]

マイナーバージョンで対応する場合は、新規追加でなければ、既存の互換性に影響が出る想定で、NGと考えます。
ブロックはプラグインや普段の運用でも結構追加されたりもするので、その点の影響も注意が必要かと。

[shinya]

今回、4.3へのアップデートなので取り込み対象にします。
ただし、ブロックを新しく追加する形に変更したいと思います。
そのためソースコードをいくつか変更することになるかと思います。
また、性能に変化が出そうな部分になるのでその辺りも考慮に入れて改修を行います。

[shinya]

こちら新しい要素として書き換えました。
（デフォルトのブロックとしては入れずに手動で追加すれば表示される）
確認よろしくお願いします。

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.76%. Comparing base (7786521) to head (9389cef).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff            @@
##                4.3    #4920   +/-   ##
=========================================
  Coverage     82.76%   82.76%           
- Complexity     6514     6516    +2     
=========================================
  Files           479      480    +1     
  Lines         26035    26046   +11     
=========================================
+ Hits          21547    21558   +11     
  Misses         4488     4488           

Flag	Coverage Δ
E2E	70.03% <0.00%> (-0.03%)	⬇️
Unit	79.61% <100.00%> (+<0.01%)	⬆️

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.

[chihiro-adachi]

確認しました。問題ないです。

[dotani1111]

@shinya
テストが落ちてしまう問題の修正を行いました。
こちらの取り込みと、自動表示の方へ文言変更をお願いします。