Merge pull request #2008 from malloydata/renderer-knowledge-transfer

fix: render value null check
malloydata · Nov 13, 2024 · 0b3a45e · 0b3a45e
2 parents 2dcb325 + fd32f17
commit 0b3a45e
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/packages/malloy-render/src/component/render-result-metadata.ts b/packages/malloy-render/src/component/render-result-metadata.ts
@@ -165,7 +165,10 @@ const populateFieldMeta = (data: DataArray, metadata: RenderResultMetadata) => {
       const fieldSet = maxUniqueFieldValueSets.get(getFieldKey(f))!;
 
       const value = f.isAtomicField() ? row.cell(f).value : undefined;
-      if (value === null || typeof value === 'undefined') {
+      if (
+        f.isAtomicField() &&
+        (value === null || typeof value === 'undefined')
+      ) {
         fieldMeta.values.add(NULL_SYMBOL);
         fieldSet.add(NULL_SYMBOL);
       } else if (valueIsNumber(f, value)) {