Use semicolon before the OLD.column_name, like

....
VALUES (
:OLD.Benefit,
:OLD.SortOrder,
....)

And don't use COMMIT inside your trigger. COMMIT and ROLLBACK can't be used in the trigger, unless you use AUTONOMUS_TRANSACTION pagma.