From 04ac28202fdec381ec8284c7d49c55edea39ad5f Mon Sep 17 00:00:00 2001 From: Brian Federle Date: Tue, 10 Jul 2012 14:01:38 -0700 Subject: [PATCH] Multi-edit: refactor form serialization To support header-level form items, serialize every form within the multi-edit div (not just the main table form). For reusability, make data serialization handled in separate 'getMultiData' function. --- ui/scripts/ui/widgets/multiEdit.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/ui/scripts/ui/widgets/multiEdit.js b/ui/scripts/ui/widgets/multiEdit.js index 8eac50646df..e5278db9a3a 100644 --- a/ui/scripts/ui/widgets/multiEdit.js +++ b/ui/scripts/ui/widgets/multiEdit.js @@ -15,6 +15,10 @@ // specific language governing permissions and limitations // under the License. (function($, cloudStack) { + var getMultiData = function($multi) { + return cloudStack.serializeForm($multi.find('form')); + }; + var _medit = cloudStack.ui.widgets.multiEdit = { /** * Append item to list @@ -790,7 +794,7 @@ var addItem = function(itemData) { var data = {}; - $.each(cloudStack.serializeForm($multiForm), function(key, value) { + $.each(getMultiData($multi), function(key, value) { if (value != '') { data[key] = value; }