Changeset 5535

Timestamp:

05/09/08 14:24:44 (3 days ago)

Author:

brandon.aaron

Message:

batch plugin: insure array is returned and added a couple more tests

Location:

trunk/plugins/batch

Files:

• jquery.batch.js (modified) (1 diff)
• test/unit/batch.js (modified) (2 diffs)

trunk/plugins/batch/jquery.batch.js

@@ -6 +6 @@
  * $Rev$
  *
- * Version: 1.0
+ * Version: 1.0.1
  */
 (function($){
 
 $.fn.batch = function(method) {
-    var args = $.makeArray(arguments).slice(1);
-    return this.map(function() {
-        var $this = $(this);
-        return $this[method].apply($this, args);
+    var args = $.makeArray(arguments).slice(1), results = [];
+    this.each(function() {
+        results.push( $(this)[method].apply($(this), args) );
     });
+    return results;
 };
 
 $.batch = {
-    version: "1.0",
+    version: "1.0.1",
     registerPlugin: function() {
         $.each( arguments, function( index, plugin ) {

trunk/plugins/batch/test/unit/batch.js

@@ -2 +2 @@
 
 test("batch", function() {
-    $.each( ['attr', 'styles', 'offset', 'width', 'height', 'html', 'text', 'val'], function(index, name) {
+    $.each( ['attrs', 'styles', 'offsets', 'widths', 'heights', 'htmls', 'texts', 'vals'], function(index, name) {
         ok( $.fn[name], "Make sure " + name + " exists" );
     });
@@ -8 +8 @@
 test("attrs", function() {
     isSet( $('input[value=Test]').attrs('value'), ["Test", "Test"], "$('input[value=Test]').attrs('value')" );
-    isSet( $('input[value=Test]').attrs('value', function(){ return 'Updated'; }).attrs('value'), ["Updated", "Updated"], "$('input[value=Test]').attrs('value', function(){ return 'Updated'; })" );
+    isSet( $('input[value=Test]').attr('value', function(){ return 'Updated'; }).attrs('value'), ["Updated", "Updated"], "$('input[value=Test]').attrs('value', function(){ return 'Updated'; })" );
 });
+test("vals", function() {
+    isSet( $('input[value=Test]').vals(), ["Test", "Test"], "$('input[value=Test]').vals()" );
+    isSet( $('input[value=Test]').attr('value', function(){ return 'Updated'; }).vals(), ["Updated", "Updated"], "$('input[value=Test]').attr('value', function(){ return 'Updated'; }).vals()" );
+});