? Pending

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
12 May 2016

Pull Request for Issue #10399 .

Summary of Changes

Always use htmlspecialchars($str, ENT_COMPAT, 'UTF-8')

Testing Instructions

Please do a code review or test all changed files.

avatar zero-24 zero-24 - open - 12 May 2016
avatar zero-24 zero-24 - change - 12 May 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 May 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 12 May 2016
Category Code style
avatar brianteeman brianteeman - test_item - 12 May 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 12 May 2016

I have tested this item :white_check_mark: successfully on cdd4017

on review


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

avatar alikon alikon - test_item - 12 May 2016 - Tested successfully
avatar alikon
alikon - comment - 12 May 2016

I have tested this item :white_check_mark: successfully on cdd4017


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

avatar brianteeman brianteeman - change - 12 May 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 12 May 2016

RTC


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

avatar joomla-cms-bot joomla-cms-bot - change - 12 May 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 12 May 2016
Milestone Added:
avatar joomla-cms-bot
joomla-cms-bot - comment - 13 May 2016

This PR has received new commits.

CC: @alikon, @brianteeman


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

avatar zero-24
zero-24 - comment - 13 May 2016

Travis was failing on the sysinfo problem that was fixed upstream.

avatar andrepereiradasilva
andrepereiradasilva - comment - 13 May 2016

@brianteeman please remove the RTC on thsi one to.
same parentheses problems.

avatar wojsmol
wojsmol - comment - 14 May 2016
avatar joomla-cms-bot
joomla-cms-bot - comment - 14 May 2016

This PR has received new commits.

CC: @alikon, @brianteeman


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

avatar zero-24 zero-24 - change - 14 May 2016
Status Ready to Commit Pending
avatar andrepereiradasilva
andrepereiradasilva - comment - 14 May 2016

seems fine to me now.

avatar zero-24
zero-24 - comment - 14 May 2016

Back to Pending. Thanks.


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

avatar joomla-cms-bot joomla-cms-bot - change - 14 May 2016
Labels Removed: ?
avatar joomla-cms-bot
joomla-cms-bot - comment - 14 May 2016

This PR has received new commits.

CC: @alikon, @brianteeman


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

avatar andrepereiradasilva andrepereiradasilva - test_item - 16 May 2016 - Tested successfully
avatar andrepereiradasilva
andrepereiradasilva - comment - 16 May 2016

I have tested this item :white_check_mark: successfully on 2f4ef51

on code review


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

avatar brianteeman brianteeman - test_item - 16 May 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 16 May 2016

I have tested this item :white_check_mark: successfully on 2f4ef51


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

avatar brianteeman brianteeman - change - 16 May 2016
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 16 May 2016

RTC


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

avatar joomla-cms-bot joomla-cms-bot - change - 16 May 2016
Labels Added: ?
avatar rdeutz rdeutz - change - 16 May 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-05-16 16:06:35
Closed_By rdeutz
avatar rdeutz rdeutz - close - 16 May 2016
avatar rdeutz rdeutz - merge - 16 May 2016
avatar joomla-cms-bot joomla-cms-bot - close - 16 May 2016
avatar joomla-cms-bot joomla-cms-bot - change - 16 May 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment