AngularJS TypeError is not a function

Question

I have the following code in my service:

testApp.service('detailsService',['databaseService', 'loggedService', '$http', function(databaseService, loggedService, $http){

    var details;

    this.getDetails = function(name){

        return $http({
            method : "GET",
            url : name    
        }).then(function(response) {
           details= response.data;
           console.log(response.data);
           return response.data;
        });

    };

}]);

What i want to do is call this function in my controller when the page(view) is loaded.

testApp.controller('testController', ['$scope', '$location', 'databaseService','detailsService', '$routeParams', function($scope, $location, databaseService, $routeParams, detailsService){

    $scope.details;

    var selectedDetails = function(name){
             detailsService.getDetails(name).then(function(data){
            $scope.details= data;
        });
    };

    selectedDetails(name);

}]);

I keep getting the error detailsService.getDetails is not a function. I'm using the same function from the detailsService in another controller without any problems.

Does anybody know why i keep getting this error?

Possible duplicate of Angular - TypeError: XX is not a function — Nikunj Sardhara
– Nikunj Sardhara, Commented Mar 21, 2017 at 10:07

Satpal · Accepted Answer · 2017-03-21 10:16:41Z

12

The error is expected as you are not injecting dependencies properly, You need to use the correct sequence.

testApp.controller('testController', ['$scope', 
    '$location', 
    'databaseService',
    'detailsService', 
    '$routeParams', 
    function($scope, $location, databaseService, detailsService, $routeParams ){

instead of

testApp.controller('testController', ['$scope', 
    '$location', 
    'databaseService',
    'detailsService', 
    '$routeParams', 
    function($scope, $location, databaseService, $routeParams, detailsService){

Note Both the string part and the function arguments need to match up 1:1.

edited Mar 21, 2017 at 10:16

answered Mar 21, 2017 at 9:57

Satpal

134k13 gold badges168 silver badges171 bronze badges

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

1 Comment

Brian Over a year ago

Just want to add, this question will explain what the [] are all about for those that don't know, and want to. stackoverflow.com/questions/18032068/…

Sachila Ranawaka · Accepted Answer · 2017-03-21 09:57:56Z

remove the promise in the service.Just return the request. since you are catching the promise from the controller no need to use it in the service

testApp.service('detailsService', ['databaseService', 'loggedService', '$http', function(databaseService, loggedService, $http) {
    var details;
    this.getDetails = function(name) {
        return $http({
            method: "GET",
            url: name
        })
    };
}]);

Also make sure the sequence is correct order when you are injecting services.

change this

 testApp.controller('testController', ['$scope', '$location', 'databaseService', 'detailsService', '$routeParams', function($scope, $location, databaseService, $routeParams, detailsService )

to this

 testApp.controller('testController', ['$scope', '$location', 'databaseService', 'detailsService', '$routeParams', function($scope, $location, databaseService, detailsService , $routeParams)

controller

testApp.controller('testController', ['$scope', '$location', 'databaseService', 'detailsService', '$routeParams', function($scope, $location, databaseService, detailsService , $routeParams) {
    $scope.details;
    var selectedDetails = function(name) {
        detailsService.getDetails(name).then(function(data) {
            $scope.details = data;
        });
    };
    selectedDetails(name);
}]);

Zoe - Save the data dump · Accepted Answer · 2018-12-13 15:51:51Z

0

I got this error because I had added a callback method to a component, but the callback was not used by all the pages where that component was used:

<component [myCallbackFn]="fnNameToBindToFn">

component.ts:

public onClick() {
    // some code
    this.myCallbackFn();
}

Each time it was not used I got this error which was effectively a NullPointerException (from Java). So in the other uses of <component> in my website I had to add a check in the component before the callback to say:

if (undefined !== this.myCallbackFn) this.myCallbackFn();

edited Dec 13, 2018 at 15:51

Zoe - Save the data dump

28.4k22 gold badges130 silver badges163 bronze badges

answered Dec 13, 2018 at 15:39

Scala Enthusiast

46711 silver badges21 bronze badges

Collectives™ on Stack Overflow

AngularJS TypeError is not a function

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

1 Comment

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related