This doesn't work because, color_all_players has only 2 parms and since you check if the message contains the name, there is no away someone could add the name to color_all_players unless if you think everyone is going to use a format message or add a parm to your function
other than that good tut.