Deprecate set_shipments_cost

solidusio · Mar 21, 2017 · e5fce46 · e5fce46
1 parent 85407b6
commit e5fce46
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/core/app/models/spree/order.rb b/core/app/models/spree/order.rb
@@ -570,6 +570,7 @@ def set_shipments_cost
       shipments.each(&:update_amounts)
       update!
     end
+    deprecate set_shipments_cost: :update!, deprecator: Spree::Deprecation
 
     def is_risky?
       payments.risky.count > 0

diff --git a/core/spec/models/spree/order_spec.rb b/core/spec/models/spree/order_spec.rb
@@ -109,7 +109,9 @@
       expect(shipment).to receive :update_amounts
       expect(order.updater).to receive :update
 
-      order.set_shipments_cost
+      Spree::Deprecation.silence do
+        order.set_shipments_cost
+      end
     end
   end