> Christian, can you 
> please have a look and test it?

Looking good :)

Updated from cvs and all sems to be working well.

> Now you can specifiy interceptor properties per action

I'm not quite clear on why this is a good thing(tm)...

It was already possible to have interceptor definitions of the same class,
but with different configuration:

    <interceptor name="acme-foo" type="foo.bar.MyInterceptor">
        <properties>
            <property name="acme" value="foo"/>
        </properties>
    </interceptor>
    <interceptor name="acme-bar" type="foo.bar.MyInterceptor">
        <properties>
            <property name="acme" value="bar"/>
        </properties>
    </interceptor>

    <action type="/foo">
        <interceptor name="acme-foo"/>
    </action>
    <action type="/bar">
        <interceptor name="acme-bar"/>
    </action>

Is it not better to keep the configuration on one level?

CHR



> -----Original Message-----
> From: Lars Hoss [mailto:[EMAIL PROTECTED]
> Sent: 28. november 2003 11:15
> To: [EMAIL PROTECTED]
> Subject: [Struts-apps] [SAIF] Added support for interceptor properties
> per action
> 
> 
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> Hi!
> 
> I have commited my changes. Now you can specifiy interceptor 
> properties 
> per action:
> 
> <action type="..." scope="request">
>    <interceptor name="...">
>      <properties>
>        <property name="foo" value="moo"/>
>      </properties>
>    </interceptor>
> </action>
> 
> Attention: it is important to set scope to request (or session when 
> it'll be supported) because setting
> properties in scope application is not supported because of threading 
> issues.
> 
> Furthermore I cleaned up the RequestProcessorHelper. 
> Christian, can you 
> please have a look and test it?
> 
> Best Regards,
> Lars
> 
> -----BEGIN PGP SIGNATURE-----
> Version: PGP 8.0.3
> 
> iQA/AwUBP8cgF7cyzbDWnRDCEQKA0ACgoY21PzAW2wqeNFhMsSX4eT11hS0AoPRk
> dblpjVuyPqTqrDpTUIO+NW3m
> =N4Ft
> -----END PGP SIGNATURE-----
> 
> 
> 
> -------------------------------------------------------
> This SF.net email is sponsored by: SF.net Giveback Program.
> Does SourceForge.net help you be more productive?  Does it
> help you create better code?  SHARE THE LOVE, and help us help
> YOU!  Click Here: http://sourceforge.net/donate/
> _______________________________________________
> Struts-apps mailing list
> [EMAIL PROTECTED]
> https://lists.sourceforge.net/lists/listinfo/struts-apps
> 


-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?  SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/
_______________________________________________
Struts-apps mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/struts-apps

Reply via email to