hashie 2.0.4 → 2.0.5

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # CHANGELOG
 
+## 2.0.5
+
+* make coercion work better with non-symbol keys in mash wapcaplet #96
+
 ## 2.0.4
 
 * make #fetch method consistent with normal Hash markiz #94

data/lib/hashie/extensions/coercion.rb

@@ -21,6 +21,10 @@ module Hashie
           super(key, value)
         end
 
+        def custom_writer(key, value)
+          self[key] = value
+        end
+
         def replace(other_hash)
           (keys - other_hash.keys).each { |key| delete(key) }
           other_hash.each { |key, value| self[key] = value }
@@ -31,7 +35,7 @@ module Hashie
       module ClassMethods
         # Set up a coercion rule such that any time the specified
         # key is set it will be coerced into the specified class.
-        # Coercion will occur by first attempting to call Class.coerce 
+        # Coercion will occur by first attempting to call Class.coerce
         # and then by calling Class.new with the value as an argument
         # in either case.
         #
@@ -59,7 +63,7 @@ module Hashie
         # Returns the specific key coercion for the specified key,
         # if one exists.
         def key_coercion(key)
-          key_coercions[key]
+          key_coercions[key.to_sym]
         end
 
         # Set up a coercion rule such that any time a value of the
@@ -94,7 +98,7 @@ module Hashie
             end
           end
         end
-        
+
         # Return all value coercions that have the :strict rule as true.
         def strict_value_coercions; @strict_value_coercions || {} end
         # Return all value coercions that have the :strict rule as false.
@@ -103,7 +107,7 @@ module Hashie
         # Fetch the value coercion, if any, for the specified object.
         def value_coercion(value)
           from = value.class
-          strict_value_coercions[from] || lenient_value_coercions[from] 
+          strict_value_coercions[from] || lenient_value_coercions[from]
         end
       end
     end

data/lib/hashie/mash.rb

@@ -82,7 +82,7 @@ module Hashie
 
     # Retrieves an attribute set in the Mash. Will convert
     # any key passed in to a string before retrieving.
-    def [](key)
+    def custom_reader(key)
       value = regular_reader(convert_key(key))
       yield value if block_given?
       value
@@ -91,10 +91,13 @@ module Hashie
     # Sets an attribute in the Mash. Key will be converted to
     # a string before it is set, and Hashes will be converted
     # into Mashes for nesting purposes.
-    def []=(key,value) #:nodoc:
+    def custom_writer(key,value) #:nodoc:
       regular_writer(convert_key(key), convert_value(value))
     end
 
+    alias_method :[], :custom_reader
+    alias_method :[]=, :custom_writer
+
     # This is the bang method reader, it will return a new Mash
     # if there isn't a value already assigned to the key requested.
     def initializing_reader(key)
@@ -148,11 +151,11 @@ module Hashie
       other_hash.each_pair do |k,v|
         key = convert_key(k)
         if regular_reader(key).is_a?(Mash) and v.is_a?(::Hash)
-          regular_reader(key).deep_update(v, &blk)
+          custom_reader(key).deep_update(v, &blk)
         else
           value = convert_value(v, true)
           value = blk.call(key, self[k], value) if blk
-          regular_writer(key, value)
+          custom_writer(key, value)
         end
       end
       self

data/lib/hashie/version.rb

@@ -1,3 +1,3 @@
 module Hashie
-  VERSION = '2.0.4'
+  VERSION = '2.0.5'
 end

data/spec/hashie/extensions/coercion_spec.rb

@@ -77,6 +77,38 @@ describe Hashie::Extensions::Coercion do
         instance[:hi].should == "bye"
       end
     end
+
+    context "when used with a Mash" do
+      class UserMash < Hashie::Mash
+      end
+      class TweetMash < Hashie::Mash
+        include Hashie::Extensions::Coercion
+        coerce_key :user, UserMash
+      end
+
+      it "should coerce with instance initialization" do
+        tweet = TweetMash.new(:user => {:email => 'foo@bar.com'})
+        tweet[:user].should be_a(UserMash)
+      end
+
+      it "should coerce when setting with attribute style" do
+        tweet = TweetMash.new
+        tweet.user = {:email => 'foo@bar.com'}
+        tweet[:user].should be_a(UserMash)
+      end
+
+      it "should coerce when setting with string index" do
+        tweet = TweetMash.new
+        tweet['user'] = {:email => 'foo@bar.com'}
+        tweet[:user].should be_a(UserMash)
+      end
+
+      it "should coerce when setting with symbol index" do
+        tweet = TweetMash.new
+        tweet[:user] = {:email => 'foo@bar.com'}
+        tweet[:user].should be_a(UserMash)
+      end
+    end
   end
 
   describe '.coerce_value' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hashie
 version: !ruby/object:Gem::Version
-  version: 2.0.4
+  version: 2.0.5
   prerelease: 
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-04-24 00:00:00.000000000 Z
+date: 2013-05-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -157,7 +157,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2159890714195675822
+      hash: 214342929644806912
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -166,7 +166,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2159890714195675822
+      hash: 214342929644806912
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23