Ensure ArrayField value is always an array

MuckRock · Dec 4, 2023 · 5347a6d · 5347a6d
1 parent fee57e5
commit 5347a6d
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/addons/dispatch/fields/ArrayField.svelte b/src/addons/dispatch/fields/ArrayField.svelte
@@ -1,5 +1,7 @@
 <script lang="ts">
+  import { beforeUpdate } from "svelte";
   import Plus16 from "svelte-octicons/lib/Plus16.svelte";
+
   import Checkbox from "./Checkbox.svelte";
   import X16 from "svelte-octicons/lib/X16.svelte";
   import Number from "./Number.svelte";
@@ -15,9 +17,16 @@
   };
 
   export let count: number = 1;
-  export let value = Array(count).fill(null);
+  export let value: [any] = Array(count).fill(null);
+
+  $: numItems = value?.length ?? 0;
 
-  $: numItems = value.length;
+  beforeUpdate(() => {
+    // value should always be an array
+    if (!Array.isArray(value)) {
+      value = [];
+    }
+  });
 
   // only one level of nesting allowed
   const types = {