rename to_object to to_struct

Author: elShiaLabeouf

.byebug_history

@@ -0,0 +1,5 @@
+c
+definition
+c
+definition.array?
+definition

README.md

@@ -139,7 +139,7 @@ To use a specific format, include the corresponding module in your representer a
 - `Representable::JSON#to_json`
 - `Representable::JSON#to_hash` (provides a hash instead of string)
 - `Representable::Hash#to_hash`
-- `Representable::Object#to_object` (provides a Struct-based object)
+- `Representable::Struct#to_struct` (provides a Struct-based object)
 - `Representable::XML#to_xml`
 - `Representable::YAML#to_yaml`
 

lib/representable/hash.rb

@@ -35,6 +35,7 @@ def from_hash(data, options={}, binding_builder=Binding)
 
     def to_hash(options={}, binding_builder=Binding)
       hash = create_representation_with({}, options, binding_builder)
+
       return hash if options[:wrap] == false
       return hash unless (wrap = options[:wrap] || representation_wrap(options))
 

lib/representable/object.rb

@@ -3,8 +3,6 @@
 
 module Representable
   module Object
-    autoload :Collection, 'representable/object/collection'
-
     def self.included(base)
       base.class_eval do
         include Representable
@@ -15,40 +13,17 @@ def self.included(base)
 
 
     module ClassMethods
-      def format_engine
-        Representable::Object
-      end
-
       def collection_representer_class
         Collection
       end
-
-      def cache_struct
-        @represented_struct ||= Struct.new(*representable_attrs.keys.map(&:to_sym))
-      end
-
-      def cache_wrapper_struct(wrap:)
-        struct_name = :"@_wrapper_struct_#{wrap}"
-        return instance_variable_get(struct_name) if instance_variable_defined?(struct_name)
-
-        instance_variable_set(struct_name, Struct.new(wrap))
-      end
     end
 
     def from_object(data, options={}, binding_builder=Binding)
       update_properties_from(data, options, binding_builder)
     end
 
     def to_object(options={}, binding_builder=Binding)
-      represented_struct = self.class.cache_struct
-
-      object = create_representation_with(represented_struct.new, options, binding_builder)
-      return object if options[:wrap] == false
-      return object unless (wrap = options[:wrap] || representation_wrap(options))
-
-      wrapper_struct = self.class.cache_wrapper_struct(wrap: wrap.to_sym)
-      wrapper_struct.new(object)
+      create_representation_with(nil, options, binding_builder)
     end
-
   end
 end

lib/representable/object/binding.rb

@@ -9,16 +9,16 @@ def self.build_for(definition)  # TODO: remove default arg.
         new(definition)
       end
 
-      def read(struct, as)
-        fragment = struct.send(as) # :getter? no, that's for parsing!
+      def read(hash, as)
+        fragment = hash.send(as) # :getter? no, that's for parsing!
 
         return FragmentNotFound if fragment.nil? and typed?
 
         fragment
       end
 
-      def write(struct, fragment, as)
-        struct.send("#{as}=", fragment)
+      def write(hash, fragment, as)
+        true
       end
 
       def deserialize_method

lib/representable/struct.rb

@@ -0,0 +1,54 @@
+require 'representable'
+require 'representable/struct/binding'
+
+module Representable
+  module Struct
+    autoload :Collection, 'representable/struct/collection'
+
+    def self.included(base)
+      base.class_eval do
+        include Representable
+        extend ClassMethods
+        register_feature Representable::Struct
+      end
+    end
+
+
+    module ClassMethods
+      def format_engine
+        Representable::Struct
+      end
+
+      def collection_representer_class
+        Collection
+      end
+
+      def cache_struct
+        @represented_struct ||= ::Struct.new(*representable_attrs.keys.map(&:to_sym))
+      end
+
+      def cache_wrapper_struct(wrap:)
+        struct_name = :"@_wrapper_struct_#{wrap}"
+        return instance_variable_get(struct_name) if instance_variable_defined?(struct_name)
+
+        instance_variable_set(struct_name, ::Struct.new(wrap))
+      end
+    end
+
+    def from_struct(data, options={}, binding_builder=Binding)
+      update_properties_from(data, options, binding_builder)
+    end
+
+    def to_struct(options={}, binding_builder=Binding)
+      represented_struct = self.class.cache_struct
+
+      object = create_representation_with(represented_struct.new, options, binding_builder)
+      return object if options[:wrap] == false
+      return object unless (wrap = options[:wrap] || representation_wrap(options))
+
+      wrapper_struct = self.class.cache_wrapper_struct(wrap: wrap.to_sym)
+      wrapper_struct.new(object)
+    end
+
+  end
+end

lib/representable/struct/binding.rb

@@ -0,0 +1,33 @@
+require 'representable/binding'
+
+module Representable
+  module Struct
+    class Binding < Representable::Binding
+      def self.build_for(definition)
+        return Collection.new(definition)  if definition.array?
+
+        new(definition)
+      end
+
+      def read(struct, as)
+        fragment = struct.send(as) # :getter? no, that's for parsing!
+
+        return FragmentNotFound if fragment.nil? and typed?
+
+        fragment
+      end
+
+      def write(struct, fragment, as)
+        struct.send("#{as}=", fragment)
+      end
+
+      def serialize_method
+        :to_struct
+      end
+
+      class Collection < self
+        include Representable::Binding::Collection
+      end
+    end
+  end
+end

lib/representable/object/collection.rb lib/representable/struct/collection.rblib/representable/object/collection.rb renamed to lib/representable/struct/collection.rb

@@ -1,12 +1,12 @@
 require 'representable/hash'
 
-module Representable::Object
+module Representable::Struct
   module Collection
-    include Representable::Object
+    include Representable::Struct
 
     def self.included(base)
       base.class_eval do
-        include Representable::Object
+        include Representable::Struct
         extend ClassMethods
         property(:_self, {:collection => true})
       end

test/examples/object.rb

@@ -5,8 +5,8 @@
 
 source = OpenStruct.new(
   name: "30 Years Live", songs: [
-  OpenStruct.new(id: 1, title: "Dear Beloved"), OpenStruct.new(id: 2, title: "Fuck Armageddon")
-]
+    OpenStruct.new(id: 1, title: "Dear Beloved"), OpenStruct.new(id: 2, title: "Fuck Armageddon")
+  ]
 )
 
 require "representable/object"
@@ -26,30 +26,3 @@ class AlbumRepresenter < Representable::Decorator
 target = Album.new
 
 AlbumRepresenter.new(target).from_object(source)
-
-# Representing to_object:
-
-class Animal
-  attr_accessor :name, :age, :species
-  def initialize(name, age, species)
-    @name = name
-    @age = age
-    @species = species
-  end
-end
-
-require "representable/object"
-
-class AnimalRepresenter < Representable::Decorator
-  include Representable::Object
-
-  property :name, getter: ->(represented:, **) { represented.name.upcase }
-  property :age
-end
-
-animal = Animal.new('Smokey',12,'c')
-animal_repr = AnimalRepresenter.new(animal).to_object(wrap: "wrapper")
-
-animal_array = [Animal.new('Shepard',22,'s'),Animal.new('Pickle',12,'c'),Animal.new('Rodgers',55,'e')]
-array_repr = AnimalRepresenter.for_collection.new(animal_array).to_object
-

test/examples/struct.rb

@@ -0,0 +1,27 @@
+require "representable/struct"
+
+# Representing to_struct:
+class Animal
+  attr_accessor :name, :age, :species
+  def initialize(name, age, species)
+    @name = name
+    @age = age
+    @species = species
+  end
+end
+
+class AnimalRepresenter < Representable::Decorator
+  include Representable::Struct
+
+  property :name, getter: ->(represented:, **) { represented.name.upcase }
+  property :age
+end
+
+animal = Animal.new('Smokey',12,'c')
+animal_repr = AnimalRepresenter.new(animal).to_struct(wrap: "wrapper")
+
+animal_array = [Animal.new('Shepard',22,'s'),Animal.new('Pickle',12,'c'),Animal.new('Rodgers',55,'e')]
+array_repr = AnimalRepresenter.for_collection.new(animal_array).to_struct
+
+animal_klass = Struct.new("Animal", :name, :age, :species)
+AnimalRepresenter.new(animal_klass.new("qwik", 12, "old")).to_struct(wrap: "wrapper")