Index: lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/LearningAction.java
===================================================================
diff -u -ra5725da4ffd404829baffc9c3f262f001daa41b9 -r21cb714b3017a073ffad794a260d772e6db980e4
--- lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/LearningAction.java	(.../LearningAction.java)	(revision a5725da4ffd404829baffc9c3f262f001daa41b9)
+++ lams_tool_assessment/src/java/org/lamsfoundation/lams/tool/assessment/web/action/LearningAction.java	(.../LearningAction.java)	(revision 21cb714b3017a073ffad794a260d772e6db980e4)
@@ -922,20 +922,30 @@
 	
 	for (LinkedHashSet<AssessmentQuestion> questionsForOnePage : pagedQuestions) {
 	    for (AssessmentQuestion question : questionsForOnePage) {
+		
+		//find corresponding questionResult
 		for (AssessmentQuestionResult questionResult : result.getQuestionResults()) {
 		    if (question.getUid().equals(questionResult.getAssessmentQuestion().getUid())) {
+			
+			//copy questionResult's info to the question
 			question.setMark(questionResult.getMark());
 			question.setResponseSubmitted(questionResult.getFinishDate() != null);
 			question.setPenalty(questionResult.getPenalty());
-
 			question.setQuestionFeedback(null);
 			for (AssessmentQuestionOption option : question.getOptions()) {
 			    if (option.getUid().equals(questionResult.getSubmittedOptionUid())) {
 				question.setQuestionFeedback(option.getFeedback());
 				break;
 			    }
 			}
+			//required for showing right/wrong answers icons on results page correctly 
+			if (question.getType() == AssessmentConstants.QUESTION_TYPE_SHORT_ANSWER
+				|| question.getType() == AssessmentConstants.QUESTION_TYPE_NUMERICAL) {
+			    boolean isAnsweredCorrectly = questionResult.getSubmittedOptionUid() != null;
+			    question.setAnswerBoolean(isAnsweredCorrectly);
+			}
 
+			//required for markandpenalty area and if it's on - on question's summary page
 			List<Object[]> questionResults = service.getAssessmentQuestionResultList(assessment.getUid(),
 				userId, question.getUid());
 			question.setQuestionResults(questionResults);
Index: lams_tool_assessment/web/pages/learning/parts/numerical.jsp
===================================================================
diff -u -r1668e000b0959071785e5cdd21732af39bd1505f -r21cb714b3017a073ffad794a260d772e6db980e4
--- lams_tool_assessment/web/pages/learning/parts/numerical.jsp	(.../numerical.jsp)	(revision 1668e000b0959071785e5cdd21732af39bd1505f)
+++ lams_tool_assessment/web/pages/learning/parts/numerical.jsp	(.../numerical.jsp)	(revision 21cb714b3017a073ffad794a260d772e6db980e4)
@@ -9,10 +9,10 @@
 		<c:if test="${finishedLock}">
 			<td class="complete-item-gif">
 			
-				<c:if test="${assessment.allowRightAnswersAfterQuestion && (question.mark > 0)}">
+				<c:if test="${assessment.allowRightAnswersAfterQuestion && question.answerBoolean}">
 					<img src="<html:rewrite page='/includes/images/completeitem.gif'/>">	
 				</c:if>			
-				<c:if test="${assessment.allowWrongAnswersAfterQuestion && !(question.mark > 0)}">
+				<c:if test="${assessment.allowWrongAnswersAfterQuestion && !question.answerBoolean}">
 					<img src="<html:rewrite page='/includes/images/incompleteitem.gif'/>">
 				</c:if>
 				
Index: lams_tool_assessment/web/pages/learning/parts/shortanswer.jsp
===================================================================
diff -u -r1668e000b0959071785e5cdd21732af39bd1505f -r21cb714b3017a073ffad794a260d772e6db980e4
--- lams_tool_assessment/web/pages/learning/parts/shortanswer.jsp	(.../shortanswer.jsp)	(revision 1668e000b0959071785e5cdd21732af39bd1505f)
+++ lams_tool_assessment/web/pages/learning/parts/shortanswer.jsp	(.../shortanswer.jsp)	(revision 21cb714b3017a073ffad794a260d772e6db980e4)
@@ -6,23 +6,24 @@
 
 <table class="question-table">
 	<tr>
+	
 		<c:if test="${finishedLock}">
 			<td class="complete-item-gif">
-			
-				<c:if test="${assessment.allowRightAnswersAfterQuestion && (question.mark > 0)}">
+				<c:if test="${assessment.allowRightAnswersAfterQuestion && question.answerBoolean}">
 					<img src="<html:rewrite page='/includes/images/completeitem.gif'/>">	
 				</c:if>			
-				<c:if test="${assessment.allowWrongAnswersAfterQuestion && !(question.mark > 0)}">
+				<c:if test="${assessment.allowWrongAnswersAfterQuestion && !question.answerBoolean}">
 					<img src="<html:rewrite page='/includes/images/incompleteitem.gif'/>">
-				</c:if>			
-		
-			</td>		
+				</c:if>	
+			</td>
 		</c:if>	
+		
 		<td class="reg-padding">
 			<input type="text" name="question${status.index}" value="<c:out value='${question.answerString}' />" styleClass="noBorder" size="70"
 				<c:if test="${isEditingDisabled}">disabled="disabled"</c:if>
 			/>	
 		</td>
+		
 	</tr>
 </table>