You're right! I see what happened here. InsertText is marked as deprecated in 
the 4.1 documentation so we replaced all 171 references to it in our code with 
@Insert. But seeing that everything looked wrong we had to add the BREAK mode 
to every one of them.

So I guess it would be nice if the default were configurable assuming 
InsertText really is deprecated and will be going away.

-----Original Message-----
From: Jesse Kuhnert [mailto:[EMAIL PROTECTED] 
Sent: Wednesday, February 14, 2007 12:03 AM
To: Tapestry users
Subject: Re: @Insert default mode

I think you are thinking of "InsertText", not "Insert". Having a mode
of break by default on that component would break everyone using
@Insert previously. (or am I losing my mind? )

On 2/12/07, Ben Dotte <[EMAIL PROTECTED]> wrote:
> Thanks but we try to avoid overriding Tapestry classes whenever possible.. in 
> this case we wouldn't really be changing anything except what "mode" defaults 
> to so I would consider it an override rather than a custom component. 
> Overrides make upgrading tricky and introduce subtle bugs that take forever 
> to track down.
>
> I believe Insert used to behave in 4.0 the same way it behaves in 4.1 with 
> mode="ognl:@[EMAIL PROTECTED]". It would be nice to have an application-wide 
> setting for this, if nothing else, for backwards compatibility sake.
>
> -----Original Message-----
> From: Norbert Sándor [mailto:[EMAIL PROTECTED]
> Sent: Monday, February 12, 2007 3:40 PM
> To: Tapestry users
> Subject: Re: @Insert default mode
>
> Maybe override the Insert component in the .application file with your
> custom one.
> N.
>
> Ben Dotte írta:
> > Does anyone know if there is a way to specify a default mode for the
> > Insert component? When we upgraded to Tapestry 4.1 we ended up having to
> > specify InsertMode.BREAK every place we use it but that ends up looking
> > pretty ugly in the many places we use Insert
> > (mode="ognl:@[EMAIL PROTECTED]").
> >
> >
> >
> > It would be nice if there was some way to configure a default value for
> > this if there isn't already.
> >
> >
> >
> > Thanks,
> >
> > Ben
> >
> >
> >
> > ------------------------------------------------------------------------
> >
> > No virus found in this incoming message.
> > Checked by AVG Free Edition.
> > Version: 7.5.441 / Virus Database: 268.17.37/682 - Release Date: 
> > 2007.02.12. 13:23
> >
>
> ---------------------------------------------------------------------
> 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]
>
>


-- 
Jesse Kuhnert
Tapestry/Dojo team member/developer

Open source based consulting work centered around
dojo/tapestry/tacos/hivemind. http://blog.opencomponentry.com

---------------------------------------------------------------------
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]

Reply via email to