Test instructions missing Information Required PR-5.3-dev Pending

User tests: Successful: Unsuccessful:

avatar mgscreativa
mgscreativa
17 Jul 2025

Pull Request for Issue # .

Summary of Changes

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 mgscreativa mgscreativa - open - 17 Jul 2025
avatar mgscreativa mgscreativa - change - 17 Jul 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Jul 2025
Category Modules Front End
avatar brianteeman
brianteeman - comment - 17 Jul 2025

This is not correct. The module class suffix is added by the module chrome. If you are not seeing this in your own template then you need to set the correct module chrome

avatar bembelimen bembelimen - change - 23 Jul 2025
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2025-07-23 09:29:37
Closed_By bembelimen
Labels Added: Test instructions missing Information Required PR-5.3-dev
avatar bembelimen bembelimen - close - 23 Jul 2025
avatar bembelimen
bembelimen - comment - 23 Jul 2025

Thank you for your Pull Request, really appreciate that you contributed.

As Brian said, the suffix should be managed by the chrome and was changed here by purpose:

#17447

So I will close this PR but I hope you don't feel discouraged by this and continue contributing.

Thank you for your effort!

avatar mgscreativa
mgscreativa - comment - 24 Jul 2025

Ok, so the $params->get('moduleclass_sfx should be at the chrome implementation?

avatar mgscreativa
mgscreativa - comment - 25 Jul 2025

Cool, thanks!

Add a Comment

Login with GitHub to post a comment