? ? Pending

User tests: Successful: Unsuccessful:

avatar chmst
chmst
27 Jan 2020

Summary of Changes

List Views have a method getSortFields() - here in com_content.
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_content com_workflow
avatar astridx
astridx - comment - 6 Feb 2020

I tested this PR successful

  1. git fetch origin pull/27679/head:remove-getsortfields-content

  2. git checkout remove-getsortfields-content

  3. 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_content/forms/filter_articles.xml
    /administrator/components/com_content/forms/filter_featured.xml
    /administrator/components/com_workflow/forms/filter_stages.xml
    /administrator/components/com_workflow/forms/filter_transitions.xml
    /administrator/components/com_workflow/forms/filter_workflows.xml
    are there.

  4. After that I checked that I could sort articles, categories and featured articles
    I had no Workflow items. But the drop down was there and worked.

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

I have tested this item successfully on d183db8


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

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

I have tested this item successfully on d183db8


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

avatar Quy Quy - change - 6 Feb 2020
The description was changed
Status Pending Ready to Commit
avatar Quy Quy - edited - 6 Feb 2020
avatar Quy
Quy - comment - 6 Feb 2020

RTC


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

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:48:18
Closed_By rdeutz
Labels Added: ? ?
avatar rdeutz rdeutz - close - 6 Feb 2020
avatar rdeutz rdeutz - merge - 6 Feb 2020
avatar rdeutz
rdeutz - comment - 6 Feb 2020

Seems to me we can remove this entirely from joomla getSortFields is never used

Add a Comment

Login with GitHub to post a comment