Project

General

Profile

Actions

Bug #76731

closed

Differing behaviour for handling addQueryStringMethod in Frontend and Backend

Added by Christoph Vigano almost 8 years ago. Updated about 4 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Backend API
Target version:
-
Start date:
2016-06-22
Due date:
% Done:

0%

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

Description

While comparing buildBackendUri() in TYPO3\CMS\Extbase\Mvc\Web\Routing\UriBuilder and getQueryArguments() in TYPO3\CMS\Frontend\ContentObject\ContentObjectRenderer, I found that buildBackendUri() has a fallback to GeneralUtility::_GET() if no addQueryStringMethod parameter is provided.

getQueryArguments() on the other hand has no such fallback, resulting in differing behaviour when no addQueryStringMethod is provided or can't be provided as is the case with the f:form viewhelper.

I found this out while using a Powermail form on a page with another plugin, which displays the current event and needs the UID for this. Without RealURL this works, because all parameters are still part of the QUERY_STRING environment variable, which is the default for buildFrontendUri() because of the above mentionend behaviour of getQueryArguments(), after the Powermail form is submitted.

If I deactivate RealURL for my plugin to generate a speaking URL, the uid of my event is now part of the URL and therefore part of the GET array after RealURL does its job, but not a part of QUERY_STRING anymore. This results in parameters being lost after the Powermail form has been submitted.

There are two ways to solve this in my opinion:
  1. provide a fallback in getQueryArguments(), but I have no clue what kind of global impact this change could have
  2. provide the f:form viewhelper with an additional parameter addQueryStringMethod, as it is now the case with f:uri.page and similar functions

As far as I can see, this behaviour also exists in the latest 6.2.x release.

Actions #1

Updated by Riccardo De Contardi over 5 years ago

  • Category set to Backend API
Actions #2

Updated by Christian Eßl almost 5 years ago

I think this issue is obsolete as the form viewhelper supports the "addQueryStringMethod" parameter since TYPO3 7.6.

Actions #3

Updated by Susanne Moog about 4 years ago

  • Status changed from New to Closed

See last comment.

Actions

Also available in: Atom PDF