coreshop / CoreShop

CoreShop - Pimcore enhanced eCommerce
http://www.coreshop.org
Other
276 stars 157 forks source link

Cart Rules: Solidify Cart Calculation #253

Closed solverat closed 6 years ago

solverat commented 6 years ago
dpfaffenbauer commented 6 years ago

done here: https://github.com/coreshop/CoreShop/pull/263

can you review please?

solverat commented 6 years ago

@dpfaffenbauer reviewed and merged! Only thing: discount should also respect shipping / payment fees etc!

dpfaffenbauer commented 6 years ago

should be working now, can you review again please?

solverat commented 6 years ago

@dpfaffenbauer if discount is greater than the cart amount, the negative price difference will show up as a positive total.

solverat commented 6 years ago

and also: if payment is empty in cart/order object, do we have some side effects we need to handle? (like paymentprovider is null in twig)

dpfaffenbauer commented 6 years ago

Regarding payment: now, thats optional.

solverat commented 6 years ago

i just tested the negative case: works now. so hopefully this is fixed...