Bug #27415

ViewHelper Forms with method get and action parameter do not address to right action

Added by René Wagner about 4 years ago. Updated about 1 month ago.

Status:Accepted Start date:2011-06-13
Priority:Should have Due date:
Assigned To:- % Done:

0%
Category:Fluid Spent time: -
Target version:7.4 (Backend)
TYPO3 Version:6.2 Is Regression:No
PHP Version: Sprint Focus:
Complexity:

Description

A form like the following is rendered invalid:

<f:form action="list" method="get" id="searchForm" name="searchForm">...</f:form>

The reason is, it is addressing the action as a query string in the form action attribute. This query string gets completely substituted by the form fields inside the form.

Resolution: On get formulars, render the query string as hidden form fields.

Manual workaround:

<f:form action="list" method="get" id="searchForm" name="searchForm">
<f:form.hidden name="action" value="list" />
...
</f:form>

Related issues

related to TYPO3.Fluid - Task #11636: Form viewhelper not functional with method GET Resolved 2010-12-22

History

#1 Updated by Stefan Neufeind about 4 years ago

  • Status changed from New to Accepted

#2 Updated by Stefan Neufeind about 4 years ago

At least it would be up to the browser to merge the two (get-arguments from action-URL and from the form). I agree this doesn't seem right. And checking with at least Firefox 4 those arguments are lost as you describe.

Same holds true for fluid in FLOW3.

#3 Updated by Bastian Waidelich about 4 years ago

The way we should probably implement this, would be to create the action URI as before, parse it with parse_url() and retrieve the query parameters with parse_str() And then we could create the hidden fields from them. This should happen for GET only and it should be possible to deactivate it IMO.

#4 Updated by Anja Leichsenring over 2 years ago

  • Project changed from Extbase MVC Framework to Fluid
  • Category changed from 963 to Fluid: ViewHelpers

#5 Updated by Wouter Wolters over 2 years ago

TYPO3 Fluid has an implementation for this AFAICS

#6 Updated by Clément MICHELET almost 2 years ago

Seems to be fixed in Flow3 in issue #11636
Any hope to have backport for 6.2 ?

With this bug, I can't expect to have in working state in the same time :
  • a table with results
  • a filter form for table
  • a paginate widget

Pass hidden value didn't work with 6.1.5.
Set the form method to post imply to persist form values which is an alternative solution but imply also many new checks to handle.

#7 Updated by Marc Bastian Heinrichs almost 2 years ago

  • Target version set to Fluid 6.2

We could not backport this 1:1 because we have to have realurl in mind.

#8 Updated by Alexander Opitz 11 months ago

  • Project changed from Fluid to Core
  • Category changed from Fluid: ViewHelpers to Fluid
  • Target version changed from Fluid 6.2 to 7.0
  • TYPO3 Version set to 6.2
  • Is Regression set to No

#9 Updated by Mathias Schreiber 7 months ago

  • Target version changed from 7.0 to 7.1 (Cleanup)

#10 Updated by Benjamin Mack about 1 month ago

  • Target version changed from 7.1 (Cleanup) to 7.4 (Backend)

Also available in: Atom PDF