ReferenceError: $http is not defined in angularJs

Question

I have a angular app with following structure

masterApp.js :

var masterApp = angular.module('masterApp', []);
masterApp.factory("organizationFactory", ["$http", organizationFactory(http)]);
masterApp.controller("organizationCtrl", ["$scope", "organizationFactory",organizationCtrl(scope,organizationFactory)]);

organizationFactory.js:

function organizationFactory(http) {
    return {

        getDefaultOrganization:function() {
            http.get(organizationRestUrl);
        },
        registerOrganization:function(organization) {
            http.post(organizationRestUrl, organization);
        },
        updateOrganization:function(organization) {
            http.put(organizationRestUrl, organization);
        },
        deleteOrganization:function() {
            http.delete(organizationRestUrl);
        }
    };
}

OrganizationCtrl.Js

function organizationCtrl($scope,organizationFactory) {

    alert(scope);
    scope.organization = organizationFactory.getDefaultOrganization();
}

When app runs i got an error

ReferenceError: http is not defined

masterApp.factory("organizationFactory", ["$http", organizationFactory(http)])

I changed to injecting http to $http and scope to $scope ( i think it will not cause the error because i am using appropriate minfication safe guidelines) .but still i am stuck with that error NB: I am sure that angular.js loading before the script

Khanh TO · Accepted Answer · 2014-05-11 04:13:47Z

2

You should pass a reference to the function instead of invoking the function and getting it returned value.

With this code:

organizationFactory(http)

You're invoking the function passing in http which does not exist in that context.

To pass a function reference, just organizationFactory is enough.

Replace:

masterApp.factory("organizationFactory", ["$http", organizationFactory(http)]);
masterApp.controller("organizationCtrl", ["$scope", "organizationFactory",organizationCtrl(scope,organizationFactory)])

With:

masterApp.factory("organizationFactory", ["$http", organizationFactory]);
masterApp.controller("organizationCtrl", ["$scope", "organizationFactory",organizationCtrl])

edited May 11, 2014 at 4:13

answered May 11, 2014 at 4:07

Khanh TO

49k13 gold badges101 silver badges117 bronze badges

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

1 Comment

Binson Eldhose Over a year ago

yes u r right @khanh. i forgot the basics of javascript. thanks

Collectives™ on Stack Overflow

ReferenceError: $http is not defined in angularJs

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related