The issue is not the lack of an existing tag to do the formatting. The problem is that, when using <column> body to perform formatting, sorting is done on the formatted value, rather than the underlying data.
Paul -----Original Message----- From: Andy Pruitt [mailto:[EMAIL PROTECTED] Sent: 06 November 2003 02:24 To: [EMAIL PROTECTED] Subject: RE: [displaytag-devel] [ displaytag-Feature Requests-683989 ] <di splay:column format="dd-MMM-yyyy"> needed for date propt Why not just do this with a JSTL fmt:formatDate tag? > I prefer John's way of doing things - it is better able to reuse existing > custom tags to format the data. > > I've proposed a very simple change which would allow pretty much what John > is suggesting, other than explicitly exposing the current column's value. > e.g. > > <display:column property="mydate"> > <display:format type="date" arg="YYYY-MM-DD"> > <bean:write name="${currentrow.mydate}"/> > </display:format> > </display:column> > > I'd be grateful for comments on my proposal: > http://sourceforge.net/tracker/index.php?func=detail&aid=823393&gr > oup_id=730 > 68&atid=536616 > > Paul > > -----Original Message----- > From: John York [mailto:[EMAIL PROTECTED] > Sent: 05 November 2003 13:57 > To: [EMAIL PROTECTED] > Subject: Re: [displaytag-devel] [ displaytag-Feature Requests-683989 ] > <display:column format="dd-MMM-yyyy"> needed for date propt > > > I don't think that this feature should be part of the core displaytag. > This is where the various string formatting utilities come into play. I'd > envision something like this: > > <display:column property="mydate"> > <display:format type="date" arg="YYYY-MM-DD"> > <bean:write name="mydate"/> > </display:format> > </display:column> > > > On Tue, 4 Nov 2003, SourceForge.net wrote: > > > Feature Requests item #683989, was opened at 2003-02-10 17:19 > > Message generated for change (Comment added) made by ravn > > You can respond by visiting: > > > https://sourceforge.net/tracker/?func=detail&atid=536616&aid=68398 > 9&group_id > =73068 > > > > Category: None > > Group: None > > Status: Open > > Resolution: None > > Priority: 5 > > Submitted By: Thorbjoern Ravn Andersen (ravn) > > Assigned to: Nobody/Anonymous (nobody) > > Summary: <display:column format="dd-MMM-yyyy"> needed for date propt > > > > Initial Comment: > > We have a need for providing a formatstring for dates > > as the default is not good enough. > > > > I have a patch, which I am testing out. If there is > > interest I can upload it. > > > > ---------------------------------------------------------------------- > > > > Comment By: Thorbjoern Ravn Andersen (ravn) > > Date: 2003-11-04 21:32 > > > > Message: > > Logged In: YES > > user_id=14845 > > > > Have you timed this? My experiments with this approach > > showed a substantional slowdown as noted earlier. > > > > ---------------------------------------------------------------------- > > > > Comment By: Tim Golden (luckynh) > > Date: 2003-10-27 22:34 > > > > Message: > > Logged In: YES > > user_id=704330 > > > > use a custom decorator: > > > > /* > > * <p>Simple Date Format</p> > > * @author luckynh > > * @version 1.0 $Revision: .0 $ ($Author: luckynh $) > > */ > > use a custom decorator: > > > > import java.text.SimpleDateFormat; > > import java.util.Date; > > import org.apache.taglibs.display.ColumnDecorator; > > > > public class ShortDateWrapper extends ColumnDecorator > > { > > private SimpleDateFormat sdf = new SimpleDateFormat( > > "MM/dd/yyyy" ); > > > > public String decorate( Object columnValue ) > > { > > Date t = (Date)columnValue; > > return sdf.format( t ); > > } > > } > > > > > > ---------------------------------------------------------------------- > > > > Comment By: Thorbjoern Ravn Andersen (ravn) > > Date: 2003-08-11 14:38 > > > > Message: > > Logged In: YES > > user_id=14845 > > > > I have now taken the time to merge the changes with the > > latest source from CVS, and submit patch 786617. > > > > > > ---------------------------------------------------------------------- > > > > Comment By: Thorbjoern Ravn Andersen (ravn) > > Date: 2003-08-07 13:37 > > > > Message: > > Logged In: YES > > user_id=14845 > > > > I have now tried to upgrade my old source tree to be in > > synch with the SF CVS, but Eclipse has some problems with > > this. When this is resolved, I will submit my patch which > > is in production use here. > > > > ---------------------------------------------------------------------- > > > > Comment By: Thorbjoern Ravn Andersen (ravn) > > Date: 2003-03-13 22:11 > > > > Message: > > Logged In: YES > > user_id=14845 > > > > I have found that the Wrapper system is noticably slower > > than just overriding the toString method. Unfortunately, I > > cannot do this in our scenario (objects come from JDBC). > > > > I hope to have time to prepare the patch this week. > > > > > > > > ---------------------------------------------------------------------- > > > > Comment By: Didier Dubois (didier_dubois) > > Date: 2003-03-06 10:41 > > > > Message: > > Logged In: YES > > user_id=726985 > > > > I resolved the problem doing the following step: > > - using a patch similar to the one posted by scottnsmith. > > This allow null Date comparison > > - Using a bean returning a "Date" object (which is correctly > > compared by Java) > > - Using a DateWrapper for this column that returns the > > formated Date as a String. > > > > HTH. > > > > ---------------------------------------------------------------------- > > > > Comment By: Didier Dubois (didier_dubois) > > Date: 2003-03-05 17:29 > > > > Message: > > Logged In: YES > > user_id=726985 > > > > YES! > > > > I have the same problem. But further more I need to sort my > > table on this date. Do your pathc support this? > > > > Didier > > > > ---------------------------------------------------------------------- > > > > Comment By: Matt Raible (mraible) > > Date: 2003-02-14 14:51 > > > > Message: > > Logged In: YES > > user_id=226669 > > > > I've subclassed Date and overridden the toString() method as > > a workaround. I agreet that adding a format attribute is > > probably a better solution. Please submit your patch. > > > > ---------------------------------------------------------------------- > > > > You can respond by visiting: > > > https://sourceforge.net/tracker/?func=detail&atid=536616&aid=68398 > 9&group_id > =73068 > > > > > > ------------------------------------------------------- > > This SF.net email is sponsored by: SF.net Giveback Program. > > Does SourceForge.net help you be more productive? Does it > > help you create better code? SHARE THE LOVE, and help us help > > YOU! Click Here: http://sourceforge.net/donate/ > > _______________________________________________ > > displaytag-devel mailing list > > [EMAIL PROTECTED] > > https://lists.sourceforge.net/lists/listinfo/displaytag-devel > > > > > > -- > John York > Software Engineer > CareerSite Corporation > > > > ------------------------------------------------------- > This SF.net email is sponsored by: SF.net Giveback Program. > Does SourceForge.net help you be more productive? Does it > help you create better code? SHARE THE LOVE, and help us help > YOU! Click Here: http://sourceforge.net/donate/ > _______________________________________________ > displaytag-devel mailing list > [EMAIL PROTECTED] > https://lists.sourceforge.net/lists/listinfo/displaytag-devel > > > ************************************** > Axios Email Confidentiality Footer > Privileged/Confidential Information may be contained in this > message. If you are not the addressee indicated in this message > (or responsible for delivery of the message to such person), you > may not copy or deliver this message to anyone. In such case, you > should destroy this message, and notify us immediately. If you or > your employer does not consent to Internet email messages of this > kind, please advise us immediately. Opinions, conclusions and > other information expressed in this message are not given or > endorsed by my Company or employer unless otherwise indicated by > an authorised representative independent of this message. > WARNING: > While Axios Systems Ltd takes steps to prevent computer viruses > from being transmitted via electronic mail attachments we cannot > guarantee that attachments do not contain computer virus code. > You are therefore strongly advised to undertake anti virus checks > prior to accessing the attachment to this electronic mail. Axios > Systems Ltd grants no warranties regarding performance use or > quality of any attachment and undertakes no liability for loss or > damage howsoever caused. > > > > ------------------------------------------------------- > This SF.net email is sponsored by: SF.net Giveback Program. > Does SourceForge.net help you be more productive? Does it > help you create better code? SHARE THE LOVE, and help us help > YOU! Click Here: http://sourceforge.net/donate/ > _______________________________________________ > displaytag-devel mailing list > [EMAIL PROTECTED] > https://lists.sourceforge.net/lists/listinfo/displaytag-devel ------------------------------------------------------- This SF.net email is sponsored by: SF.net Giveback Program. Does SourceForge.net help you be more productive? Does it help you create better code? SHARE THE LOVE, and help us help YOU! Click Here: http://sourceforge.net/donate/ _______________________________________________ displaytag-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/displaytag-devel ------------------------------------------------------- This SF.net email is sponsored by: SF.net Giveback Program. Does SourceForge.net help you be more productive? Does it help you create better code? SHARE THE LOVE, and help us help YOU! Click Here: http://sourceforge.net/donate/ _______________________________________________ displaytag-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/displaytag-devel
