"ASP is not C."

Please don't tag things as C if they're not within forty-two parsecs of the nearest C compiler.

I've got good news, and I've got bad news:
The universe is merely a figment of my imagination.
Now are you ready for the bad news?

I was unaware that a SqlDataSource control existed. Still, I think I'd prefer to keep as much of the ASP.Net code out of the HTML as possible. Does anyone else have an opinion on this?

P.S. I may be mistaken but I think you meant "DataSourceId" instead of "SataSourceId" in your CheckBoxList attributes.