Move method delegation from decorator to model

openfoodfoundation · Jun 26, 2020 · 4b360f1 · 4b360f1
1 parent 1d962f8
commit 4b360f1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/app/models/spree/payment.rb b/app/models/spree/payment.rb
@@ -7,6 +7,8 @@ class Payment < ActiveRecord::Base
 
     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
@@ -2,8 +2,6 @@
 
 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