Money#format: correctly format symbol according to currency when :sym…

…bol option is not given.
ismasan · Nov 5, 2009 · fd228db · fd228db
1 parent b723ad9
commit fd228db
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/lib/money/money.rb b/lib/money/money.rb
@@ -226,7 +226,7 @@ def format(*rules)
         symbol = ""
       end
     else
-      symbol = "$"
+      symbol = SYMBOLS[currency] || "$"
     end
 
     if rules[:no_cents]

diff --git a/test/money_spec.rb b/test/money_spec.rb
@@ -190,6 +190,17 @@
       money.format(:symbol => false).should == "1.00"
     end
 
+    specify "#format without :symbol assumes that :symbol is set to true" do
+      money = Money.new(100)
+      money.format.should == "$1.00"
+
+      money = Money.new(100, "GBP")
+      money.format.should == "£1.00"
+
+      money = Money.new(100, "XYZ")
+      money.format.should == "$1.00"
+    end
+
     specify "#format(:html => true) works as documented" do
       string = Money.ca_dollar(570).format(:html => true, :with_currency => true)
       string.should == "$5.70 <span class=\"currency\">CAD</span>"