From d49068ce66bbe578d6d36d69eefbe5995c24d916 Mon Sep 17 00:00:00 2001 From: Pau Perez Date: Fri, 26 Jun 2020 18:31:19 +0200 Subject: [PATCH] Move method delegation from decorator to model --- app/models/spree/payment.rb | 2 ++ app/models/spree/payment_decorator.rb | 2 -- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/app/models/spree/payment.rb b/app/models/spree/payment.rb index de13489922..a434400442 100644 --- a/app/models/spree/payment.rb +++ b/app/models/spree/payment.rb @@ -7,6 +7,8 @@ module Spree IDENTIFIER_CHARS = (('A'..'Z').to_a + ('0'..'9').to_a - %w(0 1 I O)).freeze + delegate :line_items, to: :order + belongs_to :order, class_name: 'Spree::Order' belongs_to :source, polymorphic: true belongs_to :payment_method, class_name: 'Spree::PaymentMethod' diff --git a/app/models/spree/payment_decorator.rb b/app/models/spree/payment_decorator.rb index a66346dd7e..f79529e306 100644 --- a/app/models/spree/payment_decorator.rb +++ b/app/models/spree/payment_decorator.rb @@ -2,8 +2,6 @@ require 'spree/localized_number' module Spree Payment.class_eval do - delegate :line_items, to: :order - # We bypass this after_rollback callback that is setup in Spree::Payment # The issues the callback fixes are not experienced in OFN: # if a payment fails on checkout the state "failed" is persisted correctly