Skip to content

[tags & users & libs & beez] Always use htmlspecialchars($str, ENT_COMPAT, 'UTF-8')#10440

Merged
rdeutz merged 10 commits intojoomla:stagingfrom
zero-24:patch-34
May 16, 2016
Merged

[tags & users & libs & beez] Always use htmlspecialchars($str, ENT_COMPAT, 'UTF-8')#10440
rdeutz merged 10 commits intojoomla:stagingfrom
zero-24:patch-34

Conversation

@zero-24
Copy link
Copy Markdown
Contributor

@zero-24 zero-24 commented May 12, 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.

</div>
<div class="pagination">
<?php if ($this->params->def('show_pagination_results', 1)) : ?>
<p class="counter">
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this extra tab correct?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes because itbis in a if see line 92 ;)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so shouldnt that be
if
tab div
tab tab if
tab tab tab p

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no as the div is on the same hirachie than the if ;)

so
if
tab div
tab if
tab tab p

is correct ;)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems wrong to me - but what do I know ;)

@alikon
Copy link
Copy Markdown
Contributor

alikon commented May 12, 2016

I have tested this item ✅ successfully on f346618

on review


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

@brianteeman
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on f346618


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

@brianteeman
Copy link
Copy Markdown
Contributor

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label May 13, 2016
@brianteeman brianteeman added this to the Joomla 3.6.0 milestone May 13, 2016
@brianteeman
Copy link
Copy Markdown
Contributor

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/

@joomla-cms-bot
Copy link
Copy Markdown

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.

@zero-24
Copy link
Copy Markdown
Contributor Author

zero-24 commented May 14, 2016

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

@rdeutz rdeutz merged commit 75d990e into joomla:staging May 16, 2016
@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label May 16, 2016
@zero-24 zero-24 deleted the patch-34 branch May 16, 2016 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants