Skip to content

[6.2] Use cache controller factory to create cache in BaseController#47953

Merged
laoneo merged 3 commits into
joomla:6.2-devfrom
joomdonation:factory_getcache_controller
Jun 19, 2026
Merged

[6.2] Use cache controller factory to create cache in BaseController#47953
laoneo merged 3 commits into
joomla:6.2-devfrom
joomdonation:factory_getcache_controller

Conversation

@joomdonation

Copy link
Copy Markdown
Contributor

Pull Request resolves # .

  • I read the Generative AI policy and my contribution is either not created with the help of AI or is compatible with the policy and GNU/GPL 2 or later.

Summary of Changes

Currently, BaseController still uses the deprecated method Factory::getCache() to create cache. This PR update that code to use cache controller factory to create cache instead.

Testing Instructions

  • Use Joomla 6.2
  • Go to System -> Global Configuration, System tab, set System Cache to one of the two ON options
  • Access to frontend of your site, view an article for example, and make sure it is still works like before

Actual result BEFORE applying this Pull Request

Works, but use deprecated method to cache view output

Expected result AFTER applying this Pull Request

Works, without using deprecated method

Link to documentations

Please select:

  • Documentation link for guide.joomla.org:

  • No documentation changes for guide.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@laoneo laoneo added the Architecture Pull requests which improve the architecture. label Jun 18, 2026
@laoneo laoneo merged commit cf39800 into joomla:6.2-dev Jun 19, 2026
55 of 56 checks passed
@laoneo

laoneo commented Jun 19, 2026

Copy link
Copy Markdown
Member

Thanks!

@laoneo laoneo added this to the Joomla! 6.2.0 milestone Jun 19, 2026
@joomdonation joomdonation deleted the factory_getcache_controller branch June 19, 2026 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Architecture Pull requests which improve the architecture. PR-6.2-dev

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants