Merge pull request #273 from togglepro/no_record_count_in_show

Check if record_count is supported by request before adding to meta.

Author: lgebhardt

lib/jsonapi/response_document.rb

@@ -50,7 +50,7 @@ def top_level_meta
       @operation_results.results.each do |result|
         meta.merge!(result.meta)
 
-        if JSONAPI.configuration.top_level_meta_include_record_count
+        if JSONAPI.configuration.top_level_meta_include_record_count && result.respond_to?(:record_count)
           meta[JSONAPI.configuration.top_level_meta_record_count_key] = result.record_count
         end
       end
@@ -123,4 +123,4 @@ def results_to_hash
       end
     end
   end
-end
+end

test/controllers/controller_test.rb

@@ -227,6 +227,16 @@ def test_show_single
     assert_nil json_response['included']
   end
 
+  def test_show_does_not_include_records_count_in_meta
+    JSONAPI.configuration.top_level_meta_include_record_count = true
+    get :show, { id: Post.first.id }
+    assert_response :success
+    assert_equal json_response['meta'], nil
+  ensure
+    JSONAPI.configuration.top_level_meta_include_record_count = false
+  end
+
+
   def test_show_single_with_includes
     get :show, {id: '1', include: 'comments'}
     assert_response :success
@@ -2397,4 +2407,4 @@ def test_save_model_callbacks_fail
     assert_response :unprocessable_entity
     assert_match /Save failed or was cancelled/, json_response['errors'][0]['detail']
   end
-end
+end