jquery-iframe-transport icon indicating copy to clipboard operation
jquery-iframe-transport copied to clipboard

No additional data sent

Open Luoti opened this issue 13 years ago • 4 comments

Hi!

I used this plugin with jquery 1.6.1, but when I upgraded to 1.7.1 the additional form data was not sent. With blind troubleshooting I found out that the additional data was not found in options.data, but it was in the origOptions.data. I fixed it by just adding:

options.data = origOptions.data;

to line 137.

Luoti avatar Jan 18 '12 10:01 Luoti

+1 - this fix works for me as well. Perhaps a check for nil on both options.data and origOptions.data is needed?

tallgreentree avatar May 17 '12 20:05 tallgreentree

+1 Great fix, I was looking for it. Should be added to the actual plugin!

ghost avatar Jun 18 '12 19:06 ghost

+1 Yep, can't add additional options without this

tony2nite avatar Oct 27 '12 15:10 tony2nite

Finally applied :P. Thanks!

cmlenz avatar Jul 06 '13 20:07 cmlenz

Because of this fix there seems to be a problem with data that is added to the options in a beforeSend callback. I think the line should be something like:

options.data = $.extend({}, origOptions.data, options.data);

There are still issues, as this does not reflect removing stuff from options.data in the beforeSend callback, but at least it does not ignore additional data which seems to be the more common use case.

swiftyone avatar Jul 22 '13 22:07 swiftyone