Question

I have a code snippet like below and I want to add imagebuttons into my asp:Panel during page load. But the events are firing already when I run the page. I want it to be fired when it is clicked.

Thanks in advance for all helps

    protected void Page_Load(object sender, EventArgs e)
    {...

        foreach (Gift g in bonusGifts)
        {
            ImageButton ib = new ImageButton();
            ib.ImageUrl = g.GiftBanner;
            ib.ID = g.GiftID.ToString();
            ib.Click += Purchase(g);
            BonusGiftPanel.Controls.Add(ib);

        }
    }

    private ImageClickEventHandler Purchase(Gift g)
    {
        _giftRep.Purchase(g, _userSession.CurrentUser);
        lblGifts.Text = "You have purcased " + g.GiftName + " for " + g.BonusPoints;

        return null;
    }

Answer 1

You should add the controls in the Page Init event, as other have said.

Your image click event handler does not conform to the ImageButton Click event handler signature. That should look something like this:

private void ImageButton_Click(ByVal sender As Object, ByVal e As ImageClickEventArgs)
{

}

Note that you can’t pass your “Gift” object directly to the ImageButton_Click. You will have to find another method of doing that.