ASP.NET ImageButton Onclick Event not handled

Question

I have an ImageButton in a GridView.

<asp:GridView ID="ItemGridView" runat="server" DataKeyNames="Id" 
     OnRowDataBound="ItemGridView_RowDataBound" AllowPaging="True" 
     AllowSorting="True" EnableSortingAndPagingCallbacks="True" 
     AutoGenerateEditButton="False" AutoGenerateDeleteButton="false"
     DataSourceID="ItemDataSource" EnableViewState="true" >
    ....
    <asp:TemplateField ShowHeader="False" ItemStyle-Width="40px">
      <ItemTemplate>
         <asp:ImageButton ID="btnDelete" SkinID="btnDelete"
            runat="server" CausesValidation="false"
            OnClick="btnDeleteAccountItem_Click" 
            OnClientClick="javascript:return confirm('Are you sure?');" />
       </ItemTemplate>
    </asp:TemplateField>

and a corresponding handler for the delete button event

protected void btnDeleteAccountItem_Click(object sender, EventArgs e) {
    ImageButton btnDel = sender as ImageButton;
    GridViewRow row = (GridViewRow)btnDel.NamingContainer;
    ....
}

I am using this very same construct in many places and it works fine. I have one gridview now, though, where it does not, and I am hoping to get some ideas for how to track down the problem. When I click the button, the client-side event fires and the alert box is displayed, then the post-back fires and I can hit a break point in the Page_Load method. So the client-side wiring of the button events seems to work. However, the event is not handled and the method btnDeleteAccountItem_Click does not get called.

This is a complex page and I cannot post all the code. What can I do to narrow down potential causes?

Have you checked any info in Page_Load, such as whether IsPostBack is true or the EVENTTARGET value in the form collection? — Rex M
– Rex M, Commented Oct 20, 2009 at 15:48
IsPostBack is true and the form collection appears to be empty. What does that mean? — cdonner
– cdonner, Commented Oct 20, 2009 at 16:01
I should add that the GridView is on a MultiView control, and that the form tag is in the master page. — cdonner
– cdonner, Commented Oct 20, 2009 at 16:18
See below - the EVENTTARGET now has the correct value, it seems, but still no invocation of the handler. — cdonner
– cdonner, Commented Oct 20, 2009 at 17:07

rick schott · Accepted Answer · 2009-10-20 16:36:54Z

5

Your event is defined incorrectly ImageButton.Click:

protected void btnDeleteAccountItem_Click(object sender, ImageClickEventArgs e) {
    ImageButton btnDel = sender as ImageButton;
    GridViewRow row = (GridViewRow)btnDel.NamingContainer;
    ....
}

edited Oct 20, 2009 at 16:36

answered Oct 20, 2009 at 16:22

rick schott

21.1k5 gold badges54 silver badges81 bronze badges

Sign up to request clarification or add additional context in comments.

5 Comments

cdonner Over a year ago

You are right, but like I said, the code works as is on other pages, and changing the event args type did not fix the problem. There must be something else on the page that breaks this.

rick schott Over a year ago

Is your OnClientClick doing anything other than a confirm?

rick schott Over a year ago

What happens if you completely remove the OnClientClick from the markup?

rick schott Over a year ago

None of this makes sense, so I will ask this question. Page_Load will fire first then your events, are you debugging far enough in the page lifecycle to get to your event?

cdonner Over a year ago

Yes I am, I have a breakpoint there that does not get hit. I think it has to to with calling Databind() on the grid, though. I have a popup window for editing the items. I am using Shadowbox in an IFrame, so this loads in a separate page, and I am not communicating back if something changed, so I just called Databind on the grid unconditionally. It seems that I cannot do that. Thanks for your help!

Dalbir Singh · Accepted Answer · 2009-10-20 15:52:07Z

3

I'm not sure if this will solve it, but once I placed a asp:Button control in my markup and generated the 'onClick' signature for it too.

I then changed my mind and decided to make it an Image button... ... I simply rewrote the tag myself.

After making those changes, I realised that the onClick signature wasn't working anymore... after some research I found an answer... I was using 'EventArgs' instead of 'ImageClickEventArgs'...

(object sender, ImageClickEventArgs e)

Once I changed the event arg object, it started working as normal.

answered Oct 20, 2009 at 15:52

Dalbir Singh

2,6383 gold badges26 silver badges29 bronze badges

1 Comment

cdonner Over a year ago

Making some progress - after fixing the event args, the EVENTTARGET value is now in the form collection, and it is the image button in questions.

Paul · Accepted Answer · 2009-10-20 15:50:53Z

2

rather than creating a button click event, you could use the datagrid row command event

You can then use e.commandName and e.commandArgument to find out which button was pressed and what its argument is:

 Private Sub gv1_RowCommand(ByVal sender As Object, ByVal e As System.Web.UI.WebControls.GridViewCommandEventArgs) Handles gv1.RowCommand
    If e.CommandName = "Whatever" Then
       // do something
     End If

Hope it helps

answered Oct 20, 2009 at 15:50

Paul

5,5762 gold badges32 silver badges38 bronze badges

Comments

Barry · Accepted Answer · 2014-06-04 03:31:01Z

1

make sure the CausesValidation is set to True.

answered Jun 4, 2014 at 3:31

Barry

111 bronze badge

Comments

CoderPi · Accepted Answer · 2015-12-09 06:31:08Z

0

You can try (you are creating wrong click event of ImageButton) :

<asp:TemplateField ShowHeader="False" ItemStyle-Width="40px">
 <ItemTemplate>
  <asp:ImageButton ID="btnDelete" SkinID="btnDelete" runat="server"
   CausesValidation="false" OnClick="btnDeleteAccountItem_Click"
   OnClientClick="javascript:return confirm('Are you sure?');" />
 </ItemTemplate>
</asp:TemplateField>

and for image button there are change in click event like as follow :

protected void btnDelete_Click(object sender, ImageClickEventArgs e)
{
 ImageButton btnDel = sender as ImageButton;
 GridViewRow row = (GridViewRow)btnDel.NamingContainer;
}

edited Dec 9, 2015 at 6:31

CoderPi

13.2k4 gold badges38 silver badges64 bronze badges

answered Dec 9, 2015 at 6:13

Vaibhav Chaurasiya

1669 bronze badges

Collectives™ on Stack Overflow

ASP.NET ImageButton Onclick Event not handled

5 Answers 5

5 Comments

1 Comment

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

5 Comments

1 Comment

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related