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

カート内数量減算を連続クリックしても、カートから商品が削除されないように変更 #1268 #2091

Merged
merged 3 commits into from
Feb 13, 2017

Conversation

hoand-vn
Copy link
Contributor

@hoand-vn hoand-vn commented Feb 8, 2017

Fixed : can not reduce quantity to Zero in order

ref #1268

@ryo-endo ryo-endo changed the title Bug fix #1268 カート内数量減算を連続クリックしても、カートから商品が削除され内容に変更 #1268 Feb 8, 2017
@ryo-endo
Copy link
Contributor

ryo-endo commented Feb 8, 2017

Issue #1268 にも記載しましたが,,,

初期実装から、商品数が0になるとカートから商品を削除する実装になっていました。
https://github.com/EC-CUBE/ec-cube/blame/65a3a4804805a8ad95338f4a4295ba40868d8fad/src/Eccube/Service/CartService.php#L136

連続クリック時に意図せずカートから商品が消えてしまうなど、利便性が悪いため、商品マイナス時には残り1個で止まるよう仕様変更します。

@ryo-endo ryo-endo added this to the 3.0.14 milestone Feb 8, 2017
Copy link
Contributor

@ryo-endo ryo-endo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

コード問題ないと思います。
変更した関数の利用箇所も、カート内の数量減算のボタンを押した時のみです。

@ryo-endo ryo-endo changed the title カート内数量減算を連続クリックしても、カートから商品が削除され内容に変更 #1268 カート内数量減算を連続クリックしても、カートから商品が削除されないように変更 #1268 Feb 13, 2017
@ryo-endo ryo-endo merged commit 957c9d5 into EC-CUBE:master Feb 13, 2017
@lqdung-lockon lqdung-lockon deleted the bug_fix_#1268 branch September 26, 2017 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants