Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/AssessmentServiceImpl.java =================================================================== diff -u -r40eb54374e84591563d8b6a679ac719dbc85c8f7 -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/AssessmentServiceImpl.java (.../AssessmentServiceImpl.java) (revision 40eb54374e84591563d8b6a679ac719dbc85c8f7) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/AssessmentServiceImpl.java (.../AssessmentServiceImpl.java) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -473,7 +473,7 @@ AssessmentQuestionResult questionAnswer = null; // get questionResult from DB instance of AssessmentResult for (AssessmentQuestionResult dbQuestionAnswer : assessmentResult.getQuestionResults()) { - if (question.equals(dbQuestionAnswer.getAssessmentQuestion())) { + if (question.getUid().equals(dbQuestionAnswer.getAssessmentQuestion().getUid())) { questionAnswer = dbQuestionAnswer; } } @@ -483,6 +483,7 @@ questionAnswer = new AssessmentQuestionResult(); questionAnswer.setAssessmentQuestion(question); questionAnswer.setAssessmentResult(assessmentResult); + assessmentResult.getQuestionResults().add(questionAnswer); Set optionAnswers = questionAnswer.getOptionAnswers(); for (AssessmentQuestionOption option : question.getOptions()) { Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/util/AssessmentBundler.java =================================================================== diff -u -ref851974c4dc3f5468b188fab9997a37b73c0e50 -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/util/AssessmentBundler.java (.../AssessmentBundler.java) (revision ef851974c4dc3f5468b188fab9997a37b73c0e50) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/util/AssessmentBundler.java (.../AssessmentBundler.java) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -34,6 +34,7 @@ import javax.servlet.http.HttpServletRequest; import org.lamsfoundation.lams.learning.export.web.action.Bundler; +import org.lamsfoundation.lams.tool.assessment.AssessmentConstants; import org.lamsfoundation.lams.util.Configuration; import org.lamsfoundation.lams.util.ConfigurationKeys; import org.lamsfoundation.lams.util.HttpUrlConnectionUtil; @@ -77,6 +78,7 @@ for (String directoryName : directoriesNames) { directories.add(outputDirectory + File.separator + "javascript" + File.separator + directoryName); } + directories.add(outputDirectory + File.separator + "css"); this.createDirectories(directories); String[] imageNames = new String[] { "ui-bg_flat_0_aaaaaa_40x100.png", "ui-bg_flat_55_fbec88_40x100.png", @@ -112,6 +114,10 @@ urlToConnectTo = getServerUrl() + "css" + URL_SEPARATOR + "jquery.jqGrid.css"; directoryToStoreFile = outputDirectory + File.separator + "javascript" + File.separator + "css"; HttpUrlConnectionUtil.writeResponseToFile(urlToConnectTo, directoryToStoreFile, "jquery.jqGrid.css", cookies); + + urlToConnectTo = getAssessmentToolUrlDir() + "css" + URL_SEPARATOR + "assessment.css"; + directoryToStoreFile = outputDirectory + File.separator + "css"; + HttpUrlConnectionUtil.writeResponseToFile(urlToConnectTo, directoryToStoreFile, "assessment.css", cookies); } @@ -128,5 +134,10 @@ return serverUrl; } } + + private String getAssessmentToolUrlDir() { + String toolFolder = getServerUrl() + "tool" + URL_SEPARATOR + AssessmentConstants.TOOL_SIGNATURE + URL_SEPARATOR + "includes" + URL_SEPARATOR; + return toolFolder; + } } Index: lams_tool_assessment/web/common/header.jsp =================================================================== diff -u -r3d1f93045d6b547df27e59a090cfbe618ae273fe -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/common/header.jsp (.../header.jsp) (revision 3d1f93045d6b547df27e59a090cfbe618ae273fe) +++ lams_tool_assessment/web/common/header.jsp (.../header.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -3,14 +3,13 @@ - - - + + + + + + + - - - - - Index: lams_tool_assessment/web/includes/css/assessment.css =================================================================== diff -u -rc56857991e269aa7f5bd250a05b52c767a9957ad -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/includes/css/assessment.css (.../assessment.css) (revision c56857991e269aa7f5bd250a05b52c767a9957ad) +++ lams_tool_assessment/web/includes/css/assessment.css (.../assessment.css) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -1,3 +1,43 @@ +table.forum th { + font-size: inherit; + border-left: none; +} +table.forum { + background:none; + border: 1px solid #cacdd1; + padding-top:0px; + margin-bottom: 10px; + border-spacing: 0px; + border-collapse: separate; +} +table.forum.question-summary-table { + width: 83%; + padding: 0px 15px 0px; + font-style: italic; + color:#47bc23; + border: none; +} +table.forum.question-summary-table th{ + padding-top:0px; +} +table.forum.question-summary-table th:first-child{ + width: 30px; + border-left: none; + padding-left:0px; + text-align: center; +} +table.forum.question-summary-table td{ + padding-left: 15px; + vertical-align: middle; + background: none; +} +table.forum.question-summary-table td:first-child{ + padding-left: 10px; +} +table.forum.question-summary-table tr:last-child td{ + border-bottom: none; +} + .innerforms { border: 0px; font-size: small; @@ -31,3 +71,73 @@ div.message img.icon { vertical-align: middle; } + +div.growlUI { + background: url(check48.png) no-repeat 10px 10px +} +div.growlUI h1, div.growlUI h2 { + color: white; + padding: 5px 5px 5px 0px; + text-align: center; +} + +table.alternative-color td.question-numbers { + padding: 15px 15px 15px; + width: 10px; + font-weight: bold; +} +table.question-table { + padding-bottom: 10px; + border-spacing: 0; + border-collapse: separate; +} +table.question-table td { + vertical-align:middle; + background:none; + border-bottom:0px; +} +.question-table .has-radio-button { + padding:5px 0px 2px 15px; + width: 5px; +} +.question-table .complete-item-gif { + padding:5px 0px 2px 15px; + width: 7px; +} +.question-table .complete-item-gif img { + border:1px; +} +.question-type { + padding: 10px 15px 7px; + font-style: italic; +} +.question-table .question-feedback, div.question-feedback { + padding: 10px 15px 0px; + font-style: italic; + color:#47bc23; +} +.question-table .question-option { + padding:5px 10px 2px; +} +.question-table .reg-padding { + padding:5px 0px 2px 15px; +} +#quesion-summary-title { + padding: 2px 15px 10px; + font-style: italic; + color:#47bc23; +} + + +#pager { + text-align: center; + padding-top: 60px; +} +#pager a{ + margin-left: 10px; + font-size: 130%; +} +.color-red { + color: red; +} + Index: lams_tool_assessment/web/pages/export/parts/exportlearner.jsp =================================================================== diff -u -rb9a5084dfc1510cefec305104580add4c26762ea -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/export/parts/exportlearner.jsp (.../exportlearner.jsp) (revision b9a5084dfc1510cefec305104580add4c26762ea) +++ lams_tool_assessment/web/pages/export/parts/exportlearner.jsp (.../exportlearner.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -11,6 +11,7 @@ + @@ -81,7 +82,7 @@

- +
Index: lams_tool_assessment/web/pages/export/parts/exportteacher.jsp =================================================================== diff -u -r405cf90d2646e8f7dc1aa032b28c820279e50930 -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/export/parts/exportteacher.jsp (.../exportteacher.jsp) (revision 405cf90d2646e8f7dc1aa032b28c820279e50930) +++ lams_tool_assessment/web/pages/export/parts/exportteacher.jsp (.../exportteacher.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -12,6 +12,7 @@ + @@ -163,7 +164,7 @@
- +
+ +
Index: lams_tool_assessment/web/pages/learning/learning.jsp =================================================================== diff -u -rb88cfcacfc98572e23f6040a7a4ed4060b58a1fc -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/learning/learning.jsp (.../learning.jsp) (revision b88cfcacfc98572e23f6040a7a4ed4060b58a1fc) +++ lams_tool_assessment/web/pages/learning/learning.jsp (.../learning.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -1,7 +1,6 @@ - - + <%@ include file="/common/taglibs.jsp"%> + @@ -36,12 +35,6 @@ - Index: lams_tool_assessment/web/pages/learning/parts/allquestions.jsp =================================================================== diff -u -r86f8f142891162680c6f7ed66a8c4edf574df991 -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/learning/parts/allquestions.jsp (.../allquestions.jsp) (revision 86f8f142891162680c6f7ed66a8c4edf574df991) +++ lams_tool_assessment/web/pages/learning/parts/allquestions.jsp (.../allquestions.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -1,68 +1,67 @@ <%@ include file="/common/taglibs.jsp"%> -
- - - - - - + +
- ${status.index + sessionMap.questionNumberingOffset} -
+ + + + + - - - -
+ ${status.index + sessionMap.questionNumberingOffset} + - + + -
- ${question.question} -
+
+ ${question.question} +
- - - <%@ include file="multiplechoice.jsp"%> - - - <%@ include file="matchingpairs.jsp"%> - - - <%@ include file="shortanswer.jsp"%> - - - <%@ include file="numerical.jsp"%> - - - <%@ include file="truefalse.jsp"%> - - - <%@ include file="essay.jsp"%> - - - <%@ include file="ordering.jsp"%> - - - - <%@ include file="questionsummary.jsp"%> -
-
- - - - - - + + <%@ include file="matchingpairs.jsp"%> + + + <%@ include file="shortanswer.jsp"%> + + + <%@ include file="numerical.jsp"%> + + + <%@ include file="truefalse.jsp"%> + + + <%@ include file="essay.jsp"%> + + + <%@ include file="ordering.jsp"%> + + + + <%@ include file="questionsummary.jsp"%> + +
+ + + + + + Index: lams_tool_assessment/web/pages/learning/parts/attemptsummary.jsp =================================================================== diff -u -r54671beb133fc34419cdae133e3cff23ced916f9 -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/learning/parts/attemptsummary.jsp (.../attemptsummary.jsp) (revision 54671beb133fc34419cdae133e3cff23ced916f9) +++ lams_tool_assessment/web/pages/learning/parts/attemptsummary.jsp (.../attemptsummary.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -1,7 +1,7 @@ - +
- - - - -
+ @@ -10,15 +10,15 @@
+
+ @@ -28,7 +28,7 @@
+ @@ -40,7 +40,7 @@
+ Index: lams_tool_assessment/web/pages/learning/parts/essay.jsp =================================================================== diff -u -r6c5d0c6b8ff17d661580df2c265f2dcabfe7d76c -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/learning/parts/essay.jsp (.../essay.jsp) (revision 6c5d0c6b8ff17d661580df2c265f2dcabfe7d76c) +++ lams_tool_assessment/web/pages/learning/parts/essay.jsp (.../essay.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -1,12 +1,12 @@ <%@ include file="/common/taglibs.jsp"%> -
+
- +
- -
+ @@ -20,6 +20,6 @@
+
<%@ include file="markandpenaltyarea.jsp"%> \ No newline at end of file Index: lams_tool_assessment/web/pages/learning/parts/markandpenaltyarea.jsp =================================================================== diff -u -r0c7332f878ff51630f2c84a47f419abc832e022e -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/learning/parts/markandpenaltyarea.jsp (.../markandpenaltyarea.jsp) (revision 0c7332f878ff51630f2c84a47f419abc832e022e) +++ lams_tool_assessment/web/pages/learning/parts/markandpenaltyarea.jsp (.../markandpenaltyarea.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -3,13 +3,13 @@ -
+
-
+
${question.defaultGrade} Index: lams_tool_assessment/web/pages/learning/parts/matchingpairs.jsp =================================================================== diff -u -r6e5d1ff4891084bef035cb865c1a395e3eb49ab0 -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/learning/parts/matchingpairs.jsp (.../matchingpairs.jsp) (revision 6e5d1ff4891084bef035cb865c1a395e3eb49ab0) +++ lams_tool_assessment/web/pages/learning/parts/matchingpairs.jsp (.../matchingpairs.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -1,28 +1,28 @@ <%@ include file="/common/taglibs.jsp"%> -
+
- +
- - -
+ - + - + + + Index: lams_tool_assessment/web/pages/learning/parts/multiplechoice.jsp =================================================================== diff -u -r6e5d1ff4891084bef035cb865c1a395e3eb49ab0 -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/learning/parts/multiplechoice.jsp (.../multiplechoice.jsp) (revision 6e5d1ff4891084bef035cb865c1a395e3eb49ab0) +++ lams_tool_assessment/web/pages/learning/parts/multiplechoice.jsp (.../multiplechoice.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -1,6 +1,6 @@ <%@ include file="/common/taglibs.jsp"%> -
+
@@ -11,23 +11,23 @@
- +
- + - - @@ -59,7 +59,7 @@ - @@ -71,17 +71,17 @@ -
+
-
+
-
+
Index: lams_tool_assessment/web/pages/learning/parts/numerical.jsp =================================================================== diff -u -r6c5d0c6b8ff17d661580df2c265f2dcabfe7d76c -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/learning/parts/numerical.jsp (.../numerical.jsp) (revision 6c5d0c6b8ff17d661580df2c265f2dcabfe7d76c) +++ lams_tool_assessment/web/pages/learning/parts/numerical.jsp (.../numerical.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -1,24 +1,24 @@ <%@ include file="/common/taglibs.jsp"%> -
+
-
+ - + - + - + + +
+
- -
+ - + - + + disabled="disabled" /> @@ -27,7 +27,7 @@
-
+
Index: lams_tool_assessment/web/pages/learning/parts/ordering.jsp =================================================================== diff -u -r6e5d1ff4891084bef035cb865c1a395e3eb49ab0 -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/learning/parts/ordering.jsp (.../ordering.jsp) (revision 6e5d1ff4891084bef035cb865c1a395e3eb49ab0) +++ lams_tool_assessment/web/pages/learning/parts/ordering.jsp (.../ordering.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -7,20 +7,20 @@
-
+
- +
- -
+ + " @@ -50,7 +50,7 @@
-
+
Index: lams_tool_assessment/web/pages/learning/parts/questionsummary.jsp =================================================================== diff -u -r6e5d1ff4891084bef035cb865c1a395e3eb49ab0 -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/learning/parts/questionsummary.jsp (.../questionsummary.jsp) (revision 6e5d1ff4891084bef035cb865c1a395e3eb49ab0) +++ lams_tool_assessment/web/pages/learning/parts/questionsummary.jsp (.../questionsummary.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -1,23 +1,23 @@ <%@ include file="/common/taglibs.jsp"%> -
+
- +
- - - - @@ -26,18 +26,15 @@ - - - - - - +
+ # + + +
+ ${status.index + 1} +
${currentAssessmentResult.finishDate}
+ @@ -101,16 +98,14 @@ +
\ No newline at end of file Index: lams_tool_assessment/web/pages/learning/parts/shortanswer.jsp =================================================================== diff -u -r6c5d0c6b8ff17d661580df2c265f2dcabfe7d76c -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/learning/parts/shortanswer.jsp (.../shortanswer.jsp) (revision 6c5d0c6b8ff17d661580df2c265f2dcabfe7d76c) +++ lams_tool_assessment/web/pages/learning/parts/shortanswer.jsp (.../shortanswer.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -1,24 +1,24 @@ <%@ include file="/common/taglibs.jsp"%> -
+
- +
- -
+ - + - + + disabled="disabled" /> @@ -27,7 +27,7 @@
-
+
Index: lams_tool_assessment/web/pages/learning/parts/truefalse.jsp =================================================================== diff -u -r6c5d0c6b8ff17d661580df2c265f2dcabfe7d76c -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/learning/parts/truefalse.jsp (.../truefalse.jsp) (revision 6c5d0c6b8ff17d661580df2c265f2dcabfe7d76c) +++ lams_tool_assessment/web/pages/learning/parts/truefalse.jsp (.../truefalse.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -1,68 +1,68 @@ <%@ include file="/common/taglibs.jsp"%> -
+
- +
- - - - - - @@ -72,12 +72,12 @@ -
+
-
+
Index: lams_tool_assessment/web/pages/monitoring/parts/questionsummary.jsp =================================================================== diff -u -re83e4b897113057714ced4be6d03a621c782b075 -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/monitoring/parts/questionsummary.jsp (.../questionsummary.jsp) (revision e83e4b897113057714ced4be6d03a621c782b075) +++ lams_tool_assessment/web/pages/monitoring/parts/questionsummary.jsp (.../questionsummary.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -146,7 +146,7 @@

<%@ include file="/common/messages.jsp"%> -
+ - + - + - checked="checked" - disabled="disabled" - /> + + checked="checked" + disabled="disabled" + /> +
+ - + - + - checked="checked" - disabled="disabled" - /> + + checked="checked" + disabled="disabled" + /> +
+
Index: lams_tool_assessment/web/pages/monitoring/parts/usersummary.jsp =================================================================== diff -u -rb9a5084dfc1510cefec305104580add4c26762ea -r1e33f64c06430834c80a2b01d42bcdf844c5d958 --- lams_tool_assessment/web/pages/monitoring/parts/usersummary.jsp (.../usersummary.jsp) (revision b9a5084dfc1510cefec305104580add4c26762ea) +++ lams_tool_assessment/web/pages/monitoring/parts/usersummary.jsp (.../usersummary.jsp) (revision 1e33f64c06430834c80a2b01d42bcdf844c5d958) @@ -136,7 +136,7 @@

<%@ include file="/common/messages.jsp"%> - +