Merge pull request #160 from developmentseed/patch/do-not-exclude-nul…

…l-properties Patch/do not exclude null properties
developmentseed · Jan 9, 2024 · 3d6a047 · 3d6a047
2 parents 8483d54 + b0c9d14
commit 3d6a047
Showing 1 changed file with 17 additions and 9 deletions.
diff --git a/tipg/factory.py b/tipg/factory.py
@@ -754,19 +754,27 @@ async def items(  # noqa: C901
                 MediaType.json,
                 MediaType.ndjson,
             ):
-                rows = (
-                    {
-                        k: v
-                        for k, v in {
+                if any(
+                    [f.get("geometry", None) is not None for f in item_list["items"]]
+                ):
+                    rows = (
+                        {
                             "collectionId": collection.id,
                             "itemId": f.get("id"),
                             **f.get("properties", {}),
                             "geometry": f.get("geometry", None),
-                        }.items()
-                        if v is not None
-                    }
-                    for f in item_list["items"]
-                )
+                        }
+                        for f in item_list["items"]
+                    )
+                else:
+                    rows = (
+                        {
+                            "collectionId": collection.id,
+                            "itemId": f.get("id"),
+                            **f.get("properties", {}),
+                        }
+                        for f in item_list["items"]
+                    )
 
                 # CSV Response
                 if output_type == MediaType.csv: