fix nextControlErrorKey

it returns now only the error key not a concatenation
of prefix, field name and errorkey

Author: Stefan Schubert

dist/formgroup.d.ts

@@ -43,12 +43,11 @@ export declare class TypedFormGroup<T> extends FormGroup {
      */
     isValidatorRegistered(name: Extract<keyof T, string>, validatorName: string): boolean;
     /**
-     * Returns an error key for the next error (<controlName>.<errorKey>).
+     * Returns an error key for the next error.
      *
      * @param name control key of the form group
-     * @param prefix to be prepend to the error key
      */
-    nextControlErrorKey(name: Extract<keyof T, string>, prefix?: string): string;
+    nextControlErrorKey(name: Extract<keyof T, string>): string;
     /**
      * Dispatches errors to this control and to child controls using given error map.
      *

dist/index.es.js

@@ -198,12 +198,11 @@ var TypedFormGroup = /** @class */ (function (_super) {
             this.registeredValidatorsMap[name].some(function (errorKey) { return errorKey === validatorName; }));
     };
     /**
-     * Returns an error key for the next error (<controlName>.<errorKey>).
+     * Returns an error key for the next error.
      *
      * @param name control key of the form group
-     * @param prefix to be prepend to the error key
      */
-    TypedFormGroup.prototype.nextControlErrorKey = function (name, prefix) {
+    TypedFormGroup.prototype.nextControlErrorKey = function (name) {
         var control = this.get(name);
         if (control && control.errors) {
             // try client side keys first for correct order
@@ -214,7 +213,7 @@ var TypedFormGroup = /** @class */ (function (_super) {
                 error = Object.keys(control.errors).shift();
             }
             if (error) {
-                return "" + (prefix ? prefix + "." : '') + name + "." + error;
+                return error;
             }
         }
         return '';

dist/index.js

@@ -202,12 +202,11 @@ var TypedFormGroup = /** @class */ (function (_super) {
             this.registeredValidatorsMap[name].some(function (errorKey) { return errorKey === validatorName; }));
     };
     /**
-     * Returns an error key for the next error (<controlName>.<errorKey>).
+     * Returns an error key for the next error.
      *
      * @param name control key of the form group
-     * @param prefix to be prepend to the error key
      */
-    TypedFormGroup.prototype.nextControlErrorKey = function (name, prefix) {
+    TypedFormGroup.prototype.nextControlErrorKey = function (name) {
         var control = this.get(name);
         if (control && control.errors) {
             // try client side keys first for correct order
@@ -218,7 +217,7 @@ var TypedFormGroup = /** @class */ (function (_super) {
                 error = Object.keys(control.errors).shift();
             }
             if (error) {
-                return "" + (prefix ? prefix + "." : '') + name + "." + error;
+                return error;
             }
         }
         return '';

src/formgroup.spec.ts

@@ -11,7 +11,7 @@ interface NestedType {
   deepValue: string;
 }
 
-test('Test FormGroupControl creation', () => {
+test('Test FormGroupControl creation #1', () => {
   const factory = new BaseFormControlFactory<TestType>(
     { value: 'testValue', nested: { deepValue: 'deepTestValue' } },
     { value: [['req', Validators.required]], nested: [] }
@@ -60,14 +60,14 @@ test('Test FormGroupControl creation', () => {
   expect(nestedGroup.hasControlErrors('deepValue')).toBe(true);
   expect(group.isValidatorRegistered('value', 'required')).toBe(false);
   expect(nestedGroup.isValidatorRegistered('deepValue', 'required')).toBe(false);
-  expect(group.nextControlErrorKey('value')).toBe('value.req');
-  expect(nestedGroup.nextControlErrorKey('deepValue')).toBe('deepValue.req');
+  expect(group.nextControlErrorKey('value')).toBe('req');
+  expect(nestedGroup.nextControlErrorKey('deepValue')).toBe('req');
 
   valueControl.setErrors({ other: 'Other Error!' });
-  expect(group.nextControlErrorKey('value')).toBe('value.other');
+  expect(group.nextControlErrorKey('value')).toBe('other');
 });
 
-test('Test FormGroupControl creation', () => {
+test('Test FormGroupControl creation #2', () => {
   const nestedFactory = new BaseFormControlFactory<NestedType>(
     { deepValue: 'deepTestValue' },
     { deepValue: [['req', Validators.required]] }

src/formgroup.ts

@@ -88,12 +88,11 @@ export class TypedFormGroup<T> extends FormGroup {
   }
 
   /**
-   * Returns an error key for the next error (<controlName>.<errorKey>).
+   * Returns an error key for the next error.
    *
    * @param name control key of the form group
-   * @param prefix to be prepend to the error key
    */
-  nextControlErrorKey(name: Extract<keyof T, string>, prefix?: string): string {
+  nextControlErrorKey(name: Extract<keyof T, string>): string {
     const control = this.get(name);
     if (control && control.errors) {
       // try client side keys first for correct order
@@ -105,7 +104,7 @@ export class TypedFormGroup<T> extends FormGroup {
         error = Object.keys(control.errors).shift();
       }
       if (error) {
-        return `${prefix ? `${prefix}.` : ''}${name}.${error}`;
+        return error;
       }
     }
     return '';