JQuery $.getJSON() Doesn't Work Right With Callback

Question

IT IS SOLVED - WAITING TO MARK (stackoverflow delay)

There's no error in firebug. I've done this before I just can't figure out why it is not working. asyncFetchContent() is fired from elsewhere. contentController is created elsewhere.

function ContentController() {

}

ContentController.prototype.asyncFetchContent = function(){ 

         $.getJSON( '/content/TestContent.json', 
            function(data){
                contentController.displayContentCallback( data ); 
                } 
         );
}

ContentController.prototype.displayContentCallback = function( javascriptObject ){ 
    alert( "in callback");
    $( "#testID" ).html( javascriptObject.title );
}

Firebug will accept a breakpoint at the definition of displayContentCallback() and stop but then silently fails once you stepover. javascriptObject is undefined.

Thanks-in-advance,

Guido

Me thinks it will work if you just use longer names for those functions ? — adeneo
– adeneo, Commented Aug 16, 2012 at 23:15
Are you sure you should be prototyping. From the way you call the other functions, and the fact that your starting object is empty, maybe an object literal would serve your needs better. — adeneo
– adeneo, Commented Aug 16, 2012 at 23:53

Sean Johnson · Accepted Answer · 2012-08-16 23:16:19Z

2

From the manual:

Important: As of jQuery 1.4, if the JSON file contains a syntax error, the request will usually fail silently. Avoid frequent hand-editing of JSON data for this reason. JSON is a data-interchange format with syntax rules that are stricter than those of JavaScript's object literal notation. For example, all strings represented in JSON, whether they are properties or values, must be enclosed in double-quotes. For details on the JSON format, see http://json.org/.

Does your JSON contain an error?

answered Aug 16, 2012 at 23:16

Sean Johnson

5,6172 gold badges19 silver badges23 bronze badges

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

1 Comment

Guido Anselmi Over a year ago

Holy S**t. Thanks. I had a comment in the file which was the problem. Can I have the 3 hours spent working on this back please?

Kpower · Accepted Answer · 2012-08-16 23:18:08Z

0

contentController.displayContentCallback( data ); does not exist. a) It's ContentController, not contentController and b) since you're using prototype, it has to be something like

function ContentController() {

}

ContentController.prototype.asyncFetchContent = function(){ 

     $.getJSON( '/content/TestContent.json', 
        function(data){
            var x = new ContentController();
            x.displayContentCallback( data ); 
        } 
     );
}

ContentController.prototype.displayContentCallback = function( javascriptObject ){ 
    alert( "in callback");
    $( "#testID" ).html( javascriptObject.title );
}

I'm pretty sure that should work.

answered Aug 16, 2012 at 23:18

Kpower

1,2573 gold badges11 silver badges19 bronze badges

2 Comments

jfriend00 Over a year ago

Presumably, contentController is a variable in their code that contains an instance of ContentController. This is why it's bad to use names that only differ in case as it invites this confusion.

Kpower Over a year ago

perhaps, but just in case... it would be nice if they could clear that up though

Collectives™ on Stack Overflow

JQuery $.getJSON() Doesn't Work Right With Callback

2 Answers 2

1 Comment

2 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

1 Comment

2 Comments

Your Answer

Sign up or log in

Post as a guest

Related