Index: lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/web/NbLearnerAction.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/web/NbLearnerAction.java,v diff -u -r1.3 -r1.4 --- lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/web/NbLearnerAction.java 20 Jul 2005 04:16:19 -0000 1.3 +++ lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/web/NbLearnerAction.java 22 Jul 2005 07:02:50 -0000 1.4 @@ -33,7 +33,8 @@ import javax.servlet.http.HttpServletRequest; import javax.servlet.http.HttpServletResponse; -import org.apache.struts.actions.LookupDispatchAction; +//import org.apache.struts.actions.LookupDispatchAction; +import org.lamsfoundation.lams.web.action.LamsLookupDispatchAction; import org.apache.log4j.Logger; @@ -49,6 +50,7 @@ import org.lamsfoundation.lams.learning.service.ILearnerService; import org.lamsfoundation.lams.learning.service.LearnerServiceProxy; import org.lamsfoundation.lams.usermanagement.User; +import org.lamsfoundation.lams.tool.noticeboard.util.NbWebUtil; /** @@ -58,10 +60,14 @@ * * @struts:action path="/learner" name="NbLearnerForm" scope="session" type="org.lamsfoundation.lams.tool.noticeboard.web.NbLearnerAction" * input=".learnerContent" validate="false" parameter="method" + * @struts.action-exception key="error.exception.NbApplication" scope="request" + * type="org.lamsfoundation.lams.tool.noticeboard.NbApplicationException" + * path=".error" + * handler="org.lamsfoundation.lams.tool.noticeboard.web.CustomStrutsExceptionHandler" * @struts:action-forward name="displayLearnerContent" path=".learnerContent" * ----------------XDoclet Tags-------------------- */ -public class NbLearnerAction extends LookupDispatchAction { +public class NbLearnerAction extends LamsLookupDispatchAction { static Logger logger = Logger.getLogger(NbLearnerAction.class.getName()); @@ -81,11 +87,18 @@ * @param response * @return */ - public ActionForward finish(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) { + public ActionForward finish(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) throws NbApplicationException { NbLearnerForm learnerForm = (NbLearnerForm)form; - Long toolSessionID = learnerForm.getToolSessionId(); - Long userID = learnerForm.getUserId(); + Long toolSessionID = NbWebUtil.convertToLong(learnerForm.getToolSessionId()); + Long userID = NbWebUtil.convertToLong(learnerForm.getUserId()); + + if (toolSessionID == null || userID == null) + { + String error = "Unable to continue. The parameters tool session id or user id is missing"; + logger.error(error); + throw new NbApplicationException(error); + } INoticeboardService nbService = NoticeboardServiceProxy.getNbService(getServlet().getServletContext()); /** TODO: learnerServiceProxy causes an exception, fix this up later */ // ILearnerService learnerService = LearnerServiceProxy.getLearnerService(getServlet().getServletContext()); Index: lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/web/NbMonitoringAction.java =================================================================== RCS file: /usr/local/cvsroot/lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/web/NbMonitoringAction.java,v diff -u -r1.5 -r1.6 --- lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/web/NbMonitoringAction.java 21 Jul 2005 04:37:28 -0000 1.5 +++ lams_tool_nb/src/java/org/lamsfoundation/lams/tool/noticeboard/web/NbMonitoringAction.java 22 Jul 2005 07:04:06 -0000 1.6 @@ -37,7 +37,8 @@ import org.apache.struts.action.ActionForm; import org.apache.struts.action.ActionForward; import org.apache.struts.action.ActionMapping; -import org.apache.struts.actions.LookupDispatchAction; +//import org.apache.struts.actions.LookupDispatchAction; +import org.lamsfoundation.lams.web.action.LamsLookupDispatchAction; import org.lamsfoundation.lams.tool.noticeboard.NoticeboardConstants; import org.lamsfoundation.lams.tool.noticeboard.service.INoticeboardService; import org.lamsfoundation.lams.tool.noticeboard.service.NoticeboardServiceProxy; @@ -57,10 +58,14 @@ * * @struts:action path="/monitoring" name="NbMonitoringForm" scope="session" type="org.lamsfoundation.lams.tool.noticeboard.web.NbMonitoringAction" * input=".monitoringContent" validate="false" parameter="method" + * @struts.action-exception key="error.exception.NbApplication" scope="request" + * type="org.lamsfoundation.lams.tool.noticeboard.NbApplicationException" + * path=".error" + * handler="org.lamsfoundation.lams.tool.noticeboard.web.CustomStrutsExceptionHandler" * @struts:action-forward name="monitorPage" path=".monitoringContent" * ----------------XDoclet Tags-------------------- */ -public class NbMonitoringAction extends LookupDispatchAction { +public class NbMonitoringAction extends LamsLookupDispatchAction { static Logger logger = Logger.getLogger(NbMonitoringAction.class.getName());