Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/util/SurveyWebUtils.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/util/SurveyWebUtils.java,v
diff -u -r1.2 -r1.3
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/util/SurveyWebUtils.java	21 Sep 2006 07:46:41 -0000	1.2
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/util/SurveyWebUtils.java	22 Sep 2006 00:57:21 -0000	1.3
@@ -54,6 +54,9 @@
 
 	public static String getChoicesStr(String[] choiceList) {
 		String choices = "";
+		if(choiceList == null)
+			return choices;
+		
 		for(String c: choiceList)
 			choices = choices + c + "&";
 		return choices;
Index: lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/LearningAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/LearningAction.java,v
diff -u -r1.4 -r1.5
--- lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/LearningAction.java	21 Sep 2006 07:46:41 -0000	1.4
+++ lams_tool_survey/src/java/org/lamsfoundation/lams/tool/survey/web/action/LearningAction.java	22 Sep 2006 00:57:21 -0000	1.5
@@ -406,7 +406,7 @@
 		question.setAnswer(answer);
 		//for mandatory questions, answer can not be null.
 		if(!question.isOptional() && answer == null){
-			errors.add(SurveyConstants.ERROR_MSG_KEY,new ActionMessage(SurveyConstants.ERROR_MSG_MANDATORY_QUESTION));
+			errors.add(SurveyConstants.ERROR_MSG_KEY+ question.getUid(),new ActionMessage(SurveyConstants.ERROR_MSG_MANDATORY_QUESTION));
 			addErrors(request, errors);
 		}
 
@@ -430,7 +430,7 @@
 			question.setAnswer(answer);
 			//for mandatory questions, answer can not be null.
 			if(!question.isOptional() && answer == null){
-				errors.add(SurveyConstants.ERROR_MSG_KEY,new ActionMessage(SurveyConstants.ERROR_MSG_MANDATORY_QUESTION));
+				errors.add(SurveyConstants.ERROR_MSG_KEY + question.getUid(),new ActionMessage(SurveyConstants.ERROR_MSG_MANDATORY_QUESTION));
 				addErrors(request, errors);
 			}
 		}
@@ -442,7 +442,7 @@
 		
 		String[] choiceList = request.getParameterValues(SurveyConstants.PREFIX_QUESTION_CHOICE+question.getUid());
 		String textEntry = request.getParameter(SurveyConstants.PREFIX_QUESTION_TEXT+question.getUid());
-		if(choiceList == null && textEntry == null)
+		if(choiceList == null && StringUtils.isBlank(textEntry))
 			return null;
 		
 		
Index: lams_tool_survey/web/pages/learning/learning.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_survey/web/pages/learning/learning.jsp,v
diff -u -r1.3 -r1.4
--- lams_tool_survey/web/pages/learning/learning.jsp	21 Sep 2006 07:46:41 -0000	1.3
+++ lams_tool_survey/web/pages/learning/learning.jsp	22 Sep 2006 00:57:21 -0000	1.4
@@ -45,55 +45,56 @@
 				${sessionMap.instructions}
 			</h2>
 			<BR><BR>
-			<%@ include file="/common/messages.jsp"%>
-				
 				<table cellpadding="0" cellspacing="0" class="alternative-color">
 					<c:choose>
 						<%-- Show on one page or when learner does not choose edit one question --%>
-						<c:when test="${sessionMap.showOnOnePage && empty questionSeqID}">
-						<c:forEach var="element" items="${sessionMap.questionList}">
-							<c:set var="question" value="${element.value}" />
-							<%@ include file="/pages/learning/question.jsp"%>
-						</c:forEach>
+						<c:when test="${sessionMap.showOnOnePage && (empty questionSeqID or questionSeqID == 0)}">
+							<c:forEach var="element" items="${sessionMap.questionList}">
+								<c:set var="question" value="${element.value}" />
+								<%@ include file="/pages/learning/question.jsp"%>
+							</c:forEach>
 						</c:when>
 						<c:otherwise>
-							
 							<c:set var="question" value="${sessionMap.questionList[questionSeqID]}"/>
 							<%@ include file="/pages/learning/question.jsp"%>
 						</c:otherwise>
 					</c:choose>
 				</table>
 				<%-- Display button according to different situation --%>
-				<c:if test="${sessionMap.mode != 'teacher'}">
-					<div class="right-buttons">
-						<c:choose>
-							<c:when test="${(sessionMap.showOnOnePage && empty questionSeqID) or position == 3}">
+					<c:choose>
+						<c:when test="${(sessionMap.showOnOnePage && (empty questionSeqID or questionSeqID == 0)) or position == 3}">
+							<div class="right-buttons">
 								<html:submit property="doSurvey"  disabled="${sessionMap.finishedLock}"  styleClass="button">
 									<fmt:message key="label.submit.survey" />
 								</html:submit>
-							</c:when>
-							<c:otherwise>
-								<c:if test="${position == 1 || position == 0}">
-									<html:button property="NextButton"  onclick="nextQuestion()" styleClass="button">
-										<fmt:message key="label.next" />
-									</html:button>
-								</c:if>
-								&nbsp;&nbsp;
-								<c:if test="${position == 2 || position == 0}">
-									<html:button property="PreviousButton"  onclick="previousQuestion()" styleClass="button">
-										<fmt:message key="label.previous" />
-									</html:button> 
-								</c:if>
+							</div>
+						</c:when>
+						<c:otherwise>
+							<c:set var="preChecked" value="true"/>
+							<c:if test="${position == 2 || position == 0}">
+								<c:set var="preChecked" value="false"/>
+							</c:if>
+							<c:set var="nextChecked" value="true"/>
+							<c:if test="${position == 1 || position == 0}">
+								<c:set var="nextChecked" value="false"/>
+							</c:if>
+							<span class="left-buttons">
+								<html:button property="PreviousButton"  onclick="previousQuestion()" styleClass="button" disabled="${preChecked}">
+									<fmt:message key="label.previous" />
+								</html:button> 
+							</span>
+							<span class="right-buttons">
+								<html:button property="NextButton"  onclick="nextQuestion()" styleClass="button"  disabled="${nextChecked}">
+									<fmt:message key="label.next" />
+								</html:button>
 								<c:if test="${position == 2}">
-									<html:submit property="doSurvey"  disabled="${sessionMap.finishedLock}"  styleClass="button">
-										<fmt:message key="label.submit.survey" />
-									</html:submit>										
+										<html:submit property="doSurvey"  disabled="${sessionMap.finishedLock}"  styleClass="button">
+											<fmt:message key="label.submit.survey" />
+										</html:submit>
 								</c:if>
-							</c:otherwise>
-						</c:choose>
-					</div>
-					<%-- end mode != teacher --%>
-				</c:if>
+							</span>
+						</c:otherwise>
+					</c:choose>
 		
 		</div>  <!--closes content-->
 		
Index: lams_tool_survey/web/pages/learning/question.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_survey/web/pages/learning/question.jsp,v
diff -u -r1.1 -r1.2
--- lams_tool_survey/web/pages/learning/question.jsp	21 Sep 2006 07:46:41 -0000	1.1
+++ lams_tool_survey/web/pages/learning/question.jsp	22 Sep 2006 00:57:21 -0000	1.2
@@ -6,6 +6,13 @@
 			<span style="color: #FF0000">*</span>
 		</c:if>
 		<c:out value="${question.description}" escapeXml="false"/>
+		<logic:messagesPresent property="questionError${question.uid}">
+			<p class="warning">
+		        <html:messages id="error" property="questionError${question.uid}">
+		            <c:out value="${error}" escapeXml="false"/><br/>
+		        </html:messages>
+			</p>
+		</logic:messagesPresent>		
 	</th>
 </tr>
 <tr>