Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Working on #2972 Template Conversion to Twig Format (sendthread.php) #2983

Merged
merged 2 commits into from
Jan 11, 2018
Merged

Working on #2972 Template Conversion to Twig Format (sendthread.php) #2983

merged 2 commits into from
Jan 11, 2018

Conversation

Starpaul20
Copy link
Member

#2972 for sendthread.php

Copy link
Member

@euantorano euantorano left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

sendthread.php Outdated
$form['fromname'] = htmlspecialchars_uni($mybb->input['fromname']);
$form['fromemail'] = htmlspecialchars_uni($mybb->input['fromemail']);
$form['subject'] = htmlspecialchars_uni($mybb->input['subject']);
$form['message'] = htmlspecialchars_uni($mybb->input['message']);
Copy link
Member

Choose a reason for hiding this comment

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

Double escapes - Twig will also escape the content.

@euantorano euantorano merged commit dc9f2ca into mybb:develop/1.9 Jan 11, 2018
@Starpaul20 Starpaul20 deleted the develop-1.9-sendthread branch January 11, 2018 15:56
@Starpaul20 Starpaul20 added the b:1.9 Branch: 1.9.x label Feb 18, 2018
euantorano pushed a commit that referenced this pull request Mar 16, 2018
…2983)

* Working on #2972 Template Conversion to Twig Format (sendthread.php)

* Removed double escaping
euantorano pushed a commit to euantorano/mybb that referenced this pull request Mar 23, 2019
…hp) (mybb#2983)

* Working on mybb#2972 Template Conversion to Twig Format (sendthread.php)

* Removed double escaping
euantorano pushed a commit to euantorano/mybb that referenced this pull request Mar 23, 2019
…hp) (mybb#2983)

* Working on mybb#2972 Template Conversion to Twig Format (sendthread.php)

* Removed double escaping
euantorano pushed a commit that referenced this pull request Jun 2, 2019
…2983)

* Working on #2972 Template Conversion to Twig Format (sendthread.php)

* Removed double escaping
euantorano pushed a commit that referenced this pull request Jun 2, 2019
…2983)

* Working on #2972 Template Conversion to Twig Format (sendthread.php)

* Removed double escaping
@euantorano euantorano mentioned this pull request Jan 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
b:1.9 Branch: 1.9.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants