Adding new updateParentForms (internal), using with setPristineActionReducer to update parent forms with pristine state. Fixes #429

Author: davidkpiano

src/form/is-pristine.js

@@ -1,11 +1,15 @@
 export default function isPristine(formState) {
   if (!formState) return false;
 
-  // Field is pending
+  // Form is pristine
   if (!formState.$form) {
     return formState.pristine;
   }
 
   // Every field in form is pristine
-  return Object.keys(formState).every((key) => isPristine(formState[key]));
+  return Object.keys(formState).every((key) => {
+    if (key === '$form') return true;
+
+    return isPristine(formState[key]);
+  });
 }

src/reducers/form/set-pristine-action-reducer.js

@@ -1,14 +1,20 @@
 import actionTypes from '../../action-types';
 import updateField from '../../utils/update-field';
+import isPristine from '../../form/is-pristine';
+import updateParentForms from '../../utils/update-parent-forms';
 
-export default function setValidityActionReducer(state, action, localPath) {
+export default function setPristineActionReducer(state, action, localPath) {
   if (action.type !== actionTypes.SET_PRISTINE) {
     return state;
   }
 
-  return updateField(state, localPath, {
+  const updatedField = updateField(state, localPath, {
     pristine: true,
   }, {
     pristine: true,
   });
+
+  return updateParentForms(updatedField, localPath, (form) => ({
+    pristine: isPristine(form),
+  }));
 }

src/utils/update-parent-forms.js

@@ -0,0 +1,22 @@
+import i from 'icepick';
+import get from './get';
+
+export default function updateParentForms(state, localPath, updater) {
+  const parentLocalPath = localPath.slice(0, -1);
+
+  const value = parentLocalPath.length
+    ? get(state, parentLocalPath)
+    : state;
+
+  const form = value.$form;
+
+  const updatedValue = typeof updater === 'function'
+    ? updater(value)
+    : updater;
+
+  const newState = i.setIn(state, [...parentLocalPath, '$form'], i.merge(form, updatedValue));
+
+  if (!parentLocalPath.length) return newState;
+
+  return updateParentForms(newState, parentLocalPath, updater);
+}

test/form-reducer-actions-spec.js

@@ -86,6 +86,62 @@ describe('formReducer() (V1)', () => {
         args: [],
         expectedField: { pristine: true },
       },
+      {
+        action: actions.setPristine,
+        initialState: {
+          $form: {
+            ...initialFieldState,
+            pristine: false,
+          },
+          name: {
+            ...initialFieldState,
+            pristine: false,
+          },
+        },
+        args: [],
+        expectedField: { pristine: true },
+        expectedForm: { pristine: true },
+      },
+      {
+        action: actions.setPristine,
+        initialState: {
+          $form: {
+            ...initialFieldState,
+            pristine: false,
+          },
+          name: {
+            ...initialFieldState,
+            pristine: false,
+          },
+          other: {
+            ...initialFieldState,
+            pristine: true,
+          },
+        },
+        args: [],
+        expectedField: { pristine: true },
+        expectedForm: { pristine: true },
+      },
+      {
+        action: actions.setPristine,
+        initialState: {
+          $form: {
+            ...initialFieldState,
+            pristine: false,
+          },
+          name: {
+            ...initialFieldState,
+            pristine: false,
+          },
+          other: {
+            ...initialFieldState,
+            pristine: false,
+          },
+        },
+        args: [],
+        expectedField: { pristine: true },
+        expectedForm: { pristine: false },
+      },
     ],
     [actionTypes.SET_DIRTY]: [
       {