The image and file fields where nolonger working. (#337)

* The image and file fields where nolonger working.

This PR restores that functionality.

* nergens voor

oscarapi/serializers/fields.py

@@ -199,10 +199,12 @@ def to_representation(self, value):
             return value.value.option
         elif obj_type == value.attribute.MULTI_OPTION:
             return value.value.values_list("option", flat=True)
-        elif obj_type == value.attribute.FILE:
-            return value.value.url
-        elif obj_type == value.attribute.IMAGE:
-            return value.value.url
+        elif obj_type in [value.attribute.FILE, value.attribute.IMAGE]:
+            url = value.value.url
+            request = self.context.get("request", None)
+            if request is not None:
+                url = request.build_absolute_uri(url)
+            return url
         elif obj_type == value.attribute.ENTITY:
             if hasattr(value.value, "json"):
                 return value.value.json()

oscarapi/serializers/product.py

@@ -207,6 +207,8 @@ def shortcut_to_internal_value(self, data, productclass, attributes):
                     ProductAttribute.DATE,
                     ProductAttribute.DATETIME,
                     ProductAttribute.ENTITY,
+                    ProductAttribute.FILE,
+                    ProductAttribute.IMAGE,
                 ]
             )
         }

oscarapi/utils/attributes.py

@@ -4,6 +4,7 @@
 from rest_framework.fields import MISSING_ERROR_MESSAGE
 from rest_framework.exceptions import ErrorDetail
 from oscarapi.utils.loading import get_api_class
+from oscarapi.serializers import fields as oscarapi_fields
 
 attribute_details = operator.itemgetter("code", "value")
 entity_internal_value = get_api_class("serializers.hooks", "entity_internal_value")
@@ -55,6 +56,11 @@ def to_attribute_type_value(self, attribute, code, value):
             internal_value = date_field.to_internal_value(value)
         elif attribute.type == attribute.ENTITY:
             internal_value = entity_internal_value(attribute, value)
+        elif attribute.type in [attribute.IMAGE, attribute.FILE]:
+            image_field = oscarapi_fields.ImageUrlField()
+            # pylint: disable=protected-access
+            image_field._context = self.context
+            internal_value = image_field.to_internal_value(value)
 
         return internal_value