Skip to content

Conversation

@brianteeman
Copy link
Contributor

@brianteeman brianteeman commented Nov 19, 2025

Pull Request for Issue # .

Summary of Changes

Removes the last use of the deprecated Joomla.JText

Testing Instructions

code review

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

Signed-off-by: BrianTeeman <brian@teeman.net>
@richard67 richard67 mentioned this pull request Nov 19, 2025
4 tasks
@hans2103
Copy link
Contributor

I have tested this item ✅ successfully on 2f7f1a1


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

@laoneo
Copy link
Member

laoneo commented Nov 19, 2025

I have tested this item ✅ successfully on 2f7f1a1

Code review


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

@laoneo laoneo removed the PR-5.4-dev label Nov 20, 2025
@laoneo
Copy link
Member

laoneo commented Nov 20, 2025

rtc


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Nov 20, 2025
@richard67 richard67 added this to the Joomla! 5.4.2 milestone Nov 20, 2025
@muhme
Copy link
Contributor

muhme commented Nov 26, 2025

✅ Final check before merge find . -name \*.php | xargs grep JText

  • The last two uses before this PR were found.
  • After this PR, only to be found in classmap file.

@muhme muhme merged commit 99ea2e7 into joomla:5.4-dev Nov 26, 2025
60 checks passed
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Nov 26, 2025
@muhme
Copy link
Contributor

muhme commented Nov 26, 2025

Thank you @brianteeman for your contribution. Thank you @hans2103 and @laoneo for testing.

@brianteeman
Copy link
Contributor Author

Thanks

@brianteeman brianteeman deleted the jtext54 branch November 26, 2025 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants