Feature #3317

AbstractFormViewHelper must be adjusted to be able to namespace arguments

Added by Bastian Waidelich over 12 years ago. Updated about 11 years ago.

Status:
Resolved
Priority:
Must have
Category:
-
Target version:
-
Start date:
2009-05-15
Due date:
% Done:

100%

Estimated time:
Has patch:


Files


Related issues

Related to TYPO3.Fluid - Bug #3651: Call to a member function getBackend() on a non-objectResolvedBastian Waidelich2009-06-12

Actions
#1

Updated by Jochen Rau over 12 years ago

  • Status changed from New to Accepted
  • Assignee set to Sebastian Kurfuerst
  • Priority changed from Should have to Must have
#2

Updated by Bastian Waidelich over 12 years ago

  • Assignee changed from Sebastian Kurfuerst to Bastian Waidelich
#3

Updated by Bastian Waidelich over 12 years ago

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

#4

Updated by Jochen Rau over 12 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 over 12 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 over 12 years ago

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

Applied in changeset r1096.

#7

Updated by Bastian Waidelich about 12 years ago

  • Project changed from 534 to TYPO3.Fluid
  • Category deleted (432)

Also available in: Atom PDF