Merge pull request #312 from cerebris/fix_engine_check

Fix issue where a nil engine name isn't being cached so it was being reevaluated repeatedly. Was slowing down large result sets by about 4X.

Author: lgebhardt

lib/jsonapi-resources.rb

@@ -20,3 +20,4 @@
 require 'jsonapi/operation_result'
 require 'jsonapi/operation_results'
 require 'jsonapi/callbacks'
+require 'jsonapi/link_builder'

lib/jsonapi/link_builder.rb

@@ -8,10 +8,11 @@ def initialize(config = {})
       @base_url               = config[:base_url]
       @primary_resource_klass = config[:primary_resource_klass]
       @route_formatter        = config[:route_formatter]
+      @is_engine              = !!engine_name
     end
 
     def engine?
-      !!engine_name
+      @is_engine
     end
 
     def engine_name

lib/jsonapi/resource_serializer.rb

@@ -1,5 +1,3 @@
-require_relative "link_builder"
-
 module JSONAPI
   class ResourceSerializer