Change RubyMoney bank to currencylayer

thewca · Jan 3, 2025 · cca9d8e · cca9d8e
1 parent 5b00470
commit cca9d8e
Show file tree

Hide file tree

Showing 7 changed files with 20 additions and 9 deletions.
diff --git a/.env.development b/.env.development
@@ -37,3 +37,4 @@ AVATARS_PUBLIC_STORAGE=local
 AVATARS_PRIVATE_STORAGE=local_private
 DUMP_HOST=https://assets.worldcubeassociation.org
 CRONJOB_POLLING_SECONDS=0
+CURRENCY_LAYER_API_KEY=9d254ec37e1591eec964dac947fafa09
diff --git a/.env.test b/.env.test
@@ -35,3 +35,4 @@ STAGING_OAUTH_SECRET=example-secret
 AVATARS_PUBLIC_STORAGE=local
 AVATARS_PRIVATE_STORAGE=local_private
 DUMP_HOST=https://assets.worldcubeassociation.org
+CURRENCY_LAYER_API_KEY=9d254ec37e1591eec964dac947fafa09
diff --git a/Gemfile b/Gemfile
@@ -45,6 +45,7 @@ gem 'time_will_tell', github: 'thewca/time_will_tell'
 gem 'redcarpet'
 gem 'bootstrap-table-rails'
 gem 'money-rails'
+gem 'money-currencylayer-bank'
 gem 'octokit'
 gem 'stripe'
 gem 'oauth2'

diff --git a/Gemfile.lock b/Gemfile.lock
@@ -462,6 +462,10 @@ GEM
       money (~> 6.12)
     money (6.16.0)
       i18n (>= 0.6.4, <= 2)
+    money-currencylayer-bank (0.7.2)
+      json (>= 1.8)
+      monetize (~> 1.4)
+      money (~> 6.7)
     money-rails (1.15.0)
       activesupport (>= 3.0)
       monetize (~> 1.9)
@@ -884,6 +888,7 @@ DEPENDENCIES
   mail_form
   mini_magick
   momentjs-rails!
+  money-currencylayer-bank
   money-rails
   mysql2
   newrelic_rpm

diff --git a/config/initializers/ruby_money.rb b/config/initializers/ruby_money.rb
@@ -1,8 +1,16 @@
 # frozen_string_literal: true
 
+require 'money/bank/currencylayer_bank'
+
 Money.locale_backend = :i18n
 
-eu_bank = EuCentralBank.new
-Money.default_bank = eu_bank
+mclb = Money::Bank::CurrencylayerBank.new
+mclb.access_key = ENV.fetch("CURRENCY_LAYER_API_KEY", "")
+mclb.currencylayer = true
+mclb.ttl_in_seconds = 86_400
+mclb.cache = 'rails_money_cache.txt' # To be replaced with proper cache path.
+mclb.update_rates
+
+Money.default_bank = mclb
 Money.default_currency = Money::Currency.new("USD")
 Money.rounding_mode = BigDecimal::ROUND_HALF_UP
diff --git a/env_config.rb b/env_config.rb
@@ -33,6 +33,7 @@
     mandatory :ASSET_HOST, :string
     mandatory :CDN_ASSETS_DISTRIBUTION_ID, :string
     mandatory :REGISTRATION_QUEUE, :string
+    mandatory :CURRENCY_LAYER_API_KEY, :string
 
     if is_compiling_assets
       mandatory :V2_REGISTRATIONS_POLL_URL, :string
@@ -59,6 +60,7 @@
     optional :PAYPAL_BASE_URL, :string, ''
     optional :WRC_WEBHOOK_URL, :string, ''
     optional :REGISTRATION_QUEUE, :string, ''
+    optional :CURRENCY_LAYER_API_KEY, :string, ''
 
     optional :V2_REGISTRATIONS_POLL_URL, :string, ''
     optional :V3_REGISTRATIONS_POLL_URL, :string, ''

diff --git a/lib/dues_calculator.rb b/lib/dues_calculator.rb
@@ -1,12 +1,6 @@
 # frozen_string_literal: true
 
 module DuesCalculator
-  def self.update_exchange_rates_if_needed
-    if !Money.default_bank.rates_updated_at || Money.default_bank.rates_updated_at < 1.day.ago
-      Money.default_bank.update_rates
-    end
-  end
-
   def self.dues_for_n_competitors(country_iso2, base_entry_fee_lowest_denomination, currency_code, n)
     dues_per_competitor_in_usd_money = dues_per_competitor_in_usd(country_iso2, base_entry_fee_lowest_denomination, currency_code)
     if dues_per_competitor_in_usd_money.present?
@@ -21,7 +15,6 @@ def self.dues_for_n_competitors(country_iso2, base_entry_fee_lowest_denomination
   def self.dues_per_competitor_in_usd(country_iso2, base_entry_fee_lowest_denomination, currency_code)
     country_band = CountryBand.find_by(iso2: country_iso2)&.number
 
-    DuesCalculator.update_exchange_rates_if_needed
     input_money_us_dollars = Money.new(base_entry_fee_lowest_denomination, currency_code).exchange_to("USD")
 
     registration_fee_dues_us_dollars = input_money_us_dollars * CountryBand.percent_registration_fee_used_for_due_amount(country_band)