Well, the value and the label are the same for all the options. Is there a reason not to leave out the value and let the implicit value get used? This doesn't need to be changed, so it's certainly not wrong. It was just something that stood out as I was going through.