use String constants instead of magic strings