I am not sure why this method is here. [`create_proposed_shipments` wipes out all shipments as its first thing](https://github.com/spree/spree/blob/892d33521675cb984dcd9090da87aa3248538f98/core/app/models/spree/order.rb#L504-L505). Why not just call the `create_proposed_shipments` method always?