AngularJS: factory $http.get JSON file

Okay, here's a list of things to look into:

1) If you're not running a webserver of any kind and just testing with file://index.html, then you're probably running into same-origin policy issues. See:

https://code.google.com/archive/p/browsersec/wikis/Part2.wiki#Same-origin_policy

Many browsers don't allow locally hosted files to access other locally hosted files. Firefox does allow it, but only if the file you're loading is contained in the same folder as the html file (or a subfolder).

2) The success function returned from $http.get() already splits up the result object for you:

$http({method: 'GET', url: '/someUrl'}).success(function(data, status, headers, config) {

So it's redundant to call success with function(response) and return response.data.

3) The success function does not return the result of the function you pass it, so this does not do what you think it does:

var mainInfo = $http.get('content.json').success(function(response) {        return response.data;    });

This is closer to what you intended:

var mainInfo = null;$http.get('content.json').success(function(data) {    mainInfo = data;});

4) But what you really want to do is return a reference to an object with a property that will be populated when the data loads, so something like this:

theApp.factory('mainInfo', function($http) {     var obj = {content:null};    $http.get('content.json').success(function(data) {        // you can do some processing here        obj.content = data;    });        return obj;    });

mainInfo.content will start off null, and when the data loads, it will point at it.

Alternatively you can return the actual promise the $http.get returns and use that:

theApp.factory('mainInfo', function($http) {     return $http.get('content.json');});

And then you can use the value asynchronously in calculations in a controller:

$scope.foo = "Hello World";mainInfo.success(function(data) {     $scope.foo = "Hello "+data.contentItem[0].username;});

I wanted to note that the fourth part of Accepted Answer is wrong.

theApp.factory('mainInfo', function($http) { var obj = {content:null};$http.get('content.json').success(function(data) {    // you can do some processing here    obj.content = data;});    return obj;    });

The above code as @Karl Zilles wrote will fail because obj will always be returned before it receives data (thus the value will always be null) and this is because we are making an Asynchronous call.

The details of similar questions are discussed in this post

In Angular, use $promise to deal with the fetched data when you want to make an asynchronous call.

The simplest version is

theApp.factory('mainInfo', function($http) {     return {        get:  function(){            $http.get('content.json'); // this will return a promise to controller        }});// and in controllermainInfo.get().then(function(response) {     $scope.foo = response.data.contentItem;});

The reason I don't use success and error is I just found out from the doc, these two methods are deprecated.

The $http legacy promise methods success and error have been deprecated. Use the standard then method instead.

this answer helped me out a lot and pointed me in the right direction but what worked for me, and hopefully others, is:

menuApp.controller("dynamicMenuController", function($scope, $http) {$scope.appetizers= [];$http.get('config/menu.json').success(function(data) {     console.log("success!");    $scope.appetizers = data.appetizers;        console.log(data.appetizers);    });    });