delegate_all_for 0.0.7 → 0.0.8

data/lib/delegate_all_for/version.rb

@@ -1,3 +1,3 @@
 module DelegateAllFor
-  VERSION = "0.0.7"
+  VERSION = "0.0.8"
 end

data/lib/delegate_all_for.rb

@@ -41,20 +41,7 @@ module DelegateAllFor
           class_eval <<-eoruby, __FILE__, __LINE__ + 1
             delegate :#{method},  #{delegate_opts}
             delegate :#{method}?, #{delegate_opts}
-          eoruby
-
-          # Create the setter with support for using nested attributes to set it if the delegated object is not present
-          exception = %(raise "#{self}##{method_prefix}#{method} delegated to #{to}.#{method}, but #{to} is nil: \#{self.inspect}")
-          class_eval <<-eoruby, __FILE__, __LINE__ + 1
-            def #{method_prefix}#{method}=(*args, &block)                             # def customer_name(*args, &block)
-              if #{to} || #{to}.respond_to?(:#{method})                               #   if client || client.respond_to?(:name)
-                #{to}.__send__(:#{method}=, *args, &block)                            #     client.__send__(:name, *args, &block)
-              elsif self.respond_to?(:#{to}_attributes=)                              #   elsif self.respond_to?(:client_attributes=)
-                self.__send__(:#{to}_attributes=, :#{method} => args.first, &block)   #     self.__send__(:client_attributes=, :name => args.first, &block)
-              else                                                                    #   else
-                #{exception}                                                          #     # add helpful exception
-              end                                                                     #   end
-            end                                                                       # end
+            delegate :#{method}=, #{delegate_opts.merge(allow_nil: false)} # allow_nil true leads to unintuitive behavior
           eoruby
         end
       else

data/spec/delegate_all_for/delegate_all_for_spec.rb

@@ -42,6 +42,9 @@ describe DelegateAllFor do
     describe ':allow_nil option' do
       subject { Parent.new }
       its(:four) { should be_nil }
+      it 'does not set allow_nil on setters' do
+        lambda { subject.four = 'FOUR' }.should raise_error RuntimeError
+      end
     end
 
     describe 'delegates to child attributes' do
@@ -55,29 +58,6 @@ describe DelegateAllFor do
       it 'does not delegate to timestamp attributes' do
         lambda { subject.created_at }.should raise_error NoMethodError
       end
-
-      describe 'writer' do
-        it 'sets the value using delegation if child is set' do
-          subject.child.should be_an_instance_of(Child)
-          subject.four = 'FOUR'
-          subject.four.should == 'FOUR'
-          subject.child.four.should == 'FOUR'
-        end
-
-        it 'sets the value using nested attributes if child is not set and nested attributes are supported' do
-          class NestedParent < Parent; accepts_nested_attributes_for :child end
-          subject = NestedParent.new
-          subject.four = 'FOUR'
-          subject.child.should be_an_instance_of(Child)
-          subject.four.should == 'FOUR'
-          subject.child.four.should == 'FOUR'
-        end
-
-        it 'raises an exception if child is not set' do
-          subject = Parent.new
-          lambda { subject.four = 'FOUR' }.should raise_error RuntimeError
-        end
-      end
     end
 
     describe 'guards against user error' do

metadata

@@ -1,67 +1,89 @@
---- !ruby/object:Gem::Specification
+--- !ruby/object:Gem::Specification 
 name: delegate_all_for
-version: !ruby/object:Gem::Version
-  version: 0.0.7
-  prerelease: 
+version: !ruby/object:Gem::Version 
+  prerelease: false
+  segments: 
+  - 0
+  - 0
+  - 8
+  version: 0.0.8
 platform: ruby
-authors:
+authors: 
 - Jason Rust
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-05-09 00:00:00.000000000 Z
-dependencies:
-- !ruby/object:Gem::Dependency
+
+date: 2012-05-09 00:00:00 -07:00
+default_executable: 
+dependencies: 
+- !ruby/object:Gem::Dependency 
   name: activerecord
-  requirement: &70148514385220 !ruby/object:Gem::Requirement
-    none: false
-    requirements:
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 3
+        - 2
+        - 3
         version: 3.2.3
   type: :runtime
-  prerelease: false
-  version_requirements: *70148514385220
-- !ruby/object:Gem::Dependency
+  version_requirements: *id001
+- !ruby/object:Gem::Dependency 
   name: rake
-  requirement: &70148514384040 !ruby/object:Gem::Requirement
-    none: false
-    requirements:
+  prerelease: false
+  requirement: &id002 !ruby/object:Gem::Requirement 
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 0
+        - 9
+        - 2
+        - 2
         version: 0.9.2.2
   type: :development
-  prerelease: false
-  version_requirements: *70148514384040
-- !ruby/object:Gem::Dependency
+  version_requirements: *id002
+- !ruby/object:Gem::Dependency 
   name: rspec
-  requirement: &70148514383580 !ruby/object:Gem::Requirement
-    none: false
-    requirements:
+  prerelease: false
+  requirement: &id003 !ruby/object:Gem::Requirement 
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 2
+        - 9
+        - 0
         version: 2.9.0
   type: :development
-  prerelease: false
-  version_requirements: *70148514383580
-- !ruby/object:Gem::Dependency
+  version_requirements: *id003
+- !ruby/object:Gem::Dependency 
   name: sqlite3
-  requirement: &70148514383080 !ruby/object:Gem::Requirement
-    none: false
-    requirements:
+  prerelease: false
+  requirement: &id004 !ruby/object:Gem::Requirement 
+    requirements: 
     - - ~>
-      - !ruby/object:Gem::Version
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 1
+        - 3
+        - 6
         version: 1.3.6
   type: :development
-  prerelease: false
-  version_requirements: *70148514383080
+  version_requirements: *id004
 description: Easy delegation of all columns of an ActiveRecord association
-email:
+email: 
 - jason@lessonplanet.com
 executables: []
+
 extensions: []
+
 extra_rdoc_files: []
-files:
+
+files: 
 - .gitignore
 - .rspec
 - Gemfile
@@ -74,31 +96,37 @@ files:
 - spec/delegate_all_for/delegate_all_for_spec.rb
 - spec/spec_helper.rb
 - spec/support/active_record.rb
+has_rdoc: true
 homepage: https://github.com/LessonPlanet/delegate_all_for
 licenses: []
+
 post_install_message: 
 rdoc_options: []
-require_paths:
+
+require_paths: 
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
-  requirements:
-  - - ! '>='
-    - !ruby/object:Gem::Version
-      version: '0'
-required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
-  requirements:
-  - - ! '>='
-    - !ruby/object:Gem::Version
-      version: '0'
+required_ruby_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      segments: 
+      - 0
+      version: "0"
+required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      segments: 
+      - 0
+      version: "0"
 requirements: []
+
 rubyforge_project: 
-rubygems_version: 1.8.17
+rubygems_version: 1.3.6
 signing_key: 
 specification_version: 3
 summary: Easy delegation of all columns of an ActiveRecord association
-test_files:
+test_files: 
 - spec/delegate_all_for/delegate_all_for_spec.rb
 - spec/spec_helper.rb
 - spec/support/active_record.rb