swa66 - 4:57 pm on Apr 22, 2013 (gmt 0)
I'd use the proper attribute of checked="checked" if you're going to go for the (xml) self closing tags.
Since it's going to be executed in a loop, I'd lose the id or make sure they are unique if you need them.
In your table you can use a true boolean instead of a string. just test it with if ...