Angularjs .factory method return

Question

I have been able to get this .factory to work by adding a "return" in front of both the Restangular.all statement and the response. My question is why is it needed there? Why can't I just return the response?

app.controller('MainController', ['GetIndexesFromES', '$scope', function(GetIndexesFromES, $scope) {
        $scope.indices = GetIndexesFromES.getUniqueIndexIDs();
        console.log($scope.indices);
}]);

app.factory('GetIndexesFromES', ['Restangular', function GetIndexesFromES (Restangular) {
    var GetIndexesFromES = {};
    GetIndexesFromES.getUniqueIndexIDs = function(){
        return Restangular.all('_stats/index,store').getList().then(function(response) {
            return response
        });
    }
    return GetIndexesFromES;
}]);

The main reason why I'm asking this question is that I want to modify the data (within the .factory) before sending it back to the controller/$scope.

Thank you, Gregg

Chandermani · Accepted Answer · 2015-06-07 22:57:56Z

1

If you carefully look at your code for getUniqueIndexIDs you will realize there is a callback.

The second return is not a return from getUniqueIndexIDs but from your then's callback function.

Essentially your getUniqueIndexIDs returns a promise created by then. This promise is resolved by the return value then callback function which in your case is return response.

What you are essentially doing is promise chaining.

answered Jun 7, 2015 at 22:57

Chandermani

42.7k12 gold badges88 silver badges89 bronze badges

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

2 Comments

Gregg Over a year ago

Hello, sorry for the delay. I had changed my code to this: ` getIndexesFromES.getUniqueIndexIDs = function(){ var localResponse = []; Restangular.all('_stats/index,store').getList().then(function(response) { angular.copy(listOfUniqueIndices(response), localResponse); }); return localResponse; }` In regards to your response it took me a while to understand all of what you were saying. The following link and sections helped: docs.angularjs.org/api/ng/service/$q section: The Promise API section: Chaining promises Thank you for your response/help.

Chandermani Over a year ago

Yes i realize that, need a better way to explain it.

Collectives™ on Stack Overflow

Angularjs .factory method return

1 Answer 1

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related