Index: lams_build/lib/lams/lams.jar =================================================================== diff -u -r8daec7e047f7ea8404c6f16218b63a8e855a15b2 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 Binary files differ Index: lams_common/src/java/org/lamsfoundation/lams/util/WebUtil.java =================================================================== diff -u -r73468dfa4bf7c6242bc350c790374188b5b97646 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_common/src/java/org/lamsfoundation/lams/util/WebUtil.java (.../WebUtil.java) (revision 73468dfa4bf7c6242bc350c790374188b5b97646) +++ lams_common/src/java/org/lamsfoundation/lams/util/WebUtil.java (.../WebUtil.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -22,6 +22,7 @@ import org.lamsfoundation.lams.tool.ToolAccessMode; import org.lamsfoundation.lams.tool.exception.ToolException; import org.lamsfoundation.lams.usermanagement.User; +import org.lamsfoundation.lams.web.util.AttributeNames; /** * helper methods useful for servlets @@ -353,7 +354,6 @@ * supports all three ways of looking at a learner screen. * * @param request - * @param param_mode * @return the ToolAccessMode object */ public static ToolAccessMode getToolAccessMode(String modeValue) { @@ -368,6 +368,24 @@ } throw new IllegalArgumentException("[" + modeValue + "] is not a legal mode" + "in LAMS"); } + + /** + * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. + * + * @param request + * @return + */ + public static ToolAccessMode readToolAccessModeAuthorDefaulted(HttpServletRequest request) { + String modeStr = request.getParameter(AttributeNames.ATTR_MODE); + + ToolAccessMode mode; + if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { + mode = ToolAccessMode.TEACHER; + } else { + mode = ToolAccessMode.AUTHOR; + } + return mode; + } /** *

Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/AuthoringAction.java =================================================================== diff -u -rc97b41d72e071f6293126bd85f6eba0b756f7836 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision c97b41d72e071f6293126bd85f6eba0b756f7836) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -107,13 +107,8 @@ String param = mapping.getParameter(); // -----------------------Assessment Author functions --------------------------- if (param.equals("start")) { - ToolAccessMode mode = getAccessMode(request); - // teacher mode "check for new" button enter. - if (mode != null) { - request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); - } else { - request.setAttribute(AttributeNames.ATTR_MODE, ToolAccessMode.AUTHOR.toString()); - } + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); + request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return start(mapping, form, request, response); } if (param.equals("definelater")) { @@ -298,7 +293,7 @@ throw new ServletException(e); } - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return mapping.findForward(AssessmentConstants.SUCCESS); @@ -323,7 +318,7 @@ SessionMap sessionMap = (SessionMap) request.getSession() .getAttribute(assessmentForm.getSessionMapID()); - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); Assessment assessment = assessmentForm.getAssessment(); IAssessmentService service = getAssessmentService(); @@ -1890,23 +1885,6 @@ } - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - private Set updateQuestionReferencesGrades(HttpServletRequest request, SessionMap sessionMap, boolean isFormSubmit) { Map paramMap = splitRequestParameter(request, Index: lams_tool_bbb/src/java/org/lamsfoundation/lams/tool/bbb/web/actions/AuthoringAction.java =================================================================== diff -u -r3399163940c61c9132223c758d274486e57ff9b7 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_bbb/src/java/org/lamsfoundation/lams/tool/bbb/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 3399163940c61c9132223c758d274486e57ff9b7) +++ lams_tool_bbb/src/java/org/lamsfoundation/lams/tool/bbb/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -88,7 +88,7 @@ String contentFolderID = WebUtil.readStrParam(request, AttributeNames.PARAM_CONTENT_FOLDER_ID); - ToolAccessMode mode = WebUtil.readToolAccessModeParam(request, AttributeNames.PARAM_MODE, true); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); // retrieving Bbb with given toolContentID Bbb bbb = bbbService.getBbbByContentId(toolContentID); @@ -99,7 +99,7 @@ // TODO NOTE: this causes DB orphans when LD not saved. } - if (mode != null && mode.isTeacher()) { + if (mode.isTeacher()) { // Set the defineLater flag so that learners cannot use content // while we are editing. This flag is released when updateContent is // called. @@ -112,7 +112,7 @@ copyProperties(authForm, bbb); // Set up sessionMap - SessionMap map = createSessionMap(bbb, getAccessMode(request), contentFolderID, toolContentID); + SessionMap map = createSessionMap(bbb, mode, contentFolderID, toolContentID); authForm.setSessionMapID(map.getSessionID()); // add the sessionMap to HTTPSession. @@ -211,23 +211,6 @@ } /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - - /** * Retrieve the SessionMap from the HttpSession. * * @param request Index: lams_tool_chat/src/java/org/lamsfoundation/lams/tool/chat/web/actions/AuthoringAction.java =================================================================== diff -u -r3399163940c61c9132223c758d274486e57ff9b7 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_chat/src/java/org/lamsfoundation/lams/tool/chat/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 3399163940c61c9132223c758d274486e57ff9b7) +++ lams_tool_chat/src/java/org/lamsfoundation/lams/tool/chat/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -87,7 +87,7 @@ String contentFolderID = WebUtil.readStrParam(request, AttributeNames.PARAM_CONTENT_FOLDER_ID); - ToolAccessMode mode = WebUtil.readToolAccessModeParam(request, "mode", true); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); // set up chatService if (chatService == null) { @@ -103,7 +103,7 @@ // TODO NOTE: this causes DB orphans when LD not saved. } - if (mode != null && mode.isTeacher()) { + if (mode.isTeacher()) { // Set the defineLater flag so that learners cannot use content // while we // are editing. This flag is released when updateContent is called. @@ -116,7 +116,7 @@ updateAuthForm(authForm, chat); // Set up sessionMap - SessionMap map = createSessionMap(chat, getAccessMode(request), contentFolderID, toolContentID); + SessionMap map = createSessionMap(chat, mode, contentFolderID, toolContentID); authForm.setSessionMapID(map.getSessionID()); // add the sessionMap to HTTPSession. @@ -242,23 +242,6 @@ } /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - - /** * Retrieve the SessionMap from the HttpSession. * * @param request Index: lams_tool_daco/src/java/org/lamsfoundation/lams/tool/daco/web/action/AuthoringAction.java =================================================================== diff -u -r412fe7a30d27bd7402ae872029af8dd655d4a638 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_daco/src/java/org/lamsfoundation/lams/tool/daco/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 412fe7a30d27bd7402ae872029af8dd655d4a638) +++ lams_tool_daco/src/java/org/lamsfoundation/lams/tool/daco/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -88,13 +88,8 @@ // -----------------------Daco Author function // --------------------------- if (param.equals("start")) { - ToolAccessMode mode = getAccessMode(request); - // teacher mode "check for new" button enter. - if (mode != null) { - request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); - } else { - request.setAttribute(AttributeNames.ATTR_MODE, ToolAccessMode.AUTHOR.toString()); - } + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); + request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return start(mapping, form, request); } if (param.equals("definelater")) { @@ -291,23 +286,6 @@ } /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - protected ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - - /** * Get answer options from HttpRequest * * @param request @@ -438,7 +416,7 @@ throw new ServletException(e); } - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return mapping.findForward(DacoConstants.SUCCESS); @@ -763,7 +741,7 @@ // get back sessionMAP SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(dacoForm.getSessionMapID()); - ToolAccessMode toolAccessMode = getAccessMode(request); + ToolAccessMode toolAccessMode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, toolAccessMode.toString()); ActionMessages errors = validateDacoForm(dacoForm, mapping, request); Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/util/ForumWebUtils.java =================================================================== diff -u -r3399163940c61c9132223c758d274486e57ff9b7 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/util/ForumWebUtils.java (.../ForumWebUtils.java) (revision 3399163940c61c9132223c758d274486e57ff9b7) +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/util/ForumWebUtils.java (.../ForumWebUtils.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -54,24 +54,4 @@ } } - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - public static ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode = null; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.AUTHOR.toString())) { - mode = ToolAccessMode.AUTHOR; - } else if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.LEARNER.toString())) { - mode = ToolAccessMode.LEARNER; - } - - return mode; - } - } Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java =================================================================== diff -u -rb47369533cc464d433b2a4306e8cc0bd3f8ecaa1 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision b47369533cc464d433b2a4306e8cc0bd3f8ecaa1) +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -252,7 +252,7 @@ conditionSet.clear(); conditionSet.addAll(forum.getConditions()); - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return mapping.findForward("success"); @@ -281,7 +281,7 @@ HttpServletResponse response) throws IllegalAccessException, InvocationTargetException, NoSuchMethodException { - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); ForumForm forumForm = (ForumForm) form; @@ -782,23 +782,6 @@ } /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - - /** * Forum validation method from STRUCT interface. * */ Index: lams_tool_gmap/src/java/org/lamsfoundation/lams/tool/gmap/web/actions/AuthoringAction.java =================================================================== diff -u -r3399163940c61c9132223c758d274486e57ff9b7 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_gmap/src/java/org/lamsfoundation/lams/tool/gmap/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 3399163940c61c9132223c758d274486e57ff9b7) +++ lams_tool_gmap/src/java/org/lamsfoundation/lams/tool/gmap/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -32,7 +32,6 @@ import javax.servlet.http.HttpServletResponse; import javax.servlet.http.HttpSession; -import org.apache.commons.lang.StringUtils; import org.apache.log4j.Logger; import org.apache.struts.action.ActionForm; import org.apache.struts.action.ActionForward; @@ -56,13 +55,6 @@ /** * @author - * @version - * - * - * - * - * - * */ public class AuthoringAction extends LamsDispatchAction { @@ -92,7 +84,7 @@ String contentFolderID = WebUtil.readStrParam(request, AttributeNames.PARAM_CONTENT_FOLDER_ID); - ToolAccessMode mode = WebUtil.readToolAccessModeParam(request, "mode", true); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); // set up gmapService if (gmapService == null) { @@ -108,7 +100,7 @@ // TODO NOTE: this causes DB orphans when LD not saved. } - if (mode != null && mode.isTeacher()) { + if (mode.isTeacher()) { // Set the defineLater flag so that learners cannot use content // while we // are editing. This flag is released when updateContent is called. @@ -121,7 +113,7 @@ updateAuthForm(authForm, gmap); // Set up sessionMap - SessionMap map = createSessionMap(gmap, getAccessMode(request), contentFolderID, toolContentID); + SessionMap map = createSessionMap(gmap, mode, contentFolderID, toolContentID); authForm.setSessionMapID(map.getSessionID()); authForm.setGmap(gmap); @@ -283,24 +275,6 @@ } /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR - * mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - - /** * Retrieve the SessionMap from the HttpSession using the author form. * * @param request Index: lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/AuthoringAction.java =================================================================== diff -u -r2228652cfb59ec7c0fcbb1914a94f65d22815911 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 2228652cfb59ec7c0fcbb1914a94f65d22815911) +++ lams_tool_images/src/java/org/lamsfoundation/lams/tool/imageGallery/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -87,13 +87,8 @@ String param = mapping.getParameter(); // -----------------------ImageGallery Author function --------------------------- if (param.equals("start")) { - ToolAccessMode mode = getAccessMode(request); - // teacher mode "check for new" button enter. - if (mode != null) { - request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); - } else { - request.setAttribute(AttributeNames.ATTR_MODE, ToolAccessMode.AUTHOR.toString()); - } + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); + request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return start(mapping, form, request, response); } if (param.equals("definelater")) { @@ -255,7 +250,7 @@ throw new ServletException(e); } - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return mapping.findForward(ImageGalleryConstants.SUCCESS); @@ -278,7 +273,7 @@ // get back sessionMAP SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(imageGalleryForm.getSessionMapID()); - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); ImageGallery imageGallery = imageGalleryForm.getImageGallery(); @@ -861,21 +856,4 @@ } } - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - } Index: lams_tool_imscc/src/java/org/lamsfoundation/lams/tool/commonCartridge/web/action/AuthoringAction.java =================================================================== diff -u -r412fe7a30d27bd7402ae872029af8dd655d4a638 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_imscc/src/java/org/lamsfoundation/lams/tool/commonCartridge/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 412fe7a30d27bd7402ae872029af8dd655d4a638) +++ lams_tool_imscc/src/java/org/lamsfoundation/lams/tool/commonCartridge/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -89,13 +89,8 @@ String param = mapping.getParameter(); // -----------------------CommonCartridge Author function --------------------------- if (param.equals("start")) { - ToolAccessMode mode = getAccessMode(request); - // teacher mode "check for new" button enter. - if (mode != null) { - request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); - } else { - request.setAttribute(AttributeNames.ATTR_MODE, ToolAccessMode.AUTHOR.toString()); - } + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); + request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return start(mapping, form, request, response); } if (param.equals("definelater")) { @@ -436,7 +431,7 @@ throw new ServletException(e); } - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return mapping.findForward(CommonCartridgeConstants.SUCCESS); @@ -460,7 +455,7 @@ // get back sessionMAP SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(commonCartridgeForm.getSessionMapID()); - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); CommonCartridge commonCartridge = commonCartridgeForm.getCommonCartridge(); @@ -810,23 +805,6 @@ return errors; } - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - public ActionForward initPedagogicalPlannerForm(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) { CommonCartridgePedagogicalPlannerForm plannerForm = (CommonCartridgePedagogicalPlannerForm) form; Index: lams_tool_kaltura/src/java/org/lamsfoundation/lams/tool/kaltura/web/actions/AuthoringAction.java =================================================================== diff -u -r2f725f8ef2aa09a2663b2335bf67213074426d11 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_kaltura/src/java/org/lamsfoundation/lams/tool/kaltura/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 2f725f8ef2aa09a2663b2335bf67213074426d11) +++ lams_tool_kaltura/src/java/org/lamsfoundation/lams/tool/kaltura/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -92,7 +92,7 @@ String contentFolderID = WebUtil.readStrParam(request, AttributeNames.PARAM_CONTENT_FOLDER_ID); - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); // set up kalturaService if (kalturaService == null) { @@ -107,7 +107,7 @@ kalturaService.saveOrUpdateKaltura(kaltura); } - if (mode != null && mode.isTeacher()) { + if (mode.isTeacher()) { // Set the defineLater flag so that learners cannot use content // while we // are editing. This flag is released when updateContent is called. @@ -365,20 +365,6 @@ } /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - - /** * Retrieve the SessionMap from the HttpSession. */ private SessionMap getSessionMap(HttpServletRequest request, AuthoringForm authForm) { Index: lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McAction.java =================================================================== diff -u -r2adc76b37e28820736a7a35934142d124c831392 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McAction.java (.../McAction.java) (revision 2adc76b37e28820736a7a35934142d124c831392) +++ lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McAction.java (.../McAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -131,7 +131,7 @@ McContent mcContent = mcContentTest; if (errors.isEmpty()) { - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); List deletedQuestionDTOs = (List) sessionMap.get(LIST_DELETED_QUESTION_DTOS); @@ -1582,7 +1582,7 @@ HttpServletResponse response) throws IOException, ServletException { McAuthoringForm mcAuthoringForm = (McAuthoringForm) form; - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); IMcService mcService = McServiceProxy.getMcService(getServlet().getServletContext()); @@ -2007,23 +2007,6 @@ } /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - - /** * repopulateOptionsBox */ private static List repopulateOptionDTOs(HttpServletRequest request, boolean isAddBlankOptions) { Index: lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McStarterAction.java =================================================================== diff -u -r2f725f8ef2aa09a2663b2335bf67213074426d11 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McStarterAction.java (.../McStarterAction.java) (revision 2f725f8ef2aa09a2663b2335bf67213074426d11) +++ lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McStarterAction.java (.../McStarterAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -97,7 +97,7 @@ String strToolContentID = request.getParameter(AttributeNames.PARAM_TOOL_CONTENT_ID); mcGeneralAuthoringDTO.setToolContentID(strToolContentID); - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); // request is from monitoring module if (mode.isTeacher()) { mcService.setDefineLater(strToolContentID, true); @@ -269,21 +269,4 @@ errors.add(Globals.ERROR_KEY, new ActionMessage(message)); saveErrors(request, errors); } - - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } } Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaAction.java =================================================================== diff -u -r0550d38d2c0eb5b51a510aa4234526ce08fb5c70 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaAction.java (.../QaAction.java) (revision 0550d38d2c0eb5b51a510aa4234526ce08fb5c70) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaAction.java (.../QaAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -129,7 +129,7 @@ QaContent qaContent = qaService.getQaContent(toolContentID); if (errors.isEmpty()) { - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); List deletedQuestionDTOs = (List) sessionMap.get(LIST_DELETED_QUESTION_DTOS); @@ -972,21 +972,4 @@ } return list; } - - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } } Index: lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaStarterAction.java =================================================================== diff -u -rb036e6dd6d2afe9c0e0b9143c9be3af1d53a1e0e -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaStarterAction.java (.../QaStarterAction.java) (revision b036e6dd6d2afe9c0e0b9143c9be3af1d53a1e0e) +++ lams_tool_laqa/src/java/org/lamsfoundation/lams/tool/qa/web/QaStarterAction.java (.../QaStarterAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -118,7 +118,7 @@ prepareDTOandForm(request, qaAuthoringForm, qaContent, qaService, sessionMap); - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); // request is from monitoring module if (mode.isTeacher()) { qaService.setDefineLater(strToolContentID, true); @@ -281,21 +281,4 @@ } return list; } - - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } } Index: lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java =================================================================== diff -u -r16ce604dcc21fbcf8b462fe28d660723ff7c925a -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 16ce604dcc21fbcf8b462fe28d660723ff7c925a) +++ lams_tool_larsrc/src/java/org/lamsfoundation/lams/tool/rsrc/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -98,13 +98,8 @@ // -----------------------Resource Author function // --------------------------- if (param.equals("start")) { - ToolAccessMode mode = getAccessMode(request); - // teacher mode "check for new" button enter. - if (mode != null) { - request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); - } else { - request.setAttribute(AttributeNames.ATTR_MODE, ToolAccessMode.AUTHOR.toString()); - } + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); + request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return start(mapping, form, request, response); } if (param.equals("definelater")) { @@ -487,7 +482,7 @@ throw new ServletException(e); } - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return mapping.findForward(ResourceConstants.SUCCESS); @@ -511,7 +506,7 @@ // get back sessionMAP SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(resourceForm.getSessionMapID()); - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); ActionMessages errors = validate(resourceForm, mapping, request); @@ -939,24 +934,6 @@ return errors; } - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR - * mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - private ActionMessages validate(ResourceForm resourceForm, ActionMapping mapping, HttpServletRequest request) { ActionMessages errors = new ActionMessages(); if (StringUtils.isBlank(resourceForm.getResource().getTitle())) { Index: lams_tool_leader/src/java/org/lamsfoundation/lams/tool/leaderselection/web/actions/AuthoringAction.java =================================================================== diff -u -r2f725f8ef2aa09a2663b2335bf67213074426d11 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_leader/src/java/org/lamsfoundation/lams/tool/leaderselection/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 2f725f8ef2aa09a2663b2335bf67213074426d11) +++ lams_tool_leader/src/java/org/lamsfoundation/lams/tool/leaderselection/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -76,7 +76,7 @@ String contentFolderID = WebUtil.readStrParam(request, AttributeNames.PARAM_CONTENT_FOLDER_ID); - ToolAccessMode mode = WebUtil.readToolAccessModeParam(request, "mode", true); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); // set up leaderselectionService if (leaderselectionService == null) { @@ -93,7 +93,7 @@ // TODO NOTE: this causes DB orphans when LD not saved. } - if (mode != null && mode.isTeacher()) { + if (mode.isTeacher()) { // Set the defineLater flag so that learners cannot use content // while we // are editing. This flag is released when updateContent is called. @@ -108,7 +108,7 @@ // Set up sessionMap SessionMap map = new SessionMap(); - map.put(AuthoringAction.KEY_MODE, getAccessMode(request)); + map.put(AuthoringAction.KEY_MODE, mode); map.put(AuthoringAction.KEY_CONTENT_FOLDER_ID, contentFolderID); map.put(AuthoringAction.KEY_TOOL_CONTENT_ID, toolContentID); authForm.setSessionMapID(map.getSessionID()); @@ -155,23 +155,4 @@ return mapping.findForward("success"); } - - /* ========== Private Methods ********** */ - - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } } Index: lams_tool_mindmap/src/java/org/lamsfoundation/lams/tool/mindmap/web/actions/AuthoringAction.java =================================================================== diff -u -r2f725f8ef2aa09a2663b2335bf67213074426d11 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_mindmap/src/java/org/lamsfoundation/lams/tool/mindmap/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 2f725f8ef2aa09a2663b2335bf67213074426d11) +++ lams_tool_mindmap/src/java/org/lamsfoundation/lams/tool/mindmap/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -84,7 +84,7 @@ // Extract toolContentID from parameters. Long toolContentID = new Long(WebUtil.readLongParam(request, AttributeNames.PARAM_TOOL_CONTENT_ID)); String contentFolderID = WebUtil.readStrParam(request, AttributeNames.PARAM_CONTENT_FOLDER_ID); - ToolAccessMode mode = WebUtil.readToolAccessModeParam(request, "mode", true); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); // set up mindmapService if (mindmapService == null) { @@ -111,7 +111,7 @@ mindmapService.saveMindmapNode(null, rootMindmapNode, 3l, childNodeName2, "ffffff", null, mindmap, null); } - if (mode != null && mode.isTeacher()) { + if (mode.isTeacher()) { // Set the defineLater flag so that learners cannot use content // while we are editing. This flag is released when updateContent is called. mindmap.setDefineLater(true); @@ -139,7 +139,7 @@ updateAuthForm(authForm, mindmap); // Set up sessionMap - SessionMap map = createSessionMap(mindmap, getAccessMode(request), contentFolderID, + SessionMap map = createSessionMap(mindmap, mode, contentFolderID, toolContentID); authForm.setSessionMapID(map.getSessionID()); @@ -337,22 +337,6 @@ } /** - * Gets ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - - /** * Retrieve the SessionMap from the HttpSession. * * @param request Index: lams_tool_notebook/src/java/org/lamsfoundation/lams/tool/notebook/web/actions/AuthoringAction.java =================================================================== diff -u -rdba506c6b1663eba922793a44b6ad5726f2931d1 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_notebook/src/java/org/lamsfoundation/lams/tool/notebook/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision dba506c6b1663eba922793a44b6ad5726f2931d1) +++ lams_tool_notebook/src/java/org/lamsfoundation/lams/tool/notebook/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -86,7 +86,7 @@ String contentFolderID = WebUtil.readStrParam(request, AttributeNames.PARAM_CONTENT_FOLDER_ID); - ToolAccessMode mode = WebUtil.readToolAccessModeParam(request, "mode", true); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); // set up notebookService if (notebookService == null) { @@ -102,7 +102,7 @@ // TODO NOTE: this causes DB orphans when LD not saved. } - if (mode != null && mode.isTeacher()) { + if (mode.isTeacher()) { // Set the defineLater flag so that learners cannot use content while we are editing. This flag is released // when updateContent is called. notebook.setDefineLater(true); @@ -114,7 +114,7 @@ updateAuthForm(authForm, notebook); // Set up sessionMap - SessionMap map = createSessionMap(notebook, getAccessMode(request), contentFolderID, + SessionMap map = createSessionMap(notebook, mode, contentFolderID, toolContentID); authForm.setSessionMapID(map.getSessionID()); @@ -243,23 +243,6 @@ } /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - - /** * Retrieve the SessionMap from the HttpSession. * * @param request Index: lams_tool_pixlr/src/java/org/lamsfoundation/lams/tool/pixlr/web/actions/AuthoringAction.java =================================================================== diff -u -r2f725f8ef2aa09a2663b2335bf67213074426d11 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_pixlr/src/java/org/lamsfoundation/lams/tool/pixlr/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 2f725f8ef2aa09a2663b2335bf67213074426d11) +++ lams_tool_pixlr/src/java/org/lamsfoundation/lams/tool/pixlr/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -97,11 +97,7 @@ AuthoringForm authForm = (AuthoringForm) form; - ToolAccessMode mode = null; - String modeStr = WebUtil.readStrParam(request, "mode", true); - if (modeStr != null && !modeStr.trim().equals("")) { - mode = WebUtil.readToolAccessModeParam(request, "mode", true); - } + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); // set up pixlrService if (pixlrService == null) { @@ -117,7 +113,7 @@ // TODO NOTE: this causes DB orphans when LD not saved. } - if (mode != null && mode.isTeacher()) { + if (mode.isTeacher()) { // Set the defineLater flag so that learners cannot use content // while we // are editing. This flag is released when updateContent is called. @@ -141,15 +137,11 @@ // Set up the authForm. updateAuthForm(authForm, pixlr); authForm.setToolContentID(toolContentID); - if (mode != null) { - authForm.setMode(mode.toString()); - } else { - authForm.setMode(""); - } + authForm.setMode(mode.toString()); authForm.setContentFolderID(contentFolderID); // Set up sessionMap - SessionMap map = createSessionMap(pixlr, getAccessMode(request), contentFolderID, + SessionMap map = createSessionMap(pixlr, mode, contentFolderID, toolContentID); authForm.setSessionMapID(map.getSessionID()); @@ -288,24 +280,6 @@ } /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR - * mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - - /** * Retrieve the SessionMap from the HttpSession. * * @param request Index: lams_tool_preview/src/java/org/lamsfoundation/lams/tool/peerreview/web/action/AuthoringAction.java =================================================================== diff -u -r412fe7a30d27bd7402ae872029af8dd655d4a638 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_preview/src/java/org/lamsfoundation/lams/tool/peerreview/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 412fe7a30d27bd7402ae872029af8dd655d4a638) +++ lams_tool_preview/src/java/org/lamsfoundation/lams/tool/peerreview/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -69,13 +69,8 @@ String param = mapping.getParameter(); if (param.equals("start")) { - ToolAccessMode mode = getAccessMode(request); - // teacher mode "check for new" button enter. - if (mode != null) { - request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); - } else { - request.setAttribute(AttributeNames.ATTR_MODE, ToolAccessMode.AUTHOR.toString()); - } + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); + request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return start(mapping, form, request, response); } if (param.equals("definelater")) { @@ -178,7 +173,7 @@ throw new ServletException(e); } - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return mapping.findForward(PeerreviewConstants.SUCCESS); @@ -203,7 +198,7 @@ SessionMap sessionMap = (SessionMap) request.getSession() .getAttribute(peerreviewForm.getSessionMapID()); - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); Peerreview peerreview = peerreviewForm.getPeerreview(); @@ -270,22 +265,4 @@ return (IPeerreviewService) wac.getBean(PeerreviewConstants.PEERREVIEW_SERVICE); } - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR - * mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - } Index: lams_tool_scratchie/src/java/org/lamsfoundation/lams/tool/scratchie/web/action/AuthoringAction.java =================================================================== diff -u -r2f725f8ef2aa09a2663b2335bf67213074426d11 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_scratchie/src/java/org/lamsfoundation/lams/tool/scratchie/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 2f725f8ef2aa09a2663b2335bf67213074426d11) +++ lams_tool_scratchie/src/java/org/lamsfoundation/lams/tool/scratchie/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -90,13 +90,8 @@ // -----------------------Scratchie Author functions ----------- String param = mapping.getParameter(); if (param.equals("start")) { - ToolAccessMode mode = getAccessMode(request); - // teacher mode "check for new" button enter. - if (mode != null) { - request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); - } else { - request.setAttribute(AttributeNames.ATTR_MODE, ToolAccessMode.AUTHOR.toString()); - } + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); + request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return start(mapping, form, request, response); } if (param.equals("definelater")) { @@ -271,7 +266,7 @@ throw new ServletException(e); } - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return mapping.findForward(ScratchieConstants.SUCCESS); @@ -288,7 +283,7 @@ SessionMap sessionMap = (SessionMap) request.getSession() .getAttribute(scratchieForm.getSessionMapID()); - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); Scratchie scratchie = scratchieForm.getScratchie(); IScratchieService service = getScratchieService(); @@ -982,21 +977,4 @@ } } - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - } Index: lams_tool_scribe/src/java/org/lamsfoundation/lams/tool/scribe/web/actions/AuthoringAction.java =================================================================== diff -u -r2f725f8ef2aa09a2663b2335bf67213074426d11 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_scribe/src/java/org/lamsfoundation/lams/tool/scribe/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 2f725f8ef2aa09a2663b2335bf67213074426d11) +++ lams_tool_scribe/src/java/org/lamsfoundation/lams/tool/scribe/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -95,7 +95,7 @@ String contentFolderID = WebUtil.readStrParam(request, AttributeNames.PARAM_CONTENT_FOLDER_ID); - ToolAccessMode mode = WebUtil.readToolAccessModeParam(request, "mode", true); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); // set up scribeService if (scribeService == null) { @@ -119,7 +119,7 @@ return mapping.findForward("message_page"); } - if (mode != null && mode.isTeacher()) { + if (mode.isTeacher()) { // Set the defineLater flag so that learners cannot use content // while we // are editing. This flag is released when updateContent is called. @@ -132,7 +132,7 @@ updateAuthForm(authForm, scribe); // Set up sessionMap - SessionMap map = createSessionMap(scribe, getAccessMode(request), contentFolderID, + SessionMap map = createSessionMap(scribe, mode, contentFolderID, toolContentID); authForm.setSessionMapID(map.getSessionID()); @@ -389,24 +389,6 @@ return map; } - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR - * mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - private List getHeadingList(SessionMap map) { return (List) map.get(KEY_HEADINGS); } Index: lams_tool_spreadsheet/src/java/org/lamsfoundation/lams/tool/spreadsheet/web/action/AuthoringAction.java =================================================================== diff -u -r412fe7a30d27bd7402ae872029af8dd655d4a638 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_spreadsheet/src/java/org/lamsfoundation/lams/tool/spreadsheet/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 412fe7a30d27bd7402ae872029af8dd655d4a638) +++ lams_tool_spreadsheet/src/java/org/lamsfoundation/lams/tool/spreadsheet/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -66,13 +66,8 @@ String param = mapping.getParameter(); // -----------------------Spreadsheet Author function --------------------------- if (param.equals("start")) { - ToolAccessMode mode = getAccessMode(request); - // teacher mode "check for new" button enter. - if (mode != null) { - request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); - } else { - request.setAttribute(AttributeNames.ATTR_MODE, ToolAccessMode.AUTHOR.toString()); - } + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); + request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return start(mapping, form, request, response); } if (param.equals("definelater")) { @@ -169,7 +164,7 @@ throw new ServletException(e); } - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return mapping.findForward(SpreadsheetConstants.SUCCESS); @@ -189,7 +184,7 @@ HttpServletResponse response) throws Exception { SpreadsheetForm spreadsheetForm = (SpreadsheetForm) (form); - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); Spreadsheet spreadsheet = spreadsheetForm.getSpreadsheet(); @@ -251,21 +246,4 @@ return (ISpreadsheetService) wac.getBean(SpreadsheetConstants.RESOURCE_SERVICE); } - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - } Index: lams_tool_spreadsheet/src/java/org/lamsfoundation/lams/tool/spreadsheet/web/action/LearningAction.java =================================================================== diff -u -r19a48bf35052a72202cc4867e68d5666f0d05898 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_spreadsheet/src/java/org/lamsfoundation/lams/tool/spreadsheet/web/action/LearningAction.java (.../LearningAction.java) (revision 19a48bf35052a72202cc4867e68d5666f0d05898) +++ lams_tool_spreadsheet/src/java/org/lamsfoundation/lams/tool/spreadsheet/web/action/LearningAction.java (.../LearningAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -396,21 +396,4 @@ } return spreadsheetUser; } - - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } } Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/AuthoringAction.java =================================================================== diff -u -rb47369533cc464d433b2a4306e8cc0bd3f8ecaa1 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision b47369533cc464d433b2a4306e8cc0bd3f8ecaa1) +++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -94,13 +94,8 @@ String param = mapping.getParameter(); // -----------------------Survey Author function --------------------------- if (param.equals("start")) { - ToolAccessMode mode = getAccessMode(request); - // teacher mode "check for new" button enter. - if (mode != null) { - request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); - } else { - request.setAttribute(AttributeNames.ATTR_MODE, ToolAccessMode.AUTHOR.toString()); - } + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); + request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return start(mapping, form, request, response); } if (param.equals("definelater")) { @@ -501,7 +496,7 @@ throw new ServletException(e); } - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return mapping.findForward(SurveyConstants.SUCCESS); @@ -524,7 +519,7 @@ // get back sessionMAP SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(surveyForm.getSessionMapID()); - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); Survey survey = surveyForm.getSurvey(); @@ -869,23 +864,6 @@ } /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - - /** * Ajax call, will add one more input line for new survey item instruction. * * @param mapping Index: lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/AuthoringAction.java =================================================================== diff -u -r412fe7a30d27bd7402ae872029af8dd655d4a638 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 412fe7a30d27bd7402ae872029af8dd655d4a638) +++ lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -93,13 +93,8 @@ // ----------------------- solid taskList methods --------------------------- if (param.equals("start")) { - ToolAccessMode mode = getAccessMode(request); - // teacher mode "check for new" button enter. - if (mode != null) { - request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); - } else { - request.setAttribute(AttributeNames.ATTR_MODE, ToolAccessMode.AUTHOR.toString()); - } + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); + request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return start(mapping, form, request, response); } if (param.equals("definelater")) { @@ -264,7 +259,7 @@ throw new ServletException(e); } - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); return mapping.findForward(TaskListConstants.SUCCESS); @@ -287,7 +282,7 @@ // get back sessionMAP SessionMap sessionMap = (SessionMap) request.getSession().getAttribute(taskListForm.getSessionMapID()); - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); TaskList taskList = taskListForm.getTaskList(); @@ -769,23 +764,6 @@ return errors; } - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - public ActionForward initPedagogicalPlannerForm(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) { TaskListPedagogicalPlannerForm plannerForm = (TaskListPedagogicalPlannerForm) form; Index: lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/AuthoringTaskListConditionAction.java =================================================================== diff -u -r2f725f8ef2aa09a2663b2335bf67213074426d11 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/AuthoringTaskListConditionAction.java (.../AuthoringTaskListConditionAction.java) (revision 2f725f8ef2aa09a2663b2335bf67213074426d11) +++ lams_tool_task/src/java/org/lamsfoundation/lams/tool/taskList/web/action/AuthoringTaskListConditionAction.java (.../AuthoringTaskListConditionAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -123,7 +123,7 @@ throw new ServletException(e); } - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); if (mode.isAuthor()) { return mapping.findForward(TaskListConstants.SUCCESS); } else { @@ -533,23 +533,6 @@ return errors; } - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - private ActionMessages validate(TaskListForm taskListForm, ActionMapping mapping, HttpServletRequest request) { ActionMessages errors = new ActionMessages(); // if (StringUtils.isBlank(taskListForm.getTaskList().getTitle())) { Index: lams_tool_vote/src/java/org/lamsfoundation/lams/tool/vote/web/AuthoringAction.java =================================================================== diff -u -r4ff01ddd9be0b8c91ac92569f68d02ca33a18f46 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_vote/src/java/org/lamsfoundation/lams/tool/vote/web/AuthoringAction.java (.../AuthoringAction.java) (revision 4ff01ddd9be0b8c91ac92569f68d02ca33a18f46) +++ lams_tool_vote/src/java/org/lamsfoundation/lams/tool/vote/web/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -159,7 +159,7 @@ } voteAuthoringForm.setMaxInputs(new Short(maxInputs)); - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); } @@ -757,7 +757,7 @@ } if (errors.isEmpty()) { - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); request.setAttribute(AttributeNames.ATTR_MODE, mode.toString()); List deletedQuestionDTOs = (List) sessionMap @@ -832,23 +832,6 @@ return mapping.findForward(VoteAppConstants.LOAD_QUESTIONS); } - /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - ToolAccessMode mode; - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - mode = ToolAccessMode.TEACHER; - } else { - mode = ToolAccessMode.AUTHOR; - } - return mode; - } - public ActionForward start(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) throws IOException, ServletException, VoteApplicationException { @@ -901,7 +884,7 @@ return mapping.findForward(VoteAppConstants.ERROR_LIST); } - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); // request is from monitoring module if (mode.isTeacher()) { VoteUtils.setDefineLater(request, true, strToolContentId, voteService); Index: lams_tool_wiki/src/java/org/lamsfoundation/lams/tool/wiki/web/actions/AuthoringAction.java =================================================================== diff -u -r2f725f8ef2aa09a2663b2335bf67213074426d11 -r23b5b0a587e3c4dc659729270eb0d608fb4dd982 --- lams_tool_wiki/src/java/org/lamsfoundation/lams/tool/wiki/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 2f725f8ef2aa09a2663b2335bf67213074426d11) +++ lams_tool_wiki/src/java/org/lamsfoundation/lams/tool/wiki/web/actions/AuthoringAction.java (.../AuthoringAction.java) (revision 23b5b0a587e3c4dc659729270eb0d608fb4dd982) @@ -95,7 +95,7 @@ String contentFolderID = WebUtil.readStrParam(request, AttributeNames.PARAM_CONTENT_FOLDER_ID); - ToolAccessMode mode = getAccessMode(request); + ToolAccessMode mode = WebUtil.readToolAccessModeAuthorDefaulted(request); // Set up the authForm. AuthoringForm authForm = (AuthoringForm) form; @@ -116,7 +116,7 @@ // TODO NOTE: this causes DB orphans when LD not saved. } - if (mode != null && mode.isTeacher()) { + if (mode.isTeacher()) { // Set the defineLater flag so that learners cannot use content // while we // are editing. This flag is released when updateContent is called. @@ -333,20 +333,6 @@ } /** - * Get ToolAccessMode from HttpRequest parameters. Default value is AUTHOR mode. - * - * @param request - * @return - */ - private ToolAccessMode getAccessMode(HttpServletRequest request) { - String modeStr = request.getParameter(AttributeNames.ATTR_MODE); - if (StringUtils.equalsIgnoreCase(modeStr, ToolAccessMode.TEACHER.toString())) { - return ToolAccessMode.TEACHER; - } - return ToolAccessMode.AUTHOR; - } - - /** * Retrieve the SessionMap from the HttpSession. * * @param request