Switch to next if target is EXIT

oat-sa · Sep 22, 2015 · 9fa58ea · 9fa58ea
1 parent f5f1697
commit 9fa58ea
Showing 1 changed file with 19 additions and 27 deletions.
diff --git a/qtism/runtime/tests/AssessmentTestSession.php b/qtism/runtime/tests/AssessmentTestSession.php
@@ -1585,9 +1585,11 @@ protected function nextRouteItem($ignoreBranchings = false, $ignorePreConditions
 	                        $this->endTestSession();
 	                    }
 	                    else if ($target === 'EXIT_TESTPART') {
+                            $route->next();
 	                        $this->moveNextTestPart();
 	                    }
 	                    else if ($target === 'EXIT_SECTION') {
+                            $route->next();
 	                        $this->moveNextAssessmentSection();
 	                    }
 	                    else {
@@ -1653,49 +1655,39 @@ public function moveNextTestPart() {
 	    $route = $this->getRoute();
 	    $from = $route->current();
 
-        if ($route->isLast() === true) {
-            $this->endTestSession();
-        } else {
-            $route->next();
-            while ($route->valid() === true && $route->current()->getTestPart() === $from->getTestPart()) {
-                $this->nextRouteItem();
-            }
-
-            if ($this->isRunning() === true) {
-                $this->interactWithItemSession();
-            }
+        while ($route->valid() === true && $route->current()->getTestPart() === $from->getTestPart()) {
+            $this->nextRouteItem();
+        }
+
+        if ($this->isRunning() === true) {
+            $this->interactWithItemSession();
         }
 	}
-	
+
 	/**
 	 * Set the position in the Route at the very next assessmentSection in the route sequence.
-	 * 
+	 *
 	 * * If there is no assessmentSection left in the flow, the test session ends gracefully.
 	 * * If there are still pending responses, they are processed.
-	 * 
+	 *
 	 * @throws AssessmentTestSessionException If the test is not running.
 	 */
 	public function moveNextAssessmentSection() {
-	    
+
 	    if ($this->isRunning() === false) {
 	        $msg = "Cannot move to the next assessmentSection while the state of the test session is INITIAL or CLOSED.";
 	        throw new AssessmentTestSessionException($msg, AssessmentTestSessionException::STATE_VIOLATION);
 	    }
-	    
+
 	    $route = $this->getRoute();
 	    $from = $route->current();
 
-        if ($route->isLast() === true) {
-            $this->endTestSession();
-        } else {
-            $route->next();
-            while ($route->valid() === true && $route->current()->getAssessmentSection() === $from->getAssessmentSection()) {
-                $this->nextRouteItem();
-            }
-
-            if ($this->isRunning() === true) {
-                $this->interactWithItemSession();
-            }
+        while ($route->valid() === true && $route->current()->getAssessmentSection() === $from->getAssessmentSection()) {
+            $this->nextRouteItem();
+        }
+
+        if ($this->isRunning() === true) {
+            $this->interactWithItemSession();
         }
 	}