Bug Tracker

Changeset 3293

Show
Ignore:
Timestamp:
09/15/07 00:23:21 (1 year ago)
Author:
jeresig
Message:

Fix for bug #1600 - multiple selects were being serialized incorrectly.

Location:
trunk/jquery
Files:
2 modified

Legend:

Unmodified
Added
Removed

  • trunk/jquery/src/ajax.js

    r3178 r3293  
    7979            return val == null ? null : 
    8080                val.constructor == Array ? 
    81                     jQuery.map( val, function(i, val){ 
     81                    jQuery.map( val, function(val, i){ 
    8282                        return {name: elem.name, value: val}; 
    8383                    }) : 

  • trunk/jquery/test/unit/ajax.js

    r3139 r3293  
    167167     
    168168    equals( $('#form').serialize(), 
    169         "action=Test&radio2=on&check=on&hidden=&foo%5Bbar%5D=&name=name&select1=&select2=3&select3=0&select3=1", 
     169        "action=Test&radio2=on&check=on&hidden=&foo%5Bbar%5D=&name=name&select1=&select2=3&select3=1&select3=2", 
    170170        'Check form serialization as query string'); 
    171171         
    172172    equals( $('#form :input').serialize(), 
    173         "action=Test&radio2=on&check=on&hidden=&foo%5Bbar%5D=&name=name&select1=&select2=3&select3=0&select3=1", 
     173        "action=Test&radio2=on&check=on&hidden=&foo%5Bbar%5D=&name=name&select1=&select2=3&select3=1&select3=2", 
    174174        'Check input serialization as query string'); 
    175175     
    … …  
    183183         
    184184    equals( $('#form, #testForm').serialize(), 
    185         "action=Test&radio2=on&check=on&hidden=&foo%5Bbar%5D=&name=name&select1=&select2=3&select3=0&select3=1&T3=%3F%0AZ&H1=x&H2=&PWD=&T1=&T2=YES&My+Name=me&S1=abc&S3=YES&S4=", 
     185        "action=Test&radio2=on&check=on&hidden=&foo%5Bbar%5D=&name=name&select1=&select2=3&select3=1&select3=2&T3=%3F%0AZ&H1=x&H2=&PWD=&T1=&T2=YES&My+Name=me&S1=abc&S3=YES&S4=", 
    186186        'Multiple form serialization as query string'); 
    187187         
    188188    equals( $('#form, #testForm :input').serialize(), 
    189         "action=Test&radio2=on&check=on&hidden=&foo%5Bbar%5D=&name=name&select1=&select2=3&select3=0&select3=1&T3=%3F%0AZ&H1=x&H2=&PWD=&T1=&T2=YES&My+Name=me&S1=abc&S3=YES&S4=", 
     189        "action=Test&radio2=on&check=on&hidden=&foo%5Bbar%5D=&name=name&select1=&select2=3&select3=1&select3=2&T3=%3F%0AZ&H1=x&H2=&PWD=&T1=&T2=YES&My+Name=me&S1=abc&S3=YES&S4=", 
    190190        'Mixed form/input serialization as query string'); 
    191191});