SC_CartSession.phpについて

[kimoto]

 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の動作に関わるような不具合ではないような気もしますがご報告させて頂きます。

[undertree]

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

[undertree]

23613で対応しました。

[undertree]

r23613で対応しました。

[kim]

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

[shinichi_takahashi]

undertree様

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