Mailing-List: contact dev-help@struts.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Struts Developers List" <dev@struts.apache.org>
Received-SPF: pass (nike.apache.org: domain of ken.mcwilliams@gmail.com
 designates 209.85.214.182 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAMopvkNwmzGozu7JjmjcVY0Ct+D=iW7bA=-wKDn5GhL-Xcu6YQ@mail.gmail.com>
References: 
 <CAMopvkOTTw=bp+OYahcB11W2jROSOn0=UeUDkiV9YEqgsKfu3A@mail.gmail.com>
	<CAPS4wT6u6YjBcQ-QcvTYLpBqFRk5ekVNh1dBtA1ZJBSy+RvCOQ@mail.gmail.com>
	<CAMopvkOZdD6AwyiWg_UEXtV18rmB1S2-LQQsPOm+EUxs8Y_EYg@mail.gmail.com>
	<CABLGb9xFy+cXjwJDn54egbHUVNPE4N8JTf2=b3804+AUjQhn2A@mail.gmail.com>
	<CAPS4wT7+dywBKRGztEzixPk0cjS-5qXDAu6-fHLNbygfxtDX-w@mail.gmail.com>
	<CAMopvkNwmzGozu7JjmjcVY0Ct+D=iW7bA=-wKDn5GhL-Xcu6YQ@mail.gmail.com>
Date: Wed, 9 Oct 2013 16:58:59 -0600
Message-ID: 
 <CAPcVg7nkFPD1UCJ8QaAhUK9+xvOeHQK84FgMeEHAaPvL1biNKw@mail.gmail.com>
Subject: Re: Strict DMI
From: Ken McWilliams <ken.mcwilliams@gmail.com>
To: Struts Developers List <dev@struts.apache.org>
Content-Type: multipart/alternative; boundary=001a113334a0a7254a04e856d408

--001a113334a0a7254a04e856d408
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

What am I missing? Why not just the @action annotation? The whole method
annotation seems to have risen out of a poor definition of "action". I
consider the action the entire follow of execution. From mapping to result
(Interceptors and the Action class too).

>From the DefaultActionMapper documentation:

*With method-prefix, instead of calling baz action's execute() method (by
default if it isn't overriden in struts.xml to be something else), the baz
action's anotherMethod() will be called. A very elegant way determine which
button is clicked. Alternatively, one would have submit button set a
particular value on the action when clicked, and the execute() method
decides on what to do with the setted value depending on which button is
clicked. *

If you need an annotation on "anotherMethod" @action would be functionally
equivalent to @method. Of course you wouldn't be able to use the "method:"
prefix but then you wouldn't have any need.


On Sun, Oct 6, 2013 at 11:23 PM, Lukasz Lenart <lukaszlenart@apache.org>wro=
te:

> I think @ActionMethod or @Method is very handy. I'm still wondering
> about how to map which actions are allowed to be used with "action:"
> prefix - what about dropping "action:" prefix and stick only with
> "method:" and "<s:form method=3D...>" ?
>
>
> Regards
> --
> =C5=81ukasz
> + 48 606 323 122 http://www.lenart.org.pl/
>
> 2013/10/4 Steven Benitez <steven.benitez@gmail.com>:
> > I suggested this because I wrote an interceptor to require the
> > @ActionMethod annotation years ago to lock down DMI. The upside to a
> > separate annotation was that it was completely compatible with XML
> > configuration (which I use). It also had a nice benefit of being
> > documentation, as well. No ambiguity as to whether an method was an
> > invocable action method or just a method that returned a String.
> >
> >
> > On Fri, Oct 4, 2013 at 10:37 AM, Paul Benedict <pbenedict@apache.org>
> wrote:
> >
> >> I like that WAY better. Instead of using opaque strings in @Action, us=
e
> >> @ActionMethod on the destination methods. +1
> >>
> >>
> >> On Fri, Oct 4, 2013 at 4:31 AM, Lukasz Lenart <lukaszlenart@apache.org
> >> >wrote:
> >>
> >> > 2013/10/3 Steven Benitez <steven.benitez@gmail.com>:
> >> > > Why not just have an @ActionMethod annotation? If its on the actio=
n
> >> > method,
> >> > > you can invoke it, if not, you can't. The global config option for
> >> > allowed
> >> > > methods sounds reasonable (e.g., execute, input, etc.)
> >> >
> >> > Nice idea and quite simple :-) What about "allowedActions" ? Maybe
> >> > extend @Action annotation and add "callable =3D true|false" which wi=
ll
> >> > indicate if action can be called by action: prefix.
> >> >
> >> >
> >> > Regards
> >> > --
> >> > =C5=81ukasz
> >> > + 48 606 323 122 http://www.lenart.org.pl/
> >> >
> >> > --------------------------------------------------------------------=
-
> >> > To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
> >> > For additional commands, e-mail: dev-help@struts.apache.org
> >> >
> >> >
> >>
> >>
> >> --
> >> Cheers,
> >> Paul
> >>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
> For additional commands, e-mail: dev-help@struts.apache.org
>
>

--001a113334a0a7254a04e856d408--