attribution 0.9.0 → 0.9.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a7fbd76f70d8e5491d55e03c1ac6f0b38b095b6b
-  data.tar.gz: afe56be4e9efe8c16595942dd2378917728c0abe
+  metadata.gz: 465049007b0878be5f3011b44be9eae8e2a45e79
+  data.tar.gz: 377c3d034d4a4a3794531c53ef7764a32519b42b
 SHA512:
-  metadata.gz: aad19801c643833ee388d1a389d9f74664a3b02262982e3af5566142c5e0cdcbcb2427ef29bc52579ddd6995d880f98e210a745ecc98074d1dae9e7e93bd364c
-  data.tar.gz: b4d49688cb6eeb36337a32a0c2d2b32301f478c1ee0abae579e2ce2280fdf2f17f479cb07b68ed345b19e61cfdffd8d457725f325799d019ba1b23645db6a86b
+  metadata.gz: 8e0daba150f2620356590745e3d072c37f05f675a68ae91172e57adbf1005fb511516be606f059c56d5233e324fdf229dfcc0bad8d1e459a8192c058b652e274
+  data.tar.gz: 8a94de2fda927d95e33d968125a167f0a38ffd7e70769036283a6c0300bc790d938a204753e3188f820e00671d24e14c8ed46141883fcb1d262bbf2655d13460

data/attribution.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |gem|
   gem.name          = "attribution"
-  gem.version       = "0.9.0"
+  gem.version       = "0.9.1"
   gem.authors       = ["Paul Barry"]
   gem.email         = ["mail@paulbarry.com"]
   gem.description   = %q{Add attributes to Ruby objects}

data/lib/attribution/version.rb

@@ -1,3 +1,3 @@
 module Attribution
-  VERSION = "0.9.0"
+  VERSION = "0.9.1"
 end

data/lib/attribution.rb

@@ -216,7 +216,7 @@ module Attribution
     def hash_attr(attr, metadata={})
       add_attribute(attr, :hash, metadata)
       define_method("#{attr}=") do |arg|
-        instance_variable_set("@#{attr}", Hash(arg))
+        instance_variable_set("@#{attr}", arg || {})
       end
     end
 
@@ -296,12 +296,7 @@ module Attribution
         if instance_variable_defined?("@#{association_name}")
           instance_variable_get("@#{association_name}")
         elsif id = instance_variable_get("@#{association_name}_id")
-
-          begin
-            association_class = Object.const_get(association_class_name)
-          rescue NameError => ex
-            raise ArgumentError.new("Association #{association_name} in #{self.class} is invalid because #{association_class_name} does not exist")
-          end
+          association_class = association_class_name.constantize
 
           if self.class.autoload_associations? && association_class.respond_to?(:find)
             instance_variable_set("@#{association_name}", association_class.find(id))
@@ -313,11 +308,7 @@ module Attribution
 
       # foo=
       define_method("#{association_name}=") do |arg|
-        begin
-          association_class = Object.const_get(association_class_name)
-        rescue NameError => ex
-          raise ArgumentError.new("Association #{association_name} in #{self.class} is invalid because #{association_class_name} does not exist")
-        end
+        association_class = association_class_name.constantize
 
         if instance_variable_defined?("@#{association_name}_id")
           remove_instance_variable("@#{association_name}_id")
@@ -340,13 +331,9 @@ module Attribution
 
       # foos
       define_method(association_name) do |*query|
-        # TODO: Support a more generic version of lazy-loading
-        begin
-          association_class = Object.const_get(association_class_name)
-        rescue NameError => ex
-          raise ArgumentError.new("Association #{association_name} in #{self.class} is invalid because #{association_class_name} does not exist")
-        end
+        association_class = association_class_name.constantize
 
+        # TODO: Support a more generic version of lazy-loading
         if query.empty? # Ex: Books.all, so we want to cache it.
           ivar = "@#{association_name}"
           if instance_variable_defined?(ivar)
@@ -363,12 +350,7 @@ module Attribution
 
       # foos=
       define_method("#{association_name}=") do |arg|
-        # TODO: put this in method
-        begin
-          association_class = Object.const_get(association_class_name)
-        rescue NameError => ex
-          raise ArgumentError.new("Association #{association_name} in #{self.class} is invalid because #{association_class_name} does not exist")
-        end
+        association_class = association_class_name.constantize
 
         attr_name = self.class.name.demodulize.underscore
         objs = (arg.is_a?(Hash) ? arg.values : Array(arg)).map do |obj|

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: attribution
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.9.1
 platform: ruby
 authors:
 - Paul Barry
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-25 00:00:00.000000000 Z
+date: 2013-10-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport