maps: fix GeoJsonPointMixin with dict as input

adhocracy4/maps/mixins.py

@@ -41,7 +41,10 @@ def unpack_geojson(self, data):
         if self.Meta.geo_field and self.Meta.geo_field in data:
             geo_field = data[self.Meta.geo_field]
             if geo_field:
-                point = json.loads(geo_field)
+                if isinstance(geo_field, dict):
+                    point = geo_field
+                else:
+                    point = json.loads(geo_field)
                 data = data.copy()
 
                 if "geometry" in point:

changelog/_0004.md

@@ -0,0 +1,4 @@
+### fixed
+
+- GeoJsonPointMixin can accept also dict payload
+

tests/maps/test_map_mixins.py

@@ -28,6 +28,17 @@ def test_valid_point_correctly_serialized_to_internal_value(geojson_point, geos_
     assert "street_name" not in data
 
 
+@pytest.mark.django_db
+def test_valid_point_as_dict_correctly_serialized_to_internal_value(
+    geojson_point, geos_point
+):
+    del geojson_point["properties"]
+    serializer = TestPointSerializer()
+    data = serializer.to_internal_value(data={"point": geojson_point})
+    assert data["point"].equals(geos_point)
+    assert "street_name" not in data
+
+
 @pytest.mark.django_db
 def test_valid_point_with_properties_to_internal_value(geojson_point, geos_point):
     serializer = TestPointSerializer()