apache/cloudstack
1
0
Fork 0
mirror of https://github.com/apache/cloudstack.git synced 2025-11-04 00:02:37 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fix delete domain button in the UI to update only after job finishes (#2905)

Browse Source 
When the “deleteDomain” command is sent via UI, the HTML is already updated assuming that the “deleteDomain” command will actually delete it. However, there are cases (domain with allocated resources) that will not allow ACS to remove the command; as a consequence, the UI will not be in sync with ACS state, which can lead to misunderstandings.
This commit is contained in:
Rafael Weingärtner 2018-11-11 18:41:37 -02:00 committed by GitHub
parent f36e0c77aa
commit 63094b2fbf
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
ui/scripts/domains.js
View File

@ -69,11 +69,23 @@
async: false, async: false,
success: function(json) { success: function(json) {
var jid = json.deletedomainresponse.jobid; var jid = json.deletedomainresponse.jobid;
var wasDomainDeletedWithSuccess = false;
$.ajax({
url: createURL("queryAsyncJobResult&jobId=" + jid),
dataType: "json",
async: false,
success: function(json) {
wasDomainDeletedWithSuccess = json.queryasyncjobresultresponse.jobresultcode == 0;
}
});
args.response.success({ args.response.success({
_custom: { _custom: {
jobId: jid jobId: jid
} }
}); });
if(!wasDomainDeletedWithSuccess){
return;
}
// Quick fix for proper UI reaction to delete domain // Quick fix for proper UI reaction to delete domain
var $item = $('.name.selected').closest('li'); var $item = $('.name.selected').closest('li');