Permalink
Browse files

fix indifferent hash. The lambdas were acting funny in a running rail…

…s app. closes #2176

git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@3987 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
  • Loading branch information...
technoweenie committed Mar 20, 2006
1 parent 4e7c6f5 commit afde9943eaec9c7370c2638316128e236222df82
Showing with 15 additions and 5 deletions.
  1. +15 −5 activesupport/lib/active_support/core_ext/hash/indifferent_access.rb
@@ -8,12 +8,22 @@ def initialize(constructor = {})
super(constructor)
end
end
-
- convert_key_and_hashes_and_call_super = lambda { |key, *args| super(convert_key(key), *args.map{|arg| convert_value(arg) }) }
- convert_other_hash_and_call_super = lambda { |other_hash| super(convert_hash(other_hash)) }
- %w( [] []= fetch store delete has_key? include? key? member? ).each{ |method_name| define_method method_name, &convert_key_and_hashes_and_call_super }
- %w( == eql? replace initialize_copy merge merge! update ).each{ |method_name| define_method method_name, &convert_other_hash_and_call_super }
+ %w( [] []= fetch store delete has_key? include? key? member? ).each do |method_name|
+ class_eval %(
+ def #{method_name}(key, *args)
+ super(convert_key(key), *args.map { |arg| convert_value(arg) })
+ end
+ )
+ end
+ #define_method method_name, &convert_key_and_hashes_and_call_super }
+ %w( == eql? replace initialize_copy merge merge! update ).each do |method_name|
+ class_eval %(
+ def #{method_name}(other_hash)
+ super(convert_hash(other_hash))
+ end
+ )
+ end
def invert
self.class.new.replace(super)

0 comments on commit afde994

Please sign in to comment.