etan39
/
cloudstack
mirror of https://github.com/apache/cloudstack.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

CS-16344: cloudstack UI - multiEdit widget - remove isOptional property, isDisabled property from field level since they are supposed to be on field.range level.

This commit is contained in:
Jessica Wang 2012-09-14 15:26:41 -07:00
parent fde4fee5d7
commit e73cbfe6a9
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
ui/scripts/ui/widgets/multiEdit.js
View File

@ -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)
);