we looked the one from wicket-extensions
Eyal Golan egola...@gmail.com Visit: http://jvdrums.sourceforge.net/ LinkedIn: http://www.linkedin.com/in/egolan74 P Save a tree. Please don't print this e-mail unless it's really necessary On Wed, May 6, 2009 at 6:43 PM, Igor Vaynberg <igor.vaynb...@gmail.com>wrote: > what is the full name of this class? there are two DateTextField > classes in wicket codebase. > > -igor > > On Tue, May 5, 2009 at 9:51 AM, Eyal Golan <egola...@gmail.com> wrote: > > Hello, > > We use Wicket 1.3.5 and I found something annoying with the > DateTextField. > > In the constructor of that class, the converter is created internally. > > If I want to use my own converter, I need to inherit DateTextField, add a > > converter as a member, and return it in the getConverter method. > > > > Why not have a protected method (that can be overridden) that returns the > > converter: > > Instead of: > > public DateTextField(String id, IModel model, String datePattern) > > { > > super(id, model, Date.class); > > this.datePattern = datePattern; > > *this.converter = new DateConverter() > > { > > private static final long serialVersionUID = 1L; > > > > /** > > * @see > > > org.apache.wicket.util.convert.converters.DateConverter#getDateFormat(java.util.Locale) > > */ > > public DateFormat getDateFormat(Locale locale) > > { > > return new > SimpleDateFormat(DateTextField.this.datePattern); > > } > > };* > > } > > > > Do something like: > > public DateTextField(String id, IModel model, String datePattern) > > { > > super(id, model, Date.class); > > this.datePattern = datePattern; > > *this.converter = newDateConverter();* > > } > > and > > > > protected newDateConverter() { > > return new DateConverter() > > { > > private static final long serialVersionUID = 1L; > > > > /** > > * @see > > > org.apache.wicket.util.convert.converters.DateConverter#getDateFormat(java.util.Locale) > > */ > > public DateFormat getDateFormat(Locale locale) > > { > > return new > SimpleDateFormat(DateTextField.this.datePattern); > > } > > }; > > } > > > > BTW, I know that we can also use the newConverterLocator() in our > > application. > > > > Do you think I should open a JIRA issue with 'wish' for that? > > > > > > Eyal Golan > > egola...@gmail.com > > > > Visit: http://jvdrums.sourceforge.net/ > > LinkedIn: http://www.linkedin.com/in/egolan74 > > > > P Save a tree. Please don't print this e-mail unless it's really > necessary > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org > For additional commands, e-mail: users-h...@wicket.apache.org > >