I think I'd create my own component in my application.

I could even name it InsertText and it would just delegate to Insert using the desired break mode.

Ben Dotte wrote:
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]






---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to