Ticket #1333 (closed バグ指摘: 修正済)
謎な処理 SC_Helper_Purchase#shippingItemTempToCart
Reported by: | Seasoft | Owned by: | Seasoft |
---|---|---|---|
Priority: | 中 | Milestone: | EC-CUBE2.11.2 |
Component: | フロント | Version: | 2.11.1 |
Keywords: | Cc: | ||
修正済み: | yes |
Change History
comment:1 Changed 12 years ago by Seasoft
- Owner changed from somebody to Seasoft
- Status changed from new to assigned
- Description modified (diff)
comment:2 follow-up: ↓ 3 Changed 12 years ago by nanasess
複数配送設定時に数量を変更した場合, それをカートにも反映させる処理ですが, 問題ありませんでしたでしょうか?
comment:3 in reply to: ↑ 2 ; follow-up: ↓ 6 Changed 12 years ago by Seasoft
もしかして、複数配送設定って、カート中の数量との整合をチェックしていなかったりします?
そうだとしたら、その処理を改めた方が良い気もします。
comment:4 follow-up: ↓ 5 Changed 12 years ago by nanasess
単純に, product_class_id をキーにして, カートの中の数量を上書きしているのですが, 具体的にどのような不具合が発生しているのでしょうか?
本関数を削除する対応だと, まずい気がしますが, いかがでしょうか?
comment:5 in reply to: ↑ 4 Changed 12 years ago by Seasoft
単純に, product_class_id をキーにして, カートの中の数量を上書きしているのですが, 具体的にどのような不具合が発生しているのでしょうか?
具体的な不具合はカスタマイズにおける発現で、前述の観点では未検証です。
本関数を削除する対応だと, まずい気がしますが, いかがでしょうか?
前述のとおり、カート中の数量との整合をチェックしていないならマズいと思います。
comment:6 in reply to: ↑ 3 ; follow-ups: ↓ 7 ↓ 8 Changed 12 years ago by Seasoft
もしかして、複数配送設定って、カート中の数量との整合をチェックしていなかったりします?
ローカル環境でチェックしてみましたが、チェックしていないようですね。
個人的には、結構衝撃的なのですが、この動作は「固定された仕様」でしょうか? それとも、「現状そうなっている」という段階でしょうか?
いずれにしても、現状のソースではマズいという認識は持ちました。
comment:7 in reply to: ↑ 6 Changed 12 years ago by nanasess
Seasoft への返信
個人的には、結構衝撃的なのですが、この動作は「固定された仕様」でしょうか? それとも、「現状そうなっている」という段階でしょうか?
「現状そうなっている」という段階です.
関数を削除するのみでは, 別の不具合が発生すると思いますので, 具体的にどんな不具合が発生するかを明確にした上で修正する必要があると思います.
comment:9 Changed 12 years ago by kajiwara
Seasoftさん
本件、ご提案と修正いただき、ありがとうございます。
現状、カゴと複数配送設定の合計数が異なる場合エラーになり、動作的には問題ないと思います。
特に他で問題がでないようならこちらでクローズします。
もう少し確認します。
comment:10 Changed 12 years ago by kajiwara
- Status changed from assigned to closed
- Resolution set to 修正済
r20971
一旦削除してみましたので、現状で不具合が確認されましたら、報告をお願いいたします。