deep_unrest 0.1.67 → 0.1.71

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: abe2ede2503cab87add17ea53a5478017b900c6c66cc46945d6e9b2059cd8969
-  data.tar.gz: 656dc1d24558ea3f58971dcb9d51773679fbb90390d7b5bec6a97f12e796332b
+  metadata.gz: b194593cf43cb152738a7a45f748af7ebebcdc8e1eef0e65cb4e66cb5afead0b
+  data.tar.gz: c64a30a21918ad72e44269e652962ac73c11129b7a43b8ae8fe430e1ff081dd9
 SHA512:
-  metadata.gz: 29999e2452ac95d6efc8502ea36d3e0d6d21fe5dafc518c357b7d85ddf244d45acfdfac6b97e22ac497a2e62476ac17f0545b8082e83c61db66a77d9b759375c
-  data.tar.gz: e83b27db74652ec85bcff243455b81206b63680bc434813e20e4b29dd95298c44e6e41c73beea8f4623555412ff56ee74c4621afd29b2ef54496cd98983c7df2
+  metadata.gz: 0cbc2b6412e32c94173f074d205d5d04b8b24f50acaa036e5ec6aa9faa70edda2cd5811ee9b07dfc72e5e1dae59b7b8386a97a83f53cf6bca989d005cffff1d4
+  data.tar.gz: 206e06dbbb0fa9991acb0ac8dac8ce14f2348b7aa57d7cb79688312d3c0f073ede6570605e1db49f216b1ae52e9225ce1879d6b3a4cba9c77e0be87ad2136277

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/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) || {}
@@ -111,28 +116,37 @@ module DeepUnrest
       paginator = get_paginator(query, parent)
       resource = item[:resource]
 
-      # monkey patch the resource to only show authorized records
       r_metaclass = class << resource; self; end
-      r_metaclass.send(:alias_method, :records_original, :records)
+      if r_metaclass.method_defined? :records
+        r_metaclass.class_eval do
+          alias_method :records_original, :records
+        end
+      end
+
       # TODO: find a way to do this that doesn't blow out the original :records method
-      r_metaclass.define_singleton_method(:records) { |ctx|
-        item[:scope].merge(records_original(ctx))
+      resource.define_singleton_method(:records) { |ctx|
+        full_scope = if self.respond_to? :records_original
+          records_original(ctx)
+        else
+          super(ctx)
+        end
+
+        item[:scope].merge(full_scope)
       }
 
       # 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
-
-      # un-monkey patch the resource :records method
-      r_metaclass.send(:alias_method, :records, :records_original)
+      resource_results = format_processor_results(resource, jsonapi_result)
 
       meta << {
         addr: [*addr, item[:key], 'meta'],
@@ -146,14 +160,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,
@@ -164,6 +178,15 @@ module DeepUnrest
         included << result
         recurse_included_queries(ctx, result, mappings, parent_context, included, meta, [*next_addr, :include])
       end
+    ensure
+      # un-monkey patch the resource :records method
+      if r_metaclass.method_defined? :records_original
+        r_metaclass.class_eval do
+          alias_method :records, :records_original
+        end
+      else
+        r_metaclass.undef_method :records
+      end
     end
 
     def self.get_query_type(item)

data/lib/deep_unrest/version.rb

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

data/lib/deep_unrest/write.rb

@@ -73,16 +73,16 @@ 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
+        unpermitted_keys = attributes.keys.map(&:underscore).map(&:to_sym) - opts
         item[:errors] = unpermitted_keys.each_with_object({}) do |attr_key, memo|
           memo[attr_key] = 'Unpermitted parameter'
         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"
@@ -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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: deep_unrest
 version: !ruby/object:Gem::Version
-  version: 0.1.67
+  version: 0.1.71
 platform: ruby
 authors:
 - Lynn Hurley
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-02-25 00:00:00.000000000 Z
+date: 2021-08-30 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: []