fixed action.  wrong operator in loop