RTC PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar Hackwar
Hackwar
3 Aug 2025

Summary of Changes

DatabaseQuery::castAsChar() has been deprecated in version 2 of the Joomla database framework package for removal in version 3. We missed that at that time and will now do that in version 4 of the framework. This PR converts all calls to the correct method DatabaseQuery::castAs() with the parameter CHAR instead. The update to version 4 of the framework will come in Joomla 6, but we want to have this change as early as possible.

Testing Instructions

Codereview

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 - 3 Aug 2025
avatar Hackwar Hackwar - change - 3 Aug 2025
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Aug 2025
Category Administration com_finder Front End com_contact com_content Libraries Plugins
avatar muhme muhme - change - 4 Aug 2025
Labels Added: PR-5.4-dev
avatar richard67 richard67 - test_item - 4 Aug 2025 - Tested successfully
avatar richard67
richard67 - comment - 4 Aug 2025

I have tested this item ✅ successfully on 629a6b4

Tested by review.

The PR does like it is documented here: https://github.com/joomla-framework/database/blob/3.x-dev/src/DatabaseQuery.php#L574

It catches all occurrences of castAsChar calls of the database query.


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

avatar chmst chmst - test_item - 4 Aug 2025 - Tested successfully
avatar chmst
chmst - comment - 4 Aug 2025

I have tested this item ✅ successfully on 629a6b4

Tested by review and applied the changes to one of my applications also with own components. No error found.


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

avatar richard67 richard67 - change - 4 Aug 2025
Status Pending Ready to Commit
avatar richard67
richard67 - comment - 4 Aug 2025

RTC


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

avatar muhme
muhme - comment - 4 Aug 2025

✅ Final check before merge:

  • Checked 20 places substituted, nothing else changed
  • Followed @richard67 advice to check all places are found with:
    find . -type f -name "*\.php" -exec grep -Hn "\->castAsChar(" {} \;
avatar muhme muhme - change - 4 Aug 2025
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2025-08-04 18:06:06
Closed_By muhme
Labels Added: RTC
avatar muhme muhme - close - 4 Aug 2025
avatar muhme muhme - merge - 4 Aug 2025
avatar muhme
muhme - comment - 4 Aug 2025

Thank you @Hackwar for your contribution. Thank you @richard67 and @chmst for testing.

avatar obuisard
obuisard - comment - 4 Sep 2025

Hello Hannes @Hackwar,
great catch about castAsChar, the removal was long overdue, but since it is code that has been removed, shouldn't that be documented? Many extensions still use it and that should be mentioned in the manual.

avatar richard67
richard67 - comment - 5 Sep 2025

There is some documentation about it in the framework where it actually has been removed: https://github.com/joomla-framework/database/blob/4.x-dev/docs/v3-to-v4-update.md

avatar obuisard
obuisard - comment - 5 Sep 2025

There is some documentation about it in the framework where it actually has been removed: https://github.com/joomla-framework/database/blob/4.x-dev/docs/v3-to-v4-update.md

Thank you Richard! But should these changes be documented in the general Joomla 5.4 to 6.0 manual as well? Since it is affecting Joomla 6. Just wondering.

Add a Comment

Login with GitHub to post a comment