Index: lams_tool_forum/conf/language/ApplicationResources.properties =================================================================== diff -u -r40fa2bc8b8b488c068a505747fe6c7580fc2a3c7 -rb2d0900461fb1911e1881b92bea75edb39c6401b --- lams_tool_forum/conf/language/ApplicationResources.properties (.../ApplicationResources.properties) (revision 40fa2bc8b8b488c068a505747fe6c7580fc2a3c7) +++ lams_tool_forum/conf/language/ApplicationResources.properties (.../ApplicationResources.properties) (revision b2d0900461fb1911e1881b92bea75edb39c6401b) @@ -143,3 +143,4 @@ #======= End labels: Exported 134 labels for en AU ===== +button.close=Close \ No newline at end of file Index: lams_tool_forum/conf/xdoclet/struts-actions.xml =================================================================== diff -u -red34ad93e720126f7473b282585c411c8acf7dbe -rb2d0900461fb1911e1881b92bea75edb39c6401b --- lams_tool_forum/conf/xdoclet/struts-actions.xml (.../struts-actions.xml) (revision ed34ad93e720126f7473b282585c411c8acf7dbe) +++ lams_tool_forum/conf/xdoclet/struts-actions.xml (.../struts-actions.xml) (revision b2d0900461fb1911e1881b92bea75edb39c6401b) @@ -266,7 +266,7 @@ @@ -277,7 +277,7 @@ input="tiles:/monitoring/forum/editUserMark" name="markForm" scope="request" - validate="true" + validate="false" parameter="updateMark" > Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/util/ForumConstants.java =================================================================== diff -u -rfb96ff9dba9dfa8cf9646fddcca425851075c80a -rb2d0900461fb1911e1881b92bea75edb39c6401b --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/util/ForumConstants.java (.../ForumConstants.java) (revision fb96ff9dba9dfa8cf9646fddcca425851075c80a) +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/util/ForumConstants.java (.../ForumConstants.java) (revision b2d0900461fb1911e1881b92bea75edb39c6401b) @@ -85,5 +85,9 @@ public static final String FORUM_TITLE = "forum_title"; public static final String ATTR_TOOL_CONTENT_TOPICS = "ToolContentTopicList"; + public static final String ATTR_TOPIC = "topic"; + public static final String ATTR_USER= "user"; + public static final String ATTR_TOPIC_LIST = "topicList"; + } Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/MonitoringAction.java =================================================================== diff -u -r1e07e9ab37e8d8232222346df99a5c86afb33c49 -rb2d0900461fb1911e1881b92bea75edb39c6401b --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/MonitoringAction.java (.../MonitoringAction.java) (revision 1e07e9ab37e8d8232222346df99a5c86afb33c49) +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/actions/MonitoringAction.java (.../MonitoringAction.java) (revision b2d0900461fb1911e1881b92bea75edb39c6401b) @@ -40,6 +40,8 @@ import javax.servlet.http.HttpServletRequest; import javax.servlet.http.HttpServletResponse; +import org.apache.commons.lang.StringUtils; +import org.apache.commons.lang.math.NumberUtils; import org.apache.log4j.Logger; import org.apache.poi.hssf.usermodel.HSSFCell; import org.apache.poi.hssf.usermodel.HSSFRow; @@ -336,8 +338,8 @@ ForumUser user = forumService.getUser(userUid); // each back to web page - request.setAttribute("topicList", messageList); - request.setAttribute("user", user); + request.setAttribute(ForumConstants.ATTR_TOPIC_LIST, messageList); + request.setAttribute(ForumConstants.ATTR_USER, user); request.setAttribute(AttributeNames.PARAM_TOOL_SESSION_ID, sessionId); return mapping.findForward("success"); } @@ -374,9 +376,12 @@ markForm.setMark(""); markForm.setComment(msg.getReport().getComment()); } - markForm.setUser(user); - markForm.setMessageDto(MessageDTO.getMessageDTO(msg)); - markForm.setSessionId(sessionId); + + // each back to web page + request.setAttribute(ForumConstants.ATTR_TOPIC, MessageDTO.getMessageDTO(msg)); + request.setAttribute(ForumConstants.ATTR_USER, user); + request.setAttribute(AttributeNames.PARAM_TOOL_SESSION_ID, sessionId); + return mapping.findForward("success"); } @@ -391,36 +396,61 @@ */ private ActionForward updateMark(ActionMapping mapping, ActionForm form, HttpServletRequest request, HttpServletResponse response) { + MarkForm markForm = (MarkForm) form; Long messageId = new Long(WebUtil.readLongParam(request, ForumConstants.MESSAGE_UID)); Long userUid = new Long(WebUtil.readLongParam(request, ForumConstants.USER_UID)); Long sessionId = new Long(WebUtil.readLongParam(request, AttributeNames.PARAM_TOOL_SESSION_ID)); + String mark = markForm.getMark(); + ActionErrors errors = new ActionErrors(); + if (StringUtils.isBlank(mark)) { + ActionMessage error = new ActionMessage("error.valueReqd"); + errors.add("report.mark", error); + }else if(!NumberUtils.isNumber(mark)){ + ActionMessage error = new ActionMessage("error.mark.needNumber"); + errors.add("report.mark", error); + }else { + try{ + Float.parseFloat(mark); + }catch(Exception e){ + ActionMessage error = new ActionMessage("error.mark.invalid.number"); + errors.add("report.mark", error); + } + } + forumService = getForumService(); + // echo back to web page + List messageList = forumService.getMessagesByUserUid(userUid, sessionId); + ForumUser user = forumService.getUser(userUid); Message msg = forumService.getMessage(messageId); + + request.setAttribute(ForumConstants.ATTR_USER, user); + request.setAttribute(AttributeNames.PARAM_TOOL_SESSION_ID, sessionId); + if(!errors.isEmpty()){ + // each back to web page + request.setAttribute(ForumConstants.ATTR_TOPIC, MessageDTO.getMessageDTO(msg)); + saveMessages(request, errors); + return mapping.getInputForward(); + } - // save it into database - MarkForm markForm = (MarkForm) form; + //update message report + forumService = getForumService(); ForumReport report = msg.getReport(); if (report == null) { report = new ForumReport(); msg.setReport(report); } - report.setMark(new Float(Float.parseFloat(markForm.getMark()))); + report.setMark(new Float(Float.parseFloat(mark))); report.setComment(markForm.getComment()); + + //echo back to topic list page + request.setAttribute(ForumConstants.ATTR_TOPIC_LIST, messageList); forumService.updateTopic(msg); - // echo back to web page - forumService = getForumService(); - List messageList = forumService - .getMessagesByUserUid(userUid, sessionId); - ForumUser user = forumService.getUser(userUid); - request.setAttribute("topicList", messageList); - request.setAttribute("user", user); - request.setAttribute(AttributeNames.PARAM_TOOL_SESSION_ID, sessionId); return mapping.findForward("success"); } Index: lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/forms/MarkForm.java =================================================================== diff -u -rd9c7ddef141d19423b6ab2914d153c2cb748f187 -rb2d0900461fb1911e1881b92bea75edb39c6401b --- lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/forms/MarkForm.java (.../MarkForm.java) (revision d9c7ddef141d19423b6ab2914d153c2cb748f187) +++ lams_tool_forum/src/java/org/lamsfoundation/lams/tool/forum/web/forms/MarkForm.java (.../MarkForm.java) (revision b2d0900461fb1911e1881b92bea75edb39c6401b) @@ -25,58 +25,19 @@ package org.lamsfoundation.lams.tool.forum.web.forms; -import org.apache.commons.lang.math.NumberUtils; import org.apache.log4j.Logger; -import org.apache.struts.action.ActionErrors; -import org.apache.struts.action.ActionMapping; -import org.apache.struts.action.ActionMessage; -import org.apache.struts.validator.ValidatorForm; -import org.lamsfoundation.lams.tool.forum.dto.MessageDTO; -import org.lamsfoundation.lams.tool.forum.persistence.ForumUser; +import org.apache.struts.action.ActionForm; /** * @struts.form name="markForm" * @author Steve.Ni * * @version $Revision$ */ -public class MarkForm extends ValidatorForm{ +public class MarkForm extends ActionForm{ private static final long serialVersionUID = -4967675292027554366L; - private static Logger logger = Logger.getLogger(MarkForm.class.getName()); private String mark; private String comment; - private ForumUser user; - private MessageDTO messageDto; - private Long sessionId; - - /** - * MessageForm validation method from STRUCT interface. - * - */ - public ActionErrors validate(ActionMapping mapping, - javax.servlet.http.HttpServletRequest request) { - ActionErrors errors = super.validate(mapping, request); - try{ - if ("".equals(mark)) { - ActionMessage error = new ActionMessage("error.valueReqd"); - errors.add("report.mark", error); - }else if(!NumberUtils.isNumber(mark)){ - ActionMessage error = new ActionMessage("error.mark.needNumber"); - errors.add("report.mark", error); - }else { - try{ - Float.parseFloat(mark); - }catch(Exception e){ - ActionMessage error = new ActionMessage("error.mark.invalid.number"); - errors.add("report.mark", error); - } - } - } catch (Exception e) { - logger.error("", e); - } - return errors; - } - public String getComment() { return comment; } @@ -89,27 +50,4 @@ public void setMark(String mark) { this.mark = mark; } - - public MessageDTO getMessageDto() { - return messageDto; - } - - public void setMessageDto(MessageDTO message) { - this.messageDto = message; - } - - public ForumUser getUser() { - return user; - } - - public void setUser(ForumUser user) { - this.user = user; - } - public Long getSessionId() { - return sessionId; - } - - public void setSessionId(Long sessionId) { - this.sessionId = sessionId; - } } Index: lams_tool_forum/web/WEB-INF/struts-config.xml =================================================================== diff -u -r3306bb9a4b9c92a8539e317d5b67b3fc5bf90e8c -rb2d0900461fb1911e1881b92bea75edb39c6401b --- lams_tool_forum/web/WEB-INF/struts-config.xml (.../struts-config.xml) (revision 3306bb9a4b9c92a8539e317d5b67b3fc5bf90e8c) +++ lams_tool_forum/web/WEB-INF/struts-config.xml (.../struts-config.xml) (revision b2d0900461fb1911e1881b92bea75edb39c6401b) @@ -295,7 +295,7 @@ @@ -305,7 +305,7 @@ input="tiles:/monitoring/forum/editUserMark" name="markForm" scope="request" - validate="true" + validate="false" parameter="updateMark" > Index: lams_tool_forum/web/jsps/monitoring/updatemarks.jsp =================================================================== diff -u -r101275d99b5a739bb87df1928291e5db597a1fca -rb2d0900461fb1911e1881b92bea75edb39c6401b --- lams_tool_forum/web/jsps/monitoring/updatemarks.jsp (.../updatemarks.jsp) (revision 101275d99b5a739bb87df1928291e5db597a1fca) +++ lams_tool_forum/web/jsps/monitoring/updatemarks.jsp (.../updatemarks.jsp) (revision b2d0900461fb1911e1881b92bea75edb39c6401b) @@ -1,82 +1,71 @@ <%@ include file="/includes/taglibs.jsp"%> - " />
- + , - - + +
- + - + - + "> - + - + - - + +
- - - + + + - - + - - - - - - - - - - -
- - * : + + <%@ include file="/common/messages.jsp"%> - +
+ * - +
+ +
- -
- -
+ " class="button" /> +    + ">
Index: lams_tool_forum/web/jsps/monitoring/usermarklist.jsp =================================================================== diff -u -r101275d99b5a739bb87df1928291e5db597a1fca -rb2d0900461fb1911e1881b92bea75edb39c6401b --- lams_tool_forum/web/jsps/monitoring/usermarklist.jsp (.../usermarklist.jsp) (revision 101275d99b5a739bb87df1928291e5db597a1fca) +++ lams_tool_forum/web/jsps/monitoring/usermarklist.jsp (.../usermarklist.jsp) (revision b2d0900461fb1911e1881b92bea75edb39c6401b) @@ -56,7 +56,7 @@ - + @@ -91,5 +91,10 @@ + + + + + Index: lams_tool_forum/web/jsps/monitoring/viewallmarks.jsp =================================================================== diff -u -r101275d99b5a739bb87df1928291e5db597a1fca -rb2d0900461fb1911e1881b92bea75edb39c6401b --- lams_tool_forum/web/jsps/monitoring/viewallmarks.jsp (.../viewallmarks.jsp) (revision 101275d99b5a739bb87df1928291e5db597a1fca) +++ lams_tool_forum/web/jsps/monitoring/viewallmarks.jsp (.../viewallmarks.jsp) (revision b2d0900461fb1911e1881b92bea75edb39c6401b) @@ -100,4 +100,9 @@ + + + + + Index: lams_tool_forum/web/jsps/monitoring/viewtopic.jsp =================================================================== diff -u -rdf638eb558ae2d18b300343911aa8e07e66e9697 -rb2d0900461fb1911e1881b92bea75edb39c6401b --- lams_tool_forum/web/jsps/monitoring/viewtopic.jsp (.../viewtopic.jsp) (revision df638eb558ae2d18b300343911aa8e07e66e9697) +++ lams_tool_forum/web/jsps/monitoring/viewtopic.jsp (.../viewtopic.jsp) (revision b2d0900461fb1911e1881b92bea75edb39c6401b) @@ -4,9 +4,8 @@ -
- close +