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

Shipping Calculators still calculate for cancelled items #1837

Open
ericsaupe opened this Issue Apr 10, 2017 · 5 comments

Comments

Projects
None yet
5 participants
@ericsaupe
Copy link
Contributor

ericsaupe commented Apr 10, 2017

Steps to reproduce

  1. Complete order with many items and flexirate shipping calculator
  2. Cancel items in shipment
  3. Adjustment created does not take into account shipping cost
  4. Cost on shipping was not changed even though less items will be shipped

Expected behavior

After an item is cancelled the calculator should only calculate cost based on what is actually going to ship.

Actual behavior

Shipping calculators are calculating cost on all items even if they are cancelled:

System configuration

Solidus Version: 2.1 (seems to be in all)

Extensions in use: core, frontend, backend

@jacobherrington

This comment has been minimized.

Copy link
Member

jacobherrington commented Sep 25, 2018

This is still present in Solidus 2.7.

@kennyadsl

This comment has been minimized.

Copy link
Member

kennyadsl commented Jan 29, 2019

@ericsaupe should be closed with #3059, can you please check?

@ericsaupe

This comment has been minimized.

Copy link
Contributor Author

ericsaupe commented Feb 5, 2019

It still seems to be a little bit off. Maybe it's just the taxes? Anyway so the setup here is

Shipping Rate Calculator - Flexible Rate by item with first item $1.5 and addition items $1

  • Added three different products to the cart
  • Completed checkout
  • Canceled two of the three items
  • The adjustments created did include the additional $1 for shipping but the shipping cost, which is taxed, remained the same. Also the tax adjustments were readjusted and now show a negative adjustment of a few cents. The tax adjustment for the remaining item seems to have disappeared as well.

screen shot 2019-02-05 at 2 56 26 pm

This was done against the current 2.9.0.alpha

Correct me if I'm misunderstanding it all here but I think it should adjust the cost of shipping by the canceled item change, adjust the line item by the cost of the item, and maybe additional adjustments to remove taxes? Just thinking about it from an accounting standpoint where adjustments were created to charge money from a customer and then negative adjustments were created to restore money to the customer against what they paid for.

@mdesantis

This comment has been minimized.

Copy link
Contributor

mdesantis commented Feb 7, 2019

@kennyadsl, #3059 isn't related to this issue, since it doesn't mess with shipment adjustments.

I worked recently on Spree::UnitCancel and I can't remember any attempt there to create adjustment for shipment costs, so I suspect to be that way by design. If instead we all agree that when we go on 'Cancel Items' tab and we cancel line items the related shipment costs should be adjusted accordingly, we should work on that as it is a "feature", from a developer point of view

@mdesantis

This comment has been minimized.

Copy link
Contributor

mdesantis commented Feb 8, 2019

@ericsaupe, by my checks, cancelling items doesn't take in consideration shipment costs using any shipment method: does it match with your observations?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment