🐛 fix unpublished chart delete

owid · Nov 25, 2024 · 86fd13d · 86fd13d
1 parent 5fe6edc
commit 86fd13d
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/adminSiteServer/apiRouter.ts b/adminSiteServer/apiRouter.ts
@@ -1066,12 +1066,14 @@ deleteRouteWithRWTransaction(
     "/charts/:chartId",
     async (req, res, trx) => {
         const chart = await expectChartById(trx, req.params.chartId)
-        const links = await getPublishedLinksTo(trx, [chart.slug!])
-        if (links.length) {
-            const sources = links.map((link) => link.sourceSlug).join(", ")
-            throw new Error(
-                `Cannot delete chart in-use in the following published documents: ${sources}`
-            )
+        if (chart.slug) {
+            const links = await getPublishedLinksTo(trx, [chart.slug])
+            if (links.length) {
+                const sources = links.map((link) => link.sourceSlug).join(", ")
+                throw new Error(
+                    `Cannot delete chart in-use in the following published documents: ${sources}`
+                )
+            }
         }
 
         await db.knexRaw(trx, `DELETE FROM chart_dimensions WHERE chartId=?`, [