human_number 0.1.6 → 0.1.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f3e4c992501e9303d1d288b507757e305bbf1cd148322bb5d23046f3ca9a5703
-  data.tar.gz: 74b84f6b21e5e4d7b869f81f4ee41a1f090be8884c07f0d58fb113029afde5de
+  metadata.gz: 42fe66194bf6870029efef196173647a511774b68b13faee24f237d665972722
+  data.tar.gz: bceac8e6a9b6ff97f8737c72068b0e2e98f11c2525dc0572c42218376e0389dc
 SHA512:
-  metadata.gz: c8204b1d746ffda09e0973bc49d7418bbfb5a3355c791839a1afc6f22e2332de1fe9ceb44eee744f92d10d7e15f0e883d3ea8c1b1d171119c84e345160a1b14b
-  data.tar.gz: 4cf5346df66b1dfb6143dec1c61db38532bb172bf9a56fddc0700531941de333292081c7431fe1799e900723289947ef10b35d48252657efef9ec76fba6b6419
+  metadata.gz: d39a9b87d7e0f1a8e7bcecbf7184931bfb35c48047992772a2635aeac8079cbaeb31a766e3443afa789b4fb9f45a1a69abe194ab0f309098c43e6860b384d052
+  data.tar.gz: ea2765ffb9962d98ec8fcd52db74a877edf20cbaa0dae1903c6d8d6edc748016160695734f0f52ca4279a16e65aae3fa235bcce28ec776376440d736fda709cc

data/CHANGELOG.md

@@ -5,7 +5,23 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [0.1.0] - 2024-01-31
+## [0.1.8] - 2025-08-05
+
+### Fixed
+- **I18n fallback contamination**: Resolved currency symbol cross-contamination where Korean won symbols (원) appeared for other currencies like CAD due to I18n fallback behavior
+- **Translation missing errors**: Fixed missing translation errors for various currency/locale combinations
+- **Rails configuration warnings**: Eliminated Rails configuration warnings in test environment
+
+### Changed
+- **Currency native locale mappings**: Corrected CURRENCY_NATIVE_LOCALES to use only accurate locale associations (e.g., CAD now maps to en-CA/fr-CA only, not generic en/fr)
+- **Locale file organization**: Moved locale files from `config/locales/` to `lib/locales/` for better gem structure
+
+### Improved
+- **Rails-i18n integration**: Enhanced loading order and initialization to prevent locale conflicts
+- **Test infrastructure**: Updated RSpec mocks and test expectations for better reliability
+- **I18n isolation**: Added `fallback: false` parameter to prevent cross-locale pollution
+
+## [0.1.0] - 2025-07-31
 
 ### Added
 
@@ -86,4 +102,5 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - **Formatters::Currency**: Currency symbol and format string application
 - **Clean separation**: Number formatting independent of currency concerns
 
+[0.1.8]: https://github.com/ether-moon/human_number/releases/tag/v0.1.8
 [0.1.0]: https://github.com/ether-moon/human_number/releases/tag/v0.1.0

data/lib/human_number/locale_support.rb

@@ -12,50 +12,38 @@ module HumanNumber
     CURRENCY_NATIVE_LOCALES = {
       "KRW" => [:ko],
       "JPY" => [:ja],
-      "USD" => [:en],
+      "USD" => %i[en en-US],
       "EUR" => %i[de fr es it nl],
-      "GBP" => %i[en en-GB],
-      "CAD" => %i[en en-CA fr fr-CA],
+      "GBP" => %i[en-GB],
+      "CAD" => %i[en-CA fr-CA],
       "CNY" => %i[zh zh-CN],
       "CHF" => %i[de fr it],
-      "INR" => %i[hi en],
+      "INR" => %i[hi en-IN],
       "RUB" => [:ru],
-      "BRL" => %i[pt pt-BR],
-      "MXN" => [:es],
-      "AUD" => %i[en en-AU],
-      "NZD" => %i[en en-NZ],
+      "BRL" => %i[pt-BR],
+      "MXN" => %i[es-MX],
+      "AUD" => %i[en-AU],
+      "NZD" => %i[en-NZ],
       "SEK" => [:sv],
       "NOK" => %i[no nb],
       "DKK" => [:da],
-      "SGD" => %i[en en-SG zh zh-SG],
-      "HKD" => %i[zh zh-HK en en-HK],
-      "ZAR" => %i[en en-ZA af],
+      "SGD" => %i[en-SG zh-SG],
+      "HKD" => %i[zh-HK en-HK],
+      "ZAR" => %i[en-ZA af],
       "TRY" => [:tr],
-      "ARS" => %i[es es-AR],
+      "ARS" => %i[es-AR],
       "IDR" => [:id],
       "THB" => [:th],
-      "MYR" => %i[ms en en-MY],
-      "PHP" => %i[en en-PH tl],
+      "MYR" => %i[ms en-MY],
+      "PHP" => %i[en-PH tl],
       "VND" => [:vi],
     }.freeze
 
     # Keys supported for native currency formatting
-    SUPPORTED_NATIVE_CURRENCY_KEYS = %i[unit format native_format].freeze
+    SUPPORTED_NATIVE_CURRENCY_KEYS = %i[unit format].freeze
 
     module_function
 
-    # Returns the primary locale for displaying a currency.
-    # If the user's locale matches a native locale for the currency, use it.
-    # Otherwise, use the first native locale for the currency.
-    #
-    # @param currency_code [String] ISO 4217 currency code (e.g., 'USD', 'EUR')
-    # @param user_locale [Symbol] User's preferred locale
-    # @return [Symbol] Primary locale for currency display
-    def primary_locale_for_currency(currency_code, user_locale)
-      locales = native_locales_for_currency(currency_code)
-      (locales.include?(user_locale.to_sym) ? user_locale.to_sym : locales.first) || :en
-    end
-
     # Returns the locale to use for currency precision rules in standard formatting.
     # Always returns the currency's primary native locale to ensure consistent
     # precision (e.g., USD always shows 2 decimals, JPY shows 0 decimals).
@@ -113,16 +101,24 @@ module HumanNumber
       # Return nil for unknown currencies so formatters can fall back appropriately
       return nil if native_locales_for_currency(currency_code).empty?
 
-      # For native locales, use native currency formatting (native_unit, native_format)
-      if SUPPORTED_NATIVE_CURRENCY_KEYS.include?(key) && native_locale?(currency_code, locale)
-        native_key = native_currency_key(key)
-
-        return I18n.t(native_key, locale:) if I18n.exists?(native_key, locale)
+      # For native locales and supported keys, try native formatting first
+      # Additional validation: only currencies that actually have native symbols
+      if SUPPORTED_NATIVE_CURRENCY_KEYS.include?(key)
+        if native_locale?(currency_code, locale)
+          native_key = native_currency_key(key)
+          return I18n.t(native_key, locale:) if I18n.exists?(native_key, locale, fallback: false)
+        else
+          # For currency symbols (:unit) and format, use currency's native locale to ensure consistency
+          # (e.g., USD always shows "$1,234.56" format, regardless of display locale)
+          target_locale = native_locales_for_currency(currency_code).first || :en
+          standard_key = currency_key(key)
+          return I18n.t(standard_key, locale: target_locale, default: currency_code)
+        end
       end
 
-      # For non-native locales or standard keys, always use the currency's native locale
-      # This ensures we get the correct currency symbol regardless of user's locale
-      I18n.t(currency_key(key), locale: primary_locale_for_currency(currency_code, locale))
+      # For other keys, use user's requested locale
+      standard_key = currency_key(key)
+      I18n.t(standard_key, locale:)
     end
   end
 end

data/lib/human_number/railtie.rb

@@ -16,8 +16,8 @@ module HumanNumber
       end
     end
 
-    initializer "human_number.load_locale_data" do
-      I18n.load_path += Dir[File.expand_path("../../config/locales/*.yml", __dir__)]
+    initializer "human_number.load_locale_data", after: "rails-i18n" do
+      I18n.load_path += Dir[File.expand_path("../locales/*.yml", __dir__)]
     end
   end
 end

data/lib/human_number/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module HumanNumber
-  VERSION = "0.1.6"
+  VERSION = "0.1.8"
 end

data/lib/human_number.rb

@@ -4,9 +4,6 @@ require "i18n"
 require "action_view"
 require "rails-i18n"
 
-# Ensure Rails I18n locales are available
-I18n.load_path += Dir[File.expand_path("**/*.yml", Gem.loaded_specs["rails-i18n"].full_gem_path)]
-
 require_relative "human_number/version"
 require_relative "human_number/locale_support"
 require_relative "human_number/formatters/number"
@@ -15,9 +12,6 @@ require_relative "human_number/formatters/currency"
 module HumanNumber
   class Error < StandardError; end
 
-  # Load locale files
-  I18n.load_path += Dir[File.expand_path("../config/locales/*.yml", __dir__)]
-
   extend ActionView::Helpers::NumberHelper
 
   class << self
@@ -213,4 +207,9 @@ module HumanNumber
   end
 end
 
+# Load locale files for all environments to ensure availability
+# locale_files = Dir[File.expand_path("locales/*.yml", __dir__)]
+# I18n.load_path += locale_files unless (I18n.load_path & locale_files).any?
+# I18n.backend.reload! if I18n.backend.respond_to?(:reload!)
+
 require_relative "human_number/railtie" if defined?(Rails::Railtie)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: human_number
 version: !ruby/object:Gem::Version
-  version: 0.1.6
+  version: 0.1.8
 platform: ruby
 authors:
 - Ether Moon
@@ -77,20 +77,6 @@ files:
 - Makefile
 - README.md
 - Rakefile
-- config/locales/bn.yml
-- config/locales/de.yml
-- config/locales/en-GB.yml
-- config/locales/en-IN.yml
-- config/locales/en.yml
-- config/locales/es.yml
-- config/locales/fr.yml
-- config/locales/hi.yml
-- config/locales/ja.yml
-- config/locales/ko.yml
-- config/locales/ur.yml
-- config/locales/zh-CN.yml
-- config/locales/zh-TW.yml
-- config/locales/zh.yml
 - human_number.gemspec
 - lib/human_number.rb
 - lib/human_number/formatters/currency.rb
@@ -99,6 +85,20 @@ files:
 - lib/human_number/rails/helpers.rb
 - lib/human_number/railtie.rb
 - lib/human_number/version.rb
+- lib/locales/bn.yml
+- lib/locales/de.yml
+- lib/locales/en-GB.yml
+- lib/locales/en-IN.yml
+- lib/locales/en.yml
+- lib/locales/es.yml
+- lib/locales/fr.yml
+- lib/locales/hi.yml
+- lib/locales/ja.yml
+- lib/locales/ko.yml
+- lib/locales/ur.yml
+- lib/locales/zh-CN.yml
+- lib/locales/zh-TW.yml
+- lib/locales/zh.yml
 - scripts/build_and_publish.sh
 homepage: https://github.com/ether-moon/human_number
 licenses: