Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#45860] - [6.0] Remove static methods from frontend modules

PR-6.0-dev Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
6 Aug 2025

Summary of Changes

This PR removes static methods from the helpers of the frontend modules.

Testing Instructions

Actual result BEFORE applying this Pull Request

Expected result AFTER applying this Pull Request

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

avatar Hackwar Hackwar - open - 6 Aug 2025
avatar Hackwar Hackwar - change - 6 Aug 2025
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 6 Aug 2025
Category ⇒ Modules Front End
avatar Hackwar Hackwar - change - 6 Aug 2025
Labels Added: PR-6.0-dev
avatar hans2103 hans2103 - test_item - 3 Sep 2025 - Tested successfully
avatar hans2103
hans2103 - comment - 3 Sep 2025

I have tested this item ✅ successfully on 2b27aee

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

avatar ceford ceford - test_item - 17 Sep 2025 - Tested successfully
avatar ceford
ceford - comment - 17 Sep 2025

I have tested this item ✅ successfully on 2b27aee

Code review: I see no calls to ::getList() in J6 and I see that some of these items will be removed in 6 and others in 7. Looks OK in my limited experience!

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

avatar richard67
richard67 - comment - 18 Sep 2025

@Hackwar Any reasons why this PR is in draft mode? It has received 2 successful human tests meanwhile.

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

I have tested this item 🔴 unsuccessfully on 2b27aee

Getting this error on locally while applying the patch
The file marked for modification does not exist: phpstan-baseline.neon

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

avatar brianteeman
brianteeman - comment - 30 Jan 2026

@vishalchincholi1 if you try to test this PR via Joomla Patch Tester even with the latest Nightly Build, you will get the error: The file marked for modification does not exist: phpstan-baseline.neon Use the Download package instead :)

avatar brianteeman brianteeman - alter_testresult - 30 Jan 2026 - vishalchincholi1: Not tested
avatar brianteeman
brianteeman - comment - 30 Jan 2026

I have removed the unsuccessful test from @vishalchincholi1 as they had not tested the pr

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

avatar RickR2H RickR2H - change - 31 Jan 2026
Status Pending ⇒ Ready to Commit
avatar RickR2H
RickR2H - comment - 31 Jan 2026

RTC

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

avatar brianteeman
brianteeman - comment - 31 Jan 2026

@RickR2H please remove the RTC label. This PR is still in draft mode and it has conflicts that need to be resolved

avatar RickR2H RickR2H - change - 31 Jan 2026
Status Ready to Commit ⇒ Pending
avatar RickR2H
RickR2H - comment - 31 Jan 2026

Back to draft

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

Add a Comment

Login with GitHub to post a comment