Hi

According to John Papa Guideline, it is better to use $inject to manually 
identify dependencies for Angular components 
<https://github.com/johnpapa/angular-styleguide#style-y091>.

What should be the best practice for the following code

angular
    .module('app')
    .factory('factory1', ['$http', factory1])
    .factory('factory2', ['factory1', '$http', '$q', factory2])    
    .factory('factory3', ['factory2', '$http', '$q', factory3]);
    
var service = {};
   
function factory1($http) {
return $http.get('config.json') 
    .then (function(response) { 
    service.siteID = response.data._id;
                                     return service.siteID
    });
} 
 function factory2(factory1, $http, $q) {
return factory1
.then(function(value) {
return $http.get('/api/sites/' + value)
        .then (function(response) {
        service.template = response.data.template;
        return service.template;
        })
})
} 
 function factory3(factory2, $http, $q) {
return factory2
.then(function(value) {
return $http.get('/api/sites/' + value)
        .then (function(response) {
        service.pages = response.data.pages;
        return service.pages;
        })
})
}...


Should I do FactoryName.$inject = [] for all of them?

Regards
Greg

-- 
You received this message because you are subscribed to the Google Groups 
"AngularJS" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to angular+unsubscr...@googlegroups.com.
To post to this group, send email to angular@googlegroups.com.
Visit this group at http://groups.google.com/group/angular.
For more options, visit https://groups.google.com/d/optout.

Reply via email to