謎な処理 SC_Helper_Purchase#shippingItemTempToCart

[Seasoft]

カスタマイズ中に受注明細上の数量が不自然な数値を示すことから分析を進めたところ、このメソッドによって受注明細が乱されていた。

この処理を外したところ、意図した動作となった。この処理の必要性が分からない。

r20971

[Seasoft]

r20971

一旦削除してみましたので、現状で不具合が確認されましたら、報告をお願いいたします。

[nanasess]

複数配送設定時に数量を変更した場合, それをカートにも反映させる処理ですが, 問題ありませんでしたでしょうか?

[Seasoft]

もしかして、複数配送設定って、カート中の数量との整合をチェックしていなかったりします?

そうだとしたら、その処理を改めた方が良い気もします。

[nanasess]

単純に, product_class_id をキーにして, カートの中の数量を上書きしているのですが, 具体的にどのような不具合が発生しているのでしょうか?

本関数を削除する対応だと, まずい気がしますが, いかがでしょうか?

[Seasoft]

単純に, product_class_id をキーにして, カートの中の数量を上書きしているのですが, 具体的にどのような不具合が発生しているのでしょうか?

具体的な不具合はカスタマイズにおける発現で、前述の観点では未検証です。

本関数を削除する対応だと, まずい気がしますが, いかがでしょうか?

前述のとおり、カート中の数量との整合をチェックしていないならマズいと思います。

[Seasoft]

もしかして、複数配送設定って、カート中の数量との整合をチェックしていなかったりします?

ローカル環境でチェックしてみましたが、チェックしていないようですね。

個人的には、結構衝撃的なのですが、この動作は「固定された仕様」でしょうか? それとも、「現状そうなっている」という段階でしょうか?

いずれにしても、現状のソースではマズいという認識は持ちました。

[nanasess]

Seasoft への返信

個人的には、結構衝撃的なのですが、この動作は「固定された仕様」でしょうか? それとも、「現状そうなっている」という段階でしょうか?

「現状そうなっている」という段階です.

関数を削除するのみでは, 別の不具合が発生すると思いますので, 具体的にどんな不具合が発生するかを明確にした上で修正する必要があると思います.

[Seasoft]

Seasoft への返信

いずれにしても、現状のソースではマズいという認識は持ちました。

#1351 の実装に伴い問題は解消したと考えます。

[kajiwara]

Seasoftさん

本件、ご提案と修正いただき、ありがとうございます。

現状、カゴと複数配送設定の合計数が異なる場合エラーになり、動作的には問題ないと思います。
特に他で問題がでないようならこちらでクローズします。

もう少し確認します。