AB#101818 Fix exports for temporal relations defined as plain string

CHANGES.md

@@ -1,3 +1,9 @@
+# 2023-12-15 (5.20.1)
+
+* Some old definitions (gebieden.stadsdelen) are using temporal relations
+  defined as a plain string instead of an objects. The exporter
+  need to take this into account.
+
 # 2023-12-15 (5.20.0)
 
 * Change export to only use active records for csv and jsonlines,

setup.cfg

@@ -1,6 +1,6 @@
 [metadata]
 name = amsterdam-schema-tools
-version = 5.20.0
+version = 5.20.1
 url = https://github.com/amsterdam/schema-tools
 license = Mozilla Public 2.0
 author = Team Data Diensten, van het Dataplatform onder de Directie Digitale Voorzieningen (Gemeente Amsterdam)

src/schematools/exports/csv.py

@@ -24,7 +24,14 @@ def geo_modifier(field: DatasetFieldSchema, column):
         return gfunc.ST_AsEWKT(column).label(field.db_name)
 
     def id_modifier(field: DatasetFieldSchema, column):
-        if field.table.is_temporal and field.is_composite_key:
+        # We need an extra check for old relation definitions
+        # that are defined as plain strings in the amsterdam schema
+        # and not as objects.
+        if field.table.is_temporal and (
+            field.is_composite_key
+            or field.related_table is not None
+            and field.related_table.is_temporal
+        ):
             return func.split_part(column, ".", 1).label(field.db_name)
         return column