GeometryField containing an empty Point geometry

[kordian-kowalski]

When trying to serialize a GeometryField that contains an empty Point, the serialization fails with the following trace:

File "/usr/local/lib/python3.6/site-packages/rest_framework/serializers.py" in to_representation
  527.                 ret[field.field_name] = field.to_representation(attribute)

File "/usr/local/lib/python3.6/site-packages/rest_framework_gis/fields.py" in to_representation
  31.         geojson = GeoJsonDict(value.geojson)

File "/usr/local/lib/python3.6/site-packages/rest_framework_gis/fields.py" in __init__
  126.         super(GeoJsonDict, self).__init__(*args, **kwargs)

Here's the geometry Point object in question:

>>> geometry
<Point object at 0x7f2da4334450>
>>> geometry.wkt
'POINT EMPTY'
>>> geometry.json

Using

django==2.0.7
djangorestframework==3.8.2
djangorestframework-gis==0.13.0

How to reproduce

from django.contrib.gis.geos import Point
from rest_framework.serializers import Serializer
from rest_framework_gis.fields import GeometryField

class MySerializer(Serializer):
    geometry = GeometryField()

p = Point()
instance = {'geometry': p}
my_serializer = MySerializer()

my_serializer.to_representation(instance)

[nemesifier]

@kordian-kowalski what would you expect instead of the exception? What's the correct behaviour according to the GeoJSON spec?

[atb00ker]

A GeoJSON Geometry object of any type other than
"GeometryCollection" has a member with the name "coordinates".
The value of the "coordinates" member is an array.  The structure
of the elements in this array is determined by the type of
geometry. GeoJSON processors MAY interpret Geometry objects with
empty "coordinates" arrays as null objects.

Source: RFC #7946 - Section 3.1.

[nemesifier]

Thanks @atb00ker, so that means GeometryField should return [] instead of crashing if my understanding is correct.