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

Fix: various problems with minimum order amount enabled #8600

Closed
wants to merge 1 commit into from

Conversation

ericrisler
Copy link
Contributor

@ericrisler ericrisler commented Feb 18, 2017

See #6151 for the basics.

When minimum order amount is enabled:

  1. add 1 product to the cart; total is below min order amount -> no error.
  2. Try to remove that product -> error about minimum amount.
  3. Try to add more to the cart with total still below min order -> error about minimum amount
  4. Add a product that will bring the total cart amount over the min order amount and no errors!

I will request the Magento team takes a look at this line. I don't understand why the minimum amount is being validated here but perhaps it has something to do with multi-shipping checkouts. However, it botches the min order amount logic badly in it's current form.

See magento#6151 for the basics.

When minimum order amount is enabled:
1. add 1 product to the cart; total is below min order amount -> no error.
2. Try to remove that product -> error about minimum amount.
3. Try to add more to the cart with total still below min order -> error about minimum amount
4. Add a product that will bring the total cart amount **over** the min order amount and no errors!

I will request the Magento team takes a look at this line. I don't understand why the minimum amount is being validated here but perhaps it has something to do with multi-shipping checkouts. However, it botches the min order amount logic badly in it's current form.
@okorshenko okorshenko self-assigned this Feb 20, 2017
@okorshenko okorshenko added this to the February 2017 milestone Feb 20, 2017
@okorshenko
Copy link
Contributor

@ericrisler thank you for your contribution. Unfortunately the issue exists, but it can't be fixed in proposed way. Closing this PR.

@okorshenko okorshenko closed this Feb 21, 2017
@magento magento locked and limited conversation to collaborators Feb 21, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants