Side navigation
#1638 closed bug (fixed)
Opened September 17, 2007 11:08AM UTC
Closed October 18, 2007 12:08AM UTC
Error migrating some $.ajax(...) calls to version 1.2.x (from versions 1.1.x)
Reported by: | Piatenko | Owned by: | |
---|---|---|---|
Priority: | major | Milestone: | 1.2.2 |
Component: | ajax | Version: | 1.2.1 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
Prior to version 1.2 it was ok to pass a Number data parameter to jQuery.ajax() function.
Example:
var i = 321; $.ajax( { url: "/test.html", type:"POST", dataType:"json", data: i, processData:false, success:function(r){alert(r)} } )
in new version I have to specify
data: i+""
to pass it as string parameter
var i = 321; $.ajax( { url: "/test.html", type:"POST", dataType:"json", data: i+"", processData:false, success:function(r){alert(r)} } )
and that's only because of this new line 2216 in code (v 1.2.1):
if ( s.dataType == "json" && (s.data && s.data.match(jsre) || s.url.match(jsre)) ) {
it could be easily fixed by adding only one additional condition
typeof s.data==="string" here:
if ( s.dataType == "json" && (s.data && typeof s.data==="string" && s.data.match(jsre) || s.url.match(jsre)) ) {
Attachments (0)
Change History (1)
Changed October 18, 2007 12:08AM UTC by comment:1
milestone: | 1.2.1 → 1.2.2 |
---|---|
resolution: | → fixed |
status: | new → closed |
version: | 1.2 → 1.2.1 |
Fixed in SVN rev [3670].