1

I'm trying to get the year of a film given its title using IMDB's API and then appending the year in parenthesis next to the title.

There's some fundamental JS or AJAX thing that I'm screwing up. Any help would be much appreciated!

This is my code: jsFiddle

HTML

<ol>
    <li>Jaws</li>
    <li>The Lord of the Rings</li>
</ol>

jQuery

function getYear(title)
{
    $.ajax({
      url: "http://www.imdbapi.com/?t=" + title,
        dataType: 'jsonp',
        success: function(data){
            var year = data.Year;
        }
    });

}

$("li").each(function() {
      var text = $(this).text();
      getYear(text);
      $(this).append(" ("+year+")");
});
Danny Cooper
  • 355
  • 1
  • 8
  • 25
  • possible duplicate of [How to return the response from an AJAX call?](http://stackoverflow.com/questions/14220321/how-to-return-the-response-from-an-ajax-call) – hjpotter92 Jan 31 '14 at 07:24
  • 2
    Here's an [updated fiddle](http://jsfiddle.net/xVAbm/118/) –  Jan 31 '14 at 07:26

1 Answers1

1

Your code is calling the AJAX function, but immediately going on to update the page before the asynchronous function returns. You need to include the code that updates the page in the callback to the AJAX function so that it executes when the data is ready.

I have reworked your code to this:

function getYear(title)
{
    $.ajax({
      url: "http://www.imdbapi.com/?t=" + $(title).text(),
        dataType: 'json',
        success: function(data){
            var year = data.Year;
            var text = $( this ).text();
            $(title).append(" ("+year+")");

        }
    });

}

$( "li" ).each(function() {

      getYear(this);
});

Which works successfully on this fiddle

  • Thank you, that makes total sense now that you say it like that. – Danny Cooper Jan 31 '14 at 07:41
  • I'm getting having "Cross-Origin Request Blocked" when attempting to use this solution It does seem to work if I use this site [OMDBapi](https://www.omdbapi.com) though – Gibado Dec 03 '16 at 19:39