P.S.: Has the bug already been posted?
On Apr 6, 11:46 am, Aaron Newton <[email protected]> wrote: > Right. Sorry, I figured that out a bit after I hit send. I figured I'd add > that to lighthouse after it got created... > > > > > > > > On Wed, Apr 6, 2011 at 9:05 AM, Arian Stolwijk <[email protected]> wrote: > > `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/Fo... > > >> 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
