Hi,

>You can actually do this now using the generic "set-property" element,
>supported by the Digester.

> >     <action path="/index"
type="pl.wwtech.shared.struts.IndexAction">
> >       <set-property property="test" value="value"/>
> >       <forward name="default" path="page.index"/>
> >     </action>

Sorry, I made a mistake and copy-pasted wrong fragment, the code I sent
uses <set-parameter> because <set-property> is taken. That would be:
"<set-parameter name="test2" value="value2"/>"

It's not as redundant as it may seem. It certainly make things easier
and faster to implement. Rationale: <set-property> is meant to be used
with Java Bean properties, while my <set-parameter> is more like a
HttpServletRequest.getParameter() - it makes no assumptions about
existing property names and doesn't need custom ActionMapping
subclasses, allowing for quick changes and constructions like
"
<set-parameter name="test.number" value="2"/>
<set-parameter name="test.1.name" value="Test 1"/>
<set-parameter name="test.2.name" value="Test 2"/>
"
and similar.
Also, if for some reason mapping have to be shared by two Actions (it's
possible indirectly) I think it's easier to have a generic ActionMapping
and calling mapping.getParameter(name,default) than to deal with
different subclasses - it could lead to problems with (lack of) multiple
inheritance. Also, these <set-property> have much uses and I think
there's no reason why <set-property> and <set-parameter> couldn't
coexist. It's strict-naming plus subclassing vs loose-naming plus
generic implementation - I really think it would be usefull having both.
Besides, I'd much prefer to have it in future downloads than to patch
each beta myself locally ;>

>In Struts 1.1,  you have to set the classname attribute for the 
>ActionMapping to use your subclass. In Struts 1.0.x, there are 
>corresponding init-params for the ActionServlet.

>This works with all the configuration objects, like ActionForwards.

>I must say you certainly have the right attitude =:-) Please keep 

Thanks.

>looking for other features we may be missing ...

>-Ted.

Greetings,
     Sebastian P. Markiewicz, WWTech (aka deacon Marcus)
     [EMAIL PROTECTED]
     tel +48 602 102750
     fax +48 602 145786


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

Reply via email to