Just curious since nobody has commented on this patch, what are everyones thoughts? This effectively eliminates the need or at least minimizes the use of at a minimum column decorators, and makes it very quick/easy to roll stuff out.... I've been watching the user list and there have been tons of instances where this patch could quickly/simply get people out and rolling. Thoughts? -David
----- Original Message ----- From: "David Erickson" <[EMAIL PROTECTED]> To: <[EMAIL PROTECTED]> Sent: Monday, May 31, 2004 10:00 AM Subject: Re: [displaytag-devel] ColumnTag Patch > I forgot to mention this also almost eliminates the need to create extra > decorator classes since you can just drop this inline, another use I had for > it is when one of my column attributes is a Date. It can sort based on the > milliseconds but I can display it in any different format I want by just > using the date-time tag library... the possiblities are great =) > -David > > ----- Original Message ----- > From: "David Erickson" <[EMAIL PROTECTED]> > To: "Displaytag-Devel List" <[EMAIL PROTECTED]> > Sent: Monday, May 31, 2004 9:54 AM > Subject: [displaytag-devel] ColumnTag Patch > > > > I wanted to submit this patch awhile ago but didnt have time, I know this > > isn't exactly in patch format, sorry =) But this is the doEndTag method > of > > column.tag, I created this patch awhile ago and compared it today to the > > latest snapshot, the only diff I noticed is the new snapshot does not > throw > > an exception if your missing attribute information on a column tag, so if > > that needs to be removed from my patch thats great. > > > > Anyway the idea behind this patch is to give more freedom to displaying > > information from the tag. For example I have a userName column that I > want > > to make a sortable property of my list, but at the same time I want it to > > link me to my area where I can edit that user. Well that becomes > difficult > > with the way things currently are set. So what this patch does is change > > the priority order of how the column gets displayed. Previously (forgive > > me its been a month or two) I belive the order was: Property Value then > Body > > Value then none. With my patch the order will go Body Content if > specified, > > Property Value, then none. This gives the user freedom do so such things > > as: > > > > <display-el:column property="userName" title="UserName" sortable="true" > > class="displayTag"> > > <a href="<html-el:rewrite > > forward="employee.support.web.userAdmin.editUser"/>?id=<bean:write > > name="user" property="id"/>" target="_top"><bean:write name="user" > > property="userName"/></a> > > </display-el:column> > > > > Meanwhile maintaining sorting ability on that column, etc. Which is > > immensly useful for me anyway =) > > > > I've attached the column.tag source, I just merged in that method with the > > latest snapshot. > > Hope this helps, and I hope to be able to have more time to contribute in > > the future. > > -David Erickson > > > > > > > > > > ------------------------------------------------------- > This SF.Net email is sponsored by: Oracle 10g > Get certified on the hottest thing ever to hit the market... Oracle 10g. > Take an Oracle 10g class now, and we'll give you the exam FREE. > http://ads.osdn.com/?ad_id=3149&alloc_id=8166&op=click > _______________________________________________ > displaytag-devel mailing list > [EMAIL PROTECTED] > https://lists.sourceforge.net/lists/listinfo/displaytag-devel > ------------------------------------------------------- This SF.Net email is sponsored by the new InstallShield X. >From Windows to Linux, servers to mobile, InstallShield X is the one installation-authoring solution that does it all. Learn more and evaluate today! http://www.installshield.com/Dev2Dev/0504 _______________________________________________ displaytag-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/displaytag-devel
