PR-5.4-dev Pending

User tests: Successful: Unsuccessful:

avatar railsonrdr
railsonrdr
18 Jul 2026

[5.4] Fix: Default format to 'html' to prevent 500 error when format is empty

Pull Request resolves # .

  • 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

Check the $format variable to prevent a 500 error.

Testing Instructions

In Joomla, accessing these links without this fix will return a 500 error, which can be interpreted as a SQLi vulnerability by Security Check Tools. Example links:

Actual result BEFORE applying this Pull Request

A 500 error is returned.

Expected result AFTER applying this Pull Request

We no longer get "false positives" for SQLi in Security Check Tools and fewer error logs from SQLi tests.

Link to documentations

Please select:

  • Documentation link for guide.joomla.org:

  • [ X ] No documentation changes for guide.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • [ X ] No documentation changes for manual.joomla.org needed

avatar railsonrdr railsonrdr - open - 18 Jul 2026
avatar railsonrdr railsonrdr - change - 18 Jul 2026
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Jul 2026
Category Libraries
avatar railsonrdr railsonrdr - change - 18 Jul 2026
The description was changed
avatar railsonrdr railsonrdr - edited - 18 Jul 2026
avatar railsonrdr railsonrdr - change - 18 Jul 2026
The description was changed
avatar railsonrdr railsonrdr - edited - 18 Jul 2026
avatar railsonrdr railsonrdr - change - 18 Jul 2026
The description was changed
avatar railsonrdr railsonrdr - edited - 18 Jul 2026
avatar Fedik
Fedik - comment - 19 Jul 2026

The error is correct when you explicitly provide an empty format URL variable.
There is nothing to fix.

avatar Fedik
Fedik - comment - 19 Jul 2026

The error is correct when you explicitly providing an empty format URL variable.
There is nothing to fix.

avatar Fedik
Fedik - comment - 19 Jul 2026

The error is correct when you explicitly providing an empty or non alphanumeric format URL variable.
There is nothing to fix.

avatar HLeithner HLeithner - change - 19 Jul 2026
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2026-07-19 19:56:48
Closed_By HLeithner
Labels Added: PR-5.4-dev
avatar HLeithner HLeithner - close - 19 Jul 2026

Add a Comment

Login with GitHub to post a comment