Move logging to inside conditional

HiCommon · Aug 18, 2020 · 0c17dfb · 0c17dfb
1 parent 1094d95
commit 0c17dfb
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 10 deletions.
diff --git a/lib/components/Steps.js b/lib/components/Steps.js
@@ -60,12 +60,11 @@ var Steps = function (_Component) {
   }, {
     key: "componentDidUpdate",
     value: function componentDidUpdate(prevProps) {
-      (0, _log2.default)("Updated steps", {
-        oldSteps: this.getSteps(prevProps.children),
-        newSteps: this.getSteps()
-      });
-
       if (this.getSteps(prevProps.children).length !== this.getSteps().length) {
+        (0, _log2.default)("Updated steps", {
+          oldSteps: this.getSteps(prevProps.children),
+          newSteps: this.getSteps()
+        });
         this.context.wizard.setSteps(this.getSteps());
       }
     }

diff --git a/src/components/Steps.js b/src/components/Steps.js
@@ -23,12 +23,11 @@ class Steps extends Component {
   }
 
   componentDidUpdate(prevProps) {
-    log("Updated steps", {
-      oldSteps: this.getSteps(prevProps.children),
-      newSteps: this.getSteps(),
-    });
-
     if (this.getSteps(prevProps.children).length !== this.getSteps().length) {
+      log("Updated steps", {
+        oldSteps: this.getSteps(prevProps.children),
+        newSteps: this.getSteps(),
+      });
       this.context.wizard.setSteps(this.getSteps());
     }
   }