From e73cbfe6a90d66f725f813a3d0bdbf668a4938b1 Mon Sep 17 00:00:00 2001 From: Jessica Wang Date: Fri, 14 Sep 2012 15:26:41 -0700 Subject: [PATCH] CS-16344: cloudstack UI - multiEdit widget - remove isOptional property, isDisabled property from field level since they are supposed to be on field.range level. --- ui/scripts/ui/widgets/multiEdit.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/ui/scripts/ui/widgets/multiEdit.js b/ui/scripts/ui/widgets/multiEdit.js index e4fc026765b..de0264ad03c 100644 --- a/ui/scripts/ui/widgets/multiEdit.js +++ b/ui/scripts/ui/widgets/multiEdit.js @@ -769,14 +769,14 @@ if (field.range) { var $range = $('
').addClass('range').appendTo($td); - $(field.range).each(function() { + $(field.range).each(function() { //e.g. field.range = ['privateport', 'privateendport']; var $input = $('') .attr({ name: this, type: 'text' }) - .addClass(!field.isOptional ? 'required' : null) - .attr('disabled', field.isDisabled ? 'disabled' : false) + //.addClass(!field.isOptional ? 'required' : null) //field.range[0] might be required while field.range[1] is optional (e.g. private start port is required while private end port is optional), so "isOptional" property should be on field.range level instead of field level. + //.attr('disabled', field.isDisabled ? 'disabled' : false) //field.range[0] might be enabled while field.range[1] is disabled (e.g. private start port is enabled while private end port is disabled), so "isDisabled" property should be on field.range level instead of field level. .appendTo( $('
').addClass('range-item').appendTo($range) );