jsonapi-resources 0.10.7 → 0.11.0.beta2

data/lib/jsonapi/relationship.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 module JSONAPI
   class Relationship
     attr_reader :acts_as_set, :foreign_key, :options, :name,
-                :class_name, :polymorphic, :always_include_optional_linkage_data,
+                :class_name, :polymorphic, :always_include_optional_linkage_data, :exclude_linkage_data,
                 :parent_resource, :eager_load_on_include, :custom_methods,
-                :inverse_relationship, :allow_include, :use_related_resource_records_for_joins
+                :inverse_relationship, :allow_include, :hidden, :use_related_resource_records_for_joins
 
     attr_writer :allow_include
 
@@ -15,7 +17,7 @@ module JSONAPI
       @acts_as_set = options.fetch(:acts_as_set, false) == true
       @foreign_key = options[:foreign_key] ? options[:foreign_key].to_sym : nil
       @parent_resource = options[:parent_resource]
-      @relation_name = options.fetch(:relation_name, @name)
+      @relation_name = options[:relation_name]
       @polymorphic = options.fetch(:polymorphic, false) == true
       @polymorphic_types = options[:polymorphic_types]
       if options[:polymorphic_relations]
@@ -28,15 +30,19 @@ module JSONAPI
                                                        else
                                                          JSONAPI.configuration.use_related_resource_records_for_joins
                                                        end
-      
+
       @use_related_resource_records_for_joins = options.fetch(:use_related_resource_records_for_joins,
                                                               use_related_resource_records_for_joins_default) == true
 
+      @hidden = options.fetch(:hidden, false) == true
+
+      @exclude_linkage_data = options[:exclude_linkage_data]
       @always_include_optional_linkage_data = options.fetch(:always_include_optional_linkage_data, false) == true
-      @eager_load_on_include = options.fetch(:eager_load_on_include, false) == true
+      @eager_load_on_include = options.fetch(:eager_load_on_include, true) == true
       @allow_include = options[:allow_include]
       @class_name = nil
-      @inverse_relationship = nil
+
+      @inverse_relationship = options[:inverse_relationship]&.to_sym
 
       @_routed = false
       @_warned_missing_route = false
@@ -66,12 +72,26 @@ module JSONAPI
       # :nocov:
     end
 
+    def inverse_relationship
+      unless @inverse_relationship
+        @inverse_relationship ||= if resource_klass._relationship(@parent_resource._type.to_s.singularize).present?
+                                    @parent_resource._type.to_s.singularize.to_sym
+                                  elsif resource_klass._relationship(@parent_resource._type).present?
+                                    @parent_resource._type.to_sym
+                                  else
+                                    nil
+                                  end
+      end
+
+      @inverse_relationship
+    end
+
     def self.polymorphic_types(name)
       @poly_hash ||= {}.tap do |hash|
         ObjectSpace.each_object do |klass|
           next unless Module === klass
           if ActiveRecord::Base > klass
-            klass.reflect_on_all_associations(:has_many).select{|r| r.options[:as] }.each do |reflection|
+            klass.reflect_on_all_associations(:has_many).select { |r| r.options[:as] }.each do |reflection|
               (hash[reflection.options[:as]] ||= []) << klass.name.underscore
             end
           end
@@ -82,7 +102,7 @@ module JSONAPI
 
     def resource_types
       if polymorphic? && belongs_to?
-        @polymorphic_types ||= self.class.polymorphic_types(@relation_name).collect {|t| t.pluralize}
+        @polymorphic_types ||= self.class.polymorphic_types(_relation_name).collect { |t| t.pluralize }
       else
         [resource_klass._type.to_s.pluralize]
       end
@@ -93,15 +113,15 @@ module JSONAPI
     end
 
     def relation_name(options)
-      case @relation_name
-        when Symbol
-          # :nocov:
-          @relation_name
-          # :nocov:
-        when String
-          @relation_name.to_sym
-        when Proc
-          @relation_name.call(options)
+      case _relation_name
+      when Symbol
+        # :nocov:
+        _relation_name
+        # :nocov:
+      when String
+        _relation_name.to_sym
+      when Proc
+        _relation_name.call(options)
       end
     end
 
@@ -117,14 +137,14 @@ module JSONAPI
 
     def exclude_links(exclude)
       case exclude
-        when :default, "default"
-          @_exclude_links = [:self, :related]
-        when :none, "none"
-          @_exclude_links = []
-        when Array
-          @_exclude_links = exclude.collect {|link| link.to_sym}
-        else
-          fail "Invalid exclude_links"
+      when :default, "default"
+        @_exclude_links = [:self, :related]
+      when :none, "none"
+        @_exclude_links = []
+      when Array
+        @_exclude_links = exclude.collect { |link| link.to_sym }
+      else
+        fail "Invalid exclude_links"
       end
     end
 
@@ -136,6 +156,10 @@ module JSONAPI
       _exclude_links.include?(link.to_sym)
     end
 
+    def _relation_name
+      @relation_name || @name
+    end
+
     class ToOne < Relationship
       attr_reader :foreign_key_on
 
@@ -144,9 +168,16 @@ module JSONAPI
         @class_name = options.fetch(:class_name, name.to_s.camelize)
         @foreign_key ||= "#{name}_id".to_sym
         @foreign_key_on = options.fetch(:foreign_key_on, :self)
-        if parent_resource
-          @inverse_relationship = options.fetch(:inverse_relationship, parent_resource._type)
+        # if parent_resource
+        #   @inverse_relationship = options.fetch(:inverse_relationship, parent_resource._type)
+        # end
+
+        if options.fetch(:create_implicit_polymorphic_type_relationships, true) == true && polymorphic?
+          # Setup the implicit relationships for the polymorphic types and exclude linkage data
+          setup_implicit_relationships_for_polymorphic_types
         end
+
+        @polymorphic_type_relationship_for = options[:polymorphic_type_relationship_for]
       end
 
       def to_s
@@ -161,11 +192,30 @@ module JSONAPI
         # :nocov:
       end
 
+      def hidden?
+        @hidden || @polymorphic_type_relationship_for.present?
+      end
+
       def polymorphic_type
         "#{name}_type" if polymorphic?
       end
 
+      def setup_implicit_relationships_for_polymorphic_types(exclude_linkage_data: true)
+        types = self.class.polymorphic_types(_relation_name)
+        unless types.present?
+          warn "No polymorphic types found for #{parent_resource.name} #{_relation_name}"
+          return
+        end
+
+        types.each do |type|
+          parent_resource.has_one(type.to_s.underscore.singularize,
+                                  exclude_linkage_data: exclude_linkage_data,
+                                  polymorphic_type_relationship_for: name)
+        end
+      end
+
       def include_optional_linkage_data?
+        return false if @exclude_linkage_data
         @always_include_optional_linkage_data || JSONAPI::configuration.always_include_to_one_linkage_data
       end
 
@@ -176,10 +226,10 @@ module JSONAPI
                      @allow_include
                    end
 
-        if !!strategy == strategy #check for boolean
+        if !!strategy == strategy # check for boolean
           return strategy
         elsif strategy.is_a?(Symbol) || strategy.is_a?(String)
-          parent_resource.send(strategy, context)
+          parent_resource_klass.send(strategy, context)
         else
           strategy.call(context)
         end
@@ -194,17 +244,21 @@ module JSONAPI
         @class_name = options.fetch(:class_name, name.to_s.camelize.singularize)
         @foreign_key ||= "#{name.to_s.singularize}_ids".to_sym
         @reflect = options.fetch(:reflect, true) == true
-        if parent_resource
-          @inverse_relationship = options.fetch(:inverse_relationship, parent_resource._type.to_s.singularize.to_sym)
-        end
+        # if parent_resource
+        #   @inverse_relationship = options.fetch(:inverse_relationship, parent_resource._type.to_s.singularize.to_sym)
+        # end
       end
 
       def to_s
         # :nocov: useful for debugging
-        "#{parent_resource}.#{name}(ToMany)"
+        "#{parent_resource_klass}.#{name}(ToMany)"
         # :nocov:
       end
 
+      def hidden?
+        @hidden
+      end
+
       def include_optional_linkage_data?
         # :nocov:
         @always_include_optional_linkage_data || JSONAPI::configuration.always_include_to_many_linkage_data
@@ -218,10 +272,10 @@ module JSONAPI
                      @allow_include
                    end
 
-        if !!strategy == strategy #check for boolean
+        if !!strategy == strategy # check for boolean
           return strategy
         elsif strategy.is_a?(Symbol) || strategy.is_a?(String)
-          parent_resource.send(strategy, context)
+          parent_resource_klass.send(strategy, context)
         else
           strategy.call(context)
         end