PoC: Presenters [WIP]

core/app/helpers/spree/base_helper.rb

@@ -119,6 +119,10 @@ def available_countries(restrict_to_zone: Spree::Config[:checkout_zone])
         countries = Country.all
       end
 
+      Spree::CountryListPresenter.new(countries)
+    end
+
+    def countries_for_collection_select(countries)
       country_names = Carmen::Country.all.map do |country|
         [country.code, country.name]
       end.to_h

core/app/presenters/spree/country_list_presenter.rb

@@ -0,0 +1,15 @@
+module Spree
+  class CountryListPresenter < Spree::Presenter::Base
+    def initialize(presentee)
+      presentee = prepare_countries(presentee)
+
+      super
+    end
+
+    protected
+
+    def prepare_countries(countries)
+      countries.map { |country| Spree::CountryPresenter.new(country) }.sort_by { |c| c.name.parameterize }
+    end
+  end
+end

core/app/presenters/spree/country_presenter.rb

@@ -0,0 +1,21 @@
+module Spree
+  class CountryPresenter < Spree::Presenter::Base
+    def name
+      translated_name || carmen_name || super
+    end
+
+    def carmen_country
+      Carmen::Country.coded iso
+    end
+
+    def carmen_name
+      carmen_country.try :name
+    end
+
+    def translated_name
+      return nil unless iso
+
+      I18n.t("spree.country_names.#{iso}", default: nil)
+    end
+  end
+end

core/lib/spree/core.rb

@@ -86,6 +86,7 @@ class GatewayError < RuntimeError; end
 require 'spree/core/stock_configuration'
 require 'spree/core/validators/email'
 require 'spree/permission_sets'
+require 'spree/presenter'
 
 require 'spree/preferences/store'
 require 'spree/preferences/static_model_preferences'

core/lib/spree/presenter.rb

@@ -0,0 +1,7 @@
+module Spree
+  module Presenter
+  end
+end
+
+require 'spree/presenter/base'
+

core/lib/spree/presenter/base.rb

@@ -0,0 +1,13 @@
+module Spree
+  module Presenter
+    class Base < SimpleDelegator
+      attr_reader :presentee
+
+      def initialize(presentee)
+        @presentee = presentee
+
+        super(presentee)
+      end
+    end
+  end
+end