Bug #55499
DebugUtility shows <\u00F2 instead of </
| Status: | Resolved | Start date: | 2014-01-31 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | Michiel Roos | % Done: | 100% |
|
| Category: | - | Spent time: | - | |
| Target version: | 6.2.0 | |||
| TYPO3 Version: | 6.2 | Is Regression: | Yes | |
| PHP Version: | 5.3 | Sprint Focus: | ||
| Complexity: | easy |
Description
The new JavaScript encoder using json_encode also encodes the forward slash to unicode: \u00F2.
This creates problems, since json_encode already 'escapes' the forward slash with a backslash.
Related issues
Associated revisions
[BUGFIX] quoteJSvalue double encodes forward slash
The new JavaScript encoder using json_encode also encodes the forward
slash to unicode: \u00F2.
This creates problems, since json_encode already 'escapes' the forward
slash with a backslash.
This patch removes the ecoding of the forward slash from quoteJSvalue
Unit tests have been adjusted to match the output of json_encode:- forward slash is escaped with a backslash
Change-Id: Ibc19f69c0b68f5bbfa67a39750884dcc030f38cb
Resolves: #55499
Releases: 6.2
Reviewed-on: https://review.typo3.org/27183
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Tested-by: Daniel Siepmann
Reviewed-by: Frans Saris
Tested-by: Frans Saris
History
#1 Updated by Gerrit Code Review over 1 year 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/27183
#2 Updated by Gerrit Code Review over 1 year 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/27183
#3 Updated by Gerrit Code Review over 1 year 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/27183
#4 Updated by Gerrit Code Review over 1 year 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/27183
#5 Updated by Michiel Roos over 1 year ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset 1925ea03b0b4496f35c08df7c8496f117ea4bdd8.