Polymorphic list fixes (#402)

* support polymorphic list

Render polymorphic list of items with different fields

* pep8

* fix polymorphic models serializer

fix get_polymorphic_serializer_for_instance only for PolymorphicModelSerializer

* fix typo

* pep8

* fix circular import and pep8

* Fix import in renderers

* Fix remaining flake8 issue

* Fix code style issue in renderers.

* Add Adam Ziolkowski and Roberto Barreda as Authors

Author: adsized

AUTHORS

@@ -1,4 +1,5 @@
 Adam Wróbel <https://adamwrobel.com>
+Adam Ziolkowski <[email protected]>
 Christian Zosel <https://zosel.ch>
 Greg Aker <[email protected]>
 Jamie Bliss <[email protected]>
@@ -8,5 +9,6 @@ Matt Layman <https://www.mattlayman.com>
 Ola Tarkowska <[email protected]>
 Oliver Sauder <[email protected]>
 Raphael Cohen <[email protected]>
+Roberto Barreda <[email protected]>
 santiavenda <[email protected]>
 Yaniv Peer <[email protected]>

rest_framework_json_api/renderers.py

@@ -11,6 +11,7 @@
 from rest_framework.serializers import BaseSerializer, ListSerializer, Serializer
 from rest_framework.settings import api_settings
 
+import rest_framework_json_api
 from rest_framework_json_api import utils
 
 
@@ -543,12 +544,6 @@ def render(self, data, accepted_media_type=None, renderer_context=None):
 
         if serializer is not None:
 
-            # Get the serializer fields
-            fields = utils.get_serializer_fields(serializer)
-
-            # Determine if resource name must be resolved on each instance (polymorphic serializer)
-            force_type_resolution = getattr(serializer, '_poly_force_type_resolution', False)
-
             # Extract root meta for any type of serializer
             json_api_meta.update(self.extract_root_meta(serializer, serializer_data))
 
@@ -559,6 +554,17 @@ def render(self, data, accepted_media_type=None, renderer_context=None):
                     resource = serializer_data[position]  # Get current resource
                     resource_instance = serializer.instance[position]  # Get current instance
 
+                    if isinstance(serializer.child, rest_framework_json_api.
+                                  serializers.PolymorphicModelSerializer):
+                        resource_serializer_class = serializer.child.\
+                            get_polymorphic_serializer_for_instance(resource_instance)()
+                    else:
+                        resource_serializer_class = serializer.child
+
+                    fields = utils.get_serializer_fields(resource_serializer_class)
+                    force_type_resolution = getattr(
+                        resource_serializer_class, '_poly_force_type_resolution', False)
+
                     json_resource_obj = self.build_json_resource_obj(
                         fields, resource, resource_instance, resource_name, force_type_resolution
                     )
@@ -573,6 +579,9 @@ def render(self, data, accepted_media_type=None, renderer_context=None):
                     if included:
                         json_api_included.extend(included)
             else:
+                fields = utils.get_serializer_fields(serializer)
+                force_type_resolution = getattr(serializer, '_poly_force_type_resolution', False)
+
                 resource_instance = serializer.instance
                 json_api_data = self.build_json_resource_obj(
                     fields, serializer_data, resource_instance, resource_name, force_type_resolution