Remove error message <li> element fully

The text was being removed, but not the actual element. A rendering difference caused the whitespace in IE.

This approach seems broken.
Reduce complexity by moving notice_served into model

This causes a small non-js UI change (the radio button is included), but the end result is completely acceptable.
Integration test for notice fields JS hiding

This has not changed in the current branch, but was untested
Removed notice_served attribute from JSON output

This is only used by PDF generation
Merge pull request #453 from ministryofjustice/fix_ie7_validation_ui_bug

Fix ie7 validation ui bug