fixed onclick function setting