No issue: Fix error when importing legacy report (#4390)

beyondessential · Mar 20, 2023 · 31b4b1b · 31b4b1b
1 parent d56ba32
commit 31b4b1b
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/packages/central-server/src/apiV2/dashboardVisualisations/EditDashboardVisualisation.js b/packages/central-server/src/apiV2/dashboardVisualisations/EditDashboardVisualisation.js
@@ -84,7 +84,10 @@ export class EditDashboardVisualisation extends EditHandler {
 
   async editRecord() {
     const { report } = this.req.body;
-    await assertPermissionGroupAccess(this.accessPolicy, report.permission_group);
+    // Skip permission check as legacy report has no permission group
+    if (report.permission_group) {
+      assertPermissionGroupAccess(this.accessPolicy, report.permission_group);
+    }
     return this.models.wrapInTransaction(async transactingModels => {
       const dashboardItemRecord = this.getDashboardItemRecord();
       const reportRecord = this.getReportRecord();