Merge pull request #962 from fsmanuel/fix/processor

Prevent error if using paginator :none

Author: lgebhardt

lib/jsonapi/processor.rb

@@ -69,7 +69,7 @@ def find
                                                            include_directives: include_directives)
       end
 
-      if (JSONAPI.configuration.top_level_meta_include_page_count && page_options[:record_count])
+      if (JSONAPI.configuration.top_level_meta_include_page_count && paginator && page_options[:record_count])
         page_options[:page_count] = paginator ? paginator.calculate_page_count(page_options[:record_count]) : 1
       end
 

test/controllers/controller_test.rb

@@ -3033,6 +3033,18 @@ def test_books_page_count_in_meta
     assert_equal 'Book 0', json_response['data'][0]['attributes']['title']
   end
 
+  def test_books_no_page_count_in_meta_with_none_paginator
+    Api::V2::BookResource.paginator :none
+    JSONAPI.configuration.top_level_meta_include_page_count = true
+    assert_cacheable_get :index, params: {include: 'book-comments'}
+    JSONAPI.configuration.top_level_meta_include_page_count = false
+
+    assert_response :success
+    assert_nil json_response['meta']['page-count']
+    assert_equal 901, json_response['data'].size
+    assert_equal 'Book 0', json_response['data'][0]['attributes']['title']
+  end
+
   def test_books_record_count_in_meta_custom_name
     Api::V2::BookResource.paginator :offset
     JSONAPI.configuration.top_level_meta_include_record_count = true