Include callback functions directly in executeQueryAsync without delegates

Question

I am tryting to create module pattern with ExecuteQueryAsync like this:

    UsersMngt = (function () {

        getUserIdByEmail = function(email){

        var clientContext = new SP.ClientContext.get_current();
        var web = clientContext.get_web();

        var user = web.get_siteUsers().getByEmail(email);


        clientContext.load(user);

        clientContext.executeQueryAsync(Function.createDelegate(this, this.onQuerySucceeded), Function.createDelegate(this, this.onQueryFailed));


        clientContext.executeQueryAsync(alert('yes'),alert('no'));               
}; //getUserIdByEmail




return{
    GetUserIdByEmail:getUserIdByEmail,
};

})();

and then I want to call it like:

UsersMngt.GetUserIdByEmail('[email protected]')

but, everytime I call it, with or without success, it always gives me both "yes" and "no" (success and failure). It also retrieves data I need in case of succces.

How can I include functions directly in "executeQueryAsync" without creating actually delegates somewhere else? I would like everything to be kept within the same code structure.

So far, it only works with delegates.

Thank you for ideas and help

Answer 1

Once this clientContext.executeQueryAsync(Function.createDelegate(this, this.onQuerySucceeded), Function.createDelegate(this, this.onQueryFailed)); query is executed, it will call the onQuerySucceeded or onQueryFailed functions depending on the result of the query.

I advice you to call the executeQueryAsync just one. and write alert messages inside onQuerySucceeded or onQueryFailed functions.

Also createDelegate is too old now. Using just this would be fine:

clientContext.executeQueryAsync(onQuerySucceeded,onQueryFailed);

Answer 2

This code executes asynchronous

code does not stop when executed, the Success or Error Callback Functions execute when the Async operation is done.

It is like telling someone else to get you a coffee

What you now program

clientContext.executeQueryAsync(alert('yes'),alert('no'));     

Is: you send someone to get a coffee, and before he leaves he says "Yes"

The parameters you have to supply are Function Definitions so without the ( ),
not Function Calls you do now

That whole Function.createDelegate is old Microsoft IE8 stuff,
ditch and learn modern IE9 notation: Differing ways to make executeQueryAsync calls - Function.createDelegate (is old IE8 code)

More details: Looping jsom getItems

Answer 3

thank you, I solved it like this, because I wanted to use annonymous functions:

UsersMngt = (function () {

getUserIdByEmail = function(email){

        var clientContext = new SP.ClientContext.get_current();
        var web = clientContext.get_web();

        var user = web.get_siteUsers().getByEmail(email);


        clientContext.load(user);


        clientContext.executeQueryAsync(
            function (){
                    alert("yes")
            },
            function(){
                    alert("no")
            }            
        )}; //getUserIdByEmail


return{
    GetUserIdByEmail:getUserIdByEmail,
};

})();