`this.get('formRequest')` should get the Form.Request instance, which is
correct, however `Element.Properties.formRequest` is broken, and also you
can only do `this.get('formRequest')` because it doesn't accept multiple
arguments, like is used in formUpdate.On Wed, Apr 6, 2011 at 5:53 PM, Aaron Newton <[email protected]> wrote: > A couple of things. > > - you are setting options on the request, so you need to indent your > options object for that per the docs > eg. > form.set('send',{ > requestOptions: { > useSpinner: true, > spinnerTarget: form > } > }); > - You also need to tell it what to update (form.formUpdate(target)); > You can also do form.formUpdate(target, options); > - jsfiddle's echo/json needs you to give it some data; not really the > issue here but may prevent you from seeing a result > - finally, you've found a bug! > > Check it out: > > > https://github.com/mootools/mootools-more/blob/master/Source/Forms/Form.Request.js#L214 > > You'll note that on line 188 and 204 (and elsewhere) we store the instance > on the element as "form.request" but on line 214 it's "formRequest" so it > doesn't retrieve it. So element.formUpdate is broken. > > For the time being, you can just create the instance the regular way (var > fq = new Form.Request(form, updateElement, options)). > > Would you like to open a lighthouse ticket for this bug? (just paste in > this last part). I can do it, but if you do it you'll get emails as we work > on it and resolve it. > > https://mootools.lighthouseapp.com/projects/24057-mootoolsmore > > Aaron > > On Tue, Apr 5, 2011 at 8:41 PM, dukeofgaming <[email protected]>wrote: > >> Hi, >> >> Quick and perhaps dumb question. Why wont the Request spinner >> parameters work when set from the send property?. >> >> http://jsfiddle.net/dukeofgaming/neQ3X/5/ >> >> I'd swear I've made it work like this, am I missing something? >> >> Thanks in advance >> > >
