Undo CharField changes to CommunicationEventType

The sending of emails relies on a (too) subtle distinction between empty
strings and None; either it won't get send, or Oscar will look for a
template file.

This behaviour should be improved as part of #319, #594. Till then we
just revert to what it was like before I merged #821.