Jonathan
Jonathan

Reputation: 3024

Why is JavaScript passing into a parameter the source code text?

http://jsfiddle.net/JonnyD/H2Dpe/3/

Console log:

   [Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object]
     (index):104

   function (newFeedItems) {
       console.log(feedItems);
   } (index):113

   Uncaught TypeError: Object function (newFeedItems) {
      console.log(feedItems);
   } has no method 'sort' 

Where is it getting this from?

function (newFeedItems) {
    console.log(feedItems);
} (index):113

when it's supposed to be:

[Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object, Object]

Take a look at the functions fetchFeeds() and processLatestFeed(). The problem starts at processLatestFeed(function(newFeedItems) {

function fetchFeeds() {
    fetchVideos(function(newVideoItems) {
        fetchPodcasts(function(newPodcastItems) {
            var newFeedItems = newVideoItems.concat(newPodcastItems);
            console.log(newFeedItems);
            processLatestFeed(function(newFeedItems) {
                console.log(feedItems);
            });
        });
    });
}

function processLatestFeed(newFeedItems, callback) {
    console.log(newFeedItems);
    newFeedItems.sort(function(a,b) {return (a.date > b.date) ? -1 : ((b.date > a.date) ? 1 : 0);});

    for (i in newFeedItems) {
        var newItem = newFeedItems[i];

        if (!isItemInFeed(newItem)) {
            feedItems.push(newItem);
        }

        if (feedItems.length == 50) {
            break;   
        }
    }

    callback();
}

Upvotes: 0

Views: 35

Answers (1)

Felix Kling
Felix Kling

Reputation: 816364

You are passing a function as first argument to processLatestFeed:

processLatestFeed(function(newFeedItems) {
  console.log(feedItems);
});

but you defined that function to accept an array as first argument and the callback as second argument:

function processLatestFeed(newFeedItems, callback) {
    console.log(newFeedItems); // <- this logs the function source
    // ...
}

So you have to call the function as

processLatestFeed(newFeedItems, function() {
  // you are not passing an arguments to the callback
});

Upvotes: 1

Related Questions