fix: allow setting null and undefined values with curried version of setValue

Author: MrWolfZ

CHANGELOG.md

@@ -1,5 +1,12 @@
 ## ngrx-forms Changelog
 
+<a name="3.0.4"></a>
+### 3.0.4
+
+#### Bugfixes
+
+* allow setting `null` and `undefined` values with curried version of `setValue`, thanks @chrissena for reporting this bug, closes [#116](https://github.com/MrWolfZ/ngrx-forms/issues/116)
+
 <a name="3.0.3"></a>
 ### 3.0.3
 

src/state.ts

@@ -560,7 +560,7 @@ export type FormState<T> = InferredFormState<InferenceWrapper<T>>;
  * This function determines if a value is a form state.
  */
 export function isFormState<TValue = any>(state: any): state is FormState<TValue> {
-  return state.hasOwnProperty('id') && state.hasOwnProperty('value') && state.hasOwnProperty('errors');
+  return !!state && state.hasOwnProperty('id') && state.hasOwnProperty('value') && state.hasOwnProperty('errors');
 }
 
 /**

src/update-function/set-value.spec.ts

@@ -52,4 +52,25 @@ describe(setValue.name, () => {
 
     expect(resultState).not.toEqual(INITIAL_STATE);
   });
+
+  it('should support setting null value curried', () => {
+    const resultState = setValue<string | null>(null)(INITIAL_STATE.controls.inner);
+    expect(resultState).not.toBe(INITIAL_STATE.controls.inner);
+  });
+
+  it('should support setting null value uncurried', () => {
+    const resultState = setValue<string | null>(INITIAL_STATE.controls.inner, null);
+    expect(resultState).not.toBe(INITIAL_STATE.controls.inner);
+  });
+
+  it('should support setting undefined value curried', () => {
+    const resultState = setValue<string | undefined>(undefined)(INITIAL_STATE.controls.inner);
+    expect(resultState).not.toBe(INITIAL_STATE.controls.inner);
+  });
+
+  it('should support setting undefined value uncurried', () => {
+    const resultState = setValue<string | undefined>(INITIAL_STATE.controls.inner, undefined);
+    expect(resultState).not.toBe(INITIAL_STATE.controls.inner);
+  });
+
 });