Index: lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McAction.java =================================================================== diff -u -r525fef17717ea250350b7204056960f735ce02cb -r9dc619c585fbef88eb9340bd68dd8137298b3129 --- lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McAction.java (.../McAction.java) (revision 525fef17717ea250350b7204056960f735ce02cb) +++ lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McAction.java (.../McAction.java) (revision 9dc619c585fbef88eb9340bd68dd8137298b3129) @@ -521,10 +521,10 @@ boolean validateSingleCorrectCandidate=authoringUtil.validateSingleCorrectCandidate(caList); logger.debug("validateSingleCorrectCandidate: " + validateSingleCorrectCandidate); - /* + boolean validateOnlyOneCorrectCandidate=authoringUtil.validateOnlyOneCorrectCandidate(caList); logger.debug("validateOnlyOneCorrectCandidate: " + validateOnlyOneCorrectCandidate); - */ + ActionMessages errors = new ActionMessages(); @@ -541,14 +541,12 @@ ActionMessage error = new ActionMessage("candidates.none.correct"); errors.add(ActionMessages.GLOBAL_MESSAGE, error); } - - /* + if (!validateOnlyOneCorrectCandidate) { ActionMessage error = new ActionMessage("candidates.duplicate.correct"); errors.add(ActionMessages.GLOBAL_MESSAGE, error); } - */ logger.debug("errors: " + errors); @@ -955,11 +953,11 @@ boolean validateSingleCorrectCandidate=authoringUtil.validateSingleCorrectCandidate(caList); logger.debug("validateSingleCorrectCandidate: " + validateSingleCorrectCandidate); - /* + boolean validateOnlyOneCorrectCandidate=authoringUtil.validateOnlyOneCorrectCandidate(caList); logger.debug("validateOnlyOneCorrectCandidate: " + validateOnlyOneCorrectCandidate); - */ + ActionMessages errors = new ActionMessages(); @@ -977,13 +975,11 @@ errors.add(ActionMessages.GLOBAL_MESSAGE, error); } - /* if (!validateOnlyOneCorrectCandidate) { ActionMessage error = new ActionMessage("candidates.duplicate.correct"); errors.add(ActionMessages.GLOBAL_MESSAGE, error); } - */ logger.debug("errors: " + errors); @@ -1444,21 +1440,6 @@ mcGeneralAuthoringDTO.setTotalMarks(totalMark); - /* - String newQuestion=request.getParameter("newQuestion"); - logger.debug("newQuestion: " + newQuestion); - mcGeneralAuthoringDTO.setEditableQuestionText(newQuestion); - - String feedback=request.getParameter("feedback"); - logger.debug("feedback: " + feedback); - mcAuthoringForm.setFeedback(feedback); - - String mark=request.getParameter("mark"); - logger.debug("mark: " + mark); - mcGeneralAuthoringDTO.setMarkValue(mark); - */ - - logger.debug("before saving final mcGeneralAuthoringDTO: " + mcGeneralAuthoringDTO); request.setAttribute(MC_GENERAL_AUTHORING_DTO, mcGeneralAuthoringDTO); Index: lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McMonitoringAction.java =================================================================== diff -u -r525fef17717ea250350b7204056960f735ce02cb -r9dc619c585fbef88eb9340bd68dd8137298b3129 --- lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McMonitoringAction.java (.../McMonitoringAction.java) (revision 525fef17717ea250350b7204056960f735ce02cb) +++ lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McMonitoringAction.java (.../McMonitoringAction.java) (revision 9dc619c585fbef88eb9340bd68dd8137298b3129) @@ -1236,10 +1236,10 @@ boolean validateSingleCorrectCandidate=authoringUtil.validateSingleCorrectCandidate(caList); logger.debug("validateSingleCorrectCandidate: " + validateSingleCorrectCandidate); - /* + boolean validateOnlyOneCorrectCandidate=authoringUtil.validateOnlyOneCorrectCandidate(caList); logger.debug("validateOnlyOneCorrectCandidate: " + validateOnlyOneCorrectCandidate); - */ + ActionMessages errors = new ActionMessages(); @@ -1258,13 +1258,11 @@ errors.add(ActionMessages.GLOBAL_MESSAGE, error); } - /* if (!validateOnlyOneCorrectCandidate) { ActionMessage error = new ActionMessage("candidates.duplicate.correct"); errors.add(ActionMessages.GLOBAL_MESSAGE, error); } - */ logger.debug("errors: " + errors); @@ -1648,10 +1646,10 @@ boolean validateSingleCorrectCandidate=authoringUtil.validateSingleCorrectCandidate(caList); logger.debug("validateSingleCorrectCandidate: " + validateSingleCorrectCandidate); - /* + boolean validateOnlyOneCorrectCandidate=authoringUtil.validateOnlyOneCorrectCandidate(caList); logger.debug("validateOnlyOneCorrectCandidate: " + validateOnlyOneCorrectCandidate); - */ + ActionMessages errors = new ActionMessages(); @@ -1669,13 +1667,12 @@ errors.add(ActionMessages.GLOBAL_MESSAGE, error); } - /* if (!validateOnlyOneCorrectCandidate) { ActionMessage error = new ActionMessage("candidates.duplicate.correct"); errors.add(ActionMessages.GLOBAL_MESSAGE, error); } - */ + logger.debug("errors: " + errors); if(!errors.isEmpty()){ Index: lams_tool_lamc/web/authoring/editQuestionBox.jsp =================================================================== diff -u -r525fef17717ea250350b7204056960f735ce02cb -r9dc619c585fbef88eb9340bd68dd8137298b3129 --- lams_tool_lamc/web/authoring/editQuestionBox.jsp (.../editQuestionBox.jsp) (revision 525fef17717ea250350b7204056960f735ce02cb) +++ lams_tool_lamc/web/authoring/editQuestionBox.jsp (.../editQuestionBox.jsp) (revision 9dc619c585fbef88eb9340bd68dd8137298b3129) @@ -93,7 +93,9 @@ { var msg = ""; alert(msg); + return false; } + return true; } @@ -159,7 +161,7 @@
- Index: lams_tool_lamc/web/authoring/newQuestionBox.jsp =================================================================== diff -u -r525fef17717ea250350b7204056960f735ce02cb -r9dc619c585fbef88eb9340bd68dd8137298b3129 --- lams_tool_lamc/web/authoring/newQuestionBox.jsp (.../newQuestionBox.jsp) (revision 525fef17717ea250350b7204056960f735ce02cb) +++ lams_tool_lamc/web/authoring/newQuestionBox.jsp (.../newQuestionBox.jsp) (revision 9dc619c585fbef88eb9340bd68dd8137298b3129) @@ -75,7 +75,9 @@ { var msg = ""; alert(msg); + return false; } + return true; } @@ -138,7 +140,7 @@
- Index: lams_tool_lamc/web/monitoring/editQuestionBox.jsp =================================================================== diff -u -r525fef17717ea250350b7204056960f735ce02cb -r9dc619c585fbef88eb9340bd68dd8137298b3129 --- lams_tool_lamc/web/monitoring/editQuestionBox.jsp (.../editQuestionBox.jsp) (revision 525fef17717ea250350b7204056960f735ce02cb) +++ lams_tool_lamc/web/monitoring/editQuestionBox.jsp (.../editQuestionBox.jsp) (revision 9dc619c585fbef88eb9340bd68dd8137298b3129) @@ -76,7 +76,9 @@ { var msg = ""; alert(msg); + return false; } + return true; } @@ -171,8 +173,8 @@ Index: lams_tool_lamc/web/monitoring/newQuestionBox.jsp =================================================================== diff -u -r525fef17717ea250350b7204056960f735ce02cb -r9dc619c585fbef88eb9340bd68dd8137298b3129 --- lams_tool_lamc/web/monitoring/newQuestionBox.jsp (.../newQuestionBox.jsp) (revision 525fef17717ea250350b7204056960f735ce02cb) +++ lams_tool_lamc/web/monitoring/newQuestionBox.jsp (.../newQuestionBox.jsp) (revision 9dc619c585fbef88eb9340bd68dd8137298b3129) @@ -74,7 +74,9 @@ { var msg = ""; alert(msg); + return false; } + return true; } @@ -169,10 +171,9 @@
                     - - +
+ +
                     - - - -