Revert "fix(forms): don't mutate validators array (angular#47830)" (a…

…ngular#47845)

This reverts commit 0329c13.

PR Close angular#47845

packages/core/test/bundling/forms_reactive/bundle.golden_symbols.json

@@ -632,6 +632,12 @@
   {
     "name": "cleanUpView"
   },
+  {
+    "name": "coerceToAsyncValidator"
+  },
+  {
+    "name": "coerceToValidator"
+  },
   {
     "name": "collectNativeNodes"
   },

packages/core/test/bundling/forms_template_driven/bundle.golden_symbols.json

@@ -608,9 +608,15 @@
   {
     "name": "cleanUpView"
   },
+  {
+    "name": "coerceToAsyncValidator"
+  },
   {
     "name": "coerceToBoolean"
   },
+  {
+    "name": "coerceToValidator"
+  },
   {
     "name": "collectNativeNodes"
   },

packages/forms/src/model/abstract_model.ts

@@ -372,15 +372,15 @@ export abstract class AbstractControl<TValue = any, TRawValue extends TValue = T
    *
    * @internal
    */
-  private _composedValidatorFn!: ValidatorFn|null;
+  private _composedValidatorFn: ValidatorFn|null;
 
   /**
    * Contains the result of merging asynchronous validators into a single validator function
    * (combined using `Validators.composeAsync`).
    *
    * @internal
    */
-  private _composedAsyncValidatorFn!: AsyncValidatorFn|null;
+  private _composedAsyncValidatorFn: AsyncValidatorFn|null;
 
   /**
    * Synchronous validators as they were provided:
@@ -390,7 +390,7 @@ export abstract class AbstractControl<TValue = any, TRawValue extends TValue = T
    *
    * @internal
    */
-  private _rawValidators!: ValidatorFn|ValidatorFn[]|null;
+  private _rawValidators: ValidatorFn|ValidatorFn[]|null;
 
   /**
    * Asynchronous validators as they were provided:
@@ -401,7 +401,7 @@ export abstract class AbstractControl<TValue = any, TRawValue extends TValue = T
    *
    * @internal
    */
-  private _rawAsyncValidators!: AsyncValidatorFn|AsyncValidatorFn[]|null;
+  private _rawAsyncValidators: AsyncValidatorFn|AsyncValidatorFn[]|null;
 
   /**
    * The current value of the control.
@@ -427,8 +427,10 @@ export abstract class AbstractControl<TValue = any, TRawValue extends TValue = T
   constructor(
       validators: ValidatorFn|ValidatorFn[]|null,
       asyncValidators: AsyncValidatorFn|AsyncValidatorFn[]|null) {
-    this.setValidators(validators);
-    this.setAsyncValidators(asyncValidators);
+    this._rawValidators = validators;
+    this._rawAsyncValidators = asyncValidators;
+    this._composedValidatorFn = coerceToValidator(this._rawValidators);
+    this._composedAsyncValidatorFn = coerceToAsyncValidator(this._rawAsyncValidators);
   }
 
   /**
@@ -618,8 +620,8 @@ export abstract class AbstractControl<TValue = any, TRawValue extends TValue = T
    * using `addValidators()` method instead.
    */
   setValidators(validators: ValidatorFn|ValidatorFn[]|null): void {
-    this._rawValidators = Array.isArray(validators) ? validators.slice() : validators;
-    this._composedValidatorFn = coerceToValidator(this._rawValidators);
+    this._rawValidators = validators;
+    this._composedValidatorFn = coerceToValidator(validators);
   }
 
   /**
@@ -633,8 +635,8 @@ export abstract class AbstractControl<TValue = any, TRawValue extends TValue = T
    * using `addAsyncValidators()` method instead.
    */
   setAsyncValidators(validators: AsyncValidatorFn|AsyncValidatorFn[]|null): void {
-    this._rawAsyncValidators = Array.isArray(validators) ? validators.slice() : validators;
-    this._composedAsyncValidatorFn = coerceToAsyncValidator(this._rawAsyncValidators);
+    this._rawAsyncValidators = validators;
+    this._composedAsyncValidatorFn = coerceToAsyncValidator(validators);
   }
 
   /**

packages/forms/test/form_control_spec.ts

@@ -266,16 +266,6 @@ describe('FormControl', () => {
       expect(c.valid).toEqual(true);
     });
 
-    it('should not mutate the validators array when overriding using setValidators', () => {
-      const control = new FormControl('');
-      const originalValidators = [Validators.required];
-
-      control.setValidators(originalValidators);
-      control.addValidators(Validators.minLength(10));
-
-      expect(originalValidators.length).toBe(1);
-    });
-
     it('should override validators by setting `control.validator` field value', () => {
       const c = new FormControl('');
       expect(c.valid).toEqual(true);
@@ -367,30 +357,6 @@ describe('FormControl', () => {
       expect(c.hasValidator(Validators.required)).toEqual(false);
     });
 
-    it('should not mutate the validators array when adding/removing sync validators', () => {
-      const originalValidators = [Validators.required];
-      const control = new FormControl('', originalValidators);
-
-      control.addValidators(Validators.min(10));
-      expect(originalValidators.length).toBe(1);
-
-      control.removeValidators(Validators.required);
-      expect(originalValidators.length).toBe(1);
-    });
-
-    it('should not mutate the validators array when adding/removing async validators', () => {
-      const firstValidator = asyncValidator('one');
-      const secondValidator = asyncValidator('two');
-      const originalValidators = [firstValidator];
-      const control = new FormControl('', null, originalValidators);
-
-      control.addAsyncValidators(secondValidator);
-      expect(originalValidators.length).toBe(1);
-
-      control.removeAsyncValidators(firstValidator);
-      expect(originalValidators.length).toBe(1);
-    });
-
     it('should return false when checking presence of a validator not identical by reference',
        () => {
          const minValidator = Validators.min(5);
@@ -552,16 +518,6 @@ describe('FormControl', () => {
          expect(c.valid).toEqual(true);
        }));
 
-    it('should not mutate the validators array when overriding using setValidators', () => {
-      const control = new FormControl('');
-      const originalValidators = [asyncValidator('one')];
-
-      control.setAsyncValidators(originalValidators);
-      control.addAsyncValidators(asyncValidator('two'));
-
-      expect(originalValidators.length).toBe(1);
-    });
-
     it('should override validators by setting `control.asyncValidator` field value',
        fakeAsync(() => {
          const c = new FormControl('');