From 08e38f032dd1bf8f207465a594d5573e5cc1cb84 Mon Sep 17 00:00:00 2001 From: Thomas Lamprecht Date: Tue, 26 Nov 2019 16:58:18 +0100 Subject: [PATCH] ui: printPropertyString: skip empty string values This cannot work anyway, and it simplifies some editors for format-string backed properties, like the upcoming u2f settings integration. If such a value was passed to the backend one would get: > invalid format - missing key in comma-separated list property Signed-off-by: Thomas Lamprecht --- www/manager6/Parser.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/www/manager6/Parser.js b/www/manager6/Parser.js index 9dfabfea..0790e683 100644 --- a/www/manager6/Parser.js +++ b/www/manager6/Parser.js @@ -86,7 +86,7 @@ Ext.define('PVE.Parser', { statics: { if (defaultKey !== undefined && key === defaultKey) { gotDefaultKeyVal = true; defaultKeyVal = value; - } else { + } else if (value !== '') { stringparts.push(key + '=' + value); } });