diff options
author | Michał Gołębiowski-Owczarek <m.goleb@gmail.com> | 2023-03-20 17:08:51 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-03-20 17:08:51 +0100 |
commit | ce264e0789116e37fe371503537a217c038dfae8 (patch) | |
tree | a674fe89ba02bbaf63723d07a4976ee87fe5dcc3 /test/unit/ajax.js | |
parent | cff2899885c314d32eea42e9eef6ead6e5da5c2f (diff) | |
download | jquery-ce264e0789116e37fe371503537a217c038dfae8.tar.gz jquery-ce264e0789116e37fe371503537a217c038dfae8.zip |
Ajax: Allow `processData: true` even for binary data
The way gh-5197 implemented binary data handling, `processData`
was being explicitly set to `false`. This is expected but it made
it impossible to override it to `true`. The new logic will only
set `processData` to `false` if it wasn't explicitly passed
in original options.
Closes gh-5205
Ref gh-5197
Diffstat (limited to 'test/unit/ajax.js')
-rw-r--r-- | test/unit/ajax.js | 23 |
1 files changed, 23 insertions, 0 deletions
diff --git a/test/unit/ajax.js b/test/unit/ajax.js index 7ecedc212..bcaa7651d 100644 --- a/test/unit/ajax.js +++ b/test/unit/ajax.js @@ -3148,4 +3148,27 @@ if ( typeof window.ArrayBuffer === "undefined" || typeof new XMLHttpRequest().re }; } ); + ajaxTest( "jQuery.ajax() - non-plain object", 1, function( assert ) { + return { + url: url( "mock.php?action=name" ), + method: "post", + data: Object.create( { name: "peter" } ), + success: function( data ) { + assert.strictEqual( data, "ERROR", "Data correctly not sent" ); + } + }; + } ); + + ajaxTest( "jQuery.ajax() - non-plain object with processData: true", 1, function( assert ) { + return { + url: url( "mock.php?action=name" ), + method: "post", + processData: true, + data: Object.create( { name: "peter" } ), + success: function( data ) { + assert.strictEqual( data, "pan", "Data sent correctly" ); + } + }; + } ); + } )(); |