[
http://jira.codehaus.org/browse/DISPL-484?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_105636
]
John Newman commented on DISPL-484:
-----------------------------------
sorry the title was supposed to say Ability to set sortName == property
> 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
[email protected]
https://lists.sourceforge.net/lists/listinfo/displaytag-devel
