From 8f3c3c3ea75b7dc4d6e35f02b8daf840da2ec8e1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Toma=C5=BE=20Jerman?= Date: Wed, 23 Nov 2022 14:02:05 +0100 Subject: [PATCH] Add default report frame multi value separator --- server/system/reporting/frame_builder.go | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/server/system/reporting/frame_builder.go b/server/system/reporting/frame_builder.go index 8fe11eb41..cfc1cd44a 100644 --- a/server/system/reporting/frame_builder.go +++ b/server/system/reporting/frame_builder.go @@ -103,7 +103,12 @@ func (b *reportFrameBuilder) stringifyVal(col string, val any) string { } func (b *reportFrameBuilder) joinMultiVal(col string, vals []string) string { - return strings.Join(vals, b.attrMvDel[col]) + // @todo b.attrMvDel[col] is never actually set so it should be ALONG with this default + del, ok := b.attrMvDel[col] + if !ok { + del = "\n" + } + return strings.Join(vals, del) } func (b *reportFrameBuilder) freshFrame() {