[DSC-2599] fix error clearing on blur

Author: FrancescoMolinaro

src/app/shared/form/form.service.spec.ts

@@ -278,7 +278,7 @@ describe('FormService test suite', () => {
 
     service.removeErrorFromField(control, model, 'error.required');
 
-    expect(control.errors).toEqual({ required: null });
+    expect(control.errors).toEqual(null);
   });
 
   it('should remove errors from fields of concat group', () => {
@@ -300,7 +300,7 @@ describe('FormService test suite', () => {
 
     // the group's inputs should no longer have an error
     Object.values(control.controls).forEach((subControl: AbstractControl) => {
-      expect(control.errors).toEqual({ [messageKey]: null });
+      expect(control.errors).toEqual(null);
     });
   });
 

src/app/shared/form/form.service.ts

@@ -26,6 +26,7 @@ import {
 import { environment } from '../../../environments/environment';
 import { AppState } from '../../app.reducer';
 import {
+  hasValue,
   isEmpty,
   isNotUndefined,
 } from '../empty.util';
@@ -53,7 +54,7 @@ export class FormService {
   constructor(
     private formBuilderService: FormBuilderService,
     private store: Store<AppState>,
-    private trnslateService: TranslateService) {
+    private translateService: TranslateService) {
   }
 
   /**
@@ -164,7 +165,7 @@ export class FormService {
       .map((errorKey) => {
         const defaultErrorKey = `error.validation.${errorKey}`;
         const customErrorKey = `error.validation.${formId}.${errorKey}`;
-        const hasDefaultLabel = this.trnslateService.instant(defaultErrorKey) !== defaultErrorKey;
+        const hasDefaultLabel = this.translateService.instant(defaultErrorKey) !== defaultErrorKey;
         return hasDefaultLabel ? defaultErrorKey : customErrorKey;
       });
     errors.forEach((error) => this.addError(formId, fieldId, fieldIndex, error));
@@ -211,7 +212,7 @@ export class FormService {
       error[errorKey] = null;
       const updatedError = { ...field.errors, ...error };
       field.setErrors(updatedError);
-      field.setValidators(() => updatedError);
+      field.clearValidators();
       field.updateValueAndValidity();
     }
 
@@ -224,9 +225,10 @@ export class FormService {
       });
     }
 
-    const hasDifferentErrors = Object.keys(field.errors).filter((key) => field.errors[key]).length > 0;
+    const currentErrors = field.errors;
+    const hasDifferentErrors = hasValue(currentErrors) && Object.keys(currentErrors).filter((key) => currentErrors[key]).length > 0;
 
-    if (isEmpty(field.errors) || !hasDifferentErrors) {
+    if (!hasDifferentErrors) {
       field.markAsUntouched();
     }