[jira] [Commented] (TAP5-1632) "cancel" mode of submit component is not submit to server, if it doesn't have t:id attribute

["Howard M. Lewis Ship (JIRA)" <ji...@apache.org>]

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

Howard M. Lewis Ship commented on TAP5-1632:
--------------------------------------------

This is odd:

Using dev tools I can see a reasonable name and id for the element:


<input value=​"Cancel Form" id=​"submit" name=​"cancel" type=​"submit">​

So cancel is a property of the form:


$('form').cancel --> <input value=​"Cancel Form" id=​"submit" name=​"cancel" type=​"submit">​

But so is submit:

$('form').cancel --> <input value=​"Cancel Form" id=​"submit" name=​"cancel" type=​"submit">​

This is not how I thought HTMLFormElement properties worked; I thought the names were used, but it's looking like names and ids.

I'm seeing the same behavior in both FireFox and Chrome.


> "cancel" mode of submit component is not submit to server, if it doesn't have t:id attribute
> --------------------------------------------------------------------------------------------
>
>                 Key: TAP5-1632
>                 URL: https://issues.apache.org/jira/browse/TAP5-1632
>             Project: Tapestry 5
>          Issue Type: Bug
>          Components: tapestry-core
>    Affects Versions: 5.3, 5.2.5
>            Reporter: Yohan Yudanara
>            Assignee: Howard M. Lewis Ship
>              Labels: component, submit
>
> "cancel" mode of submit component is not submit to server, if we don't give t:id attribute on Submit component.
> Steps to reproduce:
> <t:form t:id="form">
>        	<t:textfield t:id="testField" t:value="testField" t:validate="required" />
>        	<t:submit value="normal submit"/>
>        	<t:submit mode="cancel" value="cancel submit"/>
> </t:form>
> When clicking on "cancel submit", nothing happen (form is not submitted to server).
> --------------------
> Workaround from Josh Canfield is to give t:id attribute on Submit component:
> <t:form t:id="form">
>        	<t:textfield t:id="testField" t:value="testField" t:validate="required" />
>        	<t:submit t:id="normalSubmit" value="normal submit"/>
>        	<t:submit t:id="cancelSubmit" mode="cancel" value="cancel submit"/>
> </t:form>
> --------------------
> This is caused by tapestry gives default id 'submit' to the Submit component. 
> Tapestry should never give default id 'submit' to the Submit component to avoid this defect.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira