From: Gilles van den Hoven Date: Wed, 17 May 2006 16:01:19 +0000 (+0000) Subject: Fixxed the form serialize function as seen on the mailing list. X-Git-Tag: 1.0a~72 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=f1c8b3ffc0d8dfd9dd2cb28c1e569d66d116cc9a;p=jquery.git Fixxed the form serialize function as seen on the mailing list. --- diff --git a/ajax/ajax.js b/ajax/ajax.js index 257ae02b2..d3443cc1f 100644 --- a/ajax/ajax.js +++ b/ajax/ajax.js @@ -139,13 +139,29 @@ $.fn.load = function(a,o,f) { }; /** - * function: $.fn.formValues - * usage: $('#frmLogin').formValues() - * docs: Gets the form values and creates a key=>value array of the found values (only for ENABLED elements!) + * function: $.fn.formValues + * usage: $('#frmLogin').formValues() + * docs: Gets form values and creates a key=>value array of the found values (for ENABLED elements!) */ $.fn.formValues = function() { var a = []; - this.find("input[@type='submit'],input[@type='hidden'],textarea,input[@checked],input[@type='password'],input[@type='text'],option[@selected]").filter(":enabled").each(function() { + this.find("input,textarea,option") + .filter(":enabled") + .each(function() { + // + // Skip selects with options which aren't selected + if (((this.parentNode.type == 'select-one') || (this.parentNode.type == 'select-multiple')) && + (!this.selected)) + return; + + // + // Skip radio and checkbox elements which aren't checked + if (((this.type == 'radio') || (this.type == 'checkbox')) && + (!this.checked)) + return; + + // + // All other elements are valid ;) var o = {}; o.name = this.name || this.id || this.parentNode.name || this.parentNode.id; o.value = this.value; @@ -155,11 +171,11 @@ $.fn.formValues = function() { }; /** - * function: $.update - * usage: $.update('someJQueryObject', 'someurl', 'array'); - * docs: Mimics the ajaxUpdater from prototype. Posts the key=>value array to the url and - * puts the results from that call in the jQuery object specified. - * --> If you set the blnNoEval to true, the script tags are NOT evaluated. + * function: $.update + * usage: $.update('someJQueryObject', 'someurl', 'array'); + * docs: Mimics the ajaxUpdater from prototype. Posts the key=>value array to the url and + * puts the results from that call in the jQuery object specified. + * --> If you set the blnNoEval to true, the script tags are NOT evaluated. */ $.update = function(objElement, strURL, arrValues, fncCallback) { $.post(strURL, arrValues, function(strHTML) { @@ -170,7 +186,9 @@ $.update = function(objElement, strURL, arrValues, fncCallback) { // // Evaluate the scripts objElement.html(strHTML).find("script").each(function(){ - try { $.eval( this.text || this.textContent || this.innerHTML ); } catch(e){} + try { + $.eval( this.text || this.textContent || this.innerHTML ); + } catch(e) { } }); //