Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentOptionAnswer.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentOptionAnswer.java,v diff -u -r1.2 -r1.3 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentOptionAnswer.java 10 Mar 2009 21:39:14 -0000 1.2 +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentOptionAnswer.java 7 Aug 2013 07:29:12 -0000 1.3 @@ -55,7 +55,7 @@ return uid; } - private void setUid(Long uid) { + public void setUid(Long uid) { this.uid = uid; } Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentQuestion.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentQuestion.java,v diff -u -r1.16 -r1.17 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentQuestion.java 1 Aug 2012 11:50:04 -0000 1.16 +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentQuestion.java 7 Aug 2013 07:29:12 -0000 1.17 @@ -102,6 +102,8 @@ private String questionFeedback; + private String titleEscaped; + /** * Grade acquired from QUestionReference in learner. */ @@ -506,6 +508,13 @@ return questionFeedback; } + public String getTitleEscaped() { + return titleEscaped; + } + public void setTitleEscaped(String titleEscaped) { + this.titleEscaped = titleEscaped; + } + /** * @return Grade acquired from QUestionReference in learner. */ Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentQuestionOption.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentQuestionOption.java,v diff -u -r1.2 -r1.3 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentQuestionOption.java 3 Oct 2011 13:51:47 -0000 1.2 +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentQuestionOption.java 7 Aug 2013 07:29:12 -0000 1.3 @@ -55,6 +55,10 @@ private int answerInt = -1; private boolean answerBoolean; + + private String questionEscaped; + + private String optionStringEscaped; // ********************************************************** // Get/Set methods @@ -191,7 +195,21 @@ } public void setAnswerBoolean(boolean answerBoolean) { this.answerBoolean = answerBoolean; - } + } + + public String getQuestionEscaped() { + return questionEscaped; + } + public void setQuestionEscaped(String questionEscaped) { + this.questionEscaped = questionEscaped; + } + + public String getOptionStringEscaped() { + return optionStringEscaped; + } + public void setOptionStringEscaped(String optionStringEscaped) { + this.optionStringEscaped = optionStringEscaped; + } @Override public Object clone() { Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentQuestionResult.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentQuestionResult.java,v diff -u -r1.8 -r1.9 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentQuestionResult.java 1 Jul 2009 02:39:10 -0000 1.8 +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentQuestionResult.java 7 Aug 2013 07:29:12 -0000 1.9 @@ -52,6 +52,7 @@ // DTO fields: private Date finishDate; private AssessmentUser user; + private String answerStringEscaped; public AssessmentQuestionResult() { optionAnswers = new LinkedHashSet(); @@ -209,5 +210,13 @@ public void setUser(AssessmentUser user) { this.user = user; } + + public String getAnswerStringEscaped() { + return answerStringEscaped; + } + + public void setAnswerStringEscaped(String answerStringEscaped) { + this.answerStringEscaped = answerStringEscaped; + } } Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/AssessmentServiceImpl.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/AssessmentServiceImpl.java,v diff -u -r1.28 -r1.29 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/AssessmentServiceImpl.java 29 Jul 2013 23:11:37 -0000 1.28 +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/service/AssessmentServiceImpl.java 7 Aug 2013 07:29:12 -0000 1.29 @@ -68,7 +68,6 @@ import org.lamsfoundation.lams.tool.ToolContentManager; import org.lamsfoundation.lams.tool.ToolOutput; import org.lamsfoundation.lams.tool.ToolOutputDefinition; -import org.lamsfoundation.lams.tool.ToolSession; import org.lamsfoundation.lams.tool.ToolSessionExportOutputData; import org.lamsfoundation.lams.tool.ToolSessionManager; import org.lamsfoundation.lams.tool.assessment.AssessmentConstants; @@ -654,7 +653,8 @@ AssessmentResult lastFinishedResult = assessmentResultDao.getLastFinishedAssessmentResultBySessionId(sessionId, userId); if (lastFinishedResult != null) { - SortedSet questionResults = new TreeSet(new AssessmentQuestionResultComparator()); + SortedSet questionResults = new TreeSet( + new AssessmentQuestionResultComparator()); questionResults.addAll(lastFinishedResult.getQuestionResults()); lastFinishedResult.setQuestionResults(questionResults); escapeQuotes(lastFinishedResult); @@ -849,28 +849,28 @@ private static void escapeQuotesInQuestionResult(AssessmentQuestionResult questionResult) { String answerString = questionResult.getAnswerString(); if (answerString != null) { - answerString = StringEscapeUtils.escapeJavaScript(answerString); - questionResult.setAnswerString(answerString); + String answerStringEscaped = StringEscapeUtils.escapeJavaScript(answerString); + questionResult.setAnswerStringEscaped(answerStringEscaped); } AssessmentQuestion question = questionResult.getAssessmentQuestion(); String title = question.getTitle(); if (title != null) { - title = StringEscapeUtils.escapeJavaScript(title); - question.setTitle(title); + String titleEscaped = StringEscapeUtils.escapeJavaScript(title); + question.setTitleEscaped(titleEscaped); } for (AssessmentQuestionOption questionOption : question.getQuestionOptions()) { String questionStr = questionOption.getQuestion(); if (questionStr != null) { - questionStr = StringEscapeUtils.escapeJavaScript(questionStr); - questionOption.setQuestion(questionStr); + String questionEscaped = StringEscapeUtils.escapeJavaScript(questionStr); + questionOption.setQuestionEscaped(questionEscaped); } String optionStr = questionOption.getOptionString(); if (optionStr != null) { - optionStr = StringEscapeUtils.escapeJavaScript(optionStr); - questionOption.setOptionString(optionStr); + String optionEscaped = StringEscapeUtils.escapeJavaScript(optionStr); + questionOption.setOptionStringEscaped(optionEscaped); } } } @@ -1091,16 +1091,6 @@ Assessment toContent = Assessment.newInstance(assessment, toContentId, assessmentToolContentHandler); assessmentDao.saveObject(toContent); - - // save assessment questions as well - Set questions = toContent.getQuestions(); - if (questions != null) { - Iterator iter = questions.iterator(); - while (iter.hasNext()) { - AssessmentQuestion question = (AssessmentQuestion) iter.next(); - // createRootTopic(toContent.getUid(),null,msg); - } - } } public void setAsDefineLater(Long toolContentId, boolean value) throws DataMissingException, ToolException { Index: lams_tool_assessment/web/pages/export/parts/exportlearner.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_assessment/web/pages/export/parts/exportlearner.jsp,v diff -u -r1.6 -r1.7 --- lams_tool_assessment/web/pages/export/parts/exportlearner.jsp 17 Feb 2013 16:11:59 -0000 1.6 +++ lams_tool_assessment/web/pages/export/parts/exportlearner.jsp 7 Aug 2013 07:29:11 -0000 1.7 @@ -40,7 +40,7 @@ ], multiselect: false, - caption: "${question.title}" + caption: "${question.titleEscaped}" /* resetSelection() doesn't work in this version hope it'll be fixed in the next one Index: lams_tool_assessment/web/pages/export/parts/exportteacher.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_assessment/web/pages/export/parts/exportteacher.jsp,v diff -u -r1.6 -r1.7 --- lams_tool_assessment/web/pages/export/parts/exportteacher.jsp 17 Feb 2013 16:11:59 -0000 1.6 +++ lams_tool_assessment/web/pages/export/parts/exportteacher.jsp 7 Aug 2013 07:29:11 -0000 1.7 @@ -31,7 +31,7 @@ colNames:['#', "", - "${question.title}", + "${question.titleEscaped}", ""], Index: lams_tool_assessment/web/pages/export/parts/userresponse.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_assessment/web/pages/export/parts/userresponse.jsp,v diff -u -r1.2 -r1.3 --- lams_tool_assessment/web/pages/export/parts/userresponse.jsp 22 Mar 2012 08:59:51 -0000 1.2 +++ lams_tool_assessment/web/pages/export/parts/userresponse.jsp 7 Aug 2013 07:29:11 -0000 1.3 @@ -5,7 +5,7 @@ - responseStr += "${questionOption.optionString}"; + responseStr += "${questionOption.optionStringEscaped}"; @@ -15,15 +15,15 @@ responseStr +=''; responseStr +=''; - responseStr +="${questionOption.question}"; + responseStr +="${questionOption.questionEscaped}"; responseStr +=''; responseStr +=''; responseStr +='-'; - responseStr +="${questionOption2.optionString}"; + responseStr +="${questionOption2.optionStringEscaped}"; @@ -34,26 +34,26 @@ responseStr +=''; - responseStr +="${questionResult.answerString}"; + responseStr +="${questionResult.answerStringEscaped}"; - responseStr +="${questionResult.answerString}"; + responseStr +="${questionResult.answerStringEscaped}"; responseStr +="${questionResult.answerBoolean}"; - responseStr +="${questionResult.answerString}"; + responseStr +="${questionResult.answerStringEscaped}"; - responseStr +="${questionOption.optionString}"; + responseStr +="${questionOption.optionStringEscaped}"; Index: lams_tool_assessment/web/pages/monitoring/parts/masterDetailLoadUp.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_assessment/web/pages/monitoring/parts/masterDetailLoadUp.jsp,v diff -u -r1.5 -r1.6 --- lams_tool_assessment/web/pages/monitoring/parts/masterDetailLoadUp.jsp 17 Jun 2013 07:15:15 -0000 1.5 +++ lams_tool_assessment/web/pages/monitoring/parts/masterDetailLoadUp.jsp 7 Aug 2013 07:29:12 -0000 1.6 @@ -10,7 +10,7 @@ jQuery("#userSummary${assessmentResult.sessionId}").addRowData(${i.index + 1}, { id:"${i.index + 1}", questionResultUid:"${questionResult.uid}", - title:"${questionResult.assessmentQuestion.title}", + title:"${questionResult.assessmentQuestion.titleEscaped}", response:responseStr, grade:"" }); Index: lams_tool_assessment/web/pages/monitoring/parts/userresponse.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_assessment/web/pages/monitoring/parts/userresponse.jsp,v diff -u -r1.4 -r1.5 --- lams_tool_assessment/web/pages/monitoring/parts/userresponse.jsp 22 Mar 2012 08:59:51 -0000 1.4 +++ lams_tool_assessment/web/pages/monitoring/parts/userresponse.jsp 7 Aug 2013 07:29:12 -0000 1.5 @@ -5,7 +5,7 @@ - responseStr += "${questionOption.optionString}"; + responseStr += "${questionOption.optionStringEscaped}"; @@ -14,15 +14,15 @@ responseStr +='
'; responseStr +='
'; - responseStr +="${questionOption.question}"; + responseStr +="${questionOption.questionEscaped}"; responseStr +='
'; responseStr +='
'; responseStr +=' - '; - responseStr +="${questionOption2.optionString}"; + responseStr +="${questionOption2.optionStringEscaped}"; @@ -35,26 +35,26 @@ - responseStr +="${questionResult.answerString}"; + responseStr +="${questionResult.answerStringEscaped}"; - responseStr +="${questionResult.answerString}"; + responseStr +="${questionResult.answerStringEscaped}"; responseStr +="${questionResult.answerBoolean}"; - responseStr +="${questionResult.answerString}"; + responseStr +="${questionResult.answerStringEscaped}"; - responseStr +="${questionOption.optionString}"; + responseStr +="${questionOption.optionStringEscaped}"; Index: lams_tool_assessment/web/pages/monitoring/parts/usersummary.jsp =================================================================== RCS file: /usr/local/cvsroot/lams_tool_assessment/web/pages/monitoring/parts/usersummary.jsp,v diff -u -r1.12 -r1.13 --- lams_tool_assessment/web/pages/monitoring/parts/usersummary.jsp 19 Jun 2013 14:30:03 -0000 1.12 +++ lams_tool_assessment/web/pages/monitoring/parts/usersummary.jsp 7 Aug 2013 07:29:12 -0000 1.13 @@ -52,7 +52,7 @@ ], multiselect: false, - caption: "${question.title}", + caption: "${question.titleEscaped}", cellurl: '', cellEdit: true, beforeEditCell: function (rowid,name,val,iRow,iCol){