? ? Pending

User tests: Successful: Unsuccessful:

avatar laoneo
laoneo
5 May 2017

Summary of Changes

The grid sort JS function supports to pass a form attribute. The JHTML wrapper JHtml::_('grid.sort'); should support that functionality.

This pr doesn't change an existing behavior, it just adds a new option, so the results of the test should be the same.

Testing Instructions

  • Create some articles.
  • Create an "Articles -> Category List" menu item
  • On the front open the menu item
  • Sort by the title column

Expected result

The list is sorted by the title.

Actual result

The list is sorted by the title.

avatar laoneo laoneo - open - 5 May 2017
avatar laoneo laoneo - change - 5 May 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 5 May 2017
Category Front End com_content Libraries
avatar laoneo laoneo - change - 5 May 2017
The description was changed
avatar laoneo laoneo - edited - 5 May 2017
avatar laoneo laoneo - change - 5 May 2017
The description was changed
avatar laoneo laoneo - edited - 5 May 2017
avatar RonakParmar
RonakParmar - comment - 5 May 2017

I have tested this item successfully on 9eefa0c

Before and after this PR title sorting is working fine.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15826.

avatar RonakParmar RonakParmar - test_item - 5 May 2017 - Tested successfully
avatar ciar4n
ciar4n - comment - 10 May 2017

I have tested this item successfully on 9eefa0c


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/15826.

avatar ciar4n ciar4n - test_item - 10 May 2017 - Tested successfully
avatar franz-wohlkoenig franz-wohlkoenig - change - 10 May 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 10 May 2017

RTC after two successful tests.

avatar rdeutz rdeutz - change - 22 May 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-05-22 18:36:25
Closed_By rdeutz
Labels Added: ? ?
avatar rdeutz rdeutz - close - 22 May 2017
avatar rdeutz rdeutz - merge - 22 May 2017

Add a Comment

Login with GitHub to post a comment