Skip to content

Commit

Permalink
MoveNext and MoveBack were not behaving correctly
Browse files Browse the repository at this point in the history
when $allowTimeout = false
  • Loading branch information
bugalot committed Jul 22, 2015
1 parent 60a3c45 commit 184eb57
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions qtism/runtime/tests/AssessmentTestSession.php
Original file line number Diff line number Diff line change
Expand Up @@ -1421,7 +1421,7 @@ public function moveNext($allowTimeout = false) {
if ($allowTimeout === false) {
try {
// Do not check min times, include item timings.
$this->checkTimeLimits(false, true);
$this->checkTimeLimits(false, true, false);

// No exception thrown, we found a non timed-out item to be
// presented to the candidate.
Expand Down Expand Up @@ -1474,10 +1474,10 @@ public function moveBack($allowTimeout = false) {

while ($route->isFirst() === false) {

if ($allowTimeout = false) {
if ($allowTimeout === false) {
try {
// Do not check min times, include item timings.
$this->checkTimeLimits(false, true);
$this->checkTimeLimits(false, true, false);

// No exception thrown, return.
$this->interactWithItemSession();
Expand All @@ -1499,7 +1499,7 @@ public function moveBack($allowTimeout = false) {
// of the first RouteItem of the test...
if ($allowTimeout === false) {
try {
$this->checkTimeLimits(false, true);
$this->checkTimeLimits(false, true, false);
$this->interactWithItemSession();
}
catch (AssessmentTestSessionException $e) {
Expand Down

0 comments on commit 184eb57

Please sign in to comment.