get_or_build 0.0.2 → 0.0.3

data/README.md

@@ -34,10 +34,22 @@ No longer need to call `f.object.user || f.object.build_user`
 
 ## Using with NoSQL databases
 
-Just include in your document module `GetOrBuild::AssociationBuilder` and it will attach magick methods automatically
+Just include in your document module `GetOrBuild::AssociationBuilder` and it will attach magic methods automatically
+
+## Extension for fields_for method
+
+You are able also to use option `:build_association` for `fields_for` method to build association if it doesn't exist yet:
+
+		form_for :company do |f|
+			f.fields_for :user, :build_association => true do |fu|
+				fu.text_field :name
+			f.fields_for :user, :build_association => true do |fl|
+				fl.text_field :address
+
+If you initialize associated object user for company before run `fields_for` helper then option `:build_association` won't give any effect
 
 ## Contributing
-You are welcome! Please, run test before pull request: `rake` and make sure if everything is workig correctly.
+You are welcome! Please, run test before pull request: `rake` and make sure if everything is working correctly.
 
 ## TODO
-* tests for `MongoMapper`
+* Support for NoSQL databases

data/lib/get_or_build/association_builder.rb

@@ -11,12 +11,12 @@ module GetOrBuild
     # Here the method `company_or_build` for `User` instane `u` was generated
     def method_missing(method, *args)
       if method =~ /(\w+)_or_build$/
-        if result = send($1, *args)
-          result
-        else
-          builder_method = "build_#{$1}"
-          send(builder_method, *args) if respond_to?(builder_method)
-        end
+        if result = send($1, *args)                                       # if result = send("company", *args)
+          result                                                          #   result
+        else                                                              # else
+          builder_method = "build_#{$1}"                                  #   builder_method = "build_company"
+          send(builder_method, *args) if respond_to?(builder_method)      #   send(builder_method, *args) if respond_to?(builder_method)
+        end                                                               # end
       else
         super
       end

data/lib/get_or_build/fields_for_builder.rb

@@ -5,8 +5,15 @@ module GetOrBuild
         def fields_for_with_association_builder(record_name, record_object = nil, fields_options = {}, &block)
           fields_options, record_object = record_object, nil if record_object.is_a?(Hash) && record_object.extractable_options?
 
-          if fields_options[:build_association] && @object.respond_to?("build_#{record_name}") && !record_object
-            record_object = @object.send("build_#{record_name}")
+          if fields_options.delete(:build_association)
+            association_name = case record_name
+                               when String, Symbol
+                                 record_name
+                               when Array
+                                  ActiveModel::Naming.param_key(record_name.last)
+                               end
+            build_method  = "build_#{association_name}"
+            record_object = @object.send(build_method) if !record_object && !@object.send(association_name) && @object.respond_to?(build_method)
           end
 
           fields_for_without_association_builder(record_name, record_object, fields_options, &block)

data/lib/get_or_build/version.rb

@@ -1,3 +1,3 @@
 module FieldsForBuilder
-  VERSION = "0.0.2"
+  VERSION = "0.0.3"
 end

data/test/fields_for_helper_test.rb

@@ -10,6 +10,7 @@ class FieldsFor < ActionView::TestCase
     form_for company, :url => '' do |f|
       f.fields_for :user, :build_association => true do |ff|
         assert_not_nil ff.object
+        assert_not_nil ff.text_field(:name)
       end
     end
   end
@@ -19,10 +20,43 @@ class FieldsFor < ActionView::TestCase
     assert_equal company.user, nil
     form_for company, :url => '' do |f|
       f.fields_for :user, :build_association => false do |ff|
-        assert_equal ff.object, nil
+        assert_equal nil, ff.object
+        assert_equal nil, ff.text_field(:name)
       end
       f.fields_for :user do |ff|
-        assert_equal ff.object, nil
+        assert_equal nil, ff.object
+        assert_equal nil, ff.text_field(:name)
+      end
+    end
+  end
+
+  def test_for_existing_record1
+    company = Company.new
+    user = company.build_user(:name => 'Mister Proper')
+    form_for company, :url => '' do |f|
+      f.fields_for :user, :build_association => true do |ff|
+        assert_not_nil ff.text_field(:name)
+      end
+    end
+  end
+
+  def test_for_existing_record2
+    company = Company.new
+    user = company.build_user(:name => 'Mister Proper')
+    form_for company, :url => '' do |f|
+      f.fields_for :user do |ff|
+        assert_not_nil ff.text_field(:name)
+      end
+    end
+  end
+
+  def test_nested_params
+    company = Company.new
+    user = company.build_user(:name => 'New User')
+    form_for company, :url => '' do |f|
+      f.fields_for [:admin, user], :build_association => true do |ff|
+        assert_not_nil ff.object
+        assert_equal user.name, ff.object.name
       end
     end
   end

data/test/support/models.rb

@@ -1,10 +1,13 @@
 class Company < ActiveRecord::Base
   belongs_to :location
   has_one :user
+  accepts_nested_attributes_for :user
+  accepts_nested_attributes_for :location
 end
 
 class User < ActiveRecord::Base
   belongs_to :company
+  accepts_nested_attributes_for :company
 end
 
 class Location < ActiveRecord::Base; end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: get_or_build
 version: !ruby/object:Gem::Version
-  version: 0.0.2
+  version: 0.0.3
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-03-24 00:00:00.000000000Z
+date: 2012-03-25 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
-  requirement: &70105780474200 !ruby/object:Gem::Requirement
+  requirement: &70327775366480 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70105780474200
+  version_requirements: *70327775366480
 - !ruby/object:Gem::Dependency
   name: actionpack
-  requirement: &70105780473680 !ruby/object:Gem::Requirement
+  requirement: &70327775365680 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70105780473680
+  version_requirements: *70327775365680
 - !ruby/object:Gem::Dependency
   name: sqlite3
-  requirement: &70105780473140 !ruby/object:Gem::Requirement
+  requirement: &70327775364740 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +43,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70105780473140
+  version_requirements: *70327775364740
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70105780472620 !ruby/object:Gem::Requirement
+  requirement: &70327775364040 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,7 +54,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70105780472620
+  version_requirements: *70327775364040
 description: ! 'Assume user belongs_to company and you want to get company object
   within one method call. You can do it like this: user.company_or_build'
 email:
@@ -91,7 +91,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1527763818141659073
+      hash: 2580832862823747020
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -100,7 +100,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -1527763818141659073
+      hash: 2580832862823747020
 requirements: []
 rubyforge_project: get_or_build
 rubygems_version: 1.8.10