Merge pull request #334 from solidusio/nebulab/rainerd+elia/add-payme…

…nt-info-partial-handling

Add partial handling of payment information for payment provider custom integrations.

templates/app/helpers/checkout_helper.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+module CheckoutHelper
+  def partial_name_with_fallback(prefix, partial_name, fallback_name = 'default')
+    if lookup_context.find_all("#{prefix}/_#{partial_name}").any?
+      "#{prefix}/#{partial_name}"
+    else
+      "#{prefix}/#{fallback_name}"
+    end
+  end
+end

templates/app/views/orders/_payment_info.html.erb

@@ -10,19 +10,14 @@
   ) unless @order.completed? %>
 
   <% order.payments.valid.each do |payment| %>
-    <% source = payment.source %>
-
     <ul class="payment-info__info">
-      <% if source.is_a?(Spree::CreditCard) %>
-        <li>
-          <% if source.last_digits %>
-            <%= t('spree.ending_in') %> <%= source.last_digits %>
-          <% end %>
-        </li>
-        <li><%= source.name %></li>
-      <% else %>
-        <li><%= payment.payment_method.name %></li>
-      <% end %>
+      <li>
+        <%= render partial_name_with_fallback(
+          'orders/payment_info',
+          payment.payment_method.partial_name,
+          'default'
+        ), payment: payment %>
+      </li>
       <li>
         <strong><%= payment.display_amount %></strong>
         (<%= t(payment.state, scope: 'spree.payment_states') %>)

templates/app/views/orders/payment_info/_check.html.erb

@@ -0,0 +1 @@
+<%= t('spree.check') %>

templates/app/views/orders/payment_info/_default.html.erb

@@ -0,0 +1 @@
+<%= payment.payment_method.name %>

templates/app/views/orders/payment_info/_gateway.html.erb

@@ -0,0 +1,5 @@
+<% source = payment.source %>
+
+<% if source.last_digits %>
+  <%= t('spree.ending_in') %> <%= source.last_digits %>
+<% end %>

templates/spec/helpers/checkout_helper_spec.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+
+require 'solidus_starter_frontend_spec_helper'
+
+RSpec.describe CheckoutHelper, type: :helper do
+  describe '#partial_name_with_fallback' do
+    it "uses the partial_name if it exists" do
+      expect(
+        partial_name_with_fallback('orders/payment_info', 'gateway', 'default')
+      ).to eq('orders/payment_info/gateway')
+    end
+
+    it "uses the fallback_name if it's missing" do
+      expect(
+        partial_name_with_fallback('orders/payment_info', 'foo', 'default')
+      ).to eq('orders/payment_info/default')
+    end
+  end
+end