Angularjs : use a function that returns a promise

Question

I want to do some treatment once the function I called, which performs a call to the server, is done.

I tried :

vm.myFunction().$promise.then(function(){
    //some treatment done once vm.myFunction() is finished....
})


vm.myFunction = function(){
    var deferred = $q.defer();
        myResource.get(function(result){
            vm.results = result;
            for (var i=0; i<vm.results.length;i++){
                if (vm.results[i].state == 'open'){
                    deferred.resolve();
                    return deferred.promise;
                }
            }
        })
};

But I ma getting a

angular.js:13294 TypeError: Cannot read property '$promise' of undefined

How can I do that?

you are not returning anything from the function return myResource.get(), also there is no case where you will reject it - just to make sure you dont end up in infinite wait or add a timeout — harishr
– harishr, Commented Dec 19, 2016 at 8:55
You just return that myResource.get(//code) function. And no need to use $promise as you are already returning the promise. — Sai
– Sai, Commented Dec 19, 2016 at 9:03

Aviad · Accepted Answer · 2016-12-19 09:23:52Z

3

You must return deferred.promise at the end of the function.

vm.myFunction = function(){
    var deferred = $q.defer();
    myResource.get(function(result){
        vm.results = result;
        for (var i=0; i<vm.results.length;i++){
            if (vm.results[i].state == 'open'){
                deferred.resolve();
            }
        }
    });
    return deferred.promise;
};

answered Dec 19, 2016 at 9:23

Aviad

5312 silver badges5 bronze badges

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

Comments

Manoj Lodhi · Accepted Answer · 2016-12-19 09:40:15Z

1

Try this code,

vm.myFunction().then(function(){
  //some treatment done once vm.myFunction() is finished....
})


vm.myFunction = function(){
var deferred = $q.defer();
    myResource.get(function(result){
        vm.results = result;
        for (var i=0; i<vm.results.length;i++){
            if (vm.results[i].state == 'open'){
                deferred.resolve();
            }
        }
    })
   return deferred.promise;
};

answered Dec 19, 2016 at 9:40

Manoj Lodhi

9884 silver badges10 bronze badges

Comments

user1260928 · Accepted Answer · 2016-12-19 09:41:49Z

0

thanks to all your answers, I managed to makeit work with :

vm.myFunction = function(){
        return myResource.get(function(result){
            vm.results = result;
            for (var i=0; i<vm.results.length;i++){
                if (vm.results[i].state == 'open'){
                    //....
                }
            }
        })
};

answered Dec 19, 2016 at 9:41

user1260928

3,45910 gold badges67 silver badges112 bronze badges

Collectives™ on Stack Overflow

Angularjs : use a function that returns a promise

3 Answers 3

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related