ASP.net checkbox always checked

Question

<asp:CheckBox ID="isSubscribed" runat="server" /> Subscribe to mailing list?<br /><br />
<asp:Button runat="server" CssClass="btn" ID="btnPrefUpdate" OnClick="updatePrefs" Text="Update" /><br /><br />

This fires in the code behind:

protected void updatePrefs(object sender, EventArgs e)
{
    Response.Write(isSubscribed.Checked);
    Response.End();
}

But it's always coming out as true! Whether it is checked or not! I know I'm doing it wrong, could someone show me how to access this value properly?

Do you have anything in your Page_Load event which ever sets it as true? Have you tried putting "isSubscribed.Checked.ToString"? — Curtis
– Curtis, Commented Sep 30, 2010 at 10:58
Ah yes, in my page load I have isSubscribed.Checked = Master.isSubscribed; how do I get the on click event to run before that one? — Tom Gullen
– Tom Gullen, Commented Sep 30, 2010 at 11:03
as a side note I think Respose.Write is evil and should be avoided if possible. It can make debugging your code a nightmare :( — TheLukeMcCarthy
– TheLukeMcCarthy, Commented Sep 30, 2010 at 11:13

TheLukeMcCarthy · Accepted Answer · 2010-09-30 11:07:05Z

9

like @Curt said, it looks to me like you have something in your page_load. if you set the value in Page_Load make sure it is inside the following if statement

if(!Page.isPostBack)
{
    isSubscribed.Checked = true; 
}

answered Sep 30, 2010 at 11:07

TheLukeMcCarthy

2,2932 gold badges26 silver badges35 bronze badges

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

Comments

Wim Haanstra · Accepted Answer · 2010-09-30 11:11:35Z

1

You are doing it the right way. The boolean property Checked should just say True or False (I even tested it). Is your Page_Load doing something with the checkbox? In other words, is the value of the checkbox somehow (re)set when the post back is occuring (the postback of the button click).

In your Page_Load method you could include:

if (!this.IsPostBack)
{
// Set default or loaded values for controls
}

edited Sep 30, 2010 at 11:11

answered Sep 30, 2010 at 11:02

Wim Haanstra

6,0045 gold badges45 silver badges58 bronze badges

3 Comments

Tom Gullen Over a year ago

Yes, stupidly it sets it to the profile value in page load, how can I stop that code from loading before the button click event?

TheLukeMcCarthy Over a year ago

@Tom Gullen see my answer, or you could set it in the aspx page

Wim Haanstra Over a year ago

if (!this.IsPostBack) // Do your thing.

Collectives™ on Stack Overflow

ASP.net checkbox always checked

2 Answers 2

Comments

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related