Split Form.serialize into Form.serialize and Form.serializeElements. …

…The latter can be used stand-alone to serialize an array of elements you pass in, instead of the entire form [DHH] git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@5077 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
rails · Sep 8, 2006 · 2c6747f · 2c6747f
1 parent e2d8451
commit 2c6747f
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 4 deletions.
diff --git a/actionpack/lib/action_view/helpers/javascripts/prototype.js b/actionpack/lib/action_view/helpers/javascripts/prototype.js
@@ -1594,7 +1594,10 @@ var Form = {
 
 Form.Methods = {
   serialize: function(form) {
-    var elements = Form.getElements($(form));
+    this.serializeElements(Form.getElements($(form)));
+  },
+
+  serializeElements: function(elements) {
     var queryComponents = new Array();
 
     for (var i = 0; i < elements.length; i++) {
@@ -1603,7 +1606,7 @@ Form.Methods = {
         queryComponents.push(queryComponent);
     }
 
-    return queryComponents.join('&');
+    return queryComponents.join('&');    
   },
 
   getElements: function(form) {
@@ -1615,6 +1618,7 @@ Form.Methods = {
       for (var j = 0; j < tagElements.length; j++)
         elements.push(tagElements[j]);
     }
+
     return elements;
   },
 

diff --git a/railties/html/javascripts/prototype.js b/railties/html/javascripts/prototype.js
@@ -1594,7 +1594,10 @@ var Form = {
 
 Form.Methods = {
   serialize: function(form) {
-    var elements = Form.getElements($(form));
+    this.serializeElements(Form.getElements($(form)));
+  },
+
+  serializeElements: function(elements) {
     var queryComponents = new Array();
 
     for (var i = 0; i < elements.length; i++) {
@@ -1603,7 +1606,7 @@ Form.Methods = {
         queryComponents.push(queryComponent);
     }
 
-    return queryComponents.join('&');
+    return queryComponents.join('&');    
   },
 
   getElements: function(form) {
@@ -1615,6 +1618,7 @@ Form.Methods = {
       for (var j = 0; j < tagElements.length; j++)
         elements.push(tagElements[j]);
     }
+
     return elements;
   },