deep_unrest 0.1.64 → 0.1.69

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2455324c4b8f985d12776c58a397e719261a8a67eed3de7d713527127d6e412d
-  data.tar.gz: 3bd44536c8a3a861d0ff9fbf0d11d3dac75a5bfd898e1cdcfff83d076ab3eb03
+  metadata.gz: ca959dddbde430d68949a501d37ebd9b995dfd390849fdbe40ebc53a9dc42a72
+  data.tar.gz: 9491ceca33bc4f4c6b0cacd5dee5f0dd2377ca2f3e9d00c035f7efd3f0fa9033
 SHA512:
-  metadata.gz: 1e2f633f77b6a5f34277a09280c6017f452c4d42f6cd6da86ed4569c09ad97b175c478c8a707457ba23f1ec4c9a21002df35349957b8d99ca2425664bf3d3ba9
-  data.tar.gz: 675b09eea9fd593ee92509d001d2d411506044be2e79e50ca456ff363dd7cbb430d67fa412168565269088171c89a3cd6136f7f2eccc29a82b55389ecb6e27a6
+  metadata.gz: 5b7ea668419b1b53ff670a633657424a140105c3028d6bd32fd48b39285c6749500545e8ee0ec284da6c9d0c2dec0544ed632520b4f7bf619d0433e6d8e80ea9
+  data.tar.gz: '0220088c448da93dbbba7ebc8c43312b6734e2a494a9df4d65017e0b599ba5a100d848e4c01b1367269019c2a6eae008f0f7be57d5b32f7649de1b8062fb2a8e'

data/app/controllers/deep_unrest/application_controller.rb

@@ -51,6 +51,7 @@ module DeepUnrest
     def write
       repaired_params = repair_nested_params(params[:data])
       data = repaired_params[:data]
+      data = ActionController::Parameters.new({}) if data&.blank?
       context = repaired_params[:context] || {}
       context[:uuid] = request.uuid
       context[:current_user] = current_user

data/config/routes.rb

@@ -1,5 +1,6 @@
 DeepUnrest::Engine.routes.draw do
   patch 'update', to: 'application#update'
   get 'read', to: 'application#read'
+  patch 'read', to: 'application#read'
   patch 'write', to: 'application#write'
 end

data/lib/deep_unrest.rb

@@ -162,7 +162,7 @@ module DeepUnrest
   def self.parse_attributes(type, scope_type, attributes, user)
     p = JSONAPI::RequestParser.new
     resource = get_resource(type)
-    p.resource_klass = resource
+    p.source_klass = resource
     ctx = { current_user: user }
     opts = if scope_type == :create
              resource.creatable_fields(ctx)
@@ -170,7 +170,7 @@ module DeepUnrest
              resource.updatable_fields(ctx)
            end
 
-    p.parse_params({ attributes: attributes }, opts)[:attributes]
+    p.parse_params(resource, { attributes: attributes }, opts)[:attributes]
   rescue JSONAPI::Exceptions::ParameterNotAllowed
     unpermitted_keys = attributes.keys.map(&:to_sym) - opts
     msg = "Attributes #{unpermitted_keys} of #{type.classify} not allowed"
@@ -216,7 +216,7 @@ module DeepUnrest
   end
 
   def self.parse_id(id_str)
-    return unless id_str.is_a? String
+    return unless id_str.is_a?(String) || id_str.is_a?(Integer)
     return false if id_str.nil?
     return id_str if id_str.is_a? Integer
 
@@ -558,6 +558,19 @@ module DeepUnrest
     DeepUnrest::Write.write(ctx, params, user)
   end
 
+  def self.serialize_resource(resource_klass, fields, id)
+    resource_identity = JSONAPI::ResourceIdentity.new(resource_klass, id)
+    id_tree = JSONAPI::PrimaryResourceIdTree.new
+    id_tree.add_resource_fragment(JSONAPI::ResourceFragment.new(resource_identity), {})
+    resource_set = JSONAPI::ResourceSet.new(id_tree)
+    serializer = JSONAPI::ResourceSerializer.new(
+        resource_klass,
+        fields: fields
+    )
+    resource_set.populate!(serializer, {}, {})
+    serializer.serialize_resource_set_to_hash_single(resource_set)['data'].except('links')
+  end
+
   def self.serialize_changes(diffs, user)
     ctx = { current_user: user }
     diffs.each do |diff|
@@ -582,10 +595,7 @@ module DeepUnrest
       keys = diff[:resource].keys.map(&:to_sym)
       fields[to_assoc(diff[:klass].to_s.pluralize)] = keys
 
-      JSONAPI::ResourceSerializer.new(
-        resource_klass,
-        fields: fields
-      ).serialize_to_hash(resource_klass.new(diff[:model], ctx))[:data]
+      serialize_resource(resource_klass, fields, diff[:model].id)
     end
     resources.select { |item| item.dig('attributes') }.compact
   end
@@ -692,10 +702,10 @@ module DeepUnrest
   def self.serialize_result(ctx, item)
     resource = item[:resource]
     resource_instance = resource.new(item[:record], ctx)
-    fields = item[:query][:fields].map(&:underscore).map(&:to_sym)
-    JSONAPI::ResourceSerializer.new(
-      resource,
-      fields: { "#{item[:key].underscore.pluralize}": fields }
-    ).serialize_to_hash(resource_instance)[:data]
+    fields = {}
+    keys = item[:query][:fields].map(&:underscore).map(&:to_sym)
+    fields[to_assoc(item[:key].pluralize)] = keys
+
+    serialize_resource(resource, fields, item[:record].id).except(:links)
   end
 end

data/lib/deep_unrest/read.rb

@@ -102,6 +102,11 @@ module DeepUnrest
       end
     end
 
+    def self.format_processor_results(resource_klass, processor_result)
+      results = processor_result.resource_set.resource_klasses[resource_klass]
+      results.values.map {|r| r[:resource] }
+    end
+
     def self.query_list(ctx, item, mappings, parent_context, included, meta, addr, parent)
       base_query = item[:query].deep_dup
       extension = base_query.dig(:extend, parent&.fetch(:record)&.id&.to_s&.underscore) || {}
@@ -122,15 +127,20 @@ module DeepUnrest
       # transform sort value casing for rails
       sort_criteria = query[:sort]&.map { |s| s.clone.merge(field: s[:field].underscore) }
 
+      serializer = JSONAPI::ResourceSerializer.new(resource)
+
       processor = JSONAPI::Processor.new(resource,
                                          :find,
                                          filters: query[:filter] || {},
                                          context: ctx,
                                          sort_criteria: sort_criteria,
+                                         serializer: serializer,
                                          paginator: paginator)
 
       jsonapi_result = processor.process
 
+      resource_results = format_processor_results(resource, jsonapi_result)
+
       # un-monkey patch the resource :records method
       r_metaclass.send(:alias_method, :records, :records_original)
 
@@ -146,14 +156,14 @@ module DeepUnrest
       }
 
       # make sure to return empty array if no results are found for this node
-      if jsonapi_result.resources.empty?
+      if resource_results.empty?
         meta << {
           addr: [*addr, item[:key], 'data'],
           serialized_result: []
         }
       end
 
-      jsonapi_result.resources.each_with_index do |record, i|
+      resource_results.each_with_index do |record, i|
         next_addr = [*addr, item[:key], 'data[]', i]
         result = {
           **item,

data/lib/deep_unrest/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DeepUnrest
-  VERSION = '0.1.64'
+  VERSION = '0.1.69'
 end

data/lib/deep_unrest/write.rb

@@ -3,7 +3,6 @@
 module DeepUnrest
   module Write
     def self.get_scope_type(item, path)
-      return :index if DeepUnrest.plural?(path)
       return :destroy if item[:destroy]
       return :show if item[:readOnly] || item[:attributes].blank?
       return :create if item[:id] && DeepUnrest.temp_id?(item[:id].to_s)
@@ -74,14 +73,14 @@ module DeepUnrest
         attributes = item.dig(:query, :attributes) || {}
         resource = item[:resource]
         p = JSONAPI::RequestParser.new
-        p.resource_klass = resource
+        p.source_klass = resource
         opts = if item[:scope_type] == :create
                  resource.creatable_fields(ctx)
                else
                  resource.updatable_fields(ctx)
                end
 
-        p.parse_params({ attributes: attributes }, opts)[:attributes]
+        p.parse_params(resource, { attributes: attributes }, opts)[:attributes]
       rescue JSONAPI::Exceptions::ParameterNotAllowed
         unpermitted_keys = attributes.keys.map(&:to_sym) - opts
         item[:errors] = unpermitted_keys.each_with_object({}) do |attr_key, memo|

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: deep_unrest
 version: !ruby/object:Gem::Version
-  version: 0.1.64
+  version: 0.1.69
 platform: ruby
 authors:
 - Lynn Hurley
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-01-27 00:00:00.000000000 Z
+date: 2021-05-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -16,28 +16,28 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.2.3
+        version: 6.1.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 5.2.3
+        version: 6.1.3
 - !ruby/object:Gem::Dependency
   name: jsonapi-resources
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.9.10
+        version: '0.10'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.9.10
+        version: '0.10'
 - !ruby/object:Gem::Dependency
   name: sqlite3
   requirement: !ruby/object:Gem::Requirement
@@ -58,42 +58,42 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.2.3
+        version: 2.0.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.2.3
+        version: 2.0.0
 - !ruby/object:Gem::Dependency
   name: pundit
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: 2.1.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: 2.1.0
 - !ruby/object:Gem::Dependency
   name: dragonfly
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.1.2
+        version: 1.3.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.1.2
+        version: 1.3.0
 - !ruby/object:Gem::Dependency
   name: database_cleaner
   requirement: !ruby/object:Gem::Requirement
@@ -114,14 +114,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.7.3
+        version: 2.18.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 1.7.3
+        version: 2.18.0
 description: Update multiple or deeply nested JSONAPI resources
 email:
 - lynn.dylan.hurley@gmail.com
@@ -157,7 +157,7 @@ homepage: https://github.com/graveflex/deep_unrest
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -172,8 +172,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.2.3
+signing_key:
 specification_version: 4
 summary: Update multiple or deeply nested JSONAPI resources
 test_files: []