ngSubmit should trigger after Asynchronous Validation completes
See original GitHub issueπ feature request
Relevant Package
This feature request is for @angular/formsSpecifically, Reactive Forms
Description
**in Template**
<form [formGroup]="myFormGroup" (ngSubmit)="mFG_submit()"></form>
**In Component**
mFG_submit() {
if (!myFormGroup.valid)
return;
//logic that will run if and only if the entire form is valid after both
sync and async validators have finished executing
}
The problem with the above code is that if myFormGroup has any controls with asynchronous validators, then ngSubmit is triggered after the async validators have started executing, but before they have finished executing. As as result, when the βifβ gate inside mFG_submit() is hit, it returns immediately, because the form status is βPENDINGβ. As a result, even if the asynchronous validator returns true, i,e, the control is valid, the submission logic is never executed.
Describe the solution youβd like
One solution is to check on every submit if the form status is βPENDINGβ, and then listen to status changes, and emit ngSubmit again, the moment status changes. I would like to have such behaviour provided by the framework Currently, I use this directive
@Directive({
selector: 'form[formGroup]'
})
export class ResubmitIfPendingDirective {
constructor(
private fgd: FormGroupDirective
) {
this.subscriptions.add(this.resubmission_sub);
}
private subscriptions: Subscription = new Subscription();
private resubmission_sub: Subscription = new Subscription();
ngOnInit() {
//listen to ngSubmit of the form
this.subscriptions.add(this.fgd.ngSubmit.subscribe(() => {
//if you are already subscribed to status changes, unsubscribe
this.subscriptions.remove(this.resubmission_sub);
this.resubmission_sub.unsubscribe();
//if your form is PENDING when submitted, subscribe to status changes
if (this.fgd.control.pending) {
this.resubmission_sub = this.fgd.control.statusChanges
.pipe(
//don't do anything if new emitted status is PENDING
filter(() => !this.fgd.control.pending),
//status no longer PENDING, time to resubmit
//and stop observing statusChanges
first()
// above 2 pipes can be combined, I separated for clarity
// first(() => !this.fgd.control.pending)
).subscribe(() => this.fgd.ngSubmit.emit());
this.subscriptions.add(this.resubmission_sub)
//since the validation has already been run,
//and there have no change in the values of the controls,
//the status of this follow-up ngSubmit will either emit
//VALID or INVALID
//therefore, the ngSubmit we emit will simply
//unsubscribe resubmission_sub
//and never re-enter this if() block
//Thus, No infinite loop of submits
}
}));
}
ngOnDestroy() {
//stop listening to ngSubmit and status changes
this.subscriptions.unsubscribe();
}
}
Issue Analytics
- State:
- Created 4 years ago
- Reactions:21
- Comments:21 (5 by maintainers)
Top GitHub Comments
I think we should add that flag as an option into formBuilder.group like:
form: FormGroup = this.formBuilder.group({ ...control }, { waitForAsyncValidation: true });
and have a method in FormGroup to change this flag
I agree that this is the easiest solution. However, weβd need to default to
false
to avoid making breaking changes. Then we could add a migration and flip the default in a future framework version.Would you be comfortable putting together a PR to fix this? Iβd like to fix it, but donβt have the bandwidth right now. Happy to review and merge it though.