fix: do not treat null control values as boxed values

Author: MrWolfZ

CHANGELOG.md

@@ -1,5 +1,12 @@
 ## ngrx-forms Changelog
 
+<a name="3.0.2"></a>
+### 3.0.2
+
+#### Bugfixes
+
+* do not treat `null` control values as boxed values, closes [#94](https://github.com/MrWolfZ/ngrx-forms/issues/94)
+
 <a name="3.0.1"></a>
 ### 3.0.1
 

src/control/value-converter.spec.ts

@@ -32,6 +32,16 @@ describe('NgrxValueConverters', () => {
       const stateValue = box(['A']);
       expect(NgrxValueConverters.default<typeof stateValue.value>().convertStateToViewValue(stateValue)).toEqual(stateValue.value);
     });
+
+    it('should return null view value', () => {
+      const viewValue = null;
+      expect(NgrxValueConverters.default<typeof viewValue>().convertViewToStateValue(viewValue)).toBe(viewValue);
+    });
+
+    it('should return undefined view value', () => {
+      const viewValue = undefined;
+      expect(NgrxValueConverters.default<typeof viewValue>().convertViewToStateValue(viewValue)).toBe(viewValue);
+    });
   });
 
   describe('dateToISOString', () => {

src/control/value-converter.ts

@@ -9,7 +9,7 @@ export interface NgrxValueConverter<TView, TState> {
 export const NgrxValueConverters = {
   default<T>() {
     return {
-      convertViewToStateValue: value => typeof value === 'object' ? box(value) : value,
+      convertViewToStateValue: value => typeof value === 'object' && value !== null ? box(value) : value,
       convertStateToViewValue: unbox,
     } as NgrxValueConverter<T, Boxed<T> | T>;
   },