Refactor DynamicFormControlListDataProvider to a generic interface for consistency

Author: Steve Rhoades

projects/ng-dynamic-forms/core/src/lib/model/misc/dynamic-form-control-data.model.ts

@@ -11,8 +11,8 @@ export interface DynamicFormControlDataConfig {
   service: any;
 }
 
-export interface DynamicFormControlListDataProvider {
-  fetchList(value: string): Observable<any[]>;
+export interface DynamicFormControlListDataProvider<T> {
+  fetchList(value: string): Observable<T[]>;
 }
 
 export interface DynamicFormControlOptionDataProvider<T> {

projects/ng-dynamic-forms/core/src/lib/service/dynamic-form-data.service.spec.ts

@@ -15,8 +15,8 @@ import {DynamicFormGroupModel} from '../model/form-group/dynamic-form-group.mode
 import {DynamicFormOptionConfig} from '../model/dynamic-option-control.model';
 
 @Injectable()
-class TestProvider implements DynamicFormControlListDataProvider, DynamicFormControlOptionDataProvider<string> {
-  fetchList(value: string): Observable<any[]> {
+class TestProvider implements DynamicFormControlListDataProvider<string>, DynamicFormControlOptionDataProvider<string> {
+  fetchList(value: string): Observable<string[]> {
     return of(['test']);
   }
 

projects/ng-dynamic-forms/core/src/lib/service/dynamic-form-data.service.ts

@@ -45,7 +45,7 @@ export class DynamicFormDataService {
       });
   }
 
-  isListProvider(provider: DynamicFormControlListDataProvider): boolean {
+  isListProvider(provider: DynamicFormControlListDataProvider<any>): boolean {
     return provider.fetchList !== undefined;
   }