fix onNext for multiPage steps

LinkedInAttic · Feb 6, 2013 · 72bc5d6 · 72bc5d6
1 parent 7237f76
commit 72bc5d6
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 38 deletions.
diff --git a/js/hopscotch-0.0.4.js b/js/hopscotch-0.0.4.js
@@ -1297,6 +1297,7 @@
       var bubble = getBubble(),
           self = this,
           step,
+          origStep,
           origStepNum,
           wasMultiPage,
           changeStepCb;
@@ -1305,8 +1306,9 @@
 
       doCallbacks = utils.valOrDefault(doCallbacks, true);
       step = getCurrStep();
-      wasMultiPage = step.multipage;
+      origStep = step;
       origStepNum = currStepNum;
+      wasMultiPage = step.multipage;
 
       /**
        * Callback for goToStepWithTarget
@@ -1321,11 +1323,11 @@
 
         if (doCallbacks) {
           // Step-specific callbacks
-          if (direction > 0 && step.onNext) {
-            step.onNext();
+          if (direction > 0 && origStep.onNext) {
+            origStep.onNext();
           }
-          else if (direction < 0 && step.onPrev) {
-            step.onPrev();
+          else if (direction < 0 && origStep.onPrev) {
+            origStep.onPrev();
           }
 
           // Tour-wide next/prev callbacks
@@ -1352,7 +1354,7 @@
           utils.invokeCallbacks('error', [currTour.id, currStepNum]);
           return this.endTour(true, false);
         }
-        this.changeStepCb(currStepNum);
+        changeStepCb(currStepNum);
       }
 
       return this;