From 9e47d933af70f4e3f89cc132e02a0d89af1ff5df Mon Sep 17 00:00:00 2001 From: Nick O'Leary Date: Thu, 10 Jan 2019 15:21:27 +0000 Subject: [PATCH] Handle multiple-select box when nothing selected Fixes #2021 --- .../@node-red/editor-client/src/js/ui/editor.js | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/packages/node_modules/@node-red/editor-client/src/js/ui/editor.js b/packages/node_modules/@node-red/editor-client/src/js/ui/editor.js index 519753928..443f3844e 100644 --- a/packages/node_modules/@node-red/editor-client/src/js/ui/editor.js +++ b/packages/node_modules/@node-red/editor-client/src/js/ui/editor.js @@ -1085,6 +1085,13 @@ RED.editor = (function() { var input = $("#node-input-"+d); if (input.attr('type') === "checkbox") { newValue = input.prop('checked'); + } else if (input.prop("nodeName") === "select" && input.attr("multiple") === "multiple") { + // An empty select-multiple box returns null. + // Need to treat that as an empty array. + newValue = input.val(); + if (newValue == null) { + newValue = []; + } } else if ("format" in editing_node._def.defaults[d] && editing_node._def.defaults[d].format !== "" && input[0].nodeName === "DIV") { newValue = input.text(); } else {