1. Don't use loops in events. 2. Don't use ENABLE_CLICK in combination with the set macro. This will do bad things. 3. Check if the action sound_on_click is ever called by placing a beep or a printf("Over here!"); in the first line of the action.
Always learn from history, to be sure you make the same mistakes again...