Commit 2c668a94 authored by Dietmar Maurer's avatar Dietmar Maurer

FirewallRules: delete unused properties (else we get strange verification errors).

parent 19d3cd16
...@@ -55,16 +55,13 @@ Ext.define('PVE.FirewallRulePanel', { ...@@ -55,16 +55,13 @@ Ext.define('PVE.FirewallRulePanel', {
var me = this; var me = this;
// hack: editable ComboGrid returns nothing when empty, so we need to set '' // hack: editable ComboGrid returns nothing when empty, so we need to set ''
// Also, disabled text fields return nothing, so we need to set ''
if (values.source === undefined) { Ext.Array.each(['source', 'dest', 'proto', 'sport', 'dport'], function(key) {
values.source = ''; if (values[key] === undefined) {
} values[key] = '';
if (values.dest === undefined) { }
values.dest = ''; });
}
if (values.proto === undefined) {
values.proto = '';
}
delete values.modified_marker; delete values.modified_marker;
...@@ -147,6 +144,7 @@ Ext.define('PVE.FirewallRulePanel', { ...@@ -147,6 +144,7 @@ Ext.define('PVE.FirewallRulePanel', {
} }
]); ]);
me.column2 = [ me.column2 = [
{ {
xtype: 'pvecheckbox', xtype: 'pvecheckbox',
...@@ -170,9 +168,12 @@ Ext.define('PVE.FirewallRulePanel', { ...@@ -170,9 +168,12 @@ Ext.define('PVE.FirewallRulePanel', {
me.down('field[name=dport]').setDisabled(false); me.down('field[name=dport]').setDisabled(false);
} else { } else {
me.down('field[name=proto]').setDisabled(true); me.down('field[name=proto]').setDisabled(true);
me.down('field[name=proto]').setValue('');
me.down('field[name=sport]').setDisabled(true); me.down('field[name=sport]').setDisabled(true);
me.down('field[name=sport]').setValue('');
me.down('field[name=dport]').setDisabled(true); me.down('field[name=dport]').setDisabled(true);
} me.down('field[name=dport]').setValue('');
}
} }
} }
}, },
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment