[ 
http://jira.codehaus.org/browse/DISPL-484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_113340
 ] 

Kevin Conaway commented on DISPL-484:
-------------------------------------

Would the sortProperty attribute do what you're after?

> Ability to set sortName == columnName
> -------------------------------------
>
>                 Key: DISPL-484
>                 URL: http://jira.codehaus.org/browse/DISPL-484
>             Project: DisplayTag
>          Issue Type: Wish
>          Components: Paging/Sorting
>    Affects Versions: 1.1.1
>            Reporter: John Newman
>   Original Estimate: 1 hour, 30 minutes
>  Remaining Estimate: 1 hour, 30 minutes
>
> Hi,
> I am now using the external paging & sorting with some success.  I think we 
> need a good example of this in the docs - it is not very straightforward and 
> is poorly documented (understandable).  I can post something after I get this 
> all finalized.  
> Anyway, I don't understand the idea of having the sortName attribute.  Is 
> there ever a case where you'd want <d:column property="book.author.firstName" 
> sortName="somethingBesidesThat" /> over <d:column 
> property="book.author.firstName" sortName="book.author.firstName" /> ?  
> Really I can't figure that out, why not just go off of the property?  I'd 
> imagine everyone who has used that attribute just copy & pasted the property 
> attribute and changed it to say sortName.  If so this should be looked at but 
> maybe I am overlooking another (hopefully common) use case.
> I was able to quickly hack ColumnTag - in addHeaderToTable, I just said 
> headerCell.setSortName(this.property); instead of this.sortName.  Works fine 
> for me, but I think this deserves some consideration as I'd imagine this is 
> what 99% of everyone will want.  Yes? No?
> If everyone fully agrees, why not just nix the whole sortName thing and say 
> if sort="external" just put the property in the href?  If you'd like to keep 
> it the way it is, why not add a sortByProperty="true" attribute to table tag 
> to do what I hacked in.  I like solution #2, non intrusive, keeps backwards 
> compatibility, allows other sortNames if anyone actually does that, and will 
> save most everyone a lot of typing - I think it makes a lot of sense.  I 
> could provide a patch for either of these if given the go ahead.  

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
displaytag-devel mailing list
displaytag-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/displaytag-devel

Reply via email to