If you want the new variable to be called `p4-form-commit-success-callback`, then you should rename this as `p4-form-commit-failure-callback` so that the names match grammatically. (Alternatively, call the new variable `p4-form-commit-succeed-callback`, but that's less good.)