Being there doesn't really mean that they need a location, there are
also result types on external plugins that we won't know about. We
could change the warning message, or remove it altogether.

musachy

On Sun, Jun 8, 2008 at 11:38 AM, Bob Tiernay <[EMAIL PROTECTED]> wrote:
> Can't you determine these from struts.convention.relative.result.types?
>
> --------------------------------------------------
> From: "Musachy Barroso" <[EMAIL PROTECTED]>
> Sent: Sunday, June 08, 2008 11:14 AM
> To: "Struts Developers List" <dev@struts.apache.org>
> Subject: Re: [Struts2] Convention Plugin - @Result
>
>> Well, there are a few result that require "location", like velocity,
>> freemarker and others. There is no way to know which one do or do no.
>> To play on the safe side, a warning was added, but it is kind of weak
>> indeed.
>>
>> musachy
>>
>> On Sun, Jun 8, 2008 at 10:59 AM, Bob Tiernay <[EMAIL PROTECTED]> wrote:
>>>
>>> I am trying to use the following in my LoginAction class:
>>>
>>>       @Action(value = "login!login", results = [EMAIL PROTECTED](name = 
>>> "success",
>>> type = "redirectAction", params = {"actionName", "ListAccounts",
>>> "namespace", "/account"}
>>>       )})
>>>       public String login() throws Exception {
>>>
>>> However, this results in the following message:
>>>
>>> 0    [main] WARN org.apache.struts2.convention.DefaultResultMapBuilder  -
>>> Result [success] for action class [LoginAction] is missing the location
>>>
>>> To me, the location parameter is only applicable to a result of type
>>> "dispatch" and thus the above use case should not result in a warning.
>>> Maybe
>>> there is some way to remove this without having to hardcode the
>>> relationship
>>> in the plugin?
>>>
>>> Bob
>>>
>>>
>>>
>>>
>>>
>>>
>>>
>>> ---------------------------------------------------------------------
>>> To unsubscribe, e-mail: [EMAIL PROTECTED]
>>> For additional commands, e-mail: [EMAIL PROTECTED]
>>>
>>>
>>
>>
>>
>> --
>> "Hey you! Would you help me to carry the stone?" Pink Floyd
>>
>> ---------------------------------------------------------------------
>> 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]
>
>



-- 
"Hey you! Would you help me to carry the stone?" Pink Floyd

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

Reply via email to