Remove mission_id from tags table

I don't know what I was thinking there, but we don't actually write to
mission_id in the tags table, so now it's only adding complexity.
Tidy up RegistrationsController and the like

Three cheers for specs! Hip, hip, hooray!