Bug #36664
FormElements can't handle BackendModule Requests
| Status: | Resolved | Start date: | 2012-04-27 | |
|---|---|---|---|---|
| Priority: | Should have | Due date: | ||
| Assigned To: | Bastian Waidelich | % Done: | 0% |
|
| Category: | - | |||
| Target version: | - |
Description
The current implementation of the Request handling only works for one getParentRequest level.
This fails in the Context of the Phoenix BackendModule because the Request is hidden deeper.
Attached is a diff to go as deep as it needs to go.
FormRuntime.diff
(1.1 kB)
FormRuntime.diff
(526 Bytes)
Related issues
Associated revisions
[TASK] Use plugin namespace as argument namespace for sub requests
The use of plugin arguments instead of regular arguments is needed after
the HTTP rewrite, since all arguments of sub controllers/requests are now
defined at root level. Since the form is a plugin this makes sense.
Related: #36664
Change-Id: I2b686b044b5fae25ef0d6b241ed3bd68d4e0d8c3
History
#1 Updated by Bastian Waidelich over 3 years ago
- Assigned To deleted (
Bastian Waidelich)
#2 Updated by Bastian Waidelich over 3 years ago
- Status changed from New to Needs Feedback
Could you quickly check whether https://review.typo3.org/#/c/10755/2 fixes this for you?
#3 Updated by Marc Neuhaus over 3 years ago
I've just tried that patch without success.
the $pluginArguments and the request stays empty.
This results in simply Rerendering the original form, because no submitted data is detected.
#4 Updated by Marc Neuhaus about 3 years ago
Just tried the latest version of FLOW3 again, still not working. the Form package doesn't receive any arguments :/
The request in FormRuntime.php line 161 has no arguments at all.
#5 Updated by Bastian Waidelich about 3 years ago
Marc Neuhaus wrote:
Just tried the latest version of FLOW3 again, still not working.
Did https://review.typo3.org/#/c/10333/ solve your issue (toghether with the patch from https://review.typo3.org/#/c/10755/) ?
#6 Updated by Marc Neuhaus about 3 years ago
Those 2 Patches seem to do the trick :)
You can close this issue.
#7 Updated by Marc Neuhaus about 3 years ago
- File FormRuntime.diff added
Although this additional Patch would make this thing work in any Request :)
#8 Updated by Bastian Waidelich about 3 years ago
- Status changed from Needs Feedback to Resolved
- Assigned To set to Bastian Waidelich
Thanks for your input. I incorporated your last suggestion into https://review.typo3.org/#/c/10755/5