graphiti 1.2.16 → 1.2.17

data/lib/graphiti/configuration.rb

@@ -43,7 +43,7 @@ module Graphiti
         end
 
         if (logger = ::Rails.logger)
-          self.debug = logger.level.zero? && self.debug
+          self.debug = logger.level.zero? && debug
           Graphiti.logger = logger
         end
       end

data/lib/graphiti/debugger.rb

@@ -59,10 +59,10 @@ module Graphiti
         add_chunk(payload[:resource], payload[:parent]) do |logs, json|
           logs << [" \\_ #{sideload.name}", :yellow, true]
           json[:name] = sideload.name
-          if sideload.class.scope_proc
-            query = "#{payload[:resource].class.name}: Manual sideload via .scope"
+          query = if sideload.class.scope_proc
+            "#{payload[:resource].class.name}: Manual sideload via .scope"
           else
-            query = "#{payload[:resource].class.name}.all(#{params.inspect})"
+            "#{payload[:resource].class.name}.all(#{params.inspect})"
           end
           logs << ["    #{query}", :cyan, true]
           json[:query] = query
@@ -148,7 +148,7 @@ module Graphiti
           parent: parent,
           logs: logs,
           json: json,
-          children: [],
+          children: []
         }
       end
 

data/lib/graphiti/delegates/pagination.rb

@@ -15,7 +15,7 @@ module Graphiti
           links[:last] = pagination_link(last_page)
           links[:prev] = pagination_link(current_page - 1) unless current_page == 1
           links[:next] = pagination_link(current_page + 1) unless current_page == last_page
-        end.select {|k, v| !v.nil? }
+        end.select { |k, v| !v.nil? }
       end
 
       private
@@ -29,8 +29,8 @@ module Graphiti
         uri.query = @proxy.query.hash.merge({
           page: {
             number: page,
-            size: page_size,
-          },
+            size: page_size
+          }
         }).to_query
         uri.to_s
       end

data/lib/graphiti/deserializer.rb

@@ -87,7 +87,7 @@ class Graphiti::Deserializer
       type: data[:type],
       temp_id: data[:'temp-id'],
       method: action,
-      payload_path: ["data"],
+      payload_path: ["data"]
     }
   end
 
@@ -185,10 +185,10 @@ class Graphiti::Deserializer
         jsonapi_type: datum[:type],
         temp_id: temp_id,
         method: method,
-        payload_path: ["included", included_idx],
+        payload_path: ["included", included_idx]
       },
       attributes: attributes,
-      relationships: relationships,
+      relationships: relationships
     }
   end
 

data/lib/graphiti/errors.rb

@@ -143,8 +143,13 @@ module Graphiti
       def message
         allow = @filter.values[0][:allow]
         deny = @filter.values[0][:deny]
+        value_string = if @value == "(empty)"
+          "empty value"
+        else
+          "value #{@value.inspect}"
+        end
         msg = <<-MSG
-          #{@resource.class.name}: tried to filter on #{@filter.keys[0].inspect}, but passed invalid value #{@value.inspect}.
+          #{@resource.class.name}: tried to filter on #{@filter.keys[0].inspect}, but passed invalid #{value_string}.
         MSG
         msg << "\nAllowlist: #{allow.inspect}" if allow
         msg << "\nDenylist: #{deny.inspect}" if deny
@@ -189,7 +194,7 @@ module Graphiti
 
           Make sure the endpoint "#{@sideload.resource.endpoint[:full_path]}" exists with action #{@action.inspect}, or customize the endpoint for #{@sideload.resource.class.name}.
 
-          If you do not wish to generate a link, pass link: false or set self.relationship_links_by_default = false.
+          If you do not wish to generate a link, pass link: false or set self.autolink = false.
         MSG
       end
     end
@@ -316,14 +321,14 @@ module Graphiti
             sortable: "sort on",
             filterable: "filter on",
             readable: "read",
-            writable: "write",
+            writable: "write"
           }[@flag]
         else
           {
             sortable: "add sort",
             filterable: "add filter",
             readable: "read",
-            writable: "write",
+            writable: "write"
           }[@flag]
         end
       end
@@ -722,6 +727,21 @@ module Graphiti
     end
 
     class RecordNotFound < Base
+      def initialize(resource = nil, id = nil, path = nil)
+        @resource = resource
+        @id = id
+        @path = path
+      end
+
+      def message
+        if !@resource.nil? && !@id.nil?
+          "The referenced resource '#{@resource}' with id '#{@id}' could not be found.".tap do |msg|
+            msg << " Referenced at '#{@path}'" unless @path.nil?
+          end
+        else
+          "Specified Record Not Found"
+        end
+      end
     end
 
     class RequiredFilter < Base

data/lib/graphiti/query.rb

@@ -232,7 +232,7 @@ module Graphiti
       return true if @resource.remote?
 
       if (att = @resource.get_attr(name, flag, request: true))
-        return att
+        att
       else
         not_associated_name = !@resource.class.association_names.include?(name)
         not_associated_type = !@resource.class.association_types.include?(name)
@@ -262,8 +262,8 @@ module Graphiti
     def parse_fieldset(fieldset)
       {}.tap do |hash|
         fieldset.each_pair do |type, fields|
-          type       = type.to_sym
-          fields     = fields.split(",") unless fields.is_a?(Array)
+          type = type.to_sym
+          fields = fields.split(",") unless fields.is_a?(Array)
           hash[type] = fields.map(&:to_sym)
         end
       end
@@ -282,7 +282,7 @@ module Graphiti
 
     def sort_hash(attr)
       value = attr[0] == "-" ? :desc : :asc
-      key   = attr.sub("-", "").to_sym
+      key = attr.sub("-", "").to_sym
 
       {key => value}
     end

data/lib/graphiti/railtie.rb

@@ -6,7 +6,7 @@ module Graphiti
     end
 
     generators do
-      Dir[File.expand_path("../../deprecated_generators/**/*.rb", __dir__)].each do |f|
+      Dir[File.expand_path("../../deprecated_generators/**/*.rb", __dir__)].sort.each do |f|
         require f
       end
     end

data/lib/graphiti/request_validator.rb

@@ -1,10 +1,10 @@
 module Graphiti
   class RequestValidator
     delegate :validate,
-             :validate!,
-             :errors,
-             :deserialized_payload,
-             to: :@validator
+      :validate!,
+      :errors,
+      :deserialized_payload,
+      to: :@validator
 
     def initialize(root_resource, raw_params)
       @validator = ValidatorFactory.create(root_resource, raw_params)

data/lib/graphiti/request_validators/update_validator.rb

@@ -5,7 +5,7 @@ module Graphiti
         if required_payload? && payload_matches_endpoint?
           super
         else
-          return false
+          false
         end
       end
 
@@ -36,7 +36,6 @@ module Graphiti
           attribute_mismatch([:data, :id])
         end
 
-
         meta_type = @raw_params.dig(:data, :type)
 
         # NOTE: calling #to_s and comparing 2 strings is slower than

data/lib/graphiti/request_validators/validator.rb

@@ -25,7 +25,7 @@ module Graphiti
 
       def validate!
         unless validate
-          raise @error_class || Graphiti::Errors::InvalidRequest, self.errors
+          raise @error_class || Graphiti::Errors::InvalidRequest, errors
         end
 
         true
@@ -47,7 +47,7 @@ module Graphiti
       def process_relationships(resource, relationships, payload_path)
         opts = {
           resource: resource,
-          relationships: relationships,
+          relationships: relationships
         }
 
         Graphiti::Util::RelationshipPayload.iterate(opts) do |x|

data/lib/graphiti/resource/configuration.rb

@@ -22,7 +22,7 @@ module Graphiti
         end
 
         def type=(val)
-          val = val && val.to_sym
+          val = val&.to_sym
           if (val = super)
             serializer.type(val)
           end
@@ -42,7 +42,7 @@ module Graphiti
             path: val,
             full_path: val,
             url: val,
-            actions: [:index, :show],
+            actions: [:index, :show]
           }
         end
 
@@ -82,7 +82,8 @@ module Graphiti
           :attributes_schema_by_default,
           :relationships_readable_by_default,
           :relationships_writable_by_default,
-          :filters_accept_nil_by_default
+          :filters_accept_nil_by_default,
+          :filters_deny_empty_by_default
 
         class << self
           prepend Overrides
@@ -104,6 +105,7 @@ module Graphiti
           default(klass, :relationships_readable_by_default, true)
           default(klass, :relationships_writable_by_default, true)
           default(klass, :filters_accept_nil_by_default, false)
+          default(klass, :filters_deny_empty_by_default, false)
 
           unless klass.config[:attributes][:id]
             klass.attribute :id, :integer_id
@@ -197,7 +199,7 @@ module Graphiti
               attributes: {},
               extra_attributes: {},
               sideloads: {},
-              callbacks: {},
+              callbacks: {}
             }
         end
 

data/lib/graphiti/resource/dsl.rb

@@ -32,7 +32,8 @@ module Graphiti
               dependencies: opts[:dependent],
               required: required,
               operators: operators.to_hash,
-              allow_nil: opts.fetch(:allow_nil, filters_accept_nil_by_default)
+              allow_nil: opts.fetch(:allow_nil, filters_accept_nil_by_default),
+              deny_empty: opts.fetch(:deny_empty, filters_deny_empty_by_default)
             }
           elsif (type = args[0])
             attribute name, type, only: [:filterable], allow: opts[:allow]
@@ -55,7 +56,7 @@ module Graphiti
 
           if get_attr(name, :sortable, raise_error: :only_unsupported)
             config[:sorts][name] = {
-              proc: blk,
+              proc: blk
             }.merge(opts.slice(:only))
           elsif (type = args[0])
             attribute name, type, only: [:sortable]
@@ -78,7 +79,7 @@ module Graphiti
         def default_filter(name = nil, &blk)
           name ||= :__default
           config[:default_filters][name.to_sym] = {
-            filter: blk,
+            filter: blk
           }
         end
 
@@ -131,7 +132,7 @@ module Graphiti
             readable: true,
             writable: false,
             sortable: false,
-            filterable: false,
+            filterable: false
           }
           options = defaults.merge(options)
           config[:extra_attributes][name] = options

data/lib/graphiti/resource/links.rb

@@ -39,7 +39,7 @@ module Graphiti
           path: path,
           full_path: full_path_for(path),
           url: url_for(path),
-          actions: DEFAULT_ACTIONS.dup,
+          actions: DEFAULT_ACTIONS.dup
         }
       end
 
@@ -49,7 +49,7 @@ module Graphiti
           path: path,
           full_path: full_path_for(path),
           url: url_for(path),
-          actions: actions,
+          actions: actions
         }
       end
 
@@ -60,7 +60,7 @@ module Graphiti
           path: path,
           full_path: full_path_for(path),
           url: url_for(path),
-          actions: actions,
+          actions: actions
         }]
       end
 

data/lib/graphiti/resource/persistence.rb

@@ -89,7 +89,8 @@ module Graphiti
 
       def update(update_params, meta = nil)
         model_instance = nil
-        id = update_params.delete(:id)
+        id = update_params[:id]
+        update_params = update_params.except(:id)
 
         run_callbacks :persistence, :update, update_params, meta do
           run_callbacks :attributes, :update, update_params, meta do |params|

data/lib/graphiti/resource/polymorphism.rb

@@ -67,7 +67,8 @@ module Graphiti
         end
 
         def resource_for_model(model)
-          resource = children.find { |c| model.is_a?(c.model) }
+          resource = children.find { |c| model.class == c.model } ||
+            children.find { |c| model.is_a?(c.model) }
           if resource.nil?
             raise Errors::PolymorphicResourceChildNotFound.new(self, model: model)
           else

data/lib/graphiti/resource/remote.rb

@@ -19,7 +19,7 @@ module Graphiti
       end
 
       def save(model, meta)
-        if meta[:attributes] == {} && meta[:method] == :update
+        if meta[:attributes].except(:id) == {} && meta[:method] == :update
           model
         else
           raise Errors::RemoteWrite.new(self.class)

data/lib/graphiti/runner.rb

@@ -50,7 +50,7 @@ module Graphiti
     def jsonapi_render_options
       options = {}
       options.merge!(default_jsonapi_render_options)
-      options[:meta]   ||= {}
+      options[:meta] ||= {}
       options[:expose] ||= {}
       options[:expose][:context] = jsonapi_context
       options

data/lib/graphiti/schema.rb

@@ -33,7 +33,7 @@ module Graphiti
       {
         resources: generate_resources,
         endpoints: generate_endpoints,
-        types: generate_types,
+        types: generate_types
       }
     end
 
@@ -94,7 +94,7 @@ module Graphiti
           extra_attributes: extra_attributes(r),
           sorts: sorts(r),
           filters: filters(r),
-          relationships: relationships(r),
+          relationships: relationships(r)
         }
 
         if r.default_sort
@@ -121,7 +121,7 @@ module Graphiti
           name: r.name,
           description: r.description,
           remote: r.remote_url,
-          relationships: relationships(r),
+          relationships: relationships(r)
         }
       }
 
@@ -136,7 +136,7 @@ module Graphiti
               type: config[:type].to_s,
               readable: flag(config[:readable]),
               writable: flag(config[:writable]),
-              description: resource.attribute_description(name),
+              description: resource.attribute_description(name)
             }
           end
         end
@@ -149,7 +149,7 @@ module Graphiti
           attrs[name] = {
             type: config[:type].to_s,
             readable: flag(config[:readable]),
-            description: resource.attribute_description(name),
+            description: resource.attribute_description(name)
           }
         end
       end
@@ -186,7 +186,7 @@ module Graphiti
 
           config = {
             type: filter[:type].to_s,
-            operators: filter[:operators].keys.map(&:to_s),
+            operators: filter[:operators].keys.map(&:to_s)
           }
 
           config[:single] = true if filter[:single]

data/lib/graphiti/scope.rb

@@ -3,10 +3,10 @@ module Graphiti
     attr_accessor :object, :unpaginated_object
     attr_reader :pagination
     def initialize(object, resource, query, opts = {})
-      @object    = object
-      @resource  = resource
-      @query     = query
-      @opts      = opts
+      @object = object
+      @resource = resource
+      @query = query
+      @opts = opts
 
       @object = @resource.around_scoping(@object, @query.hash) { |scope|
         apply_scoping(scope, opts)
@@ -75,7 +75,7 @@ module Graphiti
         resource: @resource,
         params: @opts[:params],
         sideload: @opts[:sideload],
-        parent: @opts[:parent],
+        parent: @opts[:parent]
         # Set once data is resolved within block
         #   results: ...
       }