#983 deactivation warning

Resolves #983.
Can this be refactored for readability? It seems to me the more efficient logic would be 'if overbooked is undefined or empty, show confirm prompt and add the deactivation_cancelled flag, otherwise [normal deactivation code]