Skip to content
Permalink
Browse files

Feeds: remove finalize function

Finalize function (and deferred objects) are no longer needed
because li items are not hidden.
  • Loading branch information...
patheard committed Jan 30, 2014
1 parent 63edc59 commit fe898e28d567abb1b13d7a3e6d35342b103f7b1b
Showing with 1 addition and 9 deletions.
  1. +1 −9 src/plugins/feeds/feeds.js
@@ -29,7 +29,6 @@ var pluginName = "wb-feeds",
var elm = event.target,
entries = [],
results = [],
deferred = [],
processEntries = function( data ) {
var k, len;

@@ -45,11 +44,6 @@ var pluginName = "wb-feeds",
last -= 1;
return last;
},
finalize = function() {

// TODO: Use CSS instead
$content.find( "li" ).show();
},
$content, limit, feeds, last, i;

// Filter out any events triggered by descendants
@@ -67,15 +61,13 @@ var pluginName = "wb-feeds",
i = last;

while ( i >= 0 ) {
deferred[ i ] = $.ajax({
$.ajax({
url: jsonRequest( feeds[ i ].href, limit ),
dataType: "json",
timeout: 1000
}).done( processEntries );
results.push( i -= 1 );
}
$.when.apply( null, deferred ).always( finalize );

$.extend( {}, results );
}
},

0 comments on commit fe898e2

Please sign in to comment.
You can’t perform that action at this time.