Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/controller/LearningController.java =================================================================== diff -u -rfd72af2c8d69fd4c43790f58b2ff7f0db72ca58b -r79db1b495adba49dd54a6250c1221a326f153633 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/controller/LearningController.java (.../LearningController.java) (revision fd72af2c8d69fd4c43790f58b2ff7f0db72ca58b) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/controller/LearningController.java (.../LearningController.java) (revision 79db1b495adba49dd54a6250c1221a326f153633) @@ -1222,6 +1222,7 @@ @RequestMapping("/showResultsForTeacher") public String showResultsForTeacher(HttpServletRequest request) { long toolContentId = WebUtil.readLongParam(request, AssessmentConstants.PARAM_TOOL_CONTENT_ID); + boolean embedded = WebUtil.readBooleanParam(request, "embedded", false); UserDTO user = LearningController.getCurrentUser(); Lesson lesson = lessonService.getLessonByToolContentId(toolContentId); @@ -1273,7 +1274,7 @@ sessionMap.put(AssessmentConstants.CONFIG_KEY_HIDE_TITLES, Boolean.valueOf(service.getConfigValue(AssessmentConstants.CONFIG_KEY_HIDE_TITLES))); - return "pages/learning/results"; + return "pages/learning/results" + (embedded ? "/allquestions" : ""); } /** Index: lams_tool_assessment/web/pages/learning/results.jsp =================================================================== diff -u -rfd72af2c8d69fd4c43790f58b2ff7f0db72ca58b -r79db1b495adba49dd54a6250c1221a326f153633 --- lams_tool_assessment/web/pages/learning/results.jsp (.../results.jsp) (revision fd72af2c8d69fd4c43790f58b2ff7f0db72ca58b) +++ lams_tool_assessment/web/pages/learning/results.jsp (.../results.jsp) (revision 79db1b495adba49dd54a6250c1221a326f153633) @@ -8,25 +8,6 @@ <%@ include file="/common/header.jsp"%> - - - - - - - - <%-- param has higher level for request attribute --%> @@ -41,47 +22,8 @@ - - - - - Index: lams_tool_assessment/web/pages/learning/results/allquestions.jsp =================================================================== diff -u -r21c27a799a8473c3a8844258ac269683831db33e -r79db1b495adba49dd54a6250c1221a326f153633 --- lams_tool_assessment/web/pages/learning/results/allquestions.jsp (.../allquestions.jsp) (revision 21c27a799a8473c3a8844258ac269683831db33e) +++ lams_tool_assessment/web/pages/learning/results/allquestions.jsp (.../allquestions.jsp) (revision 79db1b495adba49dd54a6250c1221a326f153633) @@ -1,8 +1,112 @@ <%@ include file="/common/taglibs.jsp"%> + + + + + + + + + + + + + + + + + + + + + + + + +
-
+
+ +
+ <%-- Allow disclosing correct answers only for multiple choice questions --%> + +
+ disabled="disabled">  + > + +
+
+
+ disabled="disabled">  + > + +
+
+
+

${status.index + sessionMap.questionNumberingOffset}. @@ -12,7 +116,7 @@ ${question.title}

- + " @@ -77,4 +181,4 @@
- + \ No newline at end of file Index: lams_tool_assessment/web/pages/learning/results/essay.jsp =================================================================== diff -u -r3437878235018c207f602ea0464c109dd8e077c9 -r79db1b495adba49dd54a6250c1221a326f153633 --- lams_tool_assessment/web/pages/learning/results/essay.jsp (.../essay.jsp) (revision 3437878235018c207f602ea0464c109dd8e077c9) +++ lams_tool_assessment/web/pages/learning/results/essay.jsp (.../essay.jsp) (revision 79db1b495adba49dd54a6250c1221a326f153633) @@ -68,7 +68,7 @@ - + Index: lams_tool_assessment/web/pages/tblmonitoring/assessment.jsp =================================================================== diff -u -r442b2bdacb7a49af90480f58918774b82b1868a4 -r79db1b495adba49dd54a6250c1221a326f153633 --- lams_tool_assessment/web/pages/tblmonitoring/assessment.jsp (.../assessment.jsp) (revision 442b2bdacb7a49af90480f58918774b82b1868a4) +++ lams_tool_assessment/web/pages/tblmonitoring/assessment.jsp (.../assessment.jsp) (revision 79db1b495adba49dd54a6250c1221a326f153633) @@ -28,9 +28,18 @@ //insert total learners number taken from the parent tblmonitor.jsp $("#total-learners-number").html(TOTAL_LESSON_LEARNERS_NUMBER); - // go through each AE and set up disclose buttons there - $('.assessmentPane').each(function(){ - var assessmentPane = $(this), + $('.results').each(function(){ + // load results div for the first time + loadResultsPane($(this), false); + }); + + }); + + function loadResultsPane(resultsPane, isRefresh) { + // load an embedded results list + resultsPane.load("?embedded=true&toolContentID=" + + resultsPane.data('toolContentId'), function(){ + var assessmentPane = resultsPane.closest('.assessmentPane'), // are any correct/groups buttons clickable? discloseAllCorrectEnabled = false, discloseAllGroupsEnabled = false; @@ -53,45 +62,51 @@ return; } - discloseAnswers(button); + discloseAnswers(button, resultsPane); }); }); // if disclose all correct/groups answers buttons are clickable, add a click handler // and disable if not var allCorrectButton = $('.disclose-all-correct-button', assessmentPane); if (discloseAllCorrectEnabled) { - allCorrectButton.click(function(){ - if (!confirm("")) { - return; - } - $('.disclose-correct-button', assessmentPane).not('[disabled]').each(function() { - discloseAnswers($(this)); + // do not add a handler twice + if (!isRefresh) { + allCorrectButton.click(function(){ + if (!confirm("")) { + return; + } + $('.disclose-correct-button', assessmentPane).not('[disabled]').each(function() { + discloseAnswers($(this), resultsPane); + }); + disabledAndCheckButton(allCorrectButton); }); - disabledAndCheckButton(allCorrectButton); - }); + } } else { disabledAndCheckButton(allCorrectButton); } var allGroupsButton = $('.disclose-all-groups-button', assessmentPane); if (discloseAllGroupsEnabled) { - allGroupsButton.click(function(){ - if (!confirm("")) { - return; - } - $('.disclose-groups-button', assessmentPane).not('[disabled]').each(function() { - discloseAnswers($(this)); + // do not add a handler twice + if (!isRefresh) { + allGroupsButton.click(function(){ + if (!confirm("")) { + return; + } + $('.disclose-groups-button', assessmentPane).not('[disabled]').each(function() { + discloseAnswers($(this), resultsPane); + }); + disabledAndCheckButton(allGroupsButton); }); - disabledAndCheckButton(allGroupsButton); - }); + } } else { disabledAndCheckButton(allGroupsButton); } }); - }); + } - function discloseAnswers(button) { + function discloseAnswers(button, resultsPane) { let isCorrectButton = button.hasClass('disclose-correct-button'); $.ajax({ @@ -105,19 +120,14 @@ '' : '' } }).done(function(){ - // disable the button after click - disabledAndCheckButton(button); + // reload results after disclosing answers + loadResultsPane(resultsPane, true); }); } function disabledAndCheckButton(button){ button.attr('disabled', true).html(' ' + button.text()); } - - - function showResultsForTeacher(toolContentId) { - launchPopup("?toolContentID=" + toolContentId, 'Results'); - } @@ -192,12 +202,6 @@ <%-- Release correct/groups answers for all questions in this assessment --%>
-
- - -
-
@@ -212,143 +216,7 @@
- - -
-
-
-
-
- - Q${i.index+1}) - - - - - -
- <%-- Allow disclosing correct answers only for multiple choice questions --%> - -
- disabled="disabled">  - > - -
-
-
- disabled="disabled">  - > - -
-
-
-
-
- -
-
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
- ${ALPHABET[j.index]}. - - - - - - - ${fn:replace(qbOption.name, newLineChar, ', ')} - - -
- - Possible answers -
- ${ALPHABET[j.index]}. - - - - -
- a. - - -
- b. - - -
- ${ALPHABET[j.index]}. - - -
-
-
-
-
-
-
+