Angular ng-click multiple functions

Question

<div ng-click="method1(); method2()> </div>

Here if the method1() calls the database / external service and gets the data. And method2() is some other behavior execution like UI bindings.
Suppose in case if there is a delay in getting the data from method1(), how to avoid or stop execution of method2() until method1() execution completes.

Note: I can't add method2() call from method1() due to some restrictions. Also I want to execute both these methods from the ng-click only.

ajax goes asyc so you must consider using its callback function .you cannot have 2 function calls in ng-click . — super cool
– super cool, Commented Feb 15, 2018 at 11:00
Use $q.defer() to assign an asynchronous callback. Perhaps you need to do it in method3() where you merge them and resolve your Promises — Aleksey Solovey
– Aleksey Solovey, Commented Feb 15, 2018 at 11:01
I've read your note but looks like your method2() depends on method1() so it's recommended to call it from the method1() itself using conditional structure. — Niral Munjariya
– Niral Munjariya, Commented Feb 15, 2018 at 11:09
Is anything stopping you from writing a single new function which will call method1() and method2() and call that single function from ng-click? — Ashutosh A
– Ashutosh A, Commented Feb 15, 2018 at 11:10

Rakesh Chand · Accepted Answer · 2018-02-15 11:14:44Z

1

Call the method 2 in response of method1, like this

function method1() {
    service.getData().then(function(response) {
        //got the response here, do the operation and call method2 here
        method2()
    }, function() {});
}

If you call it giving any timeout, then it is not a sure solution. So call it in response.

answered Feb 15, 2018 at 11:14

Rakesh Chand

3,1131 gold badge25 silver badges43 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Alexander Schmidt · Accepted Answer · 2018-02-15 11:59:09Z

1

Try with $timeout:

<div ng-click="method1()"></div>   
$scope.method1=function() {
    // calls the database / external service
    $timeout(function () {
        method2();
    }, 5000);
}

edited Feb 15, 2018 at 11:59

Alexander Schmidt

5,7794 gold badges43 silver badges81 bronze badges

answered Feb 15, 2018 at 11:10

samplename

94 bronze badges

1 Comment

Aleksey Solovey Over a year ago

method1 is asynchronous, meaning it can take from few milliseconds to few minutes of processing. You can't just wait for it a specific / hardcoded amount of time. It should be resolved as a Promise (with a $q service) instead

Hussain · Accepted Answer · 2018-02-15 12:11:33Z

1

you can do something like this

function method1() {
    API.getData().then(function(response) {
        // call method2 here after you receive response
        method2()
    }, function(error) {});
}

html

<div ng-click="method1()> </div>

answered Feb 15, 2018 at 12:11

Hussain

875 bronze badges

Collectives™ on Stack Overflow

Angular ng-click multiple functions

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related