Re: URGENT!!!!!!! Upgrade with Errors
The new module is for use everywhere except OSEL. You should leave the shippingmethods module assigned to OSEL.
There is no single "fix" for the existing shipping methods module, as there's a fundamental conflict between the checkout process, where we want to error if there are no available shipping methods (assuming the administrator has configured the store to do so), and displaying shipping methods pre checkout, where you want to just display nothing if the data is not present to enable calculations.
The way your modules work without the Shipping SuperMod is really the ideal way to provide the functionality, as gathering the shipping methods directly is the most efficient and foolproof way. It's unfortunate that the only way to get the Shipping SuperMod to apply its changes to the list is through the shippingmethods item.
I'd like to reiterate that this issue arose from the fact that we fixed the functionality of the "Require Shipping" store setting. The easiest solution for people that were relying on the broken behavior in core-17 and earlier is to just disable "Require Shipping".
Originally posted by wcw
View Post
There is no single "fix" for the existing shipping methods module, as there's a fundamental conflict between the checkout process, where we want to error if there are no available shipping methods (assuming the administrator has configured the store to do so), and displaying shipping methods pre checkout, where you want to just display nothing if the data is not present to enable calculations.
The way your modules work without the Shipping SuperMod is really the ideal way to provide the functionality, as gathering the shipping methods directly is the most efficient and foolproof way. It's unfortunate that the only way to get the Shipping SuperMod to apply its changes to the list is through the shippingmethods item.
I'd like to reiterate that this issue arose from the fact that we fixed the functionality of the "Require Shipping" store setting. The easiest solution for people that were relying on the broken behavior in core-17 and earlier is to just disable "Require Shipping".
Comment