Pending

User tests: Successful: Unsuccessful:

avatar Mahesh-Gite-28
Mahesh-Gite-28
27 Apr 2026

Summary of Changes

Replace str_replace("\n", ' ', print_r($values, 1)) with
json_encode($values, JSON_UNESCAPED_SLASHES) in three JDEBUG-only
debug log statements inside BaseController.

Affected methods

  • checkEditId()
  • holdEditId()
  • releaseEditId()

Testing Instructions

  1. Enable JDEBUG in Joomla configuration.

  2. Trigger actions that call:

    • checkEditId()
    • holdEditId()
    • releaseEditId()
  3. Verify log output.

Ensure that:

  • Logs are generated correctly
  • Output is in JSON format
  • No functional behavior is affected

Actual result BEFORE applying this Pull Request

Debug logs used print_r() combined with str_replace(),
producing verbose and less readable output with unnecessary processing overhead.

Expected result AFTER applying this Pull Request

Debug logs use json_encode(), producing cleaner, compact, and
more efficient single-line output without extra processing.

Link to documentations

  • No documentation changes for guide.joomla.org needed
  • No documentation changes for manual.joomla.org needed
avatar Mahesh-Gite-28 Mahesh-Gite-28 - open - 27 Apr 2026
avatar Mahesh-Gite-28 Mahesh-Gite-28 - change - 27 Apr 2026
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 27 Apr 2026
Category Libraries
avatar Mahesh-Gite-28 Mahesh-Gite-28 - change - 27 Apr 2026
The description was changed
avatar Mahesh-Gite-28 Mahesh-Gite-28 - edited - 27 Apr 2026

Add a Comment

Login with GitHub to post a comment