use enums instead of magic values