Skip to content

Commit

Permalink
Added support for breaking in an object loop (Bug #2111).
Browse files Browse the repository at this point in the history
  • Loading branch information
jeresig committed Jan 14, 2008
1 parent ff08982 commit c39bd07
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 7 deletions.
14 changes: 8 additions & 6 deletions src/core.js
Expand Up @@ -707,20 +707,22 @@ jQuery.extend({
// args is for internal usage only
each: function( object, callback, args ) {
if ( args ) {
if ( object.length == undefined )
if ( object.length == undefined ) {
for ( var name in object )
callback.apply( object[ name ], args );
else
if ( callback.apply( object[ name ], args ) === false )
break;
} else
for ( var i = 0, length = object.length; i < length; i++ )
if ( callback.apply( object[ i ], args ) === false )
break;

// A special, fast, case for the most common use of each
} else {
if ( object.length == undefined )
if ( object.length == undefined ) {
for ( var name in object )
callback.call( object[ name ], name, object[ name ] );
else
if ( callback.call( object[ name ], name, object[ name ] ) === false )
break;
} else
for ( var i = 0, length = object.length, value = object[0];
i < length && callback.call( value, i, value ) !== false; value = object[++i] ){}
}
Expand Down
15 changes: 14 additions & 1 deletion test/unit/core.js
Expand Up @@ -1314,7 +1314,7 @@ test("text(String)", function() {
});

test("$.each(Object,Function)", function() {
expect(8);
expect(12);
$.each( [0,1,2], function(i, n){
ok( i == n, "Check array iteration" );
});
Expand All @@ -1326,6 +1326,19 @@ test("$.each(Object,Function)", function() {
$.each( { name: "name", lang: "lang" }, function(i, n){
ok( i == n, "Check object iteration" );
});

var total = 0;
jQuery.each([1,2,3], function(i,v){ total += v; });
ok( total == 6, "Looping over an array" );
total = 0;
jQuery.each([1,2,3], function(i,v){ total += v; if ( i == 1 ) return false; });
ok( total == 3, "Looping over an array, with break" );
total = 0;
jQuery.each({"a":1,"b":2,"c":3}, function(i,v){ total += v; });
ok( total == 6, "Looping over an object" );
total = 0;
jQuery.each({"a":3,"b":3,"c":3}, function(i,v){ total += v; return false; });
ok( total == 3, "Looping over an object, with break" );
});

test("$.prop", function() {
Expand Down

0 comments on commit c39bd07

Please sign in to comment.