cmlenz/jquery-iframe-transport

No additional data sent

Opened this issue · 5 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.

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

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

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

Finally applied :P. Thanks!

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.