refactor out finish_declare_victory. normalize some naming: persisted object fields use underscores, other variables use camel casing