12

I'm using Angular5, and I have a list of fields, each has a name and FormControl. I try to dynamically add the controls to the group, using this code, but I got an error.

const formControlFields = [
  {name: 'field1',control: [null, []] as FormControl},
  {name: 'field2',control: [null, []] as FormControl}
];

const formGroup: FormGroup = new FormGroup({});
formControlFields.forEach( f =>  formGroup.addControl(f.name,f.control));
this.form = new FormGroup({groups:formGroup});

This is the error I get:

ERROR TypeError: control.setParent is not a function

at FormGroup.registerControl (forms.js:4352)
at FormGroup.addControl (forms.js:4372)
at eval (trade-search.component.ts:142)
matchifang
  • 5,190
  • 12
  • 47
  • 76

3 Answers3

9

Instead use the new FormControl() syntax and it will work:

ngOnInit() {
  const formControlFields = [
    { name: 'field1', control: new FormControl(null, []) },
    { name: 'field2', control: new FormControl(null, []) }
  ];
  const formGroup: FormGroup = new FormGroup({});
  formControlFields.forEach(f => formGroup.addControl(f.name, f.control));
  this.form = new FormGroup({ groups: formGroup });
  console.log(this.form);
} 

Demo: https://stackblitz.com/edit/angular-9vbsgf?file=src%2Fapp%2Fapp.component.ts

AT82
  • 71,416
  • 24
  • 140
  • 167
1

I'm using Angular 14, and for those having this problem

this worked for me:

this.formGroup?.addControl('control-name', new FormControl('', 
  [Validators.required])
);
Siddhartha Mukherjee
  • 2,703
  • 2
  • 24
  • 29
Ropoko
  • 11
  • 3
0

Seems like formState for control is not passed or correctly set when using casting with as operator:

 control: [null, []] as FormControl

However providing any value to FormControl constructor will solve the issue:

ngOnInit() {
        this.form = new FormGroup({
            'field1': new FormControl(''),
            'field1': new FormControl('')
        }
        console.log(this.form);
    }
Sanid Sa
  • 264
  • 3
  • 9