? Pending

User tests: Successful: Unsuccessful:

avatar chmst
chmst
27 Jan 2020

Summary of Changes

List Views have a method getSortFields().
This Method is not used in the core. The names of the table headers are in the respective tables as language keys.

Testing Instructions

Code Inspect. Testing this can only be playing around with table headers in list views.
Are all texts displayed corretly, does sorting work as before? Also with open search tools?

Expected result

Everything works as before

Note

There are several similiar PRs, hopefully reducin the risk of conflicts

avatar chmst chmst - open - 27 Jan 2020
avatar chmst chmst - change - 27 Jan 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 27 Jan 2020
Category Administration com_menus com_modules com_users
avatar astridx
astridx - comment - 6 Feb 2020

I tested this successful.

  1. git fetch origin pull/27677/head:remove-getsortfields-users-menus-modules

  2. git checkout remove-getsortfields-users-menus-modules

  3. I have often wondered what I need these methode for. The sort fields are inserted with an XML file. That is why I first checked if this XML-File is for every component in this PR on his place.
    The files
    /administrator/components/com_menus/forms/filter_*
    /administrator/components/com_modules/forms/filter_*
    /administrator/components/com_users/forms/filter_*
    are there.

  4. After that I checked that I could sort users, menus items and admin/site modules.

avatar astridx astridx - test_item - 6 Feb 2020 - Tested successfully
avatar astridx
astridx - comment - 6 Feb 2020

I have tested this item successfully on b74484f


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

avatar Quy Quy - test_item - 6 Feb 2020 - Tested successfully
avatar Quy
Quy - comment - 6 Feb 2020

I have tested this item successfully on b74484f


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

avatar Quy Quy - change - 6 Feb 2020
Status Pending Ready to Commit
avatar Quy
Quy - comment - 6 Feb 2020

RTC


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

avatar chmst
chmst - comment - 6 Feb 2020

@Quy , @astridx thank you for testing

avatar rdeutz rdeutz - close - 6 Feb 2020
avatar rdeutz rdeutz - merge - 6 Feb 2020
avatar rdeutz rdeutz - change - 6 Feb 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-02-06 22:50:04
Closed_By rdeutz
Labels Added: ?

Add a Comment

Login with GitHub to post a comment