[
http://jira.codehaus.org/browse/DISPL-586?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=163596#action_163596
]
Alvaro Villarroel commented on DISPL-586:
-----------------------------------------
I modify these lines in order to have de keepStatus work with sort="external" :
The original code is:
if (!this.tableModel.isLocalSort())
{
// our sort column parameter may be a string, check that first
String sortColumnName =
requestHelper.getParameter(encodeParameter(TableTagParameters.PARAMETER_SORT));
// if usename is not null, sortColumnName is the name, if not is
the column index
String usename =
requestHelper.getParameter(encodeParameter(TableTagParameters.PARAMETER_SORTUSINGNAME));
The modificated code is:
if (!this.tableModel.isLocalSort())
{
// our sort column parameter may be a string, check that first
String sortColumnName = getFromRequestOrSession(request,
requestHelper, TableTagParameters.PARAMETER_SORT);
// if usename is not null, sortColumnName is the name, if not is
the column index
String usename = getFromRequestOrSession(request, requestHelper,
TableTagParameters.PARAMETER_SORTUSINGNAME);
So I just used "getFromRequestOrSession" method instead of
"requestHelper.getParameter" for external sorting
> keepStatus not working properly with external sorting and paging
> ----------------------------------------------------------------
>
> Key: DISPL-586
> URL: http://jira.codehaus.org/browse/DISPL-586
> Project: DisplayTag
> Issue Type: Bug
> Components: Paging/Sorting
> Affects Versions: 1.2
> Reporter: Aurora Zaharia
>
> I'm using sort="external" and partialList="true" and keepStatus="true". The
> page number and the sort direction are stored in the session, but the sort
> column not is not which makes this 1.2 feature unusable with external sorting
> and paging.
--
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:
SourcForge Community
SourceForge wants to tell your story.
http://p.sf.net/sfu/sf-spreadtheword
_______________________________________________
displaytag-devel mailing list
displaytag-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/displaytag-devel
