CS-16344: cloudstack UI - multiEdit widget - remove isOptional property, isDisabled property from field level since they are supposed to be on field.range level.
Project: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/commit/e905f61b Tree: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/tree/e905f61b Diff: http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/diff/e905f61b Branch: refs/heads/junit-tests Commit: e905f61b888dbc4a53e13a7817a6d3d931ea3622 Parents: af6dd34 Author: Jessica Wang <[email protected]> Authored: Fri Sep 14 15:26:41 2012 -0700 Committer: Chip Childers <[email protected]> Committed: Mon Oct 15 16:19:18 2012 -0400 ---------------------------------------------------------------------- ui/scripts/ui/widgets/multiEdit.js | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-cloudstack/blob/e905f61b/ui/scripts/ui/widgets/multiEdit.js ---------------------------------------------------------------------- diff --git a/ui/scripts/ui/widgets/multiEdit.js b/ui/scripts/ui/widgets/multiEdit.js index e4fc026..de0264a 100644 --- a/ui/scripts/ui/widgets/multiEdit.js +++ b/ui/scripts/ui/widgets/multiEdit.js @@ -769,14 +769,14 @@ if (field.range) { var $range = $('<div>').addClass('range').appendTo($td); - $(field.range).each(function() { + $(field.range).each(function() { //e.g. field.range = ['privateport', 'privateendport']; var $input = $('<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( $('<div>').addClass('range-item').appendTo($range) );
