-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Comments
This is still present in Solidus 2.7. |
@ericsaupe should be closed with #3059, can you please check? |
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
This was done against the current 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. |
@kennyadsl, #3059 isn't related to this issue, since it doesn't mess with shipment adjustments. I worked recently on |
@ericsaupe, by my checks, cancelling items doesn't take in consideration shipment costs using any shipment method: does it match with your observations? |
Since @jacobherrington has said it was still present on 2.7, I verified how the code was since that version and it seems it hasn't changed much in regards to the described method. My guess would be that the problem still lingers since no code was changed, but I will need to attempt to reproduce. A solution would be to add a new |
Can confirm this is still an issue. To reproduce (on the demo store if you have no vanilla environment):
Usually you'd want to avoid recalculating orders post-completion, but this situation kind of demands it. I think a simple solve - assuming this doesn't introduce other issues - could be to toss an |
Steps to reproduce
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:
solidus/core/app/models/spree/calculator/shipping/per_item.rb
Line 14 in 054e4bd
System configuration
Solidus Version: 2.1 (seems to be in all)
Extensions in use: core, frontend, backend
The text was updated successfully, but these errors were encountered: