RTC bug PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
8 Aug 2026

Pull Request resolves #48094 .

  • 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

resolve deprecated message on cassiopeia offline
Using trim() instead because you're really checking whether the message contains anything meaningful, rather than specifically removing ordinary spaces. It also handles tabs and newlines.

Testing Instructions

  1. Enable php max reporting
  2. Set the site offline
  3. Offline message = Use custom message"
  4. Empty the field Custom Message
  5. Visit front end

Actual result BEFORE applying this Pull Request

image

Expected result AFTER applying this Pull Request

After applying the PR the deprecated message is not displayed

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

avatar brianteeman brianteeman - open - 8 Aug 2026
avatar brianteeman brianteeman - change - 8 Aug 2026
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Aug 2026
Category Front End Templates (site)
avatar brianteeman brianteeman - change - 8 Aug 2026
Labels Added: PR-5.4-dev
avatar richard67 richard67 - test_item - 8 Aug 2026 - Tested successfully
avatar richard67
richard67 - comment - 8 Aug 2026

I have tested this item ✅ successfully on 6dfafa9


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

avatar richard67
richard67 - comment - 8 Aug 2026

I have tested this item ✅ successfully on 6dfafa9


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

avatar SniperSister SniperSister - test_item - 8 Aug 2026 - Tested successfully
avatar SniperSister
SniperSister - comment - 8 Aug 2026

I have tested this item ✅ successfully on 6dfafa9


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

avatar SniperSister
SniperSister - comment - 8 Aug 2026

I have tested this item ✅ successfully on 6dfafa9


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

avatar richard67 richard67 - change - 8 Aug 2026
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 8 Aug 2026

RTC


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

avatar richard67
richard67 - comment - 8 Aug 2026

RTC


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

avatar brianteeman
brianteeman - comment - 8 Aug 2026

Thanks

avatar muhme muhme - change - 8 Aug 2026
Labels Added: RTC bug
avatar muhme
muhme - comment - 8 Aug 2026

✅ Final test before merge with JBT

  • Seen Deprecated: str_replace(): Passing null to parameter before PR
  • Applied PR with Patch Tester
  • Deprecation error for empty custom offline message is gone
  • Custom offline message is still working
  • Hide offline message is still working
  • Site language default message is still working
avatar muhme muhme - change - 8 Aug 2026
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2026-08-08 15:02:05
Closed_By muhme
avatar muhme muhme - close - 8 Aug 2026
avatar muhme muhme - merge - 8 Aug 2026
avatar muhme
muhme - comment - 8 Aug 2026

Thank you very much @brianteeman for your contribution. Thanks to @richard67 and @SniperSister for testing.

avatar brianteeman
brianteeman - comment - 8 Aug 2026

Thanks

Add a Comment

Login with GitHub to post a comment