From 9e12fc0203fefb672cbbc6bb3ee44c0679ec1024 Mon Sep 17 00:00:00 2001 From: Dominik Csapak Date: Fri, 22 Jul 2022 09:35:24 +0200 Subject: [PATCH] privilege role selector: fix renderer for Proxmox VE In PBS we get an array here, so the renderer is fine, but in pve it's just a long string, so add a space after commas to achieve the same effect. Without this, the second column is not visible in pve because of an error in the renderer (no 'join' function on a string) Signed-off-by: Dominik Csapak [ T: squash in code-reduction to make it a one-liner again ] Signed-off-by: Thomas Lamprecht --- src/form/RoleSelector.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/form/RoleSelector.js b/src/form/RoleSelector.js index d82c980..b4b6357 100644 --- a/src/form/RoleSelector.js +++ b/src/form/RoleSelector.js @@ -31,7 +31,7 @@ Ext.define('Proxmox.form.RoleSelector', { dataIndex: 'privs', cellWrap: true, // join manually here, as ExtJS joins without whitespace which breaks cellWrap - renderer: v => v.join(', '), + renderer: v => Ext.isArray(v) ? v.join(', ') : v.replaceAll(',', ', '), flex: 5, }, ], -- 2.39.5