Commit 09109431 authored by Dietmar Maurer's avatar Dietmar Maurer

correctly check for undefined values

parent c838c7a3
...@@ -286,7 +286,10 @@ Ext.define('PVE.qemu.HardwareView', { ...@@ -286,7 +286,10 @@ Ext.define('PVE.qemu.HardwareView', {
selModel: sm, selModel: sm,
disabled: true, disabled: true,
enableFn: function(rec) { enableFn: function(rec) {
if (!rec || rec.data.key.match(/^unused\d+/) || rec.data['pending'] || rec.data['delete']) { if (!rec || rec.data.key.match(/^unused\d+/) || rec.data['delete']) {
return false;
}
if (Ext.isDefined(rec.data.pending) && (rec.data.pending !== '')) {
return false; return false;
} }
var rowdef = rows[rec.data.key]; var rowdef = rows[rec.data.key];
...@@ -344,7 +347,7 @@ Ext.define('PVE.qemu.HardwareView', { ...@@ -344,7 +347,7 @@ Ext.define('PVE.qemu.HardwareView', {
if (!rec) { if (!rec) {
return false; return false;
} }
if(rec.data['delete']){ if (rec.data['delete']) {
return false; return false;
} }
var rowdef = rows[rec.data.key]; var rowdef = rows[rec.data.key];
...@@ -363,7 +366,7 @@ Ext.define('PVE.qemu.HardwareView', { ...@@ -363,7 +366,7 @@ Ext.define('PVE.qemu.HardwareView', {
reload(); reload();
}, },
failure: function (response, opts) { failure: function (response, opts) {
Ext.Msg.alert('Error',response.htmlStatus); Ext.Msg.alert('Error', response.htmlStatus);
} }
}); });
} }
...@@ -377,12 +380,16 @@ Ext.define('PVE.qemu.HardwareView', { ...@@ -377,12 +380,16 @@ Ext.define('PVE.qemu.HardwareView', {
if (!rec) { if (!rec) {
return false; return false;
} }
if(!rec.data['pending'] && !rec.data['delete']){
return false; if (Ext.isDefined(rec.data.pending) && (rec.data.pending !== '')) {
return true;
}
if (rec.data['delete']) {
return true;
} }
var rowdef = rows[rec.data.key];
return true; return false;
}, },
handler: function(b, e, rec) { handler: function(b, e, rec) {
PVE.Utils.API2Request({ PVE.Utils.API2Request({
......
...@@ -338,9 +338,10 @@ Ext.define('PVE.qemu.Options', { ...@@ -338,9 +338,10 @@ Ext.define('PVE.qemu.Options', {
return; return;
} }
if(rec.data['pending'] || rec.data['delete']){ if ((Ext.isDefined(rec.data.pending) && (rec.data.pending !== '')) ||
revert_btn.setDisabled(false); rec.data['delete']) {
}else { revert_btn.setDisabled(false);
} else {
revert_btn.setDisabled(true); revert_btn.setDisabled(true);
} }
......
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