Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#8920] - Always use the API for inline scripts

? Success

User tests: Successful: Unsuccessful:

avatar dgt41
dgt41
17 Jan 2016

Don't echo script tags in the body!

All inline scripts should go through Joomla's API!

Testing

Apply the PR with patchtester
Go to administrator/index.php?option=com_modules
display the search tools
and try the dropdown for position, module name and language.
If the filtering works fine then you have a successful test

avatar dgt41 dgt41 - open - 17 Jan 2016
avatar dgt41 dgt41 - change - 17 Jan 2016
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Jan 2016
Labels Added: ?
avatar 810 810 - test_item - 17 Jan 2016 - Tested successfully
avatar 810
810 - comment - 17 Jan 2016

I have tested this item :white_check_mark: successfully on 861997d

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8920.

avatar waader waader - test_item - 17 Jan 2016 - Tested successfully
avatar waader
waader - comment - 17 Jan 2016

I have tested this item :white_check_mark: successfully on 861997d

Thanks!

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8920.

avatar joomla-cms-bot
joomla-cms-bot - comment - 17 Jan 2016

This PR has received new commits.

CC: @810, @waader

This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/8920.

avatar wilsonge wilsonge - change - 17 Jan 2016
Status Pending ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2016-01-17 20:51:13
Closed_By ⇒ wilsonge
avatar wilsonge wilsonge - close - 17 Jan 2016
avatar wilsonge wilsonge - reference | 26eac54 - 17 Jan 16
avatar wilsonge wilsonge - merge - 17 Jan 2016
avatar wilsonge wilsonge - close - 17 Jan 2016
avatar wilsonge
wilsonge - comment - 17 Jan 2016

Merged on review

avatar wilsonge wilsonge - change - 17 Jan 2016
Milestone Added:
avatar dgt41 dgt41 - head_ref_deleted - 17 Jan 2016

Add a Comment

Login with GitHub to post a comment