Changes between Initial Version and Version 5 of Ticket #2475


Ignore:
Timestamp:
2014/10/09 14:44:51 (10 years ago)
Author:
kim
Comment:

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

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #2475

    • Property Owner changed from somebody to undertree
    • Property Milestone changed from EC-CUBE2.13.2 to EC-CUBE2.13.3

  • Ticket #2475 – Description

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