Bindu, Tarun | MOBN-1551 | Refactor code and fix unit tests

bahmni-msf · Jan 20, 2021 · ecaea67 · ecaea67
1 parent ca1d1d9
commit ecaea67
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/components/Container.jsx b/src/components/Container.jsx
@@ -65,7 +65,12 @@ export class Container extends addMoreDecorator(Component) {
       ...previousState,
       data: previousState.data.update(formFieldPath, value, errors),
       collapse: undefined,
-    }), () => { onActionDone(); this.onValueUpdated(); });
+    }), () => {
+      if (onActionDone) {
+        onActionDone();
+      }
+      this.onValueUpdated();
+    });
   }
 
   onValueUpdated() {
@@ -210,10 +215,10 @@ Container.propTypes = {
     version: PropTypes.string.isRequired,
   }),
   observations: PropTypes.array.isRequired,
+  onValueUpdated: PropTypes.func,
   patient: PropTypes.object.isRequired,
   translations: PropTypes.object.isRequired,
   validate: PropTypes.bool.isRequired,
   validateForm: PropTypes.bool.isRequired,
-  onValueUpdated: PropTypes.func,
 };