RTC bug PR-6.0-dev Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
6 Aug 2025

Summary of Changes

PHPStan has an issue understanding the $this class in our layouts with shortened class names. This PR removes the imports and instead uses the full class name when hinting at the type of $this. This allows to reduce the baseline file considerably.

Testing Instructions

Since this is a change in docblocks, there isn't really anything to test. The passing of phpstan would be the indicator that this is working correctly.

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 Administration com_associations com_joomlaupdate com_users Front End Plugins
avatar Hackwar Hackwar - change - 6 Aug 2025
Labels Added: PR-6.0-dev
avatar heelc29 heelc29 - test_item - 6 Aug 2025 - Tested successfully
avatar heelc29
heelc29 - comment - 6 Aug 2025

I have tested this item ✅ successfully on 06a678d

code review


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

avatar laoneo
laoneo - comment - 6 Aug 2025

Had the same issue with my extension, this is the only valid fix for now.

avatar hans2103 hans2103 - test_item - 3 Sep 2025 - Tested successfully
avatar hans2103
hans2103 - comment - 3 Sep 2025

I have tested this item ✅ successfully on 06a678d


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

avatar richard67 richard67 - change - 6 Sep 2025
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 6 Sep 2025

RTC


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

avatar richard67 richard67 - change - 6 Sep 2025
Status Ready to Commit Pending
avatar richard67
richard67 - comment - 6 Sep 2025

Back to pending as the PR has a conflicting file.


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

avatar richard67
richard67 - comment - 6 Sep 2025

@Hackwar Could you re-create the phpstan baseline file? It has conflicts due to changes in the base branch (6.0-dev).

avatar Hackwar Hackwar - change - 7 Sep 2025
Labels Added: bug
avatar richard67 richard67 - change - 7 Sep 2025
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 7 Sep 2025

RTC


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

avatar Bodge-IT Bodge-IT - change - 10 Sep 2025
Labels Added: RTC
avatar Bodge-IT Bodge-IT - change - 10 Sep 2025
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-09-10 18:18:23
Closed_By Bodge-IT
avatar Bodge-IT Bodge-IT - close - 10 Sep 2025
avatar Bodge-IT Bodge-IT - merge - 10 Sep 2025
avatar Bodge-IT
Bodge-IT - comment - 10 Sep 2025

Thanks @Hackwar for the work, @hans2103 and @heelc29 for the tests and @laoneo for review and @richard67 for the housekeeping

Add a Comment

Login with GitHub to post a comment