money_helper 0.0.2 → 0.0.3

data/lib/money_helper.rb

@@ -4,7 +4,7 @@ require 'active_support/core_ext/object/blank'
 require 'money'
 
 module MoneyHelper
-  
+
   SYMBOL_ONLY = %w{USD GBP EUR MYR TRY} #don't use ISO code
   OK_SYMBOLS = %w{
     $ £ € ¥ 元 р. L ƒ ৳ P R$ K ₡ D ლ ₵ Q G ₹ Rp ₪ ₩ ₭ R RM ₨ ₮ դր. C$ ₦ TL ₲ ₱ T ฿ T$ m ₴ ₫ ៛
@@ -27,9 +27,10 @@ module MoneyHelper
   def self.money_to_text(amount, currency, number_only = false)
     return nil unless amount.present?
     currency = "USD" if currency.blank?
-    symbol = Money::Currency.new(currency).symbol.strip
-    include_symbol = !number_only && OK_SYMBOLS.include?(symbol)
-    (number_only || SYMBOL_ONLY.include?(currency) ? "" : currency + " ") + 
+    symbol = Money::Currency.new(currency).symbol
+    symbol.strip! if symbol.present?
+    include_symbol = !number_only && symbol.present? && OK_SYMBOLS.include?(symbol)
+    (number_only || SYMBOL_ONLY.include?(currency) ? "" : currency + " ") +
       Money.parse(amount.ceil, currency).format({
         no_cents: true,
         symbol_position: :before,
@@ -66,5 +67,5 @@ module MoneyHelper
       [ money_to_text(low, currency), money_to_text(high, currency, true) ].compact.join(delimiter)
     end
   end
-  
+
 end

data/spec/money_helper_spec.rb

@@ -102,6 +102,9 @@ describe MoneyHelper do
       MoneyHelper.money_to_text(30000, "WST").should eql("WST T30,000")
       MoneyHelper.money_to_text(30000, "ZAR").should eql("ZAR R30,000")
     end
+    it "includes only ISO code for currencies that don't have a symbol" do
+      MoneyHelper.money_to_text(30000, "UZS").should eql("UZS 30,000")
+    end
     it "includes only ISO code for RTL symbols" do
       MoneyHelper.money_to_text(30000, "AFN").should eql("AFN 30,000")
       MoneyHelper.money_to_text(30000, "DZD").should eql("DZD 30,000")
@@ -127,12 +130,6 @@ describe MoneyHelper do
       MoneyHelper.money_to_text(30000, "").should eql("$30,000")
       MoneyHelper.money_to_text(30000, nil).should eql("$30,000")
     end
-    it "returns only formatted numeral when number_only = true" do
-      MoneyHelper.money_to_text(30000, "EUR", true).should eql("30.000")
-      MoneyHelper.money_to_text(30000, "AUD", true).should eql("30,000")
-      MoneyHelper.money_to_text(30000, "AMD", true).should eql("30,000")
-      MoneyHelper.money_to_text(30000, "AFN", true).should eql("30,000")
-    end
     it "returns only the formatted numeral when number_only = true" do
       MoneyHelper.money_to_text(30000, "EUR", true).should eql("30.000")
       MoneyHelper.money_to_text(30000, "AUD", true).should eql("30,000")
@@ -171,4 +168,4 @@ describe MoneyHelper do
       MoneyHelper.money_range_to_text(nil, nil, "USD").should be_nil
     end
   end
-end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: money_helper
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
   prerelease: 
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-07-16 00:00:00.000000000 Z
+date: 2013-07-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: money
@@ -84,7 +84,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -2590949666229956668
+      hash: 2615965141994932059
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -93,7 +93,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -2590949666229956668
+      hash: 2615965141994932059
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.25