b:commandButton onclick executed twice?

[chongma]

If I click the button described below it prompts me twice to confirm. Is the code being executed twice?

<b:commandButton value="Delete" iconAwesome="trash"
    action="#{testBean.delete}"
    onclick="if (! confirm('Are you sure?')) return false">
    <f:setPropertyActionListener target="#{testBean.id}"
        value="#{testItem.id}" />
    <f:ajax execute="@this" render="@form" />
</b:commandButton>

Bear in mind the code was inside a b:dataTable

[stephanrauh]

It seems the onclick handler is rendered twice:

                 	<b:commandButton value="Delete" iconAwesome="trash"
					    action="#{car.edit}"
					    onclick="debugger">
					    <f:ajax execute="@this" render="@form" />
					</b:commandButton>

<button type="submit" id="j_idt351:j_idt352:14:j_idt363"
	name="j_idt351:j_idt352:14:j_idt363" class="btn btn-default"
	onclick="debugger;debugger;BsF.ajax.callAjax(this, event,'j_idt351','j_idt351:j_idt352:14:j_idt363',null,null,null,null);;;return false;">
	<span><i class="fa fa-trash"></i></span> Delete
</button>

[stephanrauh]

The reproducer can be simplified to

<h:body>
  <h:form>           
    	<b:commandButton
	    onclick="debugger">
	    <f:ajax/>
	</b:commandButton>
  </h:form>
</h:body>

[stephanrauh]

I guess the problem is that <b:commandButton> renders the onclick attribute, and f:ajax renders it again.

[stephanrauh]

@chongma Maybe you want to review my last change. It should do the trick, but it's kind of a hack. If you've got a better solution, just tell me (or commit it yourself :)).

[chongma]

looks good to me

[stephanrauh]

I still can't think of a better solution, so I'll close this ticket. @chongma , thanks for testing!