? 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 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 f346618

on review


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

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

I have tested this item :white_check_mark: successfully on f346618


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

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

RTC


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

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

We always ignore all your comments ;)

Cant wait to share a beer or three next week

On 13 May 2016 at 21:55, andrepereiradasilva notifications@github.com
wrote:

ok ignore my last comment the travis errors as nothing to do with your PR.

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#10440 (comment)

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

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/10440.

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

I have fixed the codestyle issue. I hope travis is happy than ;)

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:21:07
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