panko_serializer 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d166380b6a032ffaf299ff6aff8dea9dabcb7354
-  data.tar.gz: 3fd7c899d24982bada152680444403b767ff2522
+  metadata.gz: ba52569ddbbd4f0de8ea0cd554f7df5fdf78340e
+  data.tar.gz: ae21f470b06c1576077e2f56211fb00b36c91118
 SHA512:
-  metadata.gz: 7055dea074e595a3d6e01c1acbdd3f18b6d89419c1750d51918e2ba4f90264acb5d985b6a42fe472fa28185b19b938cb8f906a0b69900d4cd40d5b57d71c6326
-  data.tar.gz: 9df158a3287ef8f036cfbf7417003b5e2aba2f0546cbeedfdb398be08db211525f6735cea09db5861c3e467f5a6948e0f1d39824b9aca881690f7ed6cb0e2f73
+  metadata.gz: 9acdd06de3e6268bbacdcdc97e897357534253728cbac9f8311682ccd5c468abccce8d89226cb7d1c7b811c67ff4a103951b5030128fdf0865f0dfd62823e43f
+  data.tar.gz: a4aa910864f1aa16ac0a57f4004a8b9698f06233e2ad83c8193d2b7fb22cbd26a17871248e34c2e84fe78e888456392c375ef8c73e4cec0ba00faf8d1885a00c

data/README.md

@@ -1,8 +1,9 @@
 # Panko
 
-[![Build Status](https://travis-ci.org/yosiat/panko.svg?branch=master)](https://travis-ci.org/yosiat/panko)
+[![Build Status](https://travis-ci.org/yosiat/panko_serializer.svg?branch=master)](https://travis-ci.org/yosiat/panko_serializer)
 
 
 ## License
 The gem is available as open source under the terms of the [MIT License](http://opensource.org/licenses/MIT).
 
+

data/benchmarks/type_casts/support.rb

@@ -7,7 +7,7 @@ require "pg"
 
 
 require_relative "../benchmarking_support"
-require_relative "../../lib/panko/panko"
+require_relative "../../lib/panko/panko_serializer"
 
 def assert(type_name, from, to)
   raise "#{type_name} - #{from.class} is not equals to #{to.class}" unless from.to_json == to.to_json

data/ext/panko_serializer/attributes_iterator.c

@@ -3,6 +3,10 @@
 static ID attributes_id = 0;
 static ID types_id = 0;
 static ID values_id = 0;
+static ID delegate_hash_id = 0;
+
+static ID value_before_type_cast_id = 0;
+static ID type_id = 0;
 
 VALUE read_attributes(VALUE obj) {
   return rb_ivar_get(obj, attributes_id);
@@ -30,24 +34,42 @@ void panko_read_types_and_value(VALUE attributes_hash,
 }
 
 VALUE panko_each_attribute(VALUE obj,
-                           SerializationDescriptor descriptor,
                            VALUE attributes,
                            EachAttributeFunc func,
                            VALUE context) {
   VALUE attributes_hash = panko_read_lazy_attributes_hash(obj);
-  if(attributes_hash == Qnil) {
+  if (attributes_hash == Qnil) {
     return Qnil;
   }
 
+  VALUE delegate_hash = rb_ivar_get(attributes_hash, delegate_hash_id);
+  bool tryToReadFromDelegateHash = RHASH_SIZE(delegate_hash) > 0;
+
   VALUE types, values;
   panko_read_types_and_value(attributes_hash, &types, &values);
 
   int i;
   for (i = 0; i < RARRAY_LEN(attributes); i++) {
-    VALUE member = rb_sym2str(RARRAY_AREF(attributes, i));
+    VALUE member = RARRAY_AREF(attributes, i);
+
+    VALUE value = Qundef;
+    VALUE type_metadata = Qnil;
+
+    // First try to read from delegate hash,
+    // If the object was create in memory `User.new(name: "Yosi")`
+    // it won't exist in types/values
+    if (tryToReadFromDelegateHash) {
+      VALUE attribute_metadata = rb_hash_aref(delegate_hash, member);
+      if (attribute_metadata != Qnil) {
+        value = rb_ivar_get(attribute_metadata, value_before_type_cast_id);
+        type_metadata = rb_ivar_get(attribute_metadata, type_id);
+      }
+    }
 
-    VALUE value = rb_hash_aref(values, member);
-    VALUE type_metadata = rb_hash_aref(types, member);
+    if (value == Qundef) {
+      value = rb_hash_aref(values, member);
+      type_metadata = rb_hash_aref(types, member);
+    }
 
     func(obj, member, value, type_metadata, context);
   }
@@ -57,6 +79,9 @@ VALUE panko_each_attribute(VALUE obj,
 
 void panko_init_attributes_iterator(VALUE mPanko) {
   attributes_id = rb_intern("@attributes");
+  delegate_hash_id = rb_intern("@delegate_hash");
   values_id = rb_intern("@values");
   types_id = rb_intern("@types");
+  type_id = rb_intern("@type");
+  value_before_type_cast_id = rb_intern("@value_before_type_cast");
 }

data/ext/panko_serializer/attributes_iterator.h

@@ -1,4 +1,5 @@
 #include <ruby.h>
+#include <stdbool.h>
 
 #include "serialization_descriptor.h"
 
@@ -9,7 +10,6 @@ typedef void (*EachAttributeFunc)(VALUE object,
                                   VALUE context);
 
 extern VALUE panko_each_attribute(VALUE object,
-                                  SerializationDescriptor descriptor,
                                   VALUE attributes,
                                   EachAttributeFunc func,
                                   VALUE context);

data/ext/panko_serializer/panko_serializer.c

@@ -53,8 +53,7 @@ void serialize_fields(VALUE subject,
                       VALUE str_writer,
                       SerializationDescriptor descriptor,
                       VALUE context) {
-  panko_each_attribute(subject, descriptor, descriptor->fields,
-                       panko_attributes_iter, str_writer);
+  panko_each_attribute(subject, descriptor->fields, panko_attributes_iter, str_writer);
 
   serialize_method_fields(subject, str_writer, descriptor, context);
 }

data/ext/panko_serializer/time_conversion.c

@@ -39,8 +39,7 @@ VALUE iso_ar_iso_datetime_string(const char* value) {
 
   VALUE output = Qnil;
   if (r >= 0) {
-    char buf[21];
-    sprintf(buf, "");
+    char buf[21] = "";
 
     append_region_part(buf, value, region, 1);
     strncat(buf, "-", 1);

data/lib/panko/serialization_descriptor.rb

@@ -16,6 +16,7 @@ module Panko
         serializer_only_filters,
         serializer_except_filters
       )
+      backend.fields = backend.fields.map &:to_s
 
       backend.method_fields = apply_fields_filters(
         method_fields,

data/lib/panko/serializer.rb

@@ -34,23 +34,18 @@ module Panko
     def initialize(options = {})
       @descriptor = Panko::CACHE.fetch(self.class, options)
 
-      @context = options.fetch(:context, nil)
-      @object = nil
+      @context = options[:context]
     end
 
     attr_reader :object, :context
-    attr_writer :object
 
-    def serialize(object, writer = nil)
-      Oj.load(serialize_to_json(object, writer))
+    def serialize(object)
+      Oj.load(serialize_to_json(object))
     end
 
-    def serialize_to_json(object, writer = nil)
-      @object = object
-
-      writer ||= Oj::StringWriter.new(mode: :rails)
+    def serialize_to_json(object)
+      writer = Oj::StringWriter.new(mode: :rails)
       Panko::serialize_subject(object, writer, @descriptor, @context)
-
       writer.to_s
     end
   end

data/lib/panko/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module Panko
-  VERSION = "0.1.2"
+  VERSION = "0.1.3"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: panko_serializer
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Yosi Attias
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-08-13 00:00:00.000000000 Z
+date: 2017-08-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler