Records can have null fields. (#2875)

brimdata · Nov 2, 2023 · b90a400 · b90a400
1 parent 06ce522
commit b90a400
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/apps/zui/src/app/features/inspector/views/record-view.ts b/apps/zui/src/app/features/inspector/views/record-view.ts
@@ -8,7 +8,7 @@ export class RecordView extends ContainerView<zed.Record> {
   }
 
   count() {
-    return this.value.fields.length
+    return this.value.fields?.length || 0
   }
 
   openToken() {
@@ -20,7 +20,7 @@ export class RecordView extends ContainerView<zed.Record> {
   }
 
   *iterate(n?: number) {
-    const fields = this.value.fields
+    const fields = this.value.fields || []
     const length = n ? Math.min(n, fields.length) : fields.length
 
     for (let i = 0; i < fields.length; ++i) {

diff --git a/packages/zed-js/src/values/record.ts b/packages/zed-js/src/values/record.ts
@@ -134,7 +134,7 @@ export class Record implements Value {
   }
 
   isUnset() {
-    return this.fields === null;
+    return false;
   }
 
   toJS(opts: JSOptions = {}) {