Make variables for CSV export null-proof

Jean-François Morin · Jean-François Morin · commit bc3fb27c0c36 · 2025-02-14T15:05:27.000-05:00
diff --git a/src/app/admin/admin-reports/filtered-items/filtered-items-export-csv/filtered-items-export-csv.component.ts b/src/app/admin/admin-reports/filtered-items/filtered-items-export-csv/filtered-items-export-csv.component.ts
@@ -84,14 +84,14 @@ export class FilteredItemsExportCsvComponent implements OnInit {
    */
   export() {
     const parameters = [];
-    const colls = this.reportParams.value.collections;
+    const colls = this.reportParams.value.collections || [];
     for (let i = 0; i < colls.length; i++) {
       if (colls[i]) {
         parameters.push({ name: '-c', value: colls[i] });
       }
     }
 
-    const preds = this.reportParams.value.queryPredicates;
+    const preds = this.reportParams.value.queryPredicates || [];
     for (let i = 0; i < preds.length; i++) {
       const field = preds[i].field;
       const op = preds[i].operator;
@@ -105,7 +105,7 @@ export class FilteredItemsExportCsvComponent implements OnInit {
       }
     }
 
-    const filters = FiltersComponent.toQueryString(this.reportParams.value.filters);
+    const filters = FiltersComponent.toQueryString(this.reportParams.value.filters) || [];
     if (filters.length > 0) {
       parameters.push({ name: '-f', value: filters });
     }

Original file line number	Diff line number	Diff line change
`@@ -84,14 +84,14 @@ export class FilteredItemsExportCsvComponent implements OnInit {`
`84`	`84`	`*/`
`85`	`85`	`export() {`
`86`	`86`	`const parameters = [];`
`87`		`- const colls = this.reportParams.value.collections;`
	`87`	`+ const colls = this.reportParams.value.collections \|\| [];`
`88`	`88`	`for (let i = 0; i < colls.length; i++) {`
`89`	`89`	`if (colls[i]) {`
`90`	`90`	`parameters.push({ name: '-c', value: colls[i] });`
`91`	`91`	`}`
`92`	`92`	`}`
`93`	`93`
`94`		`- const preds = this.reportParams.value.queryPredicates;`
	`94`	`+ const preds = this.reportParams.value.queryPredicates \|\| [];`
`95`	`95`	`for (let i = 0; i < preds.length; i++) {`
`96`	`96`	`const field = preds[i].field;`
`97`	`97`	`const op = preds[i].operator;`
`@@ -105,7 +105,7 @@ export class FilteredItemsExportCsvComponent implements OnInit {`
`105`	`105`	`}`
`106`	`106`	`}`
`107`	`107`
`108`		`- const filters = FiltersComponent.toQueryString(this.reportParams.value.filters);`
	`108`	`+ const filters = FiltersComponent.toQueryString(this.reportParams.value.filters) \|\| [];`
`109`	`109`	`if (filters.length > 0) {`
`110`	`110`	`parameters.push({ name: '-f', value: filters });`
`111`	`111`	`}`