Index: lams_central/web/css/_jquery.jRating_base.scss =================================================================== diff -u -r8e742e0e9689c2f26674a1b43c225694b1a41280 -reda89b58885d3360b28c25096188f47952aff317 --- lams_central/web/css/_jquery.jRating_base.scss (.../_jquery.jRating_base.scss) (revision 8e742e0e9689c2f26674a1b43c225694b1a41280) +++ lams_central/web/css/_jquery.jRating_base.scss (.../_jquery.jRating_base.scss) (revision eda89b58885d3360b28c25096188f47952aff317) @@ -65,7 +65,6 @@ margin-top: 20px; } .rating-comment{ - background: url("../images/css/greyfade_bg.jpg") repeat-x scroll 0px 0px transparent; padding: 15px; margin: 10px 0; } Index: lams_central/web/css/_learner_base.scss =================================================================== diff -u -r6600e575e5a67163cdc6bb787a078bde420328cd -reda89b58885d3360b28c25096188f47952aff317 --- lams_central/web/css/_learner_base.scss (.../_learner_base.scss) (revision 6600e575e5a67163cdc6bb787a078bde420328cd) +++ lams_central/web/css/_learner_base.scss (.../_learner_base.scss) (revision eda89b58885d3360b28c25096188f47952aff317) @@ -873,7 +873,6 @@ // Ratings .rating-comment{ background-image: none; - background-color: $gray-lighter; border: $border-thin-default-light; } Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/dto/GeneralLearnerFlowDTO.java =================================================================== diff -u -ra61647737b2a52dfc32f5d47247df8adebbc7854 -reda89b58885d3360b28c25096188f47952aff317 --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/dto/GeneralLearnerFlowDTO.java (.../GeneralLearnerFlowDTO.java) (revision a61647737b2a52dfc32f5d47247df8adebbc7854) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/dto/GeneralLearnerFlowDTO.java (.../GeneralLearnerFlowDTO.java) (revision eda89b58885d3360b28c25096188f47952aff317) @@ -73,7 +73,7 @@ protected String endLearningMessage; - protected String userNameVisible; + protected boolean userNameVisible; protected boolean showOtherAnswers; @@ -89,8 +89,6 @@ protected Set questions; - protected List userResponses; - protected String initialScreen; protected String lockWhenFinished; @@ -99,30 +97,13 @@ protected String userUid; - protected String usernameVisible; - protected String allowRichEditor; protected String useSelectLeaderToolOuput; protected boolean allowRateAnswers; /** - * @return Returns the usernameVisible. - */ - public String getUsernameVisible() { - return usernameVisible; - } - - /** - * @param usernameVisible - * The usernameVisible to set. - */ - public void setUsernameVisible(String usernameVisible) { - this.usernameVisible = usernameVisible; - } - - /** * @return Returns the currentAnswer. */ public String getCurrentAnswer() { @@ -315,18 +296,11 @@ this.endLearningMessage = endLearningMessage; } - /** - * @return Returns the userNameVisible. - */ - public String getUserNameVisible() { + public boolean getUserNameVisible() { return userNameVisible; } - /** - * @param userNameVisible - * The userNameVisible to set. - */ - public void setUserNameVisible(String userNameVisible) { + public void setUserNameVisible(boolean userNameVisible) { this.userNameVisible = userNameVisible; } @@ -386,21 +360,6 @@ /** * @return Returns the questions. */ - public List getUserResponses() { - return userResponses; - } - - /** - * @param questions - * The questions to set. - */ - public void setUserResponses(List userResponses) { - this.userResponses = userResponses; - } - - /** - * @return Returns the questions. - */ public Set getQuestions() { return questions; } Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/model/QaQueContent.java =================================================================== diff -u -r1ee503e3d0e0228ea8a45025fddf15d9623c0377 -reda89b58885d3360b28c25096188f47952aff317 --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/model/QaQueContent.java (.../QaQueContent.java) (revision 1ee503e3d0e0228ea8a45025fddf15d9623c0377) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/model/QaQueContent.java (.../QaQueContent.java) (revision eda89b58885d3360b28c25096188f47952aff317) @@ -34,6 +34,7 @@ import javax.persistence.JoinColumn; import javax.persistence.ManyToOne; import javax.persistence.Table; +import javax.persistence.Transient; import org.apache.commons.lang.builder.EqualsBuilder; import org.apache.commons.lang.builder.HashCodeBuilder; @@ -78,6 +79,10 @@ @ManyToOne(fetch = FetchType.LAZY) @JoinColumn(name = "qa_content_id") private QaContent qaContent; + + // *************** NON Persist Fields used in learning ******************** + @Transient + private QaUsrResp userResponse; /** default constructor */ public QaQueContent() { @@ -239,4 +244,14 @@ public void setFeedback(String feedback) { this.feedback = feedback; } + + // *************** NON Persist Fields used in monitoring ******************** + + public QaUsrResp getUserResponse() { + return userResponse; + } + + public void setUserResponse(QaUsrResp userResponse) { + this.userResponse = userResponse; + } } Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/util/LearningUtil.java =================================================================== diff -u -r69092337b728f903f824a74377bd0ebf9391120c -reda89b58885d3360b28c25096188f47952aff317 --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/util/LearningUtil.java (.../LearningUtil.java) (revision 69092337b728f903f824a74377bd0ebf9391120c) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/util/LearningUtil.java (.../LearningUtil.java) (revision eda89b58885d3360b28c25096188f47952aff317) @@ -71,7 +71,7 @@ generalLearnerFlowDTO.setQuestionListingMode(QUESTION_LISTING_MODE_COMBINED); } - generalLearnerFlowDTO.setUserNameVisible(new Boolean(qaContent.isUsernameVisible()).toString()); + generalLearnerFlowDTO.setUserNameVisible(qaContent.isUsernameVisible()); generalLearnerFlowDTO.setShowOtherAnswers(qaContent.isShowOtherAnswers()); generalLearnerFlowDTO.setAllowRichEditor(new Boolean(qaContent.isAllowRichEditor()).toString()); generalLearnerFlowDTO Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/controller/QaLearningController.java =================================================================== diff -u -ra61647737b2a52dfc32f5d47247df8adebbc7854 -reda89b58885d3360b28c25096188f47952aff317 --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/controller/QaLearningController.java (.../QaLearningController.java) (revision a61647737b2a52dfc32f5d47247df8adebbc7854) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/controller/QaLearningController.java (.../QaLearningController.java) (revision eda89b58885d3360b28c25096188f47952aff317) @@ -39,7 +39,6 @@ import org.apache.commons.lang.StringEscapeUtils; import org.apache.commons.lang.StringUtils; -import org.apache.log4j.Logger; import org.lamsfoundation.lams.notebook.model.NotebookEntry; import org.lamsfoundation.lams.notebook.service.CoreNotebookConstants; import org.lamsfoundation.lams.rating.dto.ItemRatingCriteriaDTO; @@ -98,6 +97,11 @@ @Autowired @Qualifier("qaMessageService") private MessageService messageService; + + @RequestMapping("/") + public String unspecified() throws IOException, ServletException, ToolException { + return null; + } @RequestMapping("/learning") public String execute(@ModelAttribute("qaLearningForm") QaLearningForm qaLearningForm, HttpServletRequest request) @@ -342,7 +346,7 @@ user, generalLearnerFlowDTO); generalLearnerFlowDTO.setIsLearnerFinished(user.isLearnerFinished()); - return "learning/learnerRep"; + return "learning/LearnerRep"; } } } @@ -613,7 +617,7 @@ generalLearnerFlowDTO.setUserUid(user.getQueUsrId().toString()); boolean usernameVisible = qaContent.isUsernameVisible(); - generalLearnerFlowDTO.setUserNameVisible(new Boolean(usernameVisible).toString()); + generalLearnerFlowDTO.setUserNameVisible(usernameVisible); NotebookEntry notebookEntry = qaService.getEntry(new Long(toolSessionID), CoreNotebookConstants.NOTEBOOK_TOOL, QaAppConstants.MY_SIGNATURE, new Integer(userID)); @@ -694,7 +698,7 @@ generalLearnerFlowDTO.setUserUid(qaQueUsr.getQueUsrId().toString()); boolean usernameVisible = qaContent.isUsernameVisible(); - generalLearnerFlowDTO.setUserNameVisible(new Boolean(usernameVisible).toString()); + generalLearnerFlowDTO.setUserNameVisible(usernameVisible); request.setAttribute(QaAppConstants.GENERAL_LEARNER_FLOW_DTO, generalLearnerFlowDTO); @@ -1012,7 +1016,6 @@ } request.setAttribute(QaAppConstants.GENERAL_LEARNER_FLOW_DTO, generalLearnerFlowDTO); - return "learning/Notebook"; } @@ -1024,13 +1027,10 @@ */ public static void refreshSummaryData(HttpServletRequest request, QaContent qaContent, QaSession qaSession, IQaService qaService, String httpSessionID, QaQueUsr user, GeneralLearnerFlowDTO generalLearnerFlowDTO) { - SessionMap sessionMap = (SessionMap) request.getSession() .getAttribute(httpSessionID); Long userId = user.getQueUsrId(); - Set questions = qaContent.getQaQueContents(); - generalLearnerFlowDTO.setQuestions(questions); - generalLearnerFlowDTO.setUserNameVisible(new Boolean(qaContent.isUsernameVisible()).toString()); + generalLearnerFlowDTO.setUserNameVisible(qaContent.isUsernameVisible()); // potentially empty list if the user starts the lesson after the time restriction has expired. List userResponses = qaService.getResponsesByUserUid(user.getUid()); @@ -1085,14 +1085,23 @@ countRatedQuestions = qaService.getCountItemsRatedByUser(qaContent.getQaContentId(), userId.intValue()); } } + + Set questions = qaContent.getQaQueContents(); + generalLearnerFlowDTO.setQuestions(questions); + //find according QaQueContent, if any + for (QaQueContent question : questions) { + for (QaUsrResp userResponse : userResponses) { + if (question.getUid().equals(userResponse.getQaQuestion().getUid())) { + question.setUserResponse(userResponse); + break; + } + } + } request.setAttribute(TOOL_SESSION_ID, qaSession.getQaSessionId()); - sessionMap.put("commentsMinWordsLimit", commentsMinWordsLimit); sessionMap.put("isCommentsEnabled", isCommentsEnabled); sessionMap.put(AttributeNames.ATTR_COUNT_RATED_ITEMS, countRatedQuestions); - - generalLearnerFlowDTO.setUserResponses(userResponses); } /** Index: lams_tool_laqa/web/learning/LearnerRep.jsp =================================================================== diff -u -ra61647737b2a52dfc32f5d47247df8adebbc7854 -reda89b58885d3360b28c25096188f47952aff317 --- lams_tool_laqa/web/learning/LearnerRep.jsp (.../LearnerRep.jsp) (revision a61647737b2a52dfc32f5d47247df8adebbc7854) +++ lams_tool_laqa/web/learning/LearnerRep.jsp (.../LearnerRep.jsp) (revision eda89b58885d3360b28c25096188f47952aff317) @@ -22,6 +22,31 @@ +