Moved supported currencies from preferencies to store

api/app/controllers/concerns/spree/api/v2/storefront/order_concern.rb

@@ -35,9 +35,7 @@ def find_spree_current_order
           end
 
           def supported_currencies
-            Spree::Config[:supported_currencies].split(',').map do |code|
-              ::Money::Currency.find(code.strip)
-            end
+            spree_current_store.supported_currencies_list
           end
 
           def serialize_order(order)

api/spec/controllers/spree/api/v1/stores_controller_spec.rb

@@ -76,7 +76,8 @@ module Spree
           name: 'Hack0rz',
           url: 'spree123.example.com',
           mail_from_address: '[email protected]',
-          default_currency: 'USD'
+          default_currency: 'USD',
+          supported_currencies: 'USD'
         }
         api_post :create, store: store_hash
         expect(response.status).to eq(201)

api/spec/requests/spree/api/v2/storefront/cart_spec.rb

@@ -2,7 +2,7 @@
 
 describe 'API V2 Storefront Cart Spec', type: :request do
   let(:default_currency) { 'USD' }
-  let(:store) { create(:store, default_currency: default_currency) }
+  let!(:store) { create(:store, default_currency: default_currency) }
   let(:currency) { store.default_currency }
   let(:user)  { create(:user) }
   let(:order) { create(:order, user: user, store: store, currency: currency) }
@@ -359,10 +359,6 @@
     end
 
     context 'for specified currency' do
-      before do
-        Spree::Config[:supported_currencies] = 'USD,EUR'
-      end
-
       context 'store default' do
         before do
           store.update!(default_currency: 'EUR')

backend/app/controllers/spree/admin/general_settings_controller.rb

@@ -27,7 +27,7 @@ def clear_cache
       end
 
       def render(*args)
-        @preferences_currency |= [:allow_currency_change, :show_currency_selector, :supported_currencies]
+        @preferences_currency |= [:allow_currency_change, :show_currency_selector]
         super
       end
 

backend/app/views/spree/admin/general_settings/_form.html.erb

@@ -4,12 +4,6 @@
   </div>
 
   <div class="panel-body">
-    <div class="form-group">
-      <label><%= Spree.t('supported_currencies.short') %></label>
-      <input type="text" class="form-control" value="<%= Spree::Config.supported_currencies %>" name="supported_currencies"/>
-      <p class="help-block"><%= Spree.t('supported_currencies.long') %></p>
-    </div>
-
     <div class="form-group">
       <div class="checkbox">
         <label>

backend/app/views/spree/admin/stores/_form.html.erb

@@ -29,6 +29,11 @@
     <%= f.text_field :mail_from_address, class: 'form-control', required: true %>
     <%= f.error_message_on :mail_from_address %>
   <% end %>
+  <%= f.field_container :supported_currencies, class: ['form-group'] do %>
+    <%= f.label :supported_currencies, Spree.t(:supported_currencies) %>
+    <%= f.text_field :supported_currencies, class: 'form-control' %>
+    <%= f.error_message_on :supported_currencies %>
+  <% end %>
   <%= f.field_container :default_currency, class: ['form-group'] do %>
     <%= f.label :default_currency, Spree.t(:default_currency) %>
     <%= f.text_field :default_currency, class: 'form-control' %>

backend/spec/features/admin/configuration/stores_spec.rb

@@ -34,6 +34,7 @@
       page.fill_in 'store_mail_from_address', with: '[email protected]'
       page.fill_in 'store_code', with: 'SPR'
       page.fill_in 'store_default_currency', with: 'EUR'
+      page.fill_in 'store_supported_currencies', with: 'EUR'
       click_button 'Create'
 
       expect(page).to have_current_path spree.admin_stores_path
@@ -53,13 +54,16 @@
       click_link 'Edit'
       page.fill_in 'store_name', with: updated_name
       page.fill_in 'store_default_currency', with: new_currency
+      page.fill_in 'store_supported_currencies', with: new_currency
       click_button 'Update'
 
       expect(page).to have_current_path spree.admin_stores_path
       store_table = page.find('table')
       expect(store_table).to have_content(updated_name)
       expect(store_table).to have_content(new_currency)
-      expect(store.reload.name).to eq updated_name
+      store.reload
+      expect(store.name).to eq updated_name
+      expect(store.supported_currencies).to eq new_currency
     end
   end
 

backend/spec/features/admin/general_settings/prices_settings_presence_spec.rb

@@ -7,7 +7,6 @@
     it 'Multi Currency settings are present' do
       visit spree.edit_admin_general_settings_path
 
-      expect(page).to have_content 'Supported Currencies'
       expect(page).to have_content 'Allow Currency Change'
       expect(page).to have_content 'Show Currency Selector'
     end

backend/spec/features/admin/general_settings/prices_settings_updating_spec.rb

@@ -1,11 +1,11 @@
 require 'spec_helper'
 
 describe 'Updating currencies settings', type: :feature, js: true do
+  let!(:store) { create(:store, default: true) }
   stub_authorization!
 
   before do
     reset_spree_preferences do |config|
-      config.supported_currencies = 'USD'
       config.allow_currency_change = false
       config.show_currency_selector = false
     end
@@ -15,19 +15,16 @@
     visit spree.edit_admin_general_settings_path
 
     # Test initial state
-    expect(page).to have_field('supported_currencies', with: 'USD')
     expect(page).to have_unchecked_field('allow_currency_change')
     expect(page).to have_unchecked_field('show_currency_selector')
 
     # Interact with the form
-    fill_in 'supported_currencies', with: 'USD,PLN'
     check('allow_currency_change')
     check('show_currency_selector')
     click_button 'Update'
 
     # Test final state
     expect(page).to have_content 'General Settings has been successfully updated!'
-    expect(page).to have_field('supported_currencies', with: 'USD,PLN')
     expect(page).to have_checked_field('allow_currency_change')
     expect(page).to have_checked_field('show_currency_selector')
   end

backend/spec/features/admin/products/variant_prices_spec.rb

@@ -2,13 +2,12 @@
 
 describe 'Variant Prices', type: :feature, js: true do
   stub_authorization!
-
+  let!(:store) { create(:store, default: true) }
   let!(:product) { create(:product) }
 
   context 'with USD and EUR as currencies' do
     before do
       reset_spree_preferences do |config|
-        config.supported_currencies = 'USD,EUR'
         config.allow_currency_change = true
       end
     end

core/app/models/spree/app_configuration.rb

@@ -69,7 +69,6 @@ class AppConfiguration < Preferences::Configuration
     # Multi currency configurations
     preference :allow_currency_change,  :boolean, default: false
     preference :show_currency_selector, :boolean, default: false
-    preference :supported_currencies,   :string,  default: 'USD'
 
     # searcher_class allows spree extension writers to provide their own Search class
     def searcher_class

core/app/models/spree/store.rb

@@ -9,6 +9,10 @@ class Store < Spree::Base
       validates :default_currency
     end
 
+    if connection.column_exists?(:spree_stores, :supported_currencies)
+      validates :supported_currencies, presence: true
+    end
+
     before_save :ensure_default_exists_and_is_unique
     before_destroy :validate_not_default
 
@@ -27,6 +31,12 @@ def self.default
       end
     end
 
+    def supported_currencies_list
+      currencies = (read_attribute(:supported_currencies).to_s.split(',') << default_currency).map(&:to_s).map do |code|
+        ::Money::Currency.find(code.strip)
+      end.uniq.compact
+    end
+
     private
 
     def ensure_default_exists_and_is_unique

core/db/migrate/20191017121054_add_supported_currencies_to_store.rb

@@ -0,0 +1,8 @@
+class AddSupportedCurrenciesToStore < ActiveRecord::Migration[6.0]
+  def change
+    add_column :spree_stores, :supported_currencies, :string
+    Spree::Store.all.each do |store|
+      store.update_attribute(:supported_currencies, store.default_currency)
+    end
+  end
+end

core/lib/spree/core/controller_helpers/currency_helpers.rb

@@ -7,7 +7,7 @@ def self.included(receiver)
         end
 
         def supported_currencies
-          Spree::Config[:supported_currencies].split(',').map { |code| ::Money::Currency.find(code.strip) }
+          current_store.supported_currencies_list
         end
       end
     end

core/lib/spree/core/controller_helpers/store.rb

@@ -20,9 +20,7 @@ def current_currency
         end
 
         def supported_currencies
-          Spree::Config[:supported_currencies].split(',').map do |code|
-            ::Money::Currency.find(code.strip)
-          end
+          current_store.supported_currencies_list
         end
 
         def current_store

core/lib/spree/permitted_attributes.rb

@@ -39,7 +39,7 @@ module PermittedAttributes
     ]
 
     @@checkout_attributes = [
-      :coupon_code, :email, :shipping_method_id, :special_instructions, :use_billing, 
+      :coupon_code, :email, :shipping_method_id, :special_instructions, :use_billing,
       :user_id, :bill_address_id, :ship_address_id
     ]
 
@@ -97,7 +97,8 @@ module PermittedAttributes
     ]
 
     @@store_attributes = [:name, :url, :seo_title, :code, :meta_keywords,
-                          :meta_description, :default_currency, :mail_from_address]
+                          :meta_description, :default_currency, :supported_currencies,
+                          :mail_from_address]
 
     @@store_credit_attributes = %i[amount currency category_id memo]
 

core/lib/spree/testing_support/factories/store_factory.rb

@@ -1,9 +1,10 @@
 FactoryBot.define do
   factory :store, class: Spree::Store do
     sequence(:code)   { |i| "spree_#{i}" }
-    name              { 'Spree Test Store' }
-    url               { 'www.example.com' }
-    mail_from_address { '[email protected]' }
-    default_currency  { 'USD' }
+    name                  { 'Spree Test Store' }
+    url                   { 'www.example.com' }
+    mail_from_address     { '[email protected]' }
+    default_currency      { 'USD' }
+    supported_currencies  { 'USD,EUR,GBP' }
   end
 end

core/spec/lib/spree/core/controller_helpers/store_spec.rb

@@ -17,6 +17,15 @@ class FakesController < ApplicationController
     end
   end
 
+  describe '#supported_currencies' do
+    let(:currency) { 'EUR' }
+    let!(:store) { create :store, default: true, supported_currencies: currency }
+
+    it 'returns supported currencies' do
+      expect(controller.supported_currencies).to include(::Money::Currency.find(currency))
+    end
+  end
+
   describe '#current_price_options' do
     subject(:current_price_options) { controller.current_price_options }
 

core/spec/models/spree/store_spec.rb

@@ -72,5 +72,15 @@
         expect(Spree::Store.default.default).to be(true)
       end
     end
+
+    describe '.supported_currencies_list' do
+      let(:currencies) { 'USD, EUR, dummy' }
+      let!(:store) { create(:store, supported_currencies: currencies) }
+      it 'returns supported currencies list' do
+        expect(store.supported_currencies_list).to contain_exactly(
+          ::Money::Currency.find('USD'), ::Money::Currency.find('EUR')
+        )
+      end
+    end
   end
 end

frontend/spec/features/order_prices_spec.rb

@@ -1,14 +1,15 @@
 require 'spec_helper'
 
 describe 'Order', type: :feature, js: true do
+  let!(:store) { create(:store, default: true) }
   let!(:product) { create(:product) }
 
   before do
     reset_spree_preferences do |config|
-      config.supported_currencies   = 'USD,EUR,GBP'
       config.allow_currency_change  = true
       config.show_currency_selector = true
     end
+
     create(:price, variant: product.master, currency: 'EUR', amount: 16.00)
     create(:price, variant: product.master, currency: 'GBP', amount: 23.00)
   end

frontend/spec/features/product_prices_spec.rb

@@ -2,11 +2,11 @@
 
 describe 'Product with prices in multiple currencies', type: :feature, js: true do
   context 'with USD, EUR and GBP as currencies' do
+    let!(:store) { create(:store, default: true) }
     let!(:product) { create(:product) }
 
     before do
       reset_spree_preferences do |config|
-        config.supported_currencies   = 'USD,EUR,GBP'
         config.allow_currency_change  = true
         config.show_currency_selector = true
       end
@@ -26,7 +26,6 @@
     context 'and :show_currency_selector is false' do
       before do
         reset_spree_preferences do |config|
-          config.supported_currencies   = 'USD,EUR,GBP'
           config.allow_currency_change  = true
           config.show_currency_selector = false
         end
@@ -43,7 +42,6 @@
       context 'and show_currency_selector is true' do
         before do
           reset_spree_preferences do |config|
-            config.supported_currencies   = 'USD,EUR,GBP'
             config.allow_currency_change  = false
             config.show_currency_selector = true
           end

frontend/spec/features/template_rendering_spec.rb

@@ -7,7 +7,8 @@
       name: 'My Spree Store',
       url: 'spreestore.example.com',
       mail_from_address: '[email protected]',
-      default_currency: 'USD'
+      default_currency: 'USD',
+      supported_currencies: 'USD'
     )
 
     visit spree.root_path