From 53140c5b503c7351c91ebdbed0f345ef03e4a18b Mon Sep 17 00:00:00 2001 From: "Grot (@grafanabot)" <43478413+grafanabot@users.noreply.github.com> Date: Thu, 25 Nov 2021 04:27:50 -0500 Subject: [PATCH] Table: Add space between values for the DefaultCell (#42246) (#42276) fixes #42024 (cherry picked from commit 86a22a914d1f8d4fd23321f6fd4275d0031b24d1) Co-authored-by: Derik Evangelista --- packages/grafana-data/src/field/displayProcessor.test.ts | 4 ++-- packages/grafana-data/src/field/displayProcessor.ts | 6 +++++- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/packages/grafana-data/src/field/displayProcessor.test.ts b/packages/grafana-data/src/field/displayProcessor.test.ts index c28c2f87be8..0c301915ba8 100644 --- a/packages/grafana-data/src/field/displayProcessor.test.ts +++ b/packages/grafana-data/src/field/displayProcessor.test.ts @@ -70,11 +70,11 @@ describe('Process simple display values', () => { }); it('array of text', () => { - assertSame(['a', 'b', 'c'], processors, { text: 'a,b,c', numeric: NaN }); + assertSame(['a', 'b', 'c'], processors, { text: 'a, b, c', numeric: NaN }); }); it('array of numbers', () => { - assertSame([1, 2, 3], processors, { text: '1,2,3', numeric: NaN }); + assertSame([1, 2, 3], processors, { text: '1, 2, 3', numeric: NaN }); }); it('empty object', () => { diff --git a/packages/grafana-data/src/field/displayProcessor.ts b/packages/grafana-data/src/field/displayProcessor.ts index 28db9b6611f..6abc7202714 100644 --- a/packages/grafana-data/src/field/displayProcessor.ts +++ b/packages/grafana-data/src/field/displayProcessor.ts @@ -1,5 +1,5 @@ // Libraries -import { toString, toNumber as _toNumber, isEmpty, isBoolean } from 'lodash'; +import { toString, toNumber as _toNumber, isEmpty, isBoolean, isArray, join } from 'lodash'; // Types import { Field, FieldType } from '../types/dataFrame'; @@ -116,6 +116,10 @@ export function getDisplayProcessor(options?: DisplayProcessorOptions): DisplayP } } + if (text == null && isArray(value)) { + text = join(value, ', '); + } + if (text == null) { text = toString(value); if (!text) {