From: Lazaridis Ilias <ilias@...>
Date: 2011-07-12T01:54:07+09:00
Subject: [ruby-core:37998] [Ruby 1.9 - Bug #5015] method_added" is called in addition to "method_undefined


Issue #5015 has been updated by Lazaridis Ilias.


Magnus Holm wrote:
> But undef_method *does* actually add a new method: it adds a method
 which prevents the superclass' method being called
[...]

Whatever "undef_method" does, is an internal implementation detail, which should *not* call "method_added".

undef method
callback: added     any_method # false callback, "any_method" was *not* added by user
callback: undefined any_method # correct callback, "any_method" was undefined by user

----------------------------------------
Bug #5015: method_added" is called in addition to "method_undefined
http://redmine.ruby-lang.org/issues/5015

Author: Lazaridis Ilias
Status: Open
Priority: Normal
Assignee: 
Category: 
Target version: 
ruby -v: 1.9.2


When a method is undefined via "undef_method :any_method", the following methods are called back:

* method_added 
* method_undefined

The expected behaviour is that only "method_undefined" is called (similar to "remove_method").

-

class String

  def self.method_added(id)
    puts "callback: added     #{id.id2name}"
  end
  
  def self.method_undefined(id)
    puts "callback: undefined #{id.id2name}"
  end
  
  def self.method_removed(id)  
    puts "callback: removed   #{id.id2name}"
  end
  
end

puts "\nundef method (buildin)"
class String
  undef_method :capitalize
end

puts "\ndefine method"
class String
  def any_method(*args)
    nil
  end
end
    
puts "\nundef method"
class String
  undef_method :any_method
end


puts "\ndefine method"
class String
  def any_method(*args)
    nil
  end
end

puts "\nremove method"
class String
  remove_method :any_method
end



-

OUTPUT:


undef method (buildin)
callback: added     capitalize
callback: undefined capitalize

define method
callback: added     any_method

undef method
callback: added     any_method
callback: undefined any_method

define method
callback: added     any_method

remove method
callback: removed   any_method



-- 
http://redmine.ruby-lang.org