[#43230] - [5.2] Remove legacy code for filtering state in backend models
- Fixed in Code Base
- 8 Aug 2024
- Medium
- Build: 5.2-dev
- # 43230
- Diff
- Hackwar:5.2-component-filters
User tests: Successful: Unsuccessful:
Summary of Changes
Quite some time ago, we introduced code to read the filter state from the filter XML for list views in the backend, making all those setState() code unnecessary. This removes all that unnecessary code.
Testing Instructions
Test if the filters in the list views still work for all the components.
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_associations com_cache com_content com_finder com_installer com_languages com_menus com_modules com_templates com_users |
| Labels |
Added:
PR-5.2-dev
|
||
| Labels |
Added:
Feature
|
||
I have tested this item ✅ successfully on 41d68aa
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43230.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
RTC
|
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2024-08-08 19:19:42 |
| Closed_By | ⇒ | pe7er |
This needs to be reverted. As well as the language overrides not working any more the update sites check and list of update sites also does NOT work
To be clear if you clear all site cookies and session data and then log back in then at least these views are empty.
I have tested this item ✅ successfully on 41d68aa
As far as I can tell removing this code has no noticable impact. Tested by checking various filters and by adding the filter to a url.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/43230.