Changeset 5478

Timestamp:

05/06/08 22:21:43 (7 months ago)

Author:

joern.zaefferer

Message:

jquery ajax: support for dynamic data arguments, see #2806; added test/data/echoQuery.php for easier testing of sent query strings

Location:

trunk/jquery

Files:

• src/ajax.js (modified) (1 diff)
• test/data/echoQuery.php (added)
• test/unit/ajax.js (modified) (1 diff)

trunk/jquery/src/ajax.js

@@ -490 +490 @@
                     });
                 else
-                    s.push( encodeURIComponent(j) + "=" + encodeURIComponent( a[j] ) );
+                    s.push( encodeURIComponent(j) + "=" + encodeURIComponent( jQuery.isFunction(a[j]) ? a[j]() : a[j] ) );
 
         // Return the resulting serialization

trunk/jquery/test/unit/ajax.js

@@ -812 +812 @@
 });
 
+test("data option: evaluate function values (#2806)", function() {
+    stop();
+    $.ajax({
+        url: "data/echoQuery.php",
+        data: {
+            key: function() {
+                return "value";
+            }
+        },
+        success: function(result) {
+            equals( result, "key=value" );
+            start();
+        }
+    })
+});
+
 }