? Pending

User tests: Successful: Unsuccessful:

avatar SniperSister
SniperSister
25 Oct 2020

Summary of Changes

Added a warning note about potential security implications to the core.js Joomla.renderMessages docblock.

Testing Instructions

Core Review

avatar SniperSister SniperSister - open - 25 Oct 2020
avatar SniperSister SniperSister - change - 25 Oct 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Oct 2020
Category JavaScript
avatar SniperSister
SniperSister - comment - 25 Oct 2020
avatar SniperSister SniperSister - change - 25 Oct 2020
Title
Added warning note to core.js render messages docblock
[3.x] Added warning note to core.js render messages docblock
avatar SniperSister SniperSister - edited - 25 Oct 2020
avatar dgrammatiko
dgrammatiko - comment - 25 Oct 2020

I can refactor the J4 messaging code to eliminate the innerHTML and I'm pretty sure I can do it without breaking b/c in the PHP side but the JS endpoint, Joomla.renderMessages() will not be backwards compatible ?‍♂️

avatar HLeithner HLeithner - close - 27 Feb 2021
avatar HLeithner HLeithner - merge - 27 Feb 2021
avatar HLeithner HLeithner - change - 27 Feb 2021
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-02-27 15:38:29
Closed_By HLeithner
Labels Added: ?

Add a Comment

Login with GitHub to post a comment