Index: lams_monitoring/web/timeLimit.jsp =================================================================== diff -u -r47dddf6db4afaba99d8f3f88fc50b16df9251a76 -r2bc8f0c2f65f02cb65aa7e90d0bbb612494a4588 --- lams_monitoring/web/timeLimit.jsp (.../timeLimit.jsp) (revision 47dddf6db4afaba99d8f3f88fc50b16df9251a76) +++ lams_monitoring/web/timeLimit.jsp (.../timeLimit.jsp) (revision 2bc8f0c2f65f02cb65aa7e90d0bbb612494a4588) @@ -58,7 +58,7 @@ // in minutes since learner entered the activity var relativeTimeLimit = , // in seconds since epoch started - absoluteTimeLimit = ; + absoluteTimeLimitFinish = ; $(document).ready(function(){ let timeLimitWidget = $('#time-limit-widget'), @@ -103,13 +103,13 @@ }); // create counter if absolute time limit is set - if (absoluteTimeLimit) { + if (absoluteTimeLimitFinish) { let timeLimitWidgetOpen = sessionStorage.getItem('lams-time-limit-widget-open') === "true"; showTimeLimitWidget(timeLimitWidgetOpen, timeLimitWidgetOpen); updateAbsoluteTimeLimitCounter(); // expand time limit panel if absolute time limit is set and not expired - if (absoluteTimeLimit > new Date().getTime() / 1000) { + if (absoluteTimeLimitFinish > new Date().getTime() / 1000) { $('#time-limit-collapse').collapse('show'); } } @@ -137,7 +137,7 @@ if (toggle === true && displayedRelativeTimeLimit > 0) { relativeTimeLimit = displayedRelativeTimeLimit; // when teacher enables relative time limit, absolute one gets disabled - absoluteTimeLimit = null; + absoluteTimeLimitFinish = null; updateTimeLimitOnServer(); } return; @@ -182,7 +182,7 @@ // start/stop if (toggle === false) { - absoluteTimeLimit = null; + absoluteTimeLimitFinish = null; updateAbsoluteTimeLimitCounter(); return; } @@ -194,7 +194,7 @@ } if (toggle === 'stop') { - absoluteTimeLimit = Math.round(new Date().getTime() / 1000); + absoluteTimeLimitFinish = Math.round(new Date().getTime() / 1000); updateAbsoluteTimeLimitCounter(); } return; @@ -243,7 +243,7 @@ function updateTimeLimitOnServer() { // absolute time limit has higher priority - if (absoluteTimeLimit != null) { + if (absoluteTimeLimitFinish != null) { relativeTimeLimit = 0; } @@ -254,7 +254,7 @@ 'data': { 'toolContentID' : '', 'relativeTimeLimit' : relativeTimeLimit, - 'absoluteTimeLimit' : absoluteTimeLimit, + 'absoluteTimeLimitFinish' : absoluteTimeLimitFinish, '' : '' }, success : function(){ @@ -273,7 +273,7 @@ $('#relative-time-limit-start').removeClass('hidden').prop('disabled', true); } - if (absoluteTimeLimit === null) { + if (absoluteTimeLimitFinish === null) { // no absolute time limit? destroy the counter $('.absolute-time-limit-counter').countdown('destroy'); $('.absolute-time-limit-value').empty(); @@ -291,7 +291,7 @@ $('#absolute-time-limit-cancel').removeClass('hidden'); $('#absolute-time-limit-enabled').removeClass('hidden'); $('#absolute-time-limit-start').addClass('hidden'); - $('#absolute-time-limit-finish-now').prop('disabled', absoluteTimeLimit <= Math.round(new Date().getTime() / 1000)); + $('#absolute-time-limit-finish-now').prop('disabled', absoluteTimeLimitFinish <= Math.round(new Date().getTime() / 1000)); } } }); @@ -300,22 +300,22 @@ function updateAbsoluteTimeLimitCounter(secondsLeft, start) { var now = Math.round(new Date().getTime() / 1000), // preset means that counter is set just on screen and the time limit is not enforced for learners - preset = start !== true && absoluteTimeLimit == null; + preset = start !== true && absoluteTimeLimitFinish == null; if (secondsLeft) { if (!preset) { // time limit is already enforced on server, so update it there now - absoluteTimeLimit = now + secondsLeft; + absoluteTimeLimitFinish = now + secondsLeft; updateTimeLimitOnServer(); } } else { - if (absoluteTimeLimit == null) { + if (absoluteTimeLimitFinish == null) { // disable the counter updateTimeLimitOnServer(); return; } // counter initialisation on page load or "finish now" - secondsLeft = absoluteTimeLimit - now; + secondsLeft = absoluteTimeLimitFinish - now; if (secondsLeft <= 0) { // finish now updateTimeLimitOnServer(); @@ -505,7 +505,7 @@ } - +