From fa1696e3e6c443857eba25b7d6e857e29b0f1d30 Mon Sep 17 00:00:00 2001 From: Brian Federle Date: Fri, 16 Nov 2012 08:37:59 -0800 Subject: [PATCH] CS-16822: multi-edit UI: Fix blank fields on add item When adding an item, show all returned fields if noSelect is true. This prevents hidden conditional form fields from having their row data hidden, such as if an ICMP rule is added and the TCP start/end port form fields are hidden due to space constraints. --- ui/scripts/ui/widgets/multiEdit.js | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/ui/scripts/ui/widgets/multiEdit.js b/ui/scripts/ui/widgets/multiEdit.js index bccbab00d36..969470472de 100755 --- a/ui/scripts/ui/widgets/multiEdit.js +++ b/ui/scripts/ui/widgets/multiEdit.js @@ -104,9 +104,12 @@ if (!itemData) itemData = [{}]; - if ($multi.find('th,td').filter(function() { - return $(this).attr('rel') == fieldName; - }).is(':hidden')) return true; + if (!options.noSelect && + $multi.find('th,td').filter(function() { + return $(this).attr('rel') == fieldName; + }).is(':hidden')) { + return true; + } if (!field.isPassword) { if (field.edit) {