ご注文手続き画面で商品種別が入れ替わる問題の修正

[dk-umebius]

概要(Overview・Refs Issue)

下記のタイミングで注文手続き画面の商品種別が入れ替わってしまう問題の修正

• 複数配送を設定する場合 (複数の販売種別がある場合、複数配送時に商品が変わってしまう #4293)
• PurchaseFlowがwarningで返ってきた場合(Issue無し)

方針(Policy)

注文フロー内でセッションに保存しているカートキーの先頭が変わらないようにする

実際は先頭にあるカートが優先されるという仕組みを変えた方がいい気がしますが、影響範囲が大きそうなので最低限の修正で暫定対応する方針

今までどんな仕組みになっていたかと思ったのですが、そう言えば３系ではカート１つしかなかったんですね‥。
もしセッションのcart_keysの順番に依存しないようにするなら、どのカートor受注がアクティブかということをセッションに持たせる等になるのでしょうか。

このPRでは $this->cartService->save();の代わりに $this->cartService->setPrimary($Cart->getCartKey()); を実行して対象のカートが先頭で保存されるようにしています。(saveはsetPrimary内部で実行されています)

そもそも注文フロー内ではOrder.pre_order_idからカートを取得すればいいのではとも思いましたが
考えてみると下記のコードのように購入処理中の受注を取得するために先にカート情報を取得しているので大変そうだなと。

ec-cube/src/Eccube/Controller/ShoppingController.php

Lines 250 to 257 in 4e213a8

	// 受注の存在チェック
	$preOrderId = $this->cartService->getPreOrderId();
	$Order = $this->orderHelper->getPurchaseProcessingOrder($preOrderId);
	if (!$Order) {
	log_info('[注文確認] 購入処理中の受注が存在しません.', [$preOrderId]);
	return $this->redirectToRoute('shopping_error');
	}

実装に関する補足(Appendix)

テスト（Test)

warningの方は@ShoppingFlowで$this->throwInvalidItemException('エラー', true);を実行してみて確認

相談（Discussion）

良い解決方法があれば教えて欲しいです。

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

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

レビュワー確認項目

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

[chihiro-adachi]

#4923 の再現手順で、事象が解消されていることを確認しました。
修正内容も問題ないかと思います。

[okazy]

落ちているテストを確認しました。
他のプルリクで改善済みが3件と、本修正とは関係ないと思われる箇所が1件でした。
テストを再実行して以下が通ればマージします。

1) EA08SysteminfoCest: Ea0807-uc01-t01 マスターデータ管理
 Test  codeception/acceptance/EA08SysteminfoCest.php:systeminfo_マスターデータ管理
 Step  See "無回答","#customer_form #admin_customer_sex"
 Fail  Failed asserting that any element by '#customer_form #admin_customer_sex' on page /admin/customer/new
Elements: 
+ <div> 男性
女性
contains text '無回答'

[okazy]

@dk-umebius ありがとうございます！マージしました。
こちらの複雑な問題をシンプルに改善されていて感動しました。