Feature #3317

AbstractFormViewHelper must be adjusted to be able to namespace arguments

Added by Bastian Waidelich about 6 years ago. Updated almost 5 years ago.

Status:Resolved Start date:2009-05-15
Priority:Must have Due date:
Assigned To:Sebastian Kurfuerst % Done:

100%
Category:-
Target version:-
Has patch:

Description

see http://forge.typo3.org/issues/show/3316

3317_AbstractFormViewHelper_must_be_adjusted_to_be_able_to_namespace_arguments.patch Magnifier (98.4 kB) Bastian Waidelich, 2009-08-09 20:24

3317_AbstractFormViewHelper_must_be_adjusted_to_be_able_to_namespace_arguments_v2.patch Magnifier (98.4 kB) Bastian Waidelich, 2009-08-10 10:02

Related issues

related to TYPO3.Fluid - Bug #3651: Call to a member function getBackend() on a non-object Resolved 2009-06-12

History

#1 Updated by Jochen Rau about 6 years ago

  • Status changed from New to Accepted
  • Assigned To set to Sebastian Kurfuerst
  • Priority changed from Should have to Must have

#2 Updated by Bastian Waidelich almost 6 years ago

  • Assigned To changed from Sebastian Kurfuerst to Bastian Waidelich

#3 Updated by Bastian Waidelich almost 6 years ago

Attached patch implements the namespacing of field names in forms and contains some refactorings and enhancements.
@Sebastian: Please check it and commit if you have no objections.

#4 Updated by Jochen Rau almost 6 years ago

Thanks a lot for the very large patch.

I tested the patch and had to change line 61 in Tx_Fluid_ViewHelpers_Form_AbstractFormViewHelper from

$fieldNamePrefix = (string)$this->viewHelperVariableContainer->get('Tx_Fluid_ViewHelpers_FormViewHelper', 'fieldNamePrefix');

to

$fieldNamePrefix = (string)$this->viewHelperVariableContainer->get('Tx_Fluid_ViewHelpers_FormViewHelper', 'prefix');

Is this a bug or did I miss something.

#5 Updated by Bastian Waidelich almost 6 years ago

Jochen Rau wrote:

Hi Jochen,

Is this a bug or did I miss something.

This was a mistake. I've renamed "prefix" to "fieldNamePrefix" recently and apparently forgot to adjust the backported code respectively..
Attached is v2 of the patch that should fix that issue.

#6 Updated by Jochen Rau almost 6 years ago

  • Status changed from Accepted to Resolved
  • % Done changed from 0 to 100

Applied in changeset r1096.

#7 Updated by Bastian Waidelich almost 6 years ago

  • Project changed from Extbase MVC Framework to TYPO3.Fluid
  • Category deleted (432)

Also available in: Atom PDF