Task #57107

Feature #56642: Missing TypoScript property to quote and escape content for use in JavaScript

Rename escapeJsValue and make it context specific

Added by Jigal van Hemert over 5 years ago. Updated about 1 year ago.

Status:
Closed
Priority:
Must have
Category:
TypoScript
Target version:
-
Start date:
2014-03-20
Due date:
% Done:

100%

TYPO3 Version:
6.2
PHP Version:
Tags:
Complexity:
no-brainer
Sprint Focus:

Description

The stdWrap property escapeJsValue actually encodes the content and should thus be called encodeJsValue. Also the value must be enclosed in quotes to prevent it from being used outside the target context of string values in JavaScript.

Associated revisions

Revision be64df41 (diff)
Added by Jigal van Hemert over 5 years ago

[TASK] Rename escapeJsValue and make it context specific

The stdWrap property escapeJsValue actually encodes the
content and should thus be called
encodeForJavaScriptValue. Also the value must be
enclosed in quotes to prevent it from being used outside
the target context of string values in JavaScript.

Releases: 6.2
Resolves: #57107
Documentation: #57108
Change-Id: Id5f850d36a87d71ca80eb58e3359ddacdd5b655e
Reviewed-on: https://review.typo3.org/28571
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers

History

#1 Updated by Gerrit Code Review over 5 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/28571

#2 Updated by Gerrit Code Review over 5 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/28571

#3 Updated by Gerrit Code Review over 5 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/28571

#4 Updated by Jigal van Hemert over 5 years ago

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

#5 Updated by Benni Mack about 1 year ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF