Turn tsconfig settings to 11 (#34)

* `"strict": true,`
* `"noUnusedLocals": true,`
* `"noUnusedParameters": true,`
* `"forceConsistentCasingInFileNames": true,`

Also updated to typescript 2.4.1.

Also simplify the implementation of composeReducers. The composed reducer will assemble initial state automatically during Redux initialization; we don't need to do all that state manipulation manually.

Fixes #26

Author: SethDavenport

.gitignore

@@ -48,3 +48,5 @@ dist
 
 source/*ngfactory.ts
 source/*ngsummary.json
+
+*.tgz

.npmignore

@@ -8,3 +8,4 @@ coverage/
 .vscode/
 docs/
 webpack/
+*.tgz

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@angular-redux/form",
-  "version": "6.5.1",
+  "version": "6.5.2",
   "description": "Build Angular 2+ forms with Redux",
   "dependencies": {
     "immutable": "^3.8.1"
@@ -42,7 +42,7 @@
     "reflect-metadata": "^0.1.3",
     "rimraf": "^2.5.4",
     "rxjs": "^5.0.1",
-    "typescript": "^2.1.0",
+    "typescript": "^2.4.1",
     "webpack": "^2.1.0-beta.25",
     "zone.js": "^0.8.4"
   },

source/compose-reducers.ts

@@ -1,30 +1,6 @@
 import {Reducer, Action} from 'redux';
 
-import {State} from './state';
-
 export const composeReducers =
-    <State>(...reducers: Reducer<State>[]): Reducer<State> => {
-  // Pull from each reducer its initial state value, from the default
-  // value of the `state' argument. So we are provided with {initialState},
-  // and then with the individual initial states of each reducer. We
-  // compose all these states together to produce a true 'initial state'
-  // for our composed reducer.
-  let state: State = null;
-
-  for (const reducer of reducers) {
-    const result = reducer.apply(null, [undefined, {type: ''}]);
-    if (result === undefined) {
-      continue;
-    }
-
-    if (state == null) {
-      state = result;
-    }
-    else {
-      state = State.inspect(state).merge(null, result);
-    }
-  }
-
-  return (s: State = state, action: Action) =>
-    reducers.reduce((st, reducer) => reducer(st, action), s);
-};
+  <State>(...reducers: Reducer<State>[]): Reducer<State> => 
+    (s: State, action: Action) =>
+      reducers.reduce((st, reducer) => reducer(st, action), s);

source/connect-array.ts

@@ -43,9 +43,9 @@ import {controlPath, selectValueAccessor} from './shims';
 
 export class ConnectArrayTemplate {
   constructor(
-    public $implicit,
+    public $implicit: any,
     public index: number,
-    public item
+    public item: any
   ) {}
 }
 
@@ -85,7 +85,7 @@ export class ConnectArray extends ControlContainer implements OnInit {
   }
 
   @Input()
-  set connectArrayOf(collection) {
+  set connectArrayOf(collection: any) {
     this.key = collection;
 
     this.resetState(this.store.getState());
@@ -131,7 +131,7 @@ export class ConnectArray extends ControlContainer implements OnInit {
     this.formDirective.form.removeControl(this.key);
   }
 
-  private resetState(state) {
+  private resetState(state: any) {
     if (this.key == null || this.key.length === 0) {
       return; // no state to retreive if no key is set
     }
@@ -174,7 +174,7 @@ export class ConnectArray extends ControlContainer implements OnInit {
     }
   }
 
-  private registerInternals(array) {
+  private registerInternals(array: any) {
     array.registerControl = () => {};
     array.registerOnChange = () => {};
 
@@ -188,7 +188,7 @@ export class ConnectArray extends ControlContainer implements OnInit {
     });
   }
 
-  private patchDescendantControls(viewRef) {
+  private patchDescendantControls(viewRef: any) {
     const groups = Object.keys(viewRef._view)
       .map(k => viewRef._view[k])
       .filter(c => c instanceof NgModelGroup);
@@ -205,7 +205,7 @@ export class ConnectArray extends ControlContainer implements OnInit {
     });
   }
 
-  private transform(parent: FormGroup | FormArray, reference): AbstractControl {
+  private transform(parent: FormGroup | FormArray, reference: any): AbstractControl {
     const emptyControl = () => {
       const control = new FormControl(null);
       control.setParent(parent);
@@ -227,7 +227,7 @@ export class ConnectArray extends ControlContainer implements OnInit {
         return emptyControl();
     }
 
-    const iterate = (iterable): FormArray => {
+    const iterate = (iterable: any): FormArray => {
       const array = new FormArray([]);
 
       this.registerInternals(array);
@@ -246,7 +246,7 @@ export class ConnectArray extends ControlContainer implements OnInit {
       return array;
     }
 
-    const associate = (value): FormGroup => {
+    const associate = (value: any): FormGroup => {
       const group = new FormGroup({});
       group.setParent(parent);
 
@@ -280,7 +280,7 @@ export class ConnectArray extends ControlContainer implements OnInit {
 
   private simpleAccessor() {
     return {
-      writeValue: value => this.control.setValue(value),
+      writeValue: (value: any) => this.control.setValue(value),
       registerOnChange() {},
       registerOnTouched() {}
     };

source/connect-base.ts

@@ -29,7 +29,7 @@ export class ConnectBase {
 
   private formSubscription: Subscription;
   protected store: FormStore;
-  protected form;
+  protected form: any;
 
   public get path(): Array<string> {
     const path = typeof this.connect === 'function'
@@ -68,12 +68,14 @@ export class ConnectBase {
       this.stateSubscription = this.store.subscribe(() => this.resetState());
 
       Promise.resolve().then(() => {
-        this.formSubscription = (<any>this.form.valueChanges).debounceTime(0).subscribe(values => this.publish(values));
+        this.formSubscription = (<any>this.form.valueChanges)
+          .debounceTime(0)
+          .subscribe((values: any) => this.publish(values));
       });
     });
   }
 
-  private descendants(path: Array<string>, formElement): Array<ControlPair> {
+  private descendants(path: Array<string>, formElement: any): Array<ControlPair> {
     const pairs = new Array<ControlPair>();
 
     if (formElement instanceof FormArray) {
@@ -122,7 +124,7 @@ export class ConnectBase {
     });
   }
 
-  private publish(value) {
+  private publish(value: any) {
     this.store.valueChanged(this.path, this.form, value);
   }
 

source/connect-reactive.ts

@@ -10,7 +10,7 @@ import {ConnectBase} from './connect-base';
 // For reactive forms (without implicit NgForm)
 @Directive({ selector: 'form[connect][formGroup]' })
 export class ReactiveConnect extends ConnectBase {
-  @Input('formGroup') form;
+  @Input('formGroup') form: any;
 
   constructor(
     protected store: FormStore

source/form-reducer.ts

@@ -7,7 +7,7 @@ import {FORM_CHANGED} from './form-store';
 import {State} from './state';
 
 export const defaultFormReducer = <RootState>(initialState?: RootState | Iterable.Keyed<string, any>) => {
-  const reducer = (state: RootState | Iterable.Keyed<string, any> = initialState, action: Action & {payload?}) => {
+  const reducer = (state: RootState | Iterable.Keyed<string, any> | undefined = initialState, action: Action & {payload?: any}) => {
     switch (action.type) {
       case FORM_CHANGED:
         return State.assign(

source/form-store.ts

@@ -8,7 +8,7 @@ import {Action, Unsubscribe} from 'redux';
 
 export interface AbstractStore<RootState> {
   /// Dispatch an action
-  dispatch(action: Action & {payload}): void;
+  dispatch(action: Action & {payload: any}): void;
 
   /// Retrieve the current application state
   getState(): RootState;
@@ -34,7 +34,7 @@ export class FormStore {
     return this.store.getState();
   }
 
-  subscribe(fn: (state) => void): Unsubscribe {
+  subscribe(fn: (state: any) => void): Unsubscribe {
     return this.store.subscribe(() => fn(this.getState()));
   }
 

source/shims.ts

@@ -14,12 +14,12 @@ export function controlPath(name: string, parent: ControlContainer): string[] {
 }
 
 export function selectValueAccessor(
-    dir: NgControl, valueAccessors: ControlValueAccessor[]): ControlValueAccessor {
+    dir: NgControl, valueAccessors: ControlValueAccessor[]): ControlValueAccessor | null {
   if (!valueAccessors) return null;
 
-  let defaultAccessor: ControlValueAccessor;
-  let builtinAccessor: ControlValueAccessor;
-  let customAccessor: ControlValueAccessor;
+  let defaultAccessor: ControlValueAccessor | null = null;
+  let builtinAccessor: ControlValueAccessor | null = null;
+  let customAccessor: ControlValueAccessor | null = null;
   valueAccessors.forEach((v: ControlValueAccessor) => {
     if (v.constructor === DefaultValueAccessor) {
       defaultAccessor = v;