ui: improve comment behaviour for datastore Summary

when we could not load the config (e.g. missing permissions)
show the comment from the global datastore-list

also show a messagebox for a load error instead of setting
the text of the comment box

Signed-off-by: Dominik Csapak <d.csapak@proxmox.com>
This commit is contained in:
Dominik Csapak 2020-11-13 15:02:36 +01:00 committed by Thomas Lamprecht
parent 172473e4de
commit 8fc63287df
2 changed files with 8 additions and 1 deletions

View File

@ -56,7 +56,7 @@ Ext.define('PBS.DataStoreNotes', {
url: me.url, url: me.url,
waitMsgTarget: me, waitMsgTarget: me,
failure: function(response, opts) { failure: function(response, opts) {
me.update(gettext('Error') + " " + response.htmlStatus); Ext.Msg.alert(gettext('Error'), response.htmlStatus);
me.setCollapsed(false); me.setCollapsed(false);
}, },
success: function(response, opts) { success: function(response, opts) {

View File

@ -264,6 +264,13 @@ Ext.define('PBS.DataStoreSummary', {
me.down('pbsDataStoreInfo').setTitle(`${me.datastore} (${path})`); me.down('pbsDataStoreInfo').setTitle(`${me.datastore} (${path})`);
me.down('pbsDataStoreNotes').setNotes(response.result.data.comment); me.down('pbsDataStoreNotes').setNotes(response.result.data.comment);
}, },
failure: function(response) {
// fallback if e.g. we have no permissions to the config
let rec = Ext.getStore('pbs-datastore-list').findRecord('store', me.datastore);
if (rec) {
me.down('pbsDataStoreNotes').setNotes(rec.data.comment || "");
}
},
}); });
me.query('proxmoxRRDChart').forEach((chart) => { me.query('proxmoxRRDChart').forEach((chart) => {