Permalink
Browse files

* ext/psych/lib/psych/visitors/yaml_tree.rb: register nodes when

  dumping objects with custom coders. [ruby-core:66215] [Bug #10496]

* test/psych/test_coder.rb: test for fix
  • Loading branch information...
1 parent 6a191f4 commit 43fa4aa38492c80080efa61aa4b7bee422792bc7 @tenderlove tenderlove committed Feb 6, 2015
Showing with 33 additions and 3 deletions.
  1. +7 −0 CHANGELOG.rdoc
  2. +4 −3 lib/psych/visitors/yaml_tree.rb
  3. +22 −0 test/psych/test_coder.rb
View
@@ -1,3 +1,10 @@
+Fri Feb 6 17:47:05 2015 Aaron Patterson <aaron@tenderlovemaking.com>
+
+ * ext/psych/lib/psych/visitors/yaml_tree.rb: register nodes when
+ dumping objects with custom coders. [ruby-core:66215] [Bug #10496]
+
+ * test/psych/test_coder.rb: test for fix
+
Fri Feb 6 16:58:31 2015 Aaron Patterson <aaron@tenderlovemaking.com>
* ext/psych/lib/psych/visitors/to_ruby.rb: fix support for regular
@@ -21,6 +21,7 @@ def initialize
end
def register target, node
+ return unless target.respond_to? :object_id
@targets << target
@obj_to_node[target.object_id] = node
end
@@ -566,10 +567,10 @@ def dump_coder o
c = Psych::Coder.new(tag)
o.encode_with(c)
- emit_coder c
+ emit_coder c, o
end
- def emit_coder c
+ def emit_coder c, o
case c.type
when :scalar
@emitter.scalar c.scalar, nil, c.tag, c.tag.nil?, false, Nodes::Scalar::ANY
@@ -580,7 +581,7 @@ def emit_coder c
end
@emitter.end_sequence
when :map
- @emitter.start_mapping nil, c.tag, c.implicit, c.style
+ register o, @emitter.start_mapping(nil, c.tag, c.implicit, c.style)
c.map.each do |k,v|
accept k
accept v
@@ -95,6 +95,28 @@ def encode_with coder
end
end
+ class Referential
+ attr_reader :a
+
+ def initialize
+ @a = self
+ end
+
+ def encode_with(c)
+ c['a'] = @a
+ end
+
+ def init_with(c)
+ @a = c['a']
+ end
+ end
+
+ def test_self_referential
+ x = Referential.new
+ copy = Psych.load Psych.dump x
+ assert_equal copy, copy.a
+ end
+
def test_represent_with_object
thing = Psych.load(Psych.dump(RepresentWithObject.new))
assert_equal 20, thing

0 comments on commit 43fa4aa

Please sign in to comment.