[patch] [cla] Additional form elements for FileInputAuto & FileInputBlind

[Ian Fouls]

Please see the attached file for modification to FileInputAuto? & FileInputBlind?. This provides a callback to append additional DOM elements to the form before submission via the iframe.

This is done out of frustration with the FileIploader? not working in a dialog, please see ticket #9996. I need to get some code out the door that works!!!

[Adam Peller]

We cannot accept patches without a CLA. Patch format is preferred.

[Ian Fouls]

Patch file attached, CLA has been in place for some time.

[dante]

would it make more sense to have a generic mixin for content:{} object, so that you may simply specify a pre-send callback like:

new FileInputBlah({
   onBeforeSend: function(){
       return {};
   }
});

then, internally FileInputBlah? would just:

dojo.io.iframe.send({ form: this._form, content: this.onBeforeSend() });

it seems actually using the formnode (which is technically private) to clone inputs etc would be harder/more expensive than just being able to specify a function which will return the intended data to mix into content: param

[Ian Fouls]

Agreed, your solution is far more elegant, I needed to get some code out to a customer and I've already overrrun deadlines by some considerable amount. With neither FileImput? or FileUploader? working (at least a day lost here!!!) I needed a quick method of resolving the problem.

[dante]

(In [21599]) fixes #10195 - adding in and documenting onBeforeSend