Pending
User tests:
Successful:
Unsuccessful:
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
-
Enable JDEBUG in Joomla configuration.
-
Trigger actions that call:
- checkEditId()
- holdEditId()
- releaseEditId()
-
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
Mahesh-Gite-28
-
open
-
27 Apr 2026
Mahesh-Gite-28
-
change
-
27 Apr 2026
joomla-cms-bot
-
change
-
27 Apr 2026
Mahesh-Gite-28
-
change
-
27 Apr 2026
The description was changed
Mahesh-Gite-28
-
edited
-
27 Apr 2026
Add a Comment
Login with GitHub to post a comment