Project

General

Profile

Actions

Bug #81757

closed

EXT:Fluid UriViewHelper needs escapeOutput = false;

Added by Aljoscha Weber over 7 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Fluid
Target version:
-
Start date:
2017-06-30
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
8
PHP Version:
7.0
Tags:
Complexity:
no-brainer
Is Regression:
Sprint Focus:

Description

The escapeOutput property of the TYPO3\CMS\Fluid\ViewHelpers\Widget\UriViewHelper should be set to false or the generated link will be wrong in the autocomplete-ViewHelper.

Current-Output of the TYPO3\CMS\Fluid\ViewHelpers\Widget\AutocompleteViewHelper:

{f:widget.uri(action:'autocomplete', ajax: 1)}

?id=15&type=7076&fluid-widget-id=59640106e78e5b3eceab2d0a402a98a4&action=autocomplete

Output with escapeOutput = false;

?id=15&type=7076&fluid-widget-id=3530c7af3431d73ca17d4e0fdd691a0b&action=autocomplete

Actions #1

Updated by Gerrit Code Review over 7 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/53368

Actions #2

Updated by Gerrit Code Review over 7 years ago

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/53369

Actions #3

Updated by Gerrit Code Review over 7 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/53369

Actions #4

Updated by Aljoscha Weber about 7 years ago

This is still broken in 8.7.5. Adding:

    /**
     * As this ViewHelper renders HTML, the output must not be escaped.
     *
     * @var bool
     */
    protected $escapeOutput = false;

like in the patch set fixes the problem.

Actions #5

Updated by Aljoscha Weber about 7 years ago

  • Complexity set to no-brainer
Actions #6

Updated by Gerrit Code Review about 7 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/53369

Actions #7

Updated by Susanne Moog over 5 years ago

  • Status changed from Under Review to Needs Feedback

Patch has been abandoned after discussion. Does this issue still exist?

Actions #8

Updated by Susanne Moog over 4 years ago

  • Status changed from Needs Feedback to Closed

No feedback and no new reports of the issue, closing it now - if it pops up again, please re-open.

Actions

Also available in: Atom PDF