Project

General

Profile

Actions

Bug #7893

closed

Issues with loading YAML and respond_to_missing?

Added by injekt (Lee Jarvis) over 11 years ago. Updated over 11 years ago.

Status:
Rejected
Target version:
-
ruby -v:
1.9.3p385
Backport:
[ruby-core:52575]

Description

This is perhaps not a bug, I could be missing something, but shouldn't the following code work?

require 'yaml'

class Person
  def initialize(name)
    @attributes = { name: name }
  end

  def method_missing(m, *args, &block)
    @attributes.key?(m) ? @attributes[m] : super
  end

  def respond_to_missing?(m, include_private = false)
    @attributes.key?(m) || super
  end
end

dump = YAML.dump Person.new("lee")
load = YAML.load dump

The YAML.dump appears to work as expected, yet when I attempt to load the dump back, I get the following exception:

/Users/lee/bin/rb:15:in `respond_to_missing?': undefined method `key?' for nil:NilClass (NoMethodError)
  from /Users/lee/.rubies/1.9.3-p385/lib/ruby/1.9.1/psych/visitors/to_ruby.rb:291:in `respond_to?'
  from /Users/lee/.rubies/1.9.3-p385/lib/ruby/1.9.1/psych/visitors/to_ruby.rb:291:in `init_with'
  from /Users/lee/.rubies/1.9.3-p385/lib/ruby/1.9.1/psych/visitors/to_ruby.rb:284:in `revive'

However doing the same thing with JSON for example, works fine: https://gist.github.com/injekt/0a4aa3e25f09ae2f8cba

Updated by Hanmac (Hans Mackowiak) over 11 years ago

the problem is in yaml: to_ruby.rb "if o.respond_to?(:init_with)" so eigher you need to define a init_with method or

def respond_to_missing?(m, include_private = false)
(@attributes && @attributes.key?(m)) || super
end

Updated by drbrain (Eric Hodel) over 11 years ago

  • Category changed from core to ext
  • Status changed from Open to Rejected
  • Assignee set to tenderlovemaking (Aaron Patterson)

As Hanmac noted, this exception is possible with Person.allocate.respond_to? which is exactly what Psych does.

Perhaps Psych needs some documentation improvement around dump and load.

Actions

Also available in: Atom PDF

Like0
Like0Like0