I am guessing that frm.oncancel and frm.onrefresh only get defined if
it is actually an tacos supplied form, rather than a regular tapestry
supplied form. So we have a situation where including any tacos
component causes the tacos.js file to force us to use the tacos
supplied methods, yet the tacos supplied methods don't actually
function on a regular Tapestry supplied form. This is a great big
huge stinkin bug that I really need to figure out a fix for ASAP. Any
guidance someone can give me would be great. I don't, to be honest,
understand why the form specific code doesn't get included in a .js or
.script file that is only included when an AjaxForm is utilized.
--sam
On 8/28/06, Sam Gendler <[EMAIL PROTECTED]> 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]
