struts-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From David Evans <dsev...@berndtgroup.net>
Subject Re: FieldErrorTag
Date Sun, 08 Jun 2008 02:59:53 GMT
Thanks for the pointer. I rewrote the implementation to use the proper
annotation. The new implementation patches both FieldError.java and
FieldErrorTag.java, in order to follow the pattern used in the other
tags, for instance TextField. My patches are very simple, and do not
require any changes to the templates. It simply adds the value of the
new fieldName attribute, if it exists, to the existing errorFieldNames
collection.

Thanks,

Dave


On Sat, 2008-06-07 at 10:39 -0400, Musachy Barroso wrote:
> Yes, TLD is generated form annotations automatically.
> 
> musachy
> 
> On Sat, Jun 7, 2008 at 9:45 AM, Dave Newton <newton.dave@yahoo.com> wrote:
> > I was under the impression the TLD was created from annotations; is this not true?
I've never actually looked.
> >
> > Dave
> >
> > --- On Fri, 6/6/08, David Evans <dsevans@berndtgroup.net> wrote:
> >
> >> From: David Evans <dsevans@berndtgroup.net>
> >> Subject: FieldErrorTag
> >> To: dev@struts.apache.org
> >> Date: Friday, June 6, 2008, 11:29 PM
> >> I have made a change to the FieldErrorTag and would like to
> >> submit it
> >> for consideration. It is a solution to my desire to use the
> >> FieldErrorTag on a single field basis in this manner:
> >> <s:fielderror fieldName="myField" />
> >>
> >> instead of the current usage:
> >> <s:fielderror><s:param>myField</s:param></s:fielderror>
> >>
> >> I have made two simple changes, one to FieldErrorTag, one
> >> to the
> >> struts-tags.tld.
> >>
> >> I have created a JIRA ticket (WW-2671) and attached the
> >> patch to
> >> FieldErrorTag.java and a updated version of the
> >> struts-tags.tld file.
> >>
> >> Is there anything else I need to do?
> >>
> >> Thanks,
> >>
> >> Dave
> >>
> >>
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
> >> For additional commands, e-mail: dev-help@struts.apache.org
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
> > For additional commands, e-mail: dev-help@struts.apache.org
> >
> >
> 
> 
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


Mime
View raw message