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.

Add a Comment

Login with GitHub to post a comment