reform 2.0.0.rc3 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1656561190a1c9784f1e2463e6c4e2c558bb78d0
-  data.tar.gz: 236cc4811f68a6e6e6e7a9e0a31d11a4afbe955d
+  metadata.gz: 247ffc53838f5e69a71f4ab5d066572b05308003
+  data.tar.gz: f6162c6a018e02ad2f9c487441dcea08623b960a
 SHA512:
-  metadata.gz: 2aa18198338829e843a002d940dfb60d0f4d318662637d064f01a668f30fd59bf20fa3fe11dd25b7a5534e93b986aabc54d19cf2f01e3e42f02c81273fe7601a
-  data.tar.gz: 4d9cea3a1c22769400c48164b2a907e0b8f713a82b2cb9e11fb0fb97bbc963f3a5445da828b4b899b2dd4b420416fb7b7d94f1f6c400378e47210c64028d99f5
+  metadata.gz: 31ee6b2d4656b94ced0c7790832e8b6be84ea2b8218866c5cb9e75c6b3099710e9e2a52a24cd54b9c40ba0ead9a9712e7e961ac4b463f48bb34131b305c22246
+  data.tar.gz: d8954f5806cdbbd701b4e8352d930f2b1bd3aef5a36d5621ef154a0132e6e46481137b5f1d9208932a0377805f81cac75ee066885cf73b2ea4778957e8c1192b

data/CHANGES.md

@@ -2,18 +2,25 @@
 
 * The `::reform_2_0!` is no longer there. Guess why.
 * Again: `:empty` doesn't exist anymore. You can choose from `:readable`, `:writeable` and `:virtual`.
-* :populator => lambda { |fragment, index, args|
-#           songs[index] or songs[index] = args.binding[:form].new(Song.new)
-#         }
-is now :populator => lambda { |fragment, index, args|
-#           songs[index] or songs.insert(index) = Song.new
-#         }
-you don't need to know about forms anymore, the twin handles that using #insert.
+* When using `:populator` the API to work against the form has changed.
+    ```ruby
+    populator: lambda { |fragment, index, args|
+      songs[index] or songs[index] = args.binding[:form].new(Song.new)
+    }
+    ```
+
+   is now
+
+   ```ruby
+   populator: lambda { |fragment, index, args|
+     songs[index] or songs.insert(index) = Song.new
+   }
+   ```
+    You don't need to know about forms anymore, the twin handles that using the [Twin](https://github.com/apotonick/disposable) API..
 
 * `:as` option removed. Use `:from`.
 * With `Composition` included, `Form#model` would give you a composition object. You can grab that using `Form#mapper` now.
 * `Form#update!` is deprecated. It still works but will remind you to override `#present!` or use pre-populators as [described here](http://trailblazerb.org/gems/reform/prepopulator.html) and in the Trailblazer book, chapter "Nested Forms".
-
 * Forms do not `include ActiveModel::Validations` anymore. This has polluted the entire gem and is not encapsulated in `Validator`. Consider using Lotus Validations instead.
 * Validation inheritance with `ActiveModel::Validations` is broken with Rails 3.2 and 4.0. Update Rails or use the `Lotus` validations.
 

data/README.md

@@ -123,7 +123,7 @@ Sometimes, you need to do saving manually.
 
 ## Saving Forms Manually
 
-Calling `#save` with a block doesn't do anything but provide you with a nested hash with all the validated input. This allows you to implement the saving yourself.
+Calling `#save` with a block will provide a nested hash of the form's properties and values. This does **not call `#save` on the models** and allows you to implement the saving yourself.
 
 The block parameter is a nested hash of the form input.
 
@@ -830,7 +830,7 @@ If you want full support for `simple_form` do as follows.
 
 ```ruby
 class SongForm < Reform::Form
-  include ModelReflections
+  include ActiveModel::ModelReflections
 ```
 
 Including this module will add `#column_for_attribute` and other methods need by form builders to automatically guess the type of a property.

data/lib/reform.rb

@@ -4,13 +4,13 @@ module Reform
   end
 end
 
-require 'reform/contract'
-require 'reform/form'
-require 'reform/form/composition'
-require 'reform/form/active_model'
-require 'reform/form/module'
+require "reform/contract"
+require "reform/form"
+require "reform/form/composition"
+require "reform/form/active_model"
+require "reform/form/module"
 
 
-if defined?(Rails) # DISCUSS: is everyone ok with this?
-  require 'reform/rails'
+if defined?(Rails)
+  require "reform/rails"
 end

data/lib/reform/active_record.rb

@@ -1,4 +1,4 @@
-require 'reform/form/active_model'
-require 'reform/form/orm'
-require 'reform/form/active_record'
-require 'reform/form/model_reflections' # only load this in AR context as simple_form currently is bound to AR.
+require "reform/form/active_model"
+require "reform/form/orm"
+require "reform/form/active_record"
+require "reform/form/active_model/model_reflections" # only load this in AR context as simple_form currently is bound to AR.

data/lib/reform/contract.rb

@@ -1,19 +1,18 @@
-require 'forwardable'
-require 'uber/inheritable_attr'
-require 'uber/delegates'
+require "uber/inheritable_attr"
+require "disposable/twin"
+require "disposable/twin/setup"
+require "disposable/twin/default"
 
 module Reform
-  # Gives you a DSL for defining the object structure and its validations.
-  require "disposable/twin"
-  require "disposable/twin/setup"
+  # Define your form structure and its validations. Instantiate it with a model,
+  # and then +validate+ this object graph.
   class Contract < Disposable::Twin
     require "disposable/twin/composition" # Expose.
     include Expose
 
     feature Setup
     feature Setup::SkipSetter
-
-    extend Uber::Delegates
+    feature Default
 
     representer_class.instance_eval do
       def default_inline_class
@@ -57,10 +56,11 @@ module Reform
       _name = definition.name
       mod.instance_eval do
         @_name = _name.singularize.camelize
-        def name # this adds Form::name for AM::Validations and I18N. i know it's retarded.
-          # something weird happens here: somewhere in Rails, this creates a constant (e.g. User). if this name doesn't represent a valid
-          # constant, the reloading in dev will fail with weird messages. i'm not sure if we should just get rid of Rails validations etc.
-          # or if i should look into this?
+        # this adds Form::name for AM::Validations and I18N.
+        # i have a feeling that this is also needed for Rails autoloader, which is scary.
+        # beside that, it is a nice way for debugging to find out which anonymous form class you're working on:
+        #   anonymous_nested_form.class.name
+        def name
           @_name
         end
       end
@@ -76,6 +76,7 @@ module Reform
        self.class.options_for(name)
       end
     end
+
     def self.options_for(name)
       representer_class.representable_attrs.get(name)
     end

data/lib/reform/contract/validate.rb

@@ -12,7 +12,7 @@ module Reform::Contract::Validate
     valid?  # calls AM/Lotus validators and invokes self.errors=.
 
     errors.merge!(self.errors, prefix) # local errors.
-    errors.merge!(nested_errors, []) #
+    errors.merge!(nested_errors, [])
   end
 
   def errors

data/lib/reform/form.rb

@@ -9,12 +9,13 @@ module Reform
     end
 
     require "reform/form/validate"
-    include Validate # extend Contract#validate with additional behaviour.
+    include Validate # override Contract#validate with additional behaviour.
 
     require "reform/form/populator"
 
     module Property
-      # add macro logic, e.g. for :populator.
+      # Add macro logic, e.g. for :populator.
+      # TODO: This will be re-structured once Declarative allows it.
       def property(name, options={}, &block)
         if deserializer = options[:deserializer] # this means someone is explicitly specifying :deserializer.
           options[:deserializer] = Representable::Cloneable::Hash[deserializer]
@@ -23,25 +24,18 @@ module Reform
         definition = super # let representable sort out inheriting of properties, and so on.
         definition.merge!(deserializer: Representable::Cloneable::Hash.new) unless definition[:deserializer] # always keep :deserializer per property.
 
-
         deserializer_options = definition[:deserializer]
 
-        # TODO: make this pluggable.
-        # DISCUSS: Populators should be a representable concept?
-
         # Populators
         # * they assign created data, no :setter (hence the name).
         # * they (ab)use :instance, this is why they need to return a twin form.
         # * they are only used in the deserializer.
-
-
-
         if populator = options.delete(:populate_if_empty)
           deserializer_options.merge!({instance: Populator::IfEmpty.new(populator)})
           deserializer_options.merge!({setter: nil})
         elsif populator = options.delete(:populator)
           deserializer_options.merge!({instance: Populator.new(populator)})
-          deserializer_options.merge!({setter: nil}) #if options[:collection] # collections don't need to get re-assigned, they don't change.
+          deserializer_options.merge!({setter: nil})
         end
 
 

data/lib/reform/form/active_model.rb

@@ -1,55 +1,14 @@
-require 'reform/form/active_model/model_validations'
+require "reform/form/active_model/model_validations"
+require "reform/form/active_model/form_builder_methods"
+require "uber/delegates"
 
 module Reform::Form::ActiveModel
-  module FormBuilderMethods # TODO: rename to FormBuilderCompat.
-    def self.included(base)
-      base.extend ClassMethods # ::model_name
-    end
-
-    module ClassMethods
-    private
-
-      # TODO: add that shit in Form#present, not by overriding ::property.
-      def property(name, options={}, &block)
-        super.tap do |definition|
-          add_nested_attribute_compat(name) if definition[:twin] # TODO: fix that in Rails FB#1832 work.
-        end
-      end
-
-      # The Rails FormBuilder "detects" nested attributes (which is what we want) by checking existance of a setter method.
-      def add_nested_attribute_compat(name)
-        define_method("#{name}_attributes=") {} # this is why i hate respond_to? in Rails.
-      end
-    end
-
-    # Modify the incoming Rails params hash to be representable compliant.
-    def deserialize!(params)
-      # this only happens in a Hash environment. other engines have to overwrite this method.
-      schema.each do |dfn|
-        rename_nested_param_for!(params, dfn)
-      end
-
-      super(params)
-    end
-
-  private
-    def rename_nested_param_for!(params, dfn)
-      nested_name = "#{dfn.name}_attributes"
-      return unless params.has_key?(nested_name)
-
-      value = params["#{dfn.name}_attributes"]
-      value = value.values if dfn[:collection]
-
-      params[dfn.name] = value
-    end
-  end # FormBuilderMethods
-
-
   def self.included(base)
     base.class_eval do
       extend ClassMethods
       register_feature ActiveModel
 
+      extend Uber::Delegates
       delegates :model, *[:persisted?, :to_key, :to_param, :id] # Uber::Delegates
 
       def to_model # this is called somewhere in FormBuilder and ActionController.

data/lib/reform/form/active_model/form_builder_methods.rb

@@ -0,0 +1,47 @@
+module Reform::Form::ActiveModel
+  # Including FormBuilderMethods will allow using form instances with form_for, simple_form, etc.
+  # in Rails. It will further try to translate Rails' suboptimal songs_attributes weirdness
+  # back to normal `songs: ` naming in +#valiate+.
+  module FormBuilderMethods
+    def self.included(base)
+      base.extend ClassMethods # ::model_name
+    end
+
+    module ClassMethods
+    private
+
+      # TODO: add that shit in Form#present, not by overriding ::property.
+      def property(name, options={}, &block)
+        super.tap do |definition|
+          add_nested_attribute_compat(name) if definition[:twin] # TODO: fix that in Rails FB#1832 work.
+        end
+      end
+
+      # The Rails FormBuilder "detects" nested attributes (which is what we want) by checking existance of a setter method.
+      def add_nested_attribute_compat(name)
+        define_method("#{name}_attributes=") {} # this is why i hate respond_to? in Rails.
+      end
+    end
+
+    # Modify the incoming Rails params hash to be representable compliant.
+    def deserialize!(params)
+      # this only happens in a Hash environment. other engines have to overwrite this method.
+      schema.each do |dfn|
+        rename_nested_param_for!(params, dfn)
+      end
+
+      super(params)
+    end
+
+  private
+    def rename_nested_param_for!(params, dfn)
+      nested_name = "#{dfn.name}_attributes"
+      return unless params.has_key?(nested_name)
+
+      value = params["#{dfn.name}_attributes"]
+      value = value.values if dfn[:collection]
+
+      params[dfn.name] = value
+    end
+  end
+end

data/lib/reform/form/active_model/model_reflections.rb

@@ -0,0 +1,41 @@
+# ModelReflections will be the interface between the form object and form builders like simple_form.
+#
+# This module is meant to collect all dependencies simple_form needs in addition to the ActiveModel ones.
+# Goal is to collect all methods and define a reflection API so simple_form works with all ORMs and Reform
+# doesn't have to "guess" what simple_form and other form helpers need.
+class Reform::Form < Reform::Contract
+  module ActiveModel::ModelReflections
+    def self.included(base)
+      base.extend ClassMethods
+      base.register_feature self # makes it work in nested forms.
+    end
+
+    module ClassMethods
+      # Delegate reflect_on_association to the model class to support simple_form's
+      # association input.
+      def reflect_on_association(*args)
+        model_name.to_s.constantize.reflect_on_association(*args)
+      end
+    end
+
+    # Delegate column for attribute to the model to support simple_form's
+    # attribute type interrogation.
+    def column_for_attribute(name)
+      model_for_property(name).column_for_attribute(name)
+    end
+
+    def has_attribute?(name)
+      model_for_property(name).has_attribute?(name)
+    end
+
+    def defined_enums
+      return model.defined_enums unless is_a?(Reform::Form::Composition)
+
+      mapper.each.with_object({}) { |m,h| h.merge! m.defined_enums }
+    end
+
+    # this should also contain to_param and friends as this is used by the form helpers.
+  end
+
+  ModelReflections = ActiveModel::ModelReflections
+end

data/lib/reform/form/active_model/validations.rb

@@ -1,5 +1,6 @@
 require "active_model"
-require "reform/contract/errors"
+require "reform/form/active_model/errors"
+require "uber/delegates"
 
 module Reform::Form::ActiveModel
   # AM::Validations for your form.
@@ -30,6 +31,8 @@ module Reform::Form::ActiveModel
     # Validators is the validatable object. On the class level, we define validations,
     # on instance, it exposes #valid?.
     class Validator
+      # current i18n scope: :activemodel.
+
       include ActiveModel::Validations
       # extend ActiveModel::Naming
 

data/lib/reform/form/coercion.rb

@@ -1,5 +1,3 @@
 require "disposable/twin/coercion"
 
-Reform::Form.class_eval do
-  Coercion = Disposable::Twin::Coercion
-end
+Reform::Form::Coercion = Disposable::Twin::Coercion

data/lib/reform/form/orm.rb

@@ -2,7 +2,7 @@ module Reform::Form::ORM
   def model_for_property(name)
     return model unless is_a?(Reform::Form::Composition) # i am too lazy for proper inheritance. there should be a ActiveRecord::Composition that handles this.
 
-    model_name = schema.representable_attrs.get(name)[:on]
+    model_name = options_for(name)[:on]
     model[model_name]
   end
 

data/lib/reform/form/populator.rb

@@ -1,5 +1,3 @@
-# TODO: move somewhere else!
-# TODO: make inheritable? and also, there's a lot of noise. shorten.
 # Implements the :populator option.
 #
 #  populator: -> (fragment, twin, options)

data/lib/reform/form/validate.rb

@@ -5,8 +5,7 @@ module Reform::Form::Validate
       include Uber::Callable
 
       def call(form, params, options)
-        # TODO: hahahahahaha.
-        # FIXME: this is a bit ridiculous.
+        # TODO: Schema should provide property names as plain list.
         properties = options.binding[:twin].representer_class.representable_attrs[:definitions].keys
 
         properties.each { |name| params[name].present? and return false }
@@ -46,7 +45,7 @@ private
   # DISCUSS: this is only called once, on the top-level form.
   def deprecate_update!(params)
     return unless self.class.instance_methods(false).include?(:update!)
-    warn "[Reform] Form#update! is deprecated and will be removed in Reform 2.1. Please use #present! or pre-populator."
+    warn "[Reform] Form#update! is deprecated and will be removed in Reform 2.1. Please use #prepopulate!"
     update!(params)
   end
 
@@ -68,7 +67,6 @@ private
 
     deserializer.apply do |dfn|
       next unless dfn[:twin]
-
       # Representer#each and #apply have to be unified.
       dfn.merge!(
         deserialize: lambda { |decorator, params, options|

data/lib/reform/version.rb

@@ -1,3 +1,3 @@
 module Reform
-  VERSION = "2.0.0.rc3"
+  VERSION = "2.0.0"
 end

data/reform.gemspec

@@ -17,7 +17,7 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.add_dependency             "disposable",     "~> 0.1.5"
+  spec.add_dependency             "disposable",     "~> 0.1.7"
   spec.add_dependency             "uber",           "~> 0.0.11"
 
   spec.add_development_dependency "bundler"

data/test/default_test.rb

@@ -0,0 +1,22 @@
+require "test_helper"
+
+class DefaultTest < Minitest::Spec
+  Song  = Struct.new(:title, :album, :composer)
+  Album = Struct.new(:name, :songs, :artist)
+  Artist = Struct.new(:name)
+
+  class AlbumForm < Reform::Form
+    property :name, default: "Wrong"
+
+    collection :songs do
+      property :title, default: "It's Catching Up"
+    end
+  end
+
+  it do
+    form = AlbumForm.new(Album.new(nil, [Song.new]))
+
+    form.name.must_equal "Wrong"
+    form.songs[0].title.must_equal "It's Catching Up"
+  end
+end

data/test/deserialize_test.rb

@@ -1,5 +1,4 @@
 require 'test_helper'
-require 'reform/form/json'
 
 class DeserializeTest < MiniTest::Spec
   Song  = Struct.new(:title, :album, :composer)

data/test/model_reflections_test.rb

@@ -4,7 +4,7 @@ require 'test_helper'
 class ModelReflectionTest < MiniTest::Spec
   class SongForm < Reform::Form
     include Reform::Form::ActiveRecord
-    include Reform::Form::ModelReflections
+    include Reform::Form::ActiveModel::ModelReflections
 
     model :song
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: reform
 version: !ruby/object:Gem::Version
-  version: 2.0.0.rc3
+  version: 2.0.0
 platform: ruby
 authors:
 - Nick Sutterer
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-07-08 00:00:00.000000000 Z
+date: 2015-07-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: disposable
@@ -17,14 +17,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.5
+        version: 0.1.7
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.1.5
+        version: 0.1.7
 - !ruby/object:Gem::Dependency
   name: uber
   requirement: !ruby/object:Gem::Requirement
@@ -205,32 +205,29 @@ files:
 - lib/reform.rb
 - lib/reform/active_record.rb
 - lib/reform/contract.rb
-- lib/reform/contract/errors.rb
 - lib/reform/contract/validate.rb
 - lib/reform/form.rb
 - lib/reform/form/active_model.rb
+- lib/reform/form/active_model/errors.rb
+- lib/reform/form/active_model/form_builder_methods.rb
+- lib/reform/form/active_model/model_reflections.rb
 - lib/reform/form/active_model/model_validations.rb
 - lib/reform/form/active_model/validations.rb
 - lib/reform/form/active_record.rb
 - lib/reform/form/coercion.rb
 - lib/reform/form/composition.rb
-- lib/reform/form/json.rb
 - lib/reform/form/lotus.rb
-- lib/reform/form/model_reflections.rb
 - lib/reform/form/module.rb
 - lib/reform/form/mongoid.rb
 - lib/reform/form/multi_parameter_attributes.rb
 - lib/reform/form/orm.rb
 - lib/reform/form/populator.rb
 - lib/reform/form/prepopulate.rb
-- lib/reform/form/save.rb
-- lib/reform/form/scalar.rb
 - lib/reform/form/validate.rb
 - lib/reform/form/validation/unique_validator.rb
 - lib/reform/mongoid.rb
 - lib/reform/rails.rb
 - lib/reform/schema.rb
-- lib/reform/twin.rb
 - lib/reform/version.rb
 - reform.gemspec
 - test/active_model_test.rb
@@ -242,6 +239,7 @@ files:
 - test/composition_test.rb
 - test/contract_test.rb
 - test/custom_validation_test.rb
+- test/default_test.rb
 - test/deserialize_test.rb
 - test/dummy/Rakefile
 - test/dummy/app/controllers/albums_controller.rb
@@ -292,7 +290,6 @@ files:
 - test/skip_if_test.rb
 - test/skip_setter_and_getter_test.rb
 - test/test_helper.rb
-- test/twin_test.rb
 - test/unique_test.rb
 - test/validate_test.rb
 - test/virtual_test.rb
@@ -312,9 +309,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.4.8
@@ -331,6 +328,7 @@ test_files:
 - test/composition_test.rb
 - test/contract_test.rb
 - test/custom_validation_test.rb
+- test/default_test.rb
 - test/deserialize_test.rb
 - test/dummy/Rakefile
 - test/dummy/app/controllers/albums_controller.rb
@@ -381,7 +379,6 @@ test_files:
 - test/skip_if_test.rb
 - test/skip_setter_and_getter_test.rb
 - test/test_helper.rb
-- test/twin_test.rb
 - test/unique_test.rb
 - test/validate_test.rb
 - test/virtual_test.rb

data/lib/reform/form/json.rb

@@ -1,13 +0,0 @@
-require 'representable/json'
-
-module Reform
-  module Form::JSON
-    def self.included(base)
-      base.representer_class.send :include, Representable::JSON
-    end
-
-    def deserialize_method
-      :from_json
-    end
-  end
-end

data/lib/reform/form/model_reflections.rb

@@ -1,37 +0,0 @@
-# ModelReflections will be the interface between the form object and form builders like simple_form.
-#
-# This module is meant to collect all dependencies simple_form needs in addition to the ActiveModel ones.
-# Goal is to collect all methods and define a reflection API so simple_form works with all ORMs and Reform
-# doesn't have to "guess" what simple_form and other form helpers need.
-module Reform::Form::ModelReflections
-  def self.included(base)
-    base.extend ClassMethods
-    base.register_feature self # makes it work in nested forms.
-  end
-
-  module ClassMethods
-    # Delegate reflect_on_association to the model class to support simple_form's
-    # association input.
-    def reflect_on_association(*args)
-      model_name.to_s.constantize.reflect_on_association(*args)
-    end
-  end
-
-  # Delegate column for attribute to the model to support simple_form's
-  # attribute type interrogation.
-  def column_for_attribute(name)
-    model_for_property(name).column_for_attribute(name)
-  end
-
-  def has_attribute?(name)
-    model_for_property(name).has_attribute?(name)
-  end
-
-  def defined_enums
-    return model.defined_enums unless is_a?(Reform::Form::Composition)
-
-    mapper.each.with_object({}) { |m,h| h.merge! m.defined_enums }
-  end
-
-  # this should also contain to_param and friends as this is used by the form helpers.
-end

data/lib/reform/form/save.rb

@@ -1,66 +0,0 @@
-module Reform::Form::Save
-  # Returns the result of that save invocation on the model.
-  def save(options={}, &block)
-    # DISCUSS: we should never hit @mapper here (which writes to the models) when a block is passed.
-    return yield to_nested_hash if block_given?
-
-    sync_models # recursion
-    save!(options)
-  end
-
-  def save!(options={}) # FIXME.
-    result = save_model
-
-    save_representer.new(fields).to_hash # save! on all nested forms.
-
-    dynamic_save!(options)
-
-    result
-  end
-
-  def save_model
-    model.save # TODO: implement nested (that should really be done by Twin/AR).
-  end
-
-
-  require "active_support/hash_with_indifferent_access" # DISCUSS: replace?
-  def to_nested_hash(*)
-    ActiveSupport::HashWithIndifferentAccess.new(nested_hash_representer.new(fields).to_hash)
-  end
-  alias_method :to_hash, :to_nested_hash
-  # NOTE: it is not recommended using #to_hash and #to_nested_hash in your code, consider them private.
-
-private
-  def save_representer
-    self.class.representer(:save) do |dfn|
-      dfn.merge!(
-        :instance  => lambda { |form, *| form },
-        :serialize => lambda { |form, args| form.save! unless args.binding[:save] === false }
-      )
-    end
-  end
-
-  def nested_hash_representer
-    self.class.representer(:nested_hash, :all => true) do |dfn|
-      dfn.merge!(:serialize => lambda { |form, args| form.to_nested_hash }) if dfn[:form]
-
-      dfn.merge!(:as => dfn[:private_name] || dfn.name)
-    end
-  end
-
-  def dynamic_save!(options)
-    names = options.keys & changed.keys.map(&:to_sym)
-    return if names.size == 0
-
-    dynamic_save_representer.new(fields).to_hash(options.merge(:include => names))
-  end
-
-  def dynamic_save_representer
-    self.class.representer(:dynamic_save, :all => true) do |dfn|
-      dfn.merge!(
-        :serialize     => lambda { |object, options| options.user_options[options.binding.name.to_sym].call(object, options) },
-        :representable => true
-      )
-    end
-  end
-end

data/lib/reform/form/scalar.rb

@@ -1,54 +0,0 @@
-module Reform::Form::Scalar
-  # IDEA: what if every "leaf" property would be represented by a Scalar form?
-  def self.included(base)
-    base.extend ClassMethods
-    base.extend Forwardable
-  end
-
-  def update!(object)
-    @fields = object # @scalar is "I came from the outside." or <ArbitraryObject>.
-  end
-
-  def scalar
-    fields
-  end
-
-  def save!
-  end
-
-  def sync!(*)
-    model.replace(fields)
-    # FIXME: how to sync that, if it's not responds to replace? or what if we don't want to write (e.g. image with paperdragon)?
-  end
-
-  def to_nested_hash
-    scalar
-  end
-
-
-  module ClassMethods
-    def validates(name, options={})
-      if name.is_a?(Hash)
-        name, options = :scalar, name # per default, validate #scalar (e.g. "Hello").
-      else
-        def_delegator :scalar, name
-      end
-
-      super(name, options)
-    end
-  end
-
-
-  # TODO: change the way i hook into ::property.
-  module Property
-    private
-
-    def property(name, options={}, &block)
-      if options[:scalar]
-        options.merge!(:features => [Reform::Form::Scalar], populate_if_empty: String)
-      end
-
-      super
-    end
-  end
-end

data/lib/reform/twin.rb

@@ -1,20 +0,0 @@
-require 'disposable/twin'
-
-module Reform
-  module Twin
-    def self.included(base)
-      base.send :include, Disposable::Twin::Builder
-      base.extend ClassMethods
-    end
-
-    module ClassMethods
-      def twin(twin_class)
-        super(twin_class) { |dfn| property dfn.name } # create readers to twin model.
-      end
-    end
-
-    def initialize(model, options={})
-      super(build_twin(model, options))
-    end
-  end
-end

data/test/twin_test.rb

@@ -1,38 +0,0 @@
-require 'test_helper'
-
-require 'test_helper'
-require 'reform/twin'
-
-# class TwinTest < MiniTest::Spec
-#   class SongForm < Reform::Form
-#     class Twin < Disposable::Twin
-#       property :title
-#       option :is_online # TODO: this should make it read-only in reform!
-#     end
-
-#     # include Reform::Twin
-#     # twin Twin
-#   end
-
-#   let (:model) { OpenStruct.new(title: "Kenny") }
-
-#   let (:form) { SongForm.new(model, is_online: true) }
-
-#   it { form.title.must_equal "Kenny" }
-#   it { form.is_online.must_equal true }
-# end
-class TwinTest < MiniTest::Spec
-  Song = Struct.new(:name)
-
-  class SongForm < Reform::Form
-    property :name
-    property :is_online, virtual: true
-  end
-
-  let (:form) { SongForm.new(Song.new("Kenny"), is_online: true) }
-
-  it do
-    form.name.must_equal "Kenny"
-    form.is_online.must_equal true
-  end
-end