Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/QaAppConstants.java =================================================================== diff -u -r4447ae1bb802ba8a16df65d880f25575d1174003 -r27a3ba4f41698f83ce6320d5b855329b1addd5eb --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/QaAppConstants.java (.../QaAppConstants.java) (revision 4447ae1bb802ba8a16df65d880f25575d1174003) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/QaAppConstants.java (.../QaAppConstants.java) (revision 27a3ba4f41698f83ce6320d5b855329b1addd5eb) @@ -101,6 +101,8 @@ public static final String USER_ID ="userId"; public static final String RICHTEXT_OFFLINEINSTRUCTIONS ="richTextOfflineInstructions"; public static final String RICHTEXT_ONLINEINSTRUCTIONS ="richTextOnlineInstructions"; + public static final String RICHTEXT_TITLE ="richTextTitle"; + public static final String RICHTEXT_INSTRUCTIONS ="richTextInstructions"; /** Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/QaUtils.java =================================================================== diff -u -r65dbef6ba8b7c63e1b36662a6ddfb2f352c0dce6 -r27a3ba4f41698f83ce6320d5b855329b1addd5eb --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/QaUtils.java (.../QaUtils.java) (revision 65dbef6ba8b7c63e1b36662a6ddfb2f352c0dce6) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/QaUtils.java (.../QaUtils.java) (revision 27a3ba4f41698f83ce6320d5b855329b1addd5eb) @@ -355,30 +355,35 @@ public static void persistRichText(HttpServletRequest request) { - String richTextOfflineInstructions=""; - String richTextOnlineInstructions=""; - richTextOfflineInstructions = (String)request.getSession().getAttribute(RICHTEXT_OFFLINEINSTRUCTIONS); - logger.debug("richTextOfflineInstructions from session: " + richTextOfflineInstructions); - - richTextOnlineInstructions = (String)request.getSession().getAttribute(RICHTEXT_ONLINEINSTRUCTIONS); - logger.debug("richTextOnlineInstructions from session: " + richTextOnlineInstructions); - - if ((richTextOfflineInstructions == null) || (richTextOfflineInstructions.equals(""))) - { - richTextOfflineInstructions=request.getParameter(RICHTEXT_OFFLINEINSTRUCTIONS); - logger.debug("read aparameter richTextOfflineInstructions: " + richTextOfflineInstructions); - } + String richTextOfflineInstructions=request.getParameter(RICHTEXT_OFFLINEINSTRUCTIONS); + logger.debug("read parameter richTextOfflineInstructions: " + richTextOfflineInstructions); + String richTextOnlineInstructions=request.getParameter(RICHTEXT_ONLINEINSTRUCTIONS); + logger.debug("read parameter richTextOnlineInstructions: " + richTextOnlineInstructions); - if ((richTextOnlineInstructions == null) || (richTextOnlineInstructions.equals(""))) - { - richTextOnlineInstructions=request.getParameter(RICHTEXT_ONLINEINSTRUCTIONS); - logger.debug("read aparameter richTextOnlineInstructions: " + richTextOnlineInstructions); - } - - logger.debug("final richTextOfflineInstructions: " + richTextOfflineInstructions); - logger.debug("final richTextOnlineInstructions: " + richTextOnlineInstructions); - request.getSession().setAttribute(RICHTEXT_OFFLINEINSTRUCTIONS,richTextOfflineInstructions); - request.getSession().setAttribute(RICHTEXT_ONLINEINSTRUCTIONS,richTextOnlineInstructions); + if ((richTextOfflineInstructions != null) && (richTextOfflineInstructions.length() > 0)) + { + request.getSession().setAttribute(RICHTEXT_OFFLINEINSTRUCTIONS,richTextOfflineInstructions); + } + + if ((richTextOnlineInstructions != null) && (richTextOnlineInstructions.length() > 0)) + { + request.getSession().setAttribute(RICHTEXT_ONLINEINSTRUCTIONS,richTextOnlineInstructions); + } + + String richTextTitle=request.getParameter(RICHTEXT_TITLE); + logger.debug("read parameter richTextTitle: " + richTextTitle); + String richTextInstructions=request.getParameter(RICHTEXT_INSTRUCTIONS); + logger.debug("read parameter richTextInstructions: " + richTextInstructions); + + if ((richTextTitle != null) && (richTextTitle.length() > 0)) + { + request.getSession().setAttribute(RICHTEXT_TITLE,richTextTitle); + } + + if ((richTextInstructions != null) && (richTextInstructions.length() > 0)) + { + request.getSession().setAttribute(RICHTEXT_INSTRUCTIONS,richTextInstructions); + } } - + } Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/AuthoringUtil.java =================================================================== diff -u -rd231fdf18f3da49f3ca42f543c784f4d09a59332 -r27a3ba4f41698f83ce6320d5b855329b1addd5eb --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/AuthoringUtil.java (.../AuthoringUtil.java) (revision d231fdf18f3da49f3ca42f543c784f4d09a59332) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/AuthoringUtil.java (.../AuthoringUtil.java) (revision 27a3ba4f41698f83ce6320d5b855329b1addd5eb) @@ -14,19 +14,18 @@ import javax.servlet.http.HttpServletRequest; import javax.servlet.http.HttpServletResponse; -import org.lamsfoundation.lams.usermanagement.User; import org.apache.log4j.Logger; import org.apache.struts.action.ActionForm; import org.apache.struts.action.ActionMapping; -import org.lamsfoundation.lams.tool.exception.DataMissingException; import org.lamsfoundation.lams.tool.exception.ToolException; import org.lamsfoundation.lams.tool.qa.QaAppConstants; import org.lamsfoundation.lams.tool.qa.QaComparator; import org.lamsfoundation.lams.tool.qa.QaContent; import org.lamsfoundation.lams.tool.qa.QaQueContent; import org.lamsfoundation.lams.tool.qa.QaUtils; import org.lamsfoundation.lams.tool.qa.service.IQaService; +import org.lamsfoundation.lams.usermanagement.User; /** * @@ -325,7 +324,7 @@ /** * read and persist rich text parameters */ - QaUtils.persistRichText(request); + //QaUtils.persistRichText(request); endLearningMessage=qaAuthoringForm.getEndLearningMessage(); } @@ -334,6 +333,7 @@ creationDate=new Date(System.currentTimeMillis()).toString(); + /** read rich text vallues */ String richTextOfflineInstructions=""; richTextOfflineInstructions = (String)request.getSession().getAttribute(RICHTEXT_OFFLINEINSTRUCTIONS); logger.debug("createContent: richTextOfflineInstructions from session: " + richTextOfflineInstructions); @@ -343,7 +343,17 @@ richTextOnlineInstructions = (String)request.getSession().getAttribute(RICHTEXT_ONLINEINSTRUCTIONS); logger.debug("createContent richTextOnlineInstructions from session: " + richTextOnlineInstructions); if (richTextOnlineInstructions == null) richTextOnlineInstructions=""; - + + String richTextTitle=""; + richTextTitle = (String)request.getSession().getAttribute(RICHTEXT_TITLE); + logger.debug("createContent richTextTitle from session: " + richTextTitle); + if (richTextTitle == null) richTextTitle=""; + + String richTextInstructions=""; + richTextInstructions = (String)request.getSession().getAttribute(RICHTEXT_INSTRUCTIONS); + logger.debug("createContent richTextInstructions from session: " + richTextInstructions); + if (richTextInstructions == null) richTextInstructions=""; + /**obtain user object from the session*/ User toolUser=(User)request.getSession().getAttribute(TOOL_USER); Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QAction.java =================================================================== diff -u -rd231fdf18f3da49f3ca42f543c784f4d09a59332 -r27a3ba4f41698f83ce6320d5b855329b1addd5eb --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QAction.java (.../QAction.java) (revision d231fdf18f3da49f3ca42f543c784f4d09a59332) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QAction.java (.../QAction.java) (revision 27a3ba4f41698f83ce6320d5b855329b1addd5eb) @@ -36,8 +36,6 @@ import org.apache.struts.action.ActionMessage; import org.apache.struts.action.ActionMessages; import org.apache.struts.actions.DispatchAction; -import org.lamsfoundation.lams.tool.exception.DataMissingException; -import org.lamsfoundation.lams.tool.exception.ToolException; import org.lamsfoundation.lams.tool.qa.QaAppConstants; import org.lamsfoundation.lams.tool.qa.QaContent; import org.lamsfoundation.lams.tool.qa.QaSession; @@ -228,12 +226,6 @@ QaAuthoringForm qaAuthoringForm = (QaAuthoringForm) form; /** - * read and persist rich text parameters - */ - QaUtils.persistRichText(request); - logger.debug("QAction: rich text set to session scope"); - - /** * the status of define later is determined from the property inspector and * by now, we know whether it is on or off * @@ -306,8 +298,7 @@ { logger.debug("user is done with this tab."); QaUtils.persistRichText(request); - logger.debug("SUBMIT_TAB_DONE: rich text set to session scope"); - qaAuthoringForm.resetUserAction(); + qaAuthoringForm.resetUserAction(); return (mapping.findForward(LOAD_QUESTIONS)); }/**submit questions contained in the Map*/ else if (userAction.equalsIgnoreCase(SUBMIT_ALL_CONTENT)) @@ -396,6 +387,7 @@ else { logger.debug("Warning!: Uncatered-for user action: " + userAction); + QaUtils.persistRichText(request); } /**reset all user actions*/ Index: lams_tool_laqa/web/AdvancedContent.jsp =================================================================== diff -u -r16e844de3f7c34b44baee1b28e67e708f42d00bf -r27a3ba4f41698f83ce6320d5b855329b1addd5eb --- lams_tool_laqa/web/AdvancedContent.jsp (.../AdvancedContent.jsp) (revision 16e844de3f7c34b44baee1b28e67e708f42d00bf) +++ lams_tool_laqa/web/AdvancedContent.jsp (.../AdvancedContent.jsp) (revision 27a3ba4f41698f83ce6320d5b855329b1addd5eb) @@ -4,6 +4,7 @@ <%@ taglib uri="/WEB-INF/c.tld" prefix="c" %> <%@ taglib uri="/WEB-INF/fmt.tld" prefix="fmt" %> +
@@ -49,7 +50,7 @@ - + @@ -83,8 +84,9 @@
:
- - + + +
Index: lams_tool_laqa/web/BasicContent.jsp =================================================================== diff -u -r16e844de3f7c34b44baee1b28e67e708f42d00bf -r27a3ba4f41698f83ce6320d5b855329b1addd5eb --- lams_tool_laqa/web/BasicContent.jsp (.../BasicContent.jsp) (revision 16e844de3f7c34b44baee1b28e67e708f42d00bf) +++ lams_tool_laqa/web/BasicContent.jsp (.../BasicContent.jsp) (revision 27a3ba4f41698f83ce6320d5b855329b1addd5eb) @@ -3,152 +3,155 @@ <%@ taglib uri="/WEB-INF/struts-logic-el.tld" prefix="logic-el" %> <%@ taglib uri="/WEB-INF/c.tld" prefix="c" %> <%@ taglib uri="/WEB-INF/fmt.tld" prefix="fmt" %> +<%@ taglib uri="fck-editor" prefix="FCK" %> - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
: - -
: - -
- : - "> -   - - - -
: " value="" - size="50" maxlength="255"> -   - - - -
- -
- - - + + + + + + + + + + - - - - - - - - -
- - - +
+ + + +
+ + + + - -
: + + +
+
: + + + +
+ : + + "> +   + + + +
- - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
: - -
: - -
- : - -
: -   -
- -
- - - + + + + + + + + + + + + + + + + - -
- - - -
: " value="" + size="50" maxlength="255"> +   + + + +
+
- - -
-
- +
+ + + + +
+ + + +
+ + + - + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + + +
: + +
: + +
+ : + +
: +   +
+ +
+ + + + +
+ + + +
+ +
+
+
+