Feature #4277

Putting form fields in a partial causes error

Added by Irene Höppner almost 6 years ago. Updated almost 5 years ago.

Status:Resolved Start date:2010-06-18
Priority:Must have Due date:2010-06-18
Assigned To:Sebastian Kurfuerst % Done:

100%

Category:ViewHelpers
Target version:-
Has patch:

Description

You often have the same form-fields in a new-template like in an edit-template. So I would like to put them into a partial which is used in the new-form and in the edit-form.
The form-tag itself would stay in the different templates.

So a partial could look like this:

<label for="titel">Titel/Tätigkeit:</label><br />
<f:form.textbox property="titel" size="70" id="titel" /><br />
<label for="personentage">Personentage:</label><br />
<f:form.textbox property="personentage" size="5" id="personentage" /><br />

Being included like this:

<f:form method="post" action="createOhneProjekt" object="{newAuftrag}" name="newAuftrag">
    <f:form.hidden name="kunde" value="{kunde}" />
        <f:render partial="auftragNewEditFormPart" arguments="{newAuftrag: newAuftrag}"/>    
    <f:form.submit value="Auftrag Anlegen" />
</f:form>

This results to the exception: #1243325768: No value found for key "F3\Fluid\ViewHelpers\FormViewHelper->formName"

Fluid-4277.patch Magnifier (1.6 kB) Sebastian Kurfuerst, 2009-08-31 15:29


Related issues

duplicated by TYPO3.Fluid - Bug #7882: wrong section rendering with <f.form.textbox /> Closed 2010-05-24

Associated revisions

Revision beaa7a25
Added by Sebastian Kurfuerst about 5 years ago

[+BUGFIX] Fluid (Core): Putting form fields in partial now works as expected. Resolves #4277.

History

#1 Updated by Sebastian Kurfuerst almost 6 years ago

Please check if the attached patch solves the issue.

#2 Updated by Irene Höppner almost 6 years ago

It works :-))

thx!

#3 Updated by Sebastian Kurfuerst almost 6 years ago

  • Category set to ViewHelpers
  • Status changed from Needs Feedback to Accepted
  • Assigned To set to Sebastian Kurfuerst
  • Priority changed from Should have to Must have

I'll fix this.

#4 Updated by Sebastian Kurfuerst over 5 years ago

[mustfixthis] until book release

#5 Updated by Peter Niederlag over 5 years ago

just ran into this one as well ...

#6 Updated by Sebastian Kurfuerst about 5 years ago

  • Due date set to 2010-06-18
  • Start date changed from 2009-08-20 to 2010-06-18

create v5 selenium test

#7 Updated by Sebastian Kurfuerst about 5 years ago

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

commmitted in r4595 for v5, and r2359 in v4.

Also available in: Atom PDF