[jira] [Commented] (TAP5-2482) BeanEditForm field name regression introduced with 5.4-beta-31

["Jochen Kemnade (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/TAP5-2482?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14701335#comment-14701335 ] 

Jochen Kemnade commented on TAP5-2482:
--------------------------------------

Perhaps, there should be an optional {{name}} parameter for {{AbstractField}}. Or we could use the informal {{name}} parameter if it is bound.
{{formSupport.allocateControlName(resources.getId())}} could still be the fallback, but {{PropertyEditBlocks}} could set the {{name}} parameter to {{context.propertyId}}.

> BeanEditForm field name regression introduced with 5.4-beta-31
> --------------------------------------------------------------
>
>                 Key: TAP5-2482
>                 URL: https://issues.apache.org/jira/browse/TAP5-2482
>             Project: Tapestry 5
>          Issue Type: Bug
>    Affects Versions: 5.4
>            Reporter: Balázs Palcsó
>              Labels: 54_release_prerequisite
>
> Issue started with: 5.4-beta-31
> Last working release: 5.4-beta-30
> I have the below BeanEditForm
> {code}
> <t:beaneditor t:object="user"
>                   include="username,lastName,firstName,email,phone,password,confirmPassword">
>                   <p:username>
>                     <div class="form-group">
>                       <t:label for="username" />
>                       <t:textfield t:id="username" value="user.username" t:mixins="OverrideFieldFocus" />
>                     </div>
>                   </p:username>
>                   <p:password>
>                     <div class="form-group">
>                       <t:label for="password" />
>                       <t:passwordfield t:id="password" value="user.password" />
>                     </div>
>                   </p:password>
>                   <p:confirmPassword>
>                     <div class="form-group">
>                       <t:label for="confirmPassword" />
>                       <t:passwordfield t:id="confirmPassword" value="user.confirmPassword" />
>                     </div>
>                   </p:confirmPassword>
>                 </t:beaneditor>
> {code}
> The {{name}} attribute of the input generated for each included fields were matching the field name until 5.4-beta-31.
> With 5.4-beta-31 the above form generates the following name attribute for {{<input />}} for
> {{name="textField"}} instead of {{name="lastName"}}
> {{name="textField_0"}} instead of {{name="firstName"}}
> {{name="textField_1"}} instead of {{name="phone"}}
> For username, email, password, confirmPassword correct name attributes are generated.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)