I would also name the fields in the insert statement. It helps self-document and would simplify the command since you could omit the field populated by the trigger. And, it will keep you from having to revise this code should another field be added to the table. Also, if one of the fields you use is renamed or removed, the error messages might be more explicit.