Task #75575

Streamline escaping for ViewHelpers of all sysexts

Added by Benni Mack over 3 years ago. Updated about 2 years ago.

Status:
Closed
Priority:
Should have
Assignee:
Category:
Fluid
Target version:
Start date:
2016-04-13
Due date:
% Done:

100%

TYPO3 Version:
8
PHP Version:
Tags:
Complexity:
Sprint Focus:

Related issues

Related to TYPO3 Core - Bug #80238: f:translate double escapes default values Closed 2017-03-11

Associated revisions

Revision a3985b9c (diff)
Added by Benni Mack over 3 years ago

[TASK] Streamline escaping for ViewHelpers of all sysexts

The Core does not strictly use $escapeChildren and
$escapeOutput as properly introduced by Fluid Standalone in 8.0.

This results in some weird HTML output when using the inline syntax
currently.

Since $escapeOutput and $escapeChildren is active by default,
some ViewHelpers even do HSC on their own (= twice) even if not
needed.

Resolves: #75575
Releases: master
Change-Id: I3bbebe75c0368519d3506360c68df92d30a3b80a
Reviewed-on: https://review.typo3.org/47644
Reviewed-by: Helmut Hummel <>
Tested-by: Helmut Hummel <>
Reviewed-by: Nicole Cordes <>
Tested-by: Nicole Cordes <>

Revision 120b8aa8 (diff)
Added by Frank Naegler almost 3 years ago

[TASK] Remove deprecated argument htmlEscape of f:translate VH

Remove usage of htmlEscape from core templates, because it is deprecated since #75575

Resolves: #79842
Related: #75575
Releases: master
Change-Id: I62c53cb9cdba585f5ead920154dcef729c8c02eb
Reviewed-on: https://review.typo3.org/51709
Tested-by: TYPO3com <>
Reviewed-by: Wouter Wolters <>
Reviewed-by: Christian Kuhn <>
Tested-by: Christian Kuhn <>
Reviewed-by: Jigal van Hemert <>
Tested-by: Jigal van Hemert <>

History

#1 Updated by Gerrit Code Review over 3 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47644

#2 Updated by Gerrit Code Review over 3 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47644

#3 Updated by Gerrit Code Review over 3 years ago

Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47644

#4 Updated by Gerrit Code Review over 3 years ago

Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47644

#5 Updated by Gerrit Code Review over 3 years ago

Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47644

#6 Updated by Gerrit Code Review over 3 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47644

#7 Updated by Gerrit Code Review over 3 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47644

#8 Updated by Gerrit Code Review over 3 years ago

Patch set 8 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47644

#9 Updated by Gerrit Code Review over 3 years ago

Patch set 9 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47644

#10 Updated by Gerrit Code Review over 3 years ago

Patch set 10 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47644

#11 Updated by Gerrit Code Review over 3 years ago

Patch set 11 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/47644

#12 Updated by Benni Mack over 3 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100

#13 Updated by Riccardo De Contardi about 2 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF