Yes, from looking at the code, the ui-custom/accountsWizard.js needs to also
have a success function. Not sure why it was omitted.
A fix to add a notification could be:
response: {
success: function() {
$(window).trigger('cloudStack.addNotification', { ... });\
}
}
Then add the correct options to trigger a notification. I don't remember the
exact options for addNotification, but looking through the code you can find
other examples as reference.
-Brian
________________________________
From: Ian Duffy [[email protected]]
Sent: Tuesday, May 13, 2014 2:11 PM
To: CloudStack Dev
Cc: Brian Federle; Jessica Wang; Stephen Turner
Subject: Re: "Accounts" -> "Add Account" UI Question
Hi Thomas,
Looks like a possible bug.
CCing the UI team. They may be able to offer more info.
On 13 May 2014 04:00, Thomas O'Dowd
<[email protected]<mailto:[email protected]>> wrote:
When I delete an account, I get a notification. Should the same not work
for adding an account? I'm using 4.3.
I've never really played with javascript, jquery or ajax before but just
looking the the code in the debugger I'm wondering if the following is
not a problem. Maybe something obvious that I don't quite understand :-)
In accountsWizard.js
} else {
$.ajax({
url: createURL('createAccount' + array1.join("")),
dataType: "json",
async: false,
success: function(json) {
var item = json.createaccountresponse.account;
args.response.success({
data: item
});
},
error: function(XMLHttpResponse) {
args.response.error(parseXMLHttpResponse(XMLHttpResponse));
}
});
}
The code calls the function "args.response.success()" but looking at the
debugger args.response has no "success" function. It has
args.response.error() though.
In ui-custom/accountsWizard.js
} else {
args.action({
context: context,
data: data,
isLdap: isLdap,
username: username,
response: {
error: function(message) {
if (message) {
cloudStack.dialog.notice({
message: message
});
}
}
}
});
}
It doesn't setup a response.success... Only response.error.
However, I would have expected that the call to the missing function
would cause some kind of javascript error on the console but nothing is
logged.
Tom.
--
Cloudian KK - http://www.cloudian.com/get-started.html
Fancy 10TB of full featured S3 Storage?
Checkout the Cloudian® Community Edition!
