Ticket #1333 (closed バグ指摘: 修正済)

Opened 9 years ago

Last modified 9 years ago

謎な処理 SC_Helper_Purchase#shippingItemTempToCart

Reported by: Seasoft Owned by: Seasoft
Priority: Milestone: EC-CUBE2.11.2
Component: フロント Version: 2.11.1
Keywords: Cc:
修正済み: yes

Description (last modified by Seasoft) (diff)

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

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

r20971

Change History

comment:1 Changed 9 years ago by Seasoft

  • Owner changed from somebody to Seasoft
  • Status changed from new to assigned
  • Description modified (diff)

r20971

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

comment:2 follow-up: ↓ 3 Changed 9 years ago by nanasess

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

comment:3 in reply to: ↑ 2 ; follow-up: ↓ 6 Changed 9 years ago by Seasoft

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

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

comment:4 follow-up: ↓ 5 Changed 9 years ago by nanasess

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

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

comment:5 in reply to: ↑ 4 Changed 9 years ago by Seasoft

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

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

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

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

comment:6 in reply to: ↑ 3 ; follow-ups: ↓ 7 ↓ 8 Changed 9 years ago by Seasoft

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

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

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

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

comment:7 in reply to: ↑ 6 Changed 9 years ago by nanasess

Seasoft への返信

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

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

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

comment:8 in reply to: ↑ 6 Changed 9 years ago by Seasoft

  • 修正済み set

Seasoft への返信

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

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

comment:9 Changed 9 years ago by kajiwara

Seasoftさん

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

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

もう少し確認します。

comment:10 Changed 9 years ago by kajiwara

  • Status changed from assigned to closed
  • Resolution set to 修正済
Note: See TracTickets for help on using tickets.