From 1d1327e38eaabda10fd7821221c3aad2ee4e0248 Mon Sep 17 00:00:00 2001 From: Elia Schito Date: Thu, 7 Mar 2019 17:47:25 +0100 Subject: [PATCH] Align deprecation messages After update! => recalculate rename. ref: - https://github.com/solidusio/solidus/pull/1689 - https://github.com/solidusio/solidus/pull/2072 --- core/app/models/spree/order.rb | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/core/app/models/spree/order.rb b/core/app/models/spree/order.rb index d0d7c7be25d..96874a1daff 100644 --- a/core/app/models/spree/order.rb +++ b/core/app/models/spree/order.rb @@ -366,11 +366,11 @@ def line_item_options_match(line_item, options) # Creates new tax charges if there are any applicable rates. If prices already # include taxes then price adjustments are created instead. - # @deprecated This now happens during #update! + # @deprecated This now happens during #recalculate def create_tax_charge! Spree::Config.tax_adjuster_class.new(self).adjust! end - deprecate create_tax_charge!: :update!, deprecator: Spree::Deprecation + deprecate create_tax_charge!: :recalculate, deprecator: Spree::Deprecation def reimbursement_total reimbursements.sum(:total) @@ -577,11 +577,12 @@ def shipping_eq_billing_address? bill_address == ship_address end + # @deprecated This now happens during #recalculate def set_shipments_cost shipments.each(&:update_amounts) recalculate end - deprecate set_shipments_cost: :update!, deprecator: Spree::Deprecation + deprecate set_shipments_cost: :recalculate, deprecator: Spree::Deprecation def is_risky? payments.risky.count > 0