Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaAction.java =================================================================== diff -u -r2327711a9c21985a1a82c5f470c05152bba8d741 -r86a3e0e50e5bb605b4c9a062996ed1a752e422f9 --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaAction.java (.../QaAction.java) (revision 2327711a9c21985a1a82c5f470c05152bba8d741) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaAction.java (.../QaAction.java) (revision 86a3e0e50e5bb605b4c9a062996ed1a752e422f9) @@ -634,7 +634,17 @@ String activeModule=request.getParameter(ACTIVE_MODULE); logger.debug("activeModule: " + activeModule); + + String onlineInstructions=request.getParameter(ONLINE_INSTRUCTIONS); + logger.debug("onlineInstructions: " + onlineInstructions); + + String offlineInstructions=request.getParameter(OFFLINE_INSTRUCTIONS); + logger.debug("offlineInstructions: " + offlineInstructions); + + sessionMap.put(ONLINE_INSTRUCTIONS_KEY, onlineInstructions); + sessionMap.put(OFFLINE_INSTRUCTIONS, offlineInstructions); + String strToolContentID=request.getParameter(AttributeNames.PARAM_TOOL_CONTENT_ID); logger.debug("strToolContentID: " + strToolContentID); @@ -654,7 +664,11 @@ QaGeneralAuthoringDTO qaGeneralAuthoringDTO= QaUtils.buildGeneralAuthoringDTO(request, qaService, qaContent, qaAuthoringForm); logger.debug("qaGeneralAuthoringDTO: " + qaGeneralAuthoringDTO); + + qaGeneralAuthoringDTO.setOnlineInstructions(onlineInstructions); + qaGeneralAuthoringDTO.setOfflineInstructions(offlineInstructions); + Map mapQuestionContent=(Map)sessionMap.get(MAP_QUESTION_CONTENT_KEY); logger.debug("mapQuestionContent: " + mapQuestionContent); logger.debug("mapQuestionContent size: " + mapQuestionContent.size()); @@ -674,14 +688,7 @@ qaGeneralAuthoringDTO.setActivityTitle(richTextTitle); qaGeneralAuthoringDTO.setActivityInstructions(richTextInstructions); - String onlineInstructions=(String)sessionMap.get(ONLINE_INSTRUCTIONS_KEY); - logger.debug("onlineInstructions: " + onlineInstructions); - qaGeneralAuthoringDTO.setOnlineInstructions(onlineInstructions); - String offlineInstructions=(String)sessionMap.get(OFFLINE_INSTRUCTIONS_KEY); - logger.debug("offlineInstructions: " + offlineInstructions); - qaGeneralAuthoringDTO.setOfflineInstructions(offlineInstructions); - List attachmentList=(List)sessionMap.get(ATTACHMENT_LIST_KEY); logger.debug("attachmentList: " + attachmentList); List deletedAttachmentList=(List)sessionMap.get(DELETED_ATTACHMENT_LIST_KEY); @@ -759,7 +766,14 @@ String activeModule=request.getParameter(ACTIVE_MODULE); logger.debug("activeModule: " + activeModule); + + + String onlineInstructions=(String) sessionMap.get(ONLINE_INSTRUCTIONS_KEY); + logger.debug("onlineInstructions: " + onlineInstructions); + String offlineInstructions=(String) sessionMap.get(OFFLINE_INSTRUCTIONS); + logger.debug("offlineInstructions: " + offlineInstructions); + String strToolContentID=request.getParameter(AttributeNames.PARAM_TOOL_CONTENT_ID); logger.debug("strToolContentID: " + strToolContentID); @@ -769,6 +783,8 @@ QaContent qaContent=qaService.loadQa(new Long(strToolContentID).longValue()); logger.debug("qaContent: " + qaContent); + + if (qaContent == null) { logger.debug("using defaultContentIdStr: " + defaultContentIdStr); @@ -780,6 +796,10 @@ QaGeneralAuthoringDTO qaGeneralAuthoringDTO= QaUtils.buildGeneralAuthoringDTO(request, qaService, qaContent, qaAuthoringForm); logger.debug("qaGeneralAuthoringDTO: " + qaGeneralAuthoringDTO); + qaGeneralAuthoringDTO.setOnlineInstructions(onlineInstructions); + qaGeneralAuthoringDTO.setOfflineInstructions(offlineInstructions); + + qaGeneralAuthoringDTO.setSbmtSuccess( new Integer(0).toString()); Map mapQuestionContent=(Map)sessionMap.get(MAP_QUESTION_CONTENT_KEY); logger.debug("mapQuestionContent: " + mapQuestionContent); @@ -797,14 +817,7 @@ qaAuthoringForm, qaGeneralAuthoringDTO, strToolContentID, defaultContentIdStr, activeModule, sessionMap, httpSessionID); - String onlineInstructions=(String)sessionMap.get(ONLINE_INSTRUCTIONS_KEY); - logger.debug("onlineInstructions: " + onlineInstructions); - qaGeneralAuthoringDTO.setOnlineInstructions(onlineInstructions); - String offlineInstructions=(String)sessionMap.get(OFFLINE_INSTRUCTIONS_KEY); - logger.debug("offlineInstructions: " + offlineInstructions); - qaGeneralAuthoringDTO.setOfflineInstructions(offlineInstructions); - String richTextTitle = (String)sessionMap.get(ACTIVITY_TITLE_KEY); String richTextInstructions = (String)sessionMap.get(ACTIVITY_INSTRUCTIONS_KEY); Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaDLStarterAction.java =================================================================== diff -u -r65166da92a6f0f4ff73acb92b95672e237b25742 -r86a3e0e50e5bb605b4c9a062996ed1a752e422f9 --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaDLStarterAction.java (.../QaDLStarterAction.java) (revision 65166da92a6f0f4ff73acb92b95672e237b25742) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaDLStarterAction.java (.../QaDLStarterAction.java) (revision 86a3e0e50e5bb605b4c9a062996ed1a752e422f9) @@ -106,7 +106,6 @@ logger.debug("init defineLater mode. removed attributes..."); IQaService qaService = QaServiceProxy.getQaService(getServlet().getServletContext()); - request.getSession().setAttribute(TOOL_SERVICE, qaService); QaStarterAction qaStarterAction= new QaStarterAction(); return qaStarterAction.executeDefineLater(mapping, form, request, response, qaService); Index: lams_tool_laqa/web/authoring/BasicContentViewOnly.jsp =================================================================== diff -u -r3ecb833c87586c3a5ea364bbb1632c872f7011cb -r86a3e0e50e5bb605b4c9a062996ed1a752e422f9 --- lams_tool_laqa/web/authoring/BasicContentViewOnly.jsp (.../BasicContentViewOnly.jsp) (revision 3ecb833c87586c3a5ea364bbb1632c872f7011cb) +++ lams_tool_laqa/web/authoring/BasicContentViewOnly.jsp (.../BasicContentViewOnly.jsp) (revision 86a3e0e50e5bb605b4c9a062996ed1a752e422f9) @@ -32,13 +32,13 @@