Unable to use angular factory cross routes

Question

I'm trying to create a factory and use it cross routes in each controller but apparently I'm doing something wrong...

The app:

var app = angular.module('sam', ['ngRoute', 'ngGrid', 'ui.bootstrap']);

The factory

app.factory("User",function(){
        return {};
    });

The routes

// configure our routes
app.config(function($routeProvider) {
        $routeProvider

            // route for the main page which will direct to the buildings page
            .when('/', {
                templateUrl : 'web/pages/buildings.html',
                controller  : 'mainController',
                controllerAs : 'buildings'
            })

    });

The controller

app.controller('mainController', ['$filter', '$http','$log', function($filter, $http, $log, User){
        $log.log('hello!!!!!', User);
}]);

This prints : hello!!!!! undefined

score 2 · Accepted Answer · edited Jul 14 '14 at 15:59

2

you are missing 'User' in your controller.

app.controller('mainController', ['$filter', '$http','$log', **'User',** function($filter, $http, $log, User){
        $log.log('hello!!!!!', User);
}]);

edited Jul 14 '14 at 15:59

mmohab

2,303
4
27
43

answered Jul 14 '14 at 15:40

alfcho

61
1
4

score 1 · Answer 2 · answered Jul 14 '14 at 15:40

You forgot to include User as part of injection array

controller('mainController', ['$filter', '$http','$log', function($filter, $http, $log, User){

Should be:

controller('mainController', ['$filter', '$http','$log','User', function($filter, $http, $log, User){

score 0 · Answer 3 · answered Jul 14 '14 at 15:58

app.factory("User",function(){
    return {
           show:function(){
                alert('Factory called')
           }
};
});




app.controller('mainController', ['$filter', '$http','$log','User', function($filter, $http, $log, User){

  //that is the way you can call your factory
  // you can call this factory function in any controller by injecting it.

      User.show(); //will popup alert window.

}]);

Unable to use angular factory cross routes

3 Answers3