Address code review comments

Mostly styling issues.
This commit is contained in:
Pau Perez
2020-07-16 15:30:28 +02:00
parent 70afcee3fc
commit dd5e679f69
2 changed files with 2 additions and 6 deletions

View File

@@ -10,6 +10,7 @@ module Spree
IDENTIFIER_CHARS = (('A'..'Z').to_a + ('0'..'9').to_a - %w(0 1 I O)).freeze
delegate :line_items, to: :order
delegate :currency, to: :order
belongs_to :order, class_name: 'Spree::Order'
belongs_to :source, polymorphic: true
@@ -68,8 +69,6 @@ module Spree
end
end
delegate :currency, to: :order
def money
Spree::Money.new(amount, currency: currency)
end
@@ -87,9 +86,6 @@ module Spree
credit_allowed.positive?
end
# see https://github.com/spree/spree/issues/981
#
# Import from future Spree v.2.3.0 d470b31798f37
def build_source
return if source_attributes.nil?
return unless payment_method.andand.payment_source_class

View File

@@ -12,7 +12,7 @@ module Spree
unless payment_method.supports?(source)
invalidate!
raise Core::GatewayError.new(Spree.t(:payment_method_not_supported))
raise Core::GatewayError, Spree.t(:payment_method_not_supported)
end
if payment_method.auto_capture?