Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SC_CartSession 内の配列要素の扱いが怪しい #87

Closed
seasoftjapan opened this issue Feb 12, 2016 · 2 comments
Closed

SC_CartSession 内の配列要素の扱いが怪しい #87

seasoftjapan opened this issue Feb 12, 2016 · 2 comments
Labels
Milestone

Comments

@seasoftjapan
Copy link
Contributor

[0]としている箇所が疑わしい。

うち1箇所は #85 で削除済み。

未検証だが、常に NULL が渡ったりしていないだろうか。その場合、商品規格を使った、商品毎の税率設定に不備を生じる懸念を感じる。

@Yangsin Yangsin added the bug label May 15, 2016
@Yangsin Yangsin added this to the eccube-2.13.6 milestone May 15, 2016
@so-amuamu so-amuamu added bug:Low and removed bug labels Jul 31, 2018
@so-amuamu so-amuamu modified the milestones: eccube-2.13.6, 2.17.0 Jul 31, 2018
@so-amuamu
Copy link

2.17に向けて対処が必要な状態が残っているか、再現確認から

@seasoftjapan
Copy link
Contributor Author

['id']には、文字列で商品規格IDが格納されている。よって、['id'][0]は、冒頭に記載した NULL ではなく、(EC-CUBE のとんでも仕様のおかげで) 商品規格IDの1文字目を取得していた。よって、商品規格ID 10 以上は正しく動作しなかった。

#232 で修正。うち1箇所目のメソッドは現状未使用。

ryo-endo added a commit that referenced this issue Sep 18, 2018
#87 SC_CartSession 内の配列要素の扱いが怪しい
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants