jquery / jquery

Source
Commits
Network (177)
Graphs
Tree: 67089ee

jQuery JavaScript Library — Read more

This URL has Read+Write access

Getting $.param working well; Patch by ben_alman

wycats (author)

Thu Sep 17 15:37:19 -0700 2009

commit 67089eedf6f84acd9c16ea2a6dadadf7b13a7c84
tree f8d8961b690ba43fa2af129c725bcd02e638e303
parent 45dfa3b0fc49718c4f00600d1e25a129601d08ee

M	src/ajax.js	61 ••••
M	test/unit/ajax.js	37 ••••

src/ajax.js

0
@@ -585,39 +585,52 @@ jQuery.extend({
0
   // Serialize an array of form elements or a set of
0
   // key/values into a query string
0
   param: function( a ) {
0
-    var s = [];
0
-
0
+    var s = [],
0
+      param_traditional = jQuery.param.traditional;
0
+    
0
     function add( key, value ){
0
+      // If value is a function, invoke it and return its value
0
+      value = jQuery.isFunction(value) ? value() : value;
0
       s[ s.length ] = encodeURIComponent(key) + '=' + encodeURIComponent(value);
0
     }
0
-
0
+    
0
     // If an array was passed in, assume that it is an array
0
     // of form elements
0
-    if ( jQuery.isArray(a) || a.jquery ) {
0
+    if ( jQuery.isArray(a) || a.jquery )
0
       // Serialize the form elements
0
       jQuery.each( a, function() {
0
         add( this.name, this.value );
0
       });
0
-    } else {
0
-      // Recursively encode parameters from object, 
0
-      // building a prefix path as we go down
0
-      function buildParams(obj, prefix)
0
-      {
0
-        if ( jQuery.isArray(obj) ) {
0
-          for ( var i = 0, length = obj.length; i < length; i++ ) {
0
-            buildParams( obj[i], prefix );
0
-          };
0
-        } else if( typeof(obj) == "object" ) {
0-          for ( var j in obj )&nbs
5bfb
p;{
0
-            var postfix = ((j.indexOf("[]") > 0) ? "[]" : ""); // move any brackets to the end
0
-            buildParams(obj[j], (prefix ? (prefix+"["+j.replace("[]", "")+"]"+postfix) : j) );
0
-          }
0
-        } else {
0
-          add( prefix, jQuery.isFunction(obj) ? obj() : obj );
0
-        }
0
-      }
0
-      buildParams(a);
0
-    }
0
+      
0
+    else
0
+      // Encode parameters from object, recursively. If
0
+      // jQuery.param.traditional is set, encode the "old" way
0
+      // (the way 1.3.2 or older did it)
0
+      jQuery.each( a, function buildParams( prefix, obj ) {
0
+        
0
+        if ( jQuery.isArray(obj) )
0
+          jQuery.each( obj, function(i,v){
0
+            // Due to rails' limited request param syntax, numeric array
0
+            // indices are not supported. To avoid serialization ambiguity
0
+            // issues, serialized arrays can only contain scalar values. php
0
+            // does not have this issue, but we should go with the lowest
0
+            // common denominator
0
+            add( prefix + ( param_traditional ? "" : "[]" ), v );
0
+          });
0
+          
0
+        else if ( typeof obj == "object" )
0
+          if ( param_traditional )
0
+            add( prefix, obj );
0
+            
0
+          else
0
+            jQuery.each( obj, function(k,v){
0
+              buildParams( prefix ? prefix + "[" + k + "]" : k, v );
0
+            });
0
+          
0
+        else
0
+          add( prefix, obj );
0
+        
0
+      });
0
 
0
     // Return the resulting serialization
0
     return s.join("&").replace(r20, "+");

test/unit/ajax.js

0
@@ -251,7 +251,30 @@ test("serialize()", function() {
0
 });
0
 
0
 test("jQuery.param()", function() {
0
-  expect(8);
0
+  expect(13);
0
+  
0
+  equals( jQuery.param.traditional, undefined, "traditional flag, undefined by default" );
0
+  
0
+  var params = {foo:"bar", baz:42, quux:"All your base are belong to us"};
0
+  equals( jQuery.param(params), "foo=bar&baz=42&quux=All+your+base+are+belong+to+us", "simple" );
0
+
0
+  params = {someName: [1, 2, 3], regularThing: "blah" };
0
+  equals( jQuery.param(params), "someName%5B%5D=1&someName%5B%5D=2&someName%5B%5D=3&regularThing=blah", "with array" );
0
+
0
+  params = {foo: ['a', 'b', 'c']};
0
+  equals( jQuery.param(params), "foo%5B%5D=a&foo%5B%5D=b&foo%5B%5D=c", "with array of strings" );
0
+
0
+  params = {foo: ["baz", 42, "All your base are belong to us"] };
0
+  equals( jQuery.param(params), "foo%5B%5D=baz&foo%5B%5D=42&foo%5B%5D=All+your+base+are+belong+to+us", "more array" );
0
+
0
+  params = {foo: { bar: 'baz', beep: 42, quux: 'All your base are belong to us' } };
0
+  equals( jQuery.param(params), "foo%5Bbar%5D=baz&foo%5Bbeep%5D=42&foo%5Bquux%5D=All+your+base+are+belong+to+us", "even more arrays" );
0
+  
0
+  params = { a:[1,2], b:{ c:3, d:[4,5], e:{ x:[6], y:7, z:[8,9] }, f:true, g:false, h:undefined }, i:[10,11], j:true, k:false, l:[undefined,0], m:"cowboy hat?" };
0
+  equals( jQuery.param(params), "a%5B%5D=1&a%5B%5D=2&b%5Bc%5D=3&b%5Bd%5D%5B%5D=4&b%5Bd%5D%5B%5D=5&b%5Be%5D%5Bx%5D%5B%5D=6&b%5Be%5D%5By%5D=7&b%5Be%5D%5Bz%5D%5B%5D=8&b%5Be%5D%5Bz%5D%5B%5D=9&b%5Bf%5D=true&b%5Bg%5D=false&b%5Bh%5D=undefined&i%5B%5D=10&i%5B%5D=11&j=true&k=false&l%5B%5D=undefined&l%5B%5D=0&m=cowboy+hat%3F", "huge structure" );
0
+  
0
+  jQuery.param.traditional = true;
0
+  
0
   var params = {foo:"bar", baz:42, quux:"All your base are belong to us"};
0
   equals( jQuery.param(params), "foo=bar&baz=42&quux=All+your+base+are+belong+to+us", "simple" );
0
 
0
@@ -266,15 +289,9 @@ test("jQuery.param()", function() {
0
 
0
   params = {"foo[bar]":"baz", "foo[beep]":42, "foo[quux]":"All your base are belong to us"};
0
   equals( jQuery.param(params), "foo%5Bbar%5D=baz&foo%5Bbeep%5D=42&foo%5Bquux%5D=All+your+base+are+belong+to+us", "even more arrays" );
0
-
0
-  params = {foo: {bar: "baz", beep: 42, quux: "All your base are belong to us"}};
0
-  equals( jQuery.param(params), "foo%5Bbar%5D=baz&foo%5Bbeep%5D=42&foo%5Bquux%5D=All+your+base+are+belong+to+us", "nested object" );
0
-
0
-  params = {foo: {"bar[]": [1,2,3]}};
0
-  equals( jQuery.param(params), "foo%5Bbar%5D%5B%5D=1&foo%5Bbar%5D%5B%5D=2&foo%5Bbar%5D%5B%5D=3", "nested array");
0
-
0
-  params = {foo: [{bar: "baz", beep: 42}, {bar: "baz2", beep: 43}]};
0
-  equals( jQuery.param(params), "foo%5Bbar%5D=baz&foo%5Bbeep%5D=42&foo%5Bbar%5D=baz2&foo%5Bbeep%5D=43", "nested array of objects" );
0
+  
0
+  params = { a:[1,2], b:{ c:3, d:[4,5], e:{ x:[6], y:7, z:[8,9] }, f:true, g:false, h:undefined }, i:[10,11], j:true, k:false, l:[undefined,0], m:"cowboy hat?" };
0
+  equals( jQuery.param(params), "a=1&a=2&b=%5Bobject+Object%5D&i=10&i=11&j=true&k=false&l=undefined&l=0&m=cowboy+hat%3F", "huge structure" );
0
 
0
 });
0

Comments

Please log in to comment.