Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java =================================================================== diff -u -r1ef4b877aea0883709c641351e07595c6be84361 -r99a9e8ffeedcb8f20796543815866501b6739792 --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java (.../LearningAction.java) (revision 1ef4b877aea0883709c641351e07595c6be84361) +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/LearningAction.java (.../LearningAction.java) (revision 99a9e8ffeedcb8f20796543815866501b6739792) @@ -604,21 +604,22 @@ */ private ActionForward newReplyTopic(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) { - String sessionMapID = request.getParameter(ForumConstants.ATTR_SESSION_MAP_ID); - MessageForm msgForm = (MessageForm) form; + String sessionMapID = request.getParameter(ForumConstants.ATTR_SESSION_MAP_ID); + SessionMap sessionMap = getSessionMap(request, msgForm); msgForm.setSessionMapID(sessionMapID); Long parentId = WebUtil.readLongParam(request, ForumConstants.ATTR_PARENT_TOPIC_ID); + sessionMap.put(ForumConstants.ATTR_PARENT_TOPIC_ID, parentId); + // get parent topic, it can decide default subject of reply. MessageDTO topic = getTopic(parentId); - if (topic != null && topic.getMessage() != null) { String reTitle = topic.getMessage().getSubject(); MessageDTO originalMessage = MessageDTO.getMessageDTO(topic.getMessage()); - request.setAttribute(ForumConstants.ATTR_ORIGINAL_MESSAGE, originalMessage); + sessionMap.put(ForumConstants.ATTR_ORIGINAL_MESSAGE, originalMessage); // echo back current topic subject to web page if (reTitle != null && !reTitle.trim().startsWith("Re:")) { @@ -627,9 +628,7 @@ msgForm.getMessage().setSubject(reTitle); } } - SessionMap sessionMap = getSessionMap(request, msgForm); - sessionMap.put(ForumConstants.ATTR_PARENT_TOPIC_ID, parentId); - + // Should we show the reflection or not? We shouldn't show it when the View Forum screen is accessed // from the Monitoring Summary screen, but we should when accessed from the Learner Progress screen. // Need to constantly past this value on, rather than hiding just the once, as the View Forum Index: lams_tool_forum/web/jsps/learning/reply.jsp =================================================================== diff -u -rdbab108fb2d8f57bc99aaeef05449e7ff59f1d66 -r99a9e8ffeedcb8f20796543815866501b6739792 --- lams_tool_forum/web/jsps/learning/reply.jsp (.../reply.jsp) (revision dbab108fb2d8f57bc99aaeef05449e7ff59f1d66) +++ lams_tool_forum/web/jsps/learning/reply.jsp (.../reply.jsp) (revision 99a9e8ffeedcb8f20796543815866501b6739792) @@ -4,8 +4,8 @@ + -