Project

General

Profile

Actions

Bug #77946

closed

Fluid page link (f:link.page) to different pid using addQueryString not working as expected

Added by Markus Mächler over 7 years ago. Updated over 4 years ago.

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

0%

Estimated time:
TYPO3 Version:
7
PHP Version:
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

If you are for instance on a page with uid 1 and want to link to a page with uid 2 and set the addQueryString option, the link will be generated for uid 1 instead of 2.

e.g.

We are on page 1
<f:link.page pageUid="2" addQueryString="1">Page 2</f:link.page>

The problem is that the url builder creates a url that looks like yourpage.com?id=2&id=1 where the second id parameter overwrites the first.
I feel like this issue should at least be addressed in TYPO3\CMS\Fluid\ViewHelpers\Link\PageViewHelper where the id parameter could be stripped away if the pageUid is set. It is very counter intuitive if there is a pageUid parameter present but not respected. It could also be fixed in TYPO3\CMS\Extbase\Mvc\Web\Routing\UriBuilder directly, although this might break other code.

Actions

Also available in: Atom PDF