Unit/System Tests b/c break PR-6.0-dev Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
14 Feb 2025

Summary of Changes

The LegacyPropertyManagementTrait has been deprecated for quite some time and will most likely be removed in 7.0. This removes its usage from the base model and view classes of the CMS. This might look like quite a big break in b/c, but if you followed Joomla core code since Joomla 2.5, it isn't actually that big. We've not been using it anywhere really, except for the ->get() method in the views, which calls the methods from the models. However this has always been an overwritten method in the view anyway and it also still exists after this change. This means that all those old view classes will still work like before. The code from the trait has been deprecated since Joomla 3.4.0, even though it has been moved around and with Joomla 4.0 has finally ended up in the legacy trait. From my perspective, it is time to do this after 11 years of it being deprecated: https://github.com/joomla/joomla-cms/blob/3.4.0/libraries/joomla/object/object.php

Testing Instructions

Codereview.

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: joomla/Manual#486

  • No documentation changes for manual.joomla.org needed

avatar Hackwar Hackwar - open - 14 Feb 2025
avatar Hackwar Hackwar - change - 14 Feb 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 14 Feb 2025
Category Libraries
avatar Hackwar Hackwar - change - 14 Feb 2025
The description was changed
avatar Hackwar Hackwar - edited - 14 Feb 2025
avatar Hackwar Hackwar - change - 19 Feb 2025
Labels Added: b/c break PR-6.0-dev
avatar joomla-cms-bot joomla-cms-bot - change - 19 Feb 2025
Category Libraries Libraries Unit Tests
avatar Hackwar Hackwar - change - 19 Feb 2025
Labels Added: Unit/System Tests
avatar heelc29
heelc29 - comment - 11 Jun 2025

tests failing because its still used here

$view->set('component_name', $this->input->get('component_name'));

avatar Hackwar
Hackwar - comment - 11 Jun 2025

Thanks, I removed that now.

avatar Hackwar Hackwar - change - 23 Jun 2025
The description was changed
avatar Hackwar Hackwar - edited - 23 Jun 2025
avatar Hackwar
Hackwar - comment - 23 Jun 2025

Now that the documentation is also done, this one is ready for testing.

avatar vishalchincholi1 vishalchincholi1 - test_item - 30 Jan 2026 - Tested successfully
avatar vishalchincholi1
vishalchincholi1 - comment - 30 Jan 2026

I have tested this item ✅ successfully on 183ed38

A final search confirmed that LegacyPropertyManagementTrait is no longer used in the libraries/src/MVC directory.


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

avatar brianteeman
brianteeman - comment - 30 Jan 2026

@vishalchincholi1 how did you test this? Reading the code is not enough

Add a Comment

Login with GitHub to post a comment