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

Opened 7 years ago

Last modified 6 years ago

SC_CartSession.phpについて

Reported by: kimoto Owned by: undertree
Priority: Milestone: EC-CUBE2.13.3
Component: その他 Version: 2.13.0
Keywords: Cc:
修正済み: no

Description (last modified by kim) (diff)

 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=13889&forum=1&post_id=62347#forumpost62347

お世話になります。ひょんな事からカート操作系APIを作成していて気づいたのですが、SC_CartSession.phpに不具合があるように見受けられます。
カート内の商品を削除するメソッド(delProductKey)ですが、
他の関数で使用しているgetMax()を使わずにcount()を使っているために、
特定の場合(カートに入れる→商品を削除、を繰り返した後など)に
カート内の商品を削除できない事象が発生する可能性があるかと思われます。data/class/SC_CartSession.php
284	        $max = count($this->cartSession[$productTypeId]);
285	        for ($i = 0; $i < $max; $i++) {
ですが、他のメソッドと同様に
		$max = $this->getMax($productTypeId);
		for ($i = 1; $i <= $max; $i++) {
とあるべきだと思います。どこに記載して良いかわからないのでこちらに記載しました。
現状、EC-CUBEの動作に関わるような不具合ではないような気もしますがご報告させて頂きます。

Change History

comment:1 Changed 6 years ago by h_yoshimoto

  • Milestone changed from EC-CUBE2.13.2 to EC-CUBE2.13.3

comment:2 Changed 6 years ago by undertree

  • Owner changed from somebody to undertree

上記のご指摘について、修正コミットを出しておきます。

comment:3 Changed 6 years ago by undertree

23613で対応しました。

comment:4 Changed 6 years ago by undertree

r23613で対応しました。

comment:5 Changed 6 years ago by kim

  • Description modified (diff)

undertree 様 r23613 で対応完了しておりましたら、修正済みにチェックをいただけますでしょうか。

comment:6 Changed 6 years ago by shinichi_takahashi

  • Status changed from new to closed
  • Resolution set to 修正済

undertree様

ご対応ありがとうございました。 ソースでの修正を確認できましたので、クローズさせていただきます。

Note: See TracTickets for help on using tickets.