? Pending

User tests: Successful: Unsuccessful:

avatar joomdonation
joomdonation
21 Apr 2017

Pull Request for Issue # .

Summary of Changes

For backend component model classes which extends JModelList, the model states are populated automatically in populateState method of JModelList.

However, we still had to set model states manually in every model lists class to support hathor template. Now, that template is removed in Joomla 4, we can remove these code

Testing Instructions

There are some components affected by this change. Since change is the same for all components, you can pick one component to test (For example.com_content).

  1. Install Joomla 4 (with sample data for easier testing)
  2. Access to Articles management screen
  3. Use search tool (enter keyword to search, select a category...), see the list of articles changed and filtered properly
  4. Navigate to a different component (for example, cpanel of administrator area)
  5. Access to Articles management screen screen again, make sure the data you entered in search tool (keyword, selected category...) still keep it's previous value.
avatar joomdonation joomdonation - open - 21 Apr 2017
avatar joomdonation joomdonation - change - 21 Apr 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 Apr 2017
Category Administration com_banners com_categories com_contact com_content com_languages com_messages com_newsfeeds com_plugins com_redirect com_search com_tags com_users
avatar joomdonation joomdonation - change - 21 Apr 2017
Title
Kill hathor support code in list models
[4.0]Kill hathor support code in list models
avatar joomdonation joomdonation - edited - 21 Apr 2017
avatar wilsonge
wilsonge - comment - 29 Apr 2017

Can you fix conflicts please?

avatar joomdonation joomdonation - change - 29 Apr 2017
Labels Added: ?
avatar joomdonation
joomdonation - comment - 29 Apr 2017

@wilsonge Done. CS error is not related to this change and the error message is not very clear to me, so I don't fix it in this PR.

avatar wilsonge
wilsonge - comment - 29 Apr 2017

Thanks. I've just fixed the CS issue in the 4.0 branch :)

avatar joomdonation
joomdonation - comment - 29 Apr 2017

Yes, good now !

avatar wilsonge wilsonge - close - 29 Apr 2017
avatar wilsonge wilsonge - merge - 29 Apr 2017
avatar wilsonge wilsonge - change - 29 Apr 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-04-29 14:34:26
Closed_By wilsonge

Add a Comment

Login with GitHub to post a comment