Changeset 8877

Timestamp:

02/15/08 23:33:43 (7 months ago)

Author:

nzkoz

Message:

Serialize BigDecimals as Floats when using to_yaml. Closes #8746 [ernesto.jimenez]

Files:

• trunk/activesupport/CHANGELOG (modified) (1 diff)
• trunk/activesupport/lib/active_support/core_ext/bigdecimal/conversions.rb (modified) (2 diffs)
• trunk/activesupport/test/core_ext/bigdecimal.rb (added)

trunk/activesupport/CHANGELOG

@@ -1 +1 @@
-*SVN*
+
+* Serialize BigDecimals as Floats when using to_yaml. #8746 [ernesto.jimenez]
-
-* Adding TimeWithZone #to_yaml, #to_datetime, #eql? and method aliases for duck-typing compatibility with Time [Geoff Buesing]

trunk/activesupport/lib/active_support/core_ext/bigdecimal/conversions.rb

@@ +1 @@
+require 'yaml'
+
-class BigDecimal #:nodoc:
-  alias :_original_to_s :to_s
@@ -4 +6 @@
-    _original_to_s(format)
-  end
+
+  yaml_as "tag:yaml.org,2002:float"
+  def to_yaml( opts = {} )
+    YAML::quick_emit( nil, opts ) do |out|
+      # This emits the number without any scientific notation.
+      # I prefer it to using self.to_f.to_s, which would lose precision.
+      #
+      # Note that YAML allows that when reconsituting floats
+      # to native types, some precision may get lost.
+      # There is no full precision real YAML tag that I am aware of.
+      str = self.to_s
+      if str == "Infinity"
+        str = ".Inf"
+      elsif str == "-Infinity"
+        str = "-.Inf"
+      elsif str == "NaN"
+        str = ".NaN"
+      end
+      out.scalar( "tag:yaml.org,2002:float", str, :plain )
+    end
+  end
-end