Index: lams_tool_assessment/conf/language/lams/ApplicationResources_en_AU.properties =================================================================== diff -u -r43fb0f7cdc24246ee4b4469e4e8f22b5035f4daa -r09147fe666e4bd39661695a2bace9b226a4b6130 --- lams_tool_assessment/conf/language/lams/ApplicationResources_en_AU.properties (.../ApplicationResources_en_AU.properties) (revision 43fb0f7cdc24246ee4b4469e4e8f22b5035f4daa) +++ lams_tool_assessment/conf/language/lams/ApplicationResources_en_AU.properties (.../ApplicationResources_en_AU.properties) (revision 09147fe666e4bd39661695a2bace9b226a4b6130) @@ -292,5 +292,6 @@ label.justify.hedging.marks =Please, justify your hedging marks. label.incorrect.answer.nullifies.mark =Only correct answer selection assigns full mark +label.ask.for.hedging.justification =Ask for hedging justification? #======= End labels: Exported 283 labels for en AU ===== Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/dbupdates/patch20150731.sql =================================================================== diff -u --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/dbupdates/patch20150731.sql (revision 0) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/dbupdates/patch20150731.sql (revision 09147fe666e4bd39661695a2bace9b226a4b6130) @@ -0,0 +1,14 @@ +-- Turn off autocommit, so nothing is committed if there is an error +SET AUTOCOMMIT = 0; +SET FOREIGN_KEY_CHECKS=0; +----------------------Put all sql statements below here------------------------- + +-- LDEV-3522 Add justification textareas to the hedging marks question type +ALTER TABLE tl_laasse10_assessment_question ADD COLUMN hedging_justification_enabled TINYINT(1) NOT NULL DEFAULT 0; + +----------------------Put all sql statements above here------------------------- + +-- If there were no errors, commit and restore autocommit to on +COMMIT; +SET AUTOCOMMIT = 1; +SET FOREIGN_KEY_CHECKS=1; \ No newline at end of file Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentQuestion.java =================================================================== diff -u -r1387fe8af57326cbd56493d2d1be5228ee98c729 -r09147fe666e4bd39661695a2bace9b226a4b6130 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentQuestion.java (.../AssessmentQuestion.java) (revision 1387fe8af57326cbd56493d2d1be5228ee98c729) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/model/AssessmentQuestion.java (.../AssessmentQuestion.java) (revision 09147fe666e4bd39661695a2bace9b226a4b6130) @@ -99,6 +99,9 @@ //only for essay type of question private int minWordsLimit; + //only for hedging mark type of question + private boolean hedgingJustificationEnabled; + // DTO fields: private String answerString; @@ -548,6 +551,18 @@ this.minWordsLimit = minWordsLimit; } + /** + * @hibernate.property column="hedging_justification_enabled" + * @return + */ + public boolean isHedgingJustificationEnabled() { + return hedgingJustificationEnabled; + } + + public void setHedgingJustificationEnabled(boolean hedgingJustificationEnabled) { + this.hedgingJustificationEnabled = hedgingJustificationEnabled; + } + public String getAnswerString() { return answerString; } Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/AuthoringAction.java =================================================================== diff -u -r1387fe8af57326cbd56493d2d1be5228ee98c729 -r09147fe666e4bd39661695a2bace9b226a4b6130 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 1387fe8af57326cbd56493d2d1be5228ee98c729) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/AuthoringAction.java (.../AuthoringAction.java) (revision 09147fe666e4bd39661695a2bace9b226a4b6130) @@ -1757,6 +1757,7 @@ form.setAllowRichEditor(question.isAllowRichEditor()); form.setMaxWordsLimit(question.getMaxWordsLimit()); form.setMinWordsLimit(question.getMinWordsLimit()); + form.setHedgingJustificationEnabled(question.isHedgingJustificationEnabled()); if (questionIdx >= 0) { form.setQuestionIndex(new Integer(questionIdx).toString()); } @@ -1857,6 +1858,7 @@ question.setFeedbackOnCorrect(questionForm.getFeedbackOnCorrect()); question.setFeedbackOnPartiallyCorrect(questionForm.getFeedbackOnPartiallyCorrect()); question.setFeedbackOnIncorrect(questionForm.getFeedbackOnIncorrect()); + question.setHedgingJustificationEnabled(questionForm.isHedgingJustificationEnabled()); } // set options Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/LearningAction.java =================================================================== diff -u -r43fb0f7cdc24246ee4b4469e4e8f22b5035f4daa -r09147fe666e4bd39661695a2bace9b226a4b6130 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/LearningAction.java (.../LearningAction.java) (revision 43fb0f7cdc24246ee4b4469e4e8f22b5035f4daa) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/LearningAction.java (.../LearningAction.java) (revision 09147fe666e4bd39661695a2bace9b226a4b6130) @@ -802,10 +802,12 @@ } } - //store justificaion of hedging - String answerString = request.getParameter(AssessmentConstants.ATTR_QUESTION_PREFIX + i); - answerString = answerString.replaceAll("[\n\r\f]", ""); - question.setAnswerString(answerString); + //store justification of hedging if enabled + if (question.isHedgingJustificationEnabled()) { + String answerString = request.getParameter(AssessmentConstants.ATTR_QUESTION_PREFIX + i); + answerString = answerString.replaceAll("[\n\r\f]", ""); + question.setAnswerString(answerString); + } } } } @@ -866,8 +868,10 @@ } isAnswered = sumMarkHedging == question.getGrade(); - //verify justification of hedging is provided - isAnswered &= StringUtils.isNotBlank(question.getAnswerString()); + //verify justification of hedging is provided if it was enabled + if (question.isHedgingJustificationEnabled()) { + isAnswered &= StringUtils.isNotBlank(question.getAnswerString()); + } } // check all questions were answered Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/form/AssessmentQuestionForm.java =================================================================== diff -u -r1387fe8af57326cbd56493d2d1be5228ee98c729 -r09147fe666e4bd39661695a2bace9b226a4b6130 --- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/form/AssessmentQuestionForm.java (.../AssessmentQuestionForm.java) (revision 1387fe8af57326cbd56493d2d1be5228ee98c729) +++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/form/AssessmentQuestionForm.java (.../AssessmentQuestionForm.java) (revision 09147fe666e4bd39661695a2bace9b226a4b6130) @@ -60,6 +60,7 @@ private boolean allowRichEditor; private int maxWordsLimit; private int minWordsLimit; + private boolean hedgingJustificationEnabled; public String getSessionMapID() { return sessionMapID; @@ -245,4 +246,11 @@ this.minWordsLimit = minWordsLimit; } + public boolean isHedgingJustificationEnabled() { + return hedgingJustificationEnabled; + } + + public void setHedgingJustificationEnabled(boolean hedgingJustificationEnabled) { + this.hedgingJustificationEnabled = hedgingJustificationEnabled; + } } Index: lams_tool_assessment/web/pages/authoring/parts/addmarkhedging.jsp =================================================================== diff -u -r0340817df8d420d1feafa8ad9a43f667023515b2 -r09147fe666e4bd39661695a2bace9b226a4b6130 --- lams_tool_assessment/web/pages/authoring/parts/addmarkhedging.jsp (.../addmarkhedging.jsp) (revision 0340817df8d420d1feafa8ad9a43f667023515b2) +++ lams_tool_assessment/web/pages/authoring/parts/addmarkhedging.jsp (.../addmarkhedging.jsp) (revision 09147fe666e4bd39661695a2bace9b226a4b6130) @@ -160,6 +160,13 @@ +
+ + +
+
Index: lams_tool_assessment/web/pages/learning/learning.jsp =================================================================== diff -u -r43fb0f7cdc24246ee4b4469e4e8f22b5035f4daa -r09147fe666e4bd39661695a2bace9b226a4b6130 --- lams_tool_assessment/web/pages/learning/learning.jsp (.../learning.jsp) (revision 43fb0f7cdc24246ee4b4469e4e8f22b5035f4daa) +++ lams_tool_assessment/web/pages/learning/learning.jsp (.../learning.jsp) (revision 09147fe666e4bd39661695a2bace9b226a4b6130) @@ -97,10 +97,15 @@ var grade = selects.length == 0 ? 0 : eval(selects.first().find('option:last-child').val()) var totalSelected = countHedgeQuestionSelectTotal(questionIndex); - var textareaValue = $("#question" + questionIndex + "__lamstextarea").val(); - var isButtonEnabled = (totalSelected == grade) && $.trim(textareaValue); + var isButtonEnabled = (totalSelected == grade); + //check if hedging justification is enabled + var justificationTextarea = $("#question" + questionIndex + "__lamstextarea"); + if( justificationTextarea.length) { + isButtonEnabled = isButtonEnabled && $.trim(justificationTextarea.val()); + } + //if totalSelected equals to question's grade - show button if (isButtonEnabled) { $("[type=button][name=submit-hedging-question" + questionIndex + "]").prop("disabled", "").removeClass("button-disabled"); @@ -450,12 +455,10 @@ } else { var grade = ${question.grade}; var totalSelected = countHedgeQuestionSelectTotal(questionIndex); - - var justificationTextareaValue = $("#question" + questionIndex + "__lamstextarea").val(); //if totalSelected not equals to question's grade OR textarea is empty or contains only white-space - show warning if (totalSelected != grade - || !$.trim(justificationTextareaValue)) { + || ${question.hedgingJustificationEnabled} && !$.trim($("#question" + questionIndex + "__lamstextarea").val())) { markHedgingWrongTotalQuestions.push("${status.index}"); } } Index: lams_tool_assessment/web/pages/learning/parts/markhedging.jsp =================================================================== diff -u -r43fb0f7cdc24246ee4b4469e4e8f22b5035f4daa -r09147fe666e4bd39661695a2bace9b226a4b6130 --- lams_tool_assessment/web/pages/learning/parts/markhedging.jsp (.../markhedging.jsp) (revision 43fb0f7cdc24246ee4b4469e4e8f22b5035f4daa) +++ lams_tool_assessment/web/pages/learning/parts/markhedging.jsp (.../markhedging.jsp) (revision 09147fe666e4bd39661695a2bace9b226a4b6130) @@ -62,19 +62,21 @@ - - - -
- -
- - - - - + + + + +
+ +
+ + + + + + Index: lams_tool_assessment/web/pages/monitoring/parts/userresponse.jsp =================================================================== diff -u -r43fb0f7cdc24246ee4b4469e4e8f22b5035f4daa -r09147fe666e4bd39661695a2bace9b226a4b6130 --- lams_tool_assessment/web/pages/monitoring/parts/userresponse.jsp (.../userresponse.jsp) (revision 43fb0f7cdc24246ee4b4469e4e8f22b5035f4daa) +++ lams_tool_assessment/web/pages/monitoring/parts/userresponse.jsp (.../userresponse.jsp) (revision 09147fe666e4bd39661695a2bace9b226a4b6130) @@ -84,7 +84,9 @@ responseStr +='
'; - responseStr +="${questionResult.answerStringEscaped}"; + + responseStr +="${questionResult.answerStringEscaped}"; + \ No newline at end of file