I am sure the fix fixes AjaxForms. Unfortunately, it breaks any non-AjaxForms that are on the same page. I have a page with a number of AjaxForms (included as part of a component) as well as a normal tapestry @Form component. Since tacos.js overrides functionality globally, for all forms, it breaks the normal @Form component.
--sam On 8/28/06, andyhot <[EMAIL PROTECTED]> wrote:
Initially, therre was this bug report: http://tacoscomponents.jot.com/BugReporter/Bug109 You can try the demo page at http://opencomponentry.com:8080/tacos/bugs/Bug109.html Enter more than six letters (there's a min length validator), hit submit, then enter one letter and then click on 'Cancel form' Sam Gendler wrote: > OK, so after pulling the latest tacos source and checking out > tacos.js, I see that the mentioned methods are indeed newer than my > current build. So I ported all my changes to the latest tacos trunk > AND I went ahead an upgraded to a custom build of dojo 0.3.1. Aside > from the (longstanding) problems I get when tacos.js calls > field.focus() or field.select(), throwing an exception and causing > other js to fail to execute (I put the two lines in a conditional > statement which skips the affected browsers), everything works as it > used to before. I can no call tacos.cancelForm(), too, except that it > still throws errors in FIrefox. Now I am getting an error on the > following line: > > if (submitType == 'cancel') frm.oncancel(event); > > The message is: > > frm.oncancel is not a function > > That is the only error I am getting. Has anyone actually tested this > code and seen it work in firefox? I really need to be able to cancel > a form that happens to be on a page with tacos components. > > --sam > > > On 8/28/06, Sam Gendler <[EMAIL PROTECTED]> wrote: >> Actually, I still don't have a fix. According to the FAQ entry you >> referred me to earlier: >> >> In ajax Forms you'll have to use tacos.refreshForm(document.form) or >> tacos.cancelForm(document.form). >> >> but neither function exists anywhere in the tacos object. I get: >> >> tacos.cancelForm is not a function >> >> Looking at tacos.js, there is no mention of either method, and >> grepping through the entire tacos source code, I never find the >> strings cancelForm or refreshForm in any .js, .script, or .java file. >> I'm thinking the faq is apparently way out of date (unless these >> methods were added fairly recently. I am dealing with a fairly old >> version of tacos right now) >> >> --sam >> >> >> On 8/28/06, Sam Gendler <[EMAIL PROTECTED]> wrote: >> > Yeah, that definitely needs to be documented somewhere REALLY obvious. >> > I am using other tacos components on the page, so tacos.js is being >> > loaded, even though it isn't using an AjaxForm. As a result, tacos.js >> > is getting loaded and it is overloading the behaviour. >> > >> > Really, if tacos is going to overload behaviour, wouldn't it be better >> > if it overloaded existing methods, so that it wouldn't break >> > functionality like this, rather than forcing a user to know about >> > internal data structure changes? >> > >> > --sam >> > >> > >> > On 8/28/06, andyhot <[EMAIL PROTECTED]> wrote: >> > > Sam Gendler wrote: >> > > > Actually, I am on Tap 4.0.2 and it isn't an AjaxForm. Unless >> you are >> > > > saying that merely the existence of tacos causes the overloaded >> > > > behaviour, even on regular tapestry forms. >> > > if tacos.js and form.js are included in a page, they indeed >> overload the >> > > cancel and >> > > refresh behaviour - they redefine some of Tapestry's javascript >> code... >> > > >> > > We might need a way to disable those overrides I guess. Perhaps >> Jesse >> > > can be of help >> > > here. >> > > > >> > > > --sam >> > > > >> > > > >> > > > On 8/28/06, andyhot <[EMAIL PROTECTED]> wrote: >> > > >> Jesse Kuhnert wrote: >> > > >> > I'm assuming you are working in 4.1 if this is the error you >> are >> > > >> getting. >> > > >> or tapestry 4.0 and tacos... >> > > >> http://tacos.sourceforge.net/faq.html#faq-N10080 >> > > >> >> > > >> -- >> > > >> Andreas Andreou - [EMAIL PROTECTED] - http://andyhot.di.uoa.gr >> > > >> Tapestry / Tacos developer >> > > >> Open Source / J2EE Consulting >> > > >> >> > > >> >> > > >> >> --------------------------------------------------------------------- >> > > >> To unsubscribe, e-mail: [EMAIL PROTECTED] >> > > >> For additional commands, e-mail: [EMAIL PROTECTED] >> > > >> >> > > >> >> > > > >> > > > >> --------------------------------------------------------------------- >> > > > To unsubscribe, e-mail: [EMAIL PROTECTED] >> > > > For additional commands, e-mail: [EMAIL PROTECTED] >> > > > >> > > > >> > > >> > > >> > > -- >> > > Andreas Andreou - [EMAIL PROTECTED] - http://andyhot.di.uoa.gr >> > > Tapestry / Tacos developer >> > > Open Source / J2EE Consulting >> > > >> > > >> > > >> --------------------------------------------------------------------- >> > > To unsubscribe, e-mail: [EMAIL PROTECTED] >> > > For additional commands, e-mail: [EMAIL PROTECTED] >> > > >> > > >> > >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > -- Andreas Andreou - [EMAIL PROTECTED] - http://andyhot.di.uoa.gr Tapestry / Tacos developer Open Source / J2EE Consulting --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
