fix(download): include Year column for explorers when its slug isn't …

…"year" (#4445)
owid · Jan 14, 2025 · 5537ae3 · 5537ae3
1 parent d36bbfb
commit 5537ae3
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/packages/@ourworldindata/core-table/src/OwidTable.ts b/packages/@ourworldindata/core-table/src/OwidTable.ts
@@ -701,9 +701,19 @@ export class OwidTable extends CoreTable<OwidRow, OwidColumnDef> {
     ): string {
         let table
         if (activeColumnSlugs?.length) {
-            table = this.select([
+            const timeColumnToInclude = [
                 OwidTableSlugs.year,
                 OwidTableSlugs.day,
+                this.timeColumn.slug, // needed for explorers, where the time column may be called anything
+            ].find((colSlug) => this.has(colSlug))
+
+            if (!timeColumnToInclude)
+                throw new Error(
+                    "Expected to find a time column to include in the CSV"
+                )
+
+            table = this.select([
+                timeColumnToInclude,
                 this.entityNameSlug,
                 ...activeColumnSlugs,
             ])