AngularJS factory function undefined

Question

I've been looking over this problem for the past 3 hours now and I'm extremely confused. The apache2 error logs say that the php script is being run, however I'm getting a an error from my factory when it's called.

TypeError: InputUpload.uploadImage(...) is undefined

I don't see how the javascript even gets that far to run the php script when the firefox debugger says it isn't defined...

var app = angular.module("app", ["ui.bootstrap"]);

//http://stackoverflow.com/questions/18571001/file-upload-using-angularjs
app.factory("API", function ($http) {
    return {
    uploadImage: function (image) {
        $http.post('/js/upload_image.php', image);
    }
    }
});

app.controller("MainController", ['$scope', '$http', 'API', function($scope, $http, 'API') {
    $scope.imageUrl = "";

    $scope.template = "";

    $scope.templates = [];

    $scope.templates.push("select an option...");
    $scope.templates.push("MakeGray");
    $scope.templates.push("Canny");

    $scope.template = $scope.templates[0];

    $scope.add = function() {
    alert("HELLO");
    var f = document.getElementById('file').files[0];
    var r = new FileReader();
    r.onloadend = function(e) {
        var data = e.target.result;

        API.uploadImage(data)
        .success(function (imgUrl) {
            $scope.imageUrl = imgUrl;
        })
        .error (function (error) {
        });
    }

    r.readAsBinaryString(f);
    }
});

there shouldn't have ' in function parameter 'API' .. remove ' from API . that's may be typo right ? — Anik Islam Abhi
– Anik Islam Abhi, Commented Dec 17, 2015 at 8:54
are you using any lazy loading functionality in your angular? — Jeeva J
– Jeeva J, Commented Dec 17, 2015 at 8:55
The code I posted is all the angularJS I'm using in my website, I'm also using just the angularJS min file. — Zypps987
– Zypps987, Commented Dec 17, 2015 at 8:58
do changes as app.controller("MainController", ['$scope', '$http', 'API', function($scope, $http, API) — vpsingh016
– vpsingh016, Commented Dec 17, 2015 at 8:58

Durgpal Singh · Accepted Answer · 2015-12-17 08:54:33Z

2

try this

    app.controller("MainController", ['$scope', '$http', 'InputUpload', 'API', function($scope, $http, API) {
//your code here..........

answered Dec 17, 2015 at 8:54

Durgpal Singh

12k4 gold badges40 silver badges50 bronze badges

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

2 Comments

Zypps987 Over a year ago

Adding return in app.factory removed the error message saying it was undefined. However there is a new problem that popped up and I'm not sure why... I'm trying to upload an image to my webserver and I followed stackoverflow.com/questions/18571001/… And it still isn't working...

Durgpal Singh Over a year ago

error in post request or in factory method? actually I am java developer and we use servlet for image saving.

AlainIb · Accepted Answer · 2015-12-17 09:01:31Z

1

you need to return $http promise :

app.factory("API", function ($http) {
    return {
        uploadImage: function (image) {
           //add return her 
           return $http.post('/js/upload_image.php', image);
        }
    }
});

answered Dec 17, 2015 at 9:01

AlainIb

4,6156 gold badges42 silver badges67 bronze badges

2 Comments

Zypps987 Over a year ago

That got rid of the undefined part but now I'm dealt with an even bigger problem xD Error: Argument 1 of FileReader.readAsBinaryString does not implement interface Blob.

Zypps987 Over a year ago

nvm I forgot to undo a stupid change before adding in what you suggested, error message is gone now, thanks!

Andrew · Accepted Answer · 2015-12-17 09:08:38Z

0

Your problem is in controller injection.

In your controller replace it to

.controller("MainController", ['$scope', '$http', 'API', function($scope, $http, API) {

and the call your function API.uploadImage(img)

answered Dec 17, 2015 at 9:08

Andrew

1,5344 gold badges20 silver badges29 bronze badges

Collectives™ on Stack Overflow

AngularJS factory function undefined

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related