Index: lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/McResources.properties
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/Attic/McResources.properties,v
diff -u -r1.17 -r1.18
--- lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/McResources.properties	28 Oct 2005 19:15:47 -0000	1.17
+++ lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/McResources.properties	29 Oct 2005 12:02:49 -0000	1.18
@@ -1,7 +1,7 @@
 # Project lams_tool_lamc11
 #Authoring mode resources
 label.authoring.mc						=Multiple Choice
-label.authoring.mc.basic 				=Please define the questions and their options. 
+label.authoring.mc.basic 				=Please define the questions. 
 label.authoring.mc.basic.editOptions	=Please define the question and/or its options.
 label.authoring.title					=Title
 label.authoring.instructions			=Instructions
Index: lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McAction.java,v
diff -u -r1.26 -r1.27
--- lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McAction.java	28 Oct 2005 19:15:47 -0000	1.26
+++ lams_tool_lamc/src/java/org/lamsfoundation/lams/tool/mc/web/McAction.java	29 Oct 2005 12:02:49 -0000	1.27
@@ -488,6 +488,9 @@
 			request.getSession().setAttribute(MAP_OPTIONS_CONTENT, mapOptionsContent);
     		logger.debug("updated Options Map: " + request.getSession().getAttribute(MAP_OPTIONS_CONTENT));
     		
+    		
+    		
+    		
     		Long selectedQuestionContentUid=(Long) request.getSession().getAttribute(SELECTED_QUESTION_CONTENT_UID);
 	 		logger.debug("selectedQuestionContentUid:" + selectedQuestionContentUid);
 	 		
@@ -514,7 +517,13 @@
 	 	        }
 	 		}
 	 		
+	 		Map mapSelectedOptions= (Map) request.getSession().getAttribute(MAP_SELECTED_OPTIONS);
+	 		mapSelectedOptions.clear();
+	 		mapSelectedOptions = repopulateCurrentCheckBoxStatesMap(request);
+	 		logger.debug("after add mapSelectedOptions: " + mapSelectedOptions);
+	 		request.getSession().setAttribute(MAP_SELECTED_OPTIONS, mapSelectedOptions);
 	 		
+	 		
 	 		mcAuthoringForm.resetUserAction();
 			return (mapping.findForward(EDIT_OPTS_CONTENT));
 	 	}
@@ -558,6 +567,14 @@
 				}
 			}
 			
+			Map mapSelectedOptions= (Map) request.getSession().getAttribute(MAP_SELECTED_OPTIONS);
+	 		mapSelectedOptions.clear();
+	 		mapSelectedOptions = repopulateCurrentCheckBoxStatesMap(request);
+	 		logger.debug("after add mapSelectedOptions: " + mapSelectedOptions);
+	 		request.getSession().setAttribute(MAP_SELECTED_OPTIONS, mapSelectedOptions);
+
+			
+			
 			mcAuthoringForm.resetUserAction();			
 			return (mapping.findForward(EDIT_OPTS_CONTENT));
 	 	}
@@ -812,23 +829,33 @@
 			McContent mcContent=mcService.retrieveMc(toolContentId);
 			logger.debug("mcContent:" + mcContent);
 			
-			logger.debug("updating mcContent title and instructions:" + mcContent);
-			mcContent.setTitle(richTextTitle);
-			mcContent.setInstructions(richTextInstructions);
+						
 
-			mcContent.setQuestionsSequenced(isQuestionsSequenced); 
-		    mcContent.setSynchInMonitor(isSynchInMonitor);
-		    mcContent.setUsernameVisible(isUsernameVisible);
-		    mcContent.setRetries(isRetries);
-		    mcContent.setPassMark(new Integer(passmark));
-		    mcContent.setShowFeedback(isShowFeedback);
-		    mcContent.setEndLearningMessage(endLearningMessage);
-		    mcContent.setReportTitle(reportTitle);
-		    mcContent.setMonitoringReportTitle(monitoringReportTitle);
-		    mcContent.setEndLearningMessage(endLearningMessage);
-		    mcContent.setOfflineInstructions(richTextOfflineInstructions);
-		    mcContent.setOnlineInstructions(richTextOnlineInstructions);
-		    
+			if (mcContent != null)
+			{
+				logger.debug("updating mcContent title and instructions:" + mcContent);
+				mcContent.setTitle(richTextTitle);
+				mcContent.setInstructions(richTextInstructions);
+
+				mcContent.setQuestionsSequenced(isQuestionsSequenced); 
+			    mcContent.setSynchInMonitor(isSynchInMonitor);
+			    mcContent.setUsernameVisible(isUsernameVisible);
+			    mcContent.setRetries(isRetries);
+			    mcContent.setPassMark(new Integer(passmark));
+			    mcContent.setShowFeedback(isShowFeedback);
+			    mcContent.setEndLearningMessage(endLearningMessage);
+			    mcContent.setReportTitle(reportTitle);
+			    mcContent.setMonitoringReportTitle(monitoringReportTitle);
+			    mcContent.setEndLearningMessage(endLearningMessage);
+			    mcContent.setOfflineInstructions(richTextOfflineInstructions);
+			    mcContent.setOnlineInstructions(richTextOnlineInstructions);
+			}
+			else
+			{
+				mcContent=createContent(request, mcAuthoringForm);		
+				logger.debug("mcContent created");
+			}
+					    
 			mcService.resetAllQuestions(mcContent.getUid());
 			logger.debug("all question reset for :" + mcContent.getUid());
 			
@@ -915,72 +942,11 @@
 	   	    return (mapping.findForward(LOAD_QUESTIONS));
         }
 	 	
-	 	
-	 	
-	 	
-	 	
 	 	mcAuthoringForm.resetUserAction();
    	    return (mapping.findForward(LOAD_QUESTIONS));
     }
     
     
-    /**
-     * shrinks the size of the Map to only used entries
-     * 
-     * @param mapQuestionContent
-     * @param request
-     * @return
-     */
-    protected Map repopulateMap(HttpServletRequest request, String parameterType)
-    {
-    	Map mapTempQuestionsContent= new TreeMap(new McComparator());
-    	logger.debug("parameterType: " + parameterType);
-    	
-    	long mapCounter=0;
-    	for (long i=1; i <= MAX_QUESTION_COUNT ; i++)
-		{
-			String candidateQuestionEntry =request.getParameter(parameterType + i);
-			if (
-				(candidateQuestionEntry != null) && 
-				(candidateQuestionEntry.length() > 0)   
-				)
-			{
-				mapCounter++;
-				mapTempQuestionsContent.put(new Long(mapCounter).toString(), candidateQuestionEntry);
-			}
-		}
-    	logger.debug("return repopulated Map: " + mapTempQuestionsContent);
-    	return mapTempQuestionsContent;
-    }
-    
-	
-    protected Map repopulateCurrentWeightsMap(HttpServletRequest request, String parameterType)
-    {
-    	Map mapTempQuestionsContent= new TreeMap(new McComparator());
-    	logger.debug("parameterType: " + parameterType);
-    	
-    	long mapCounter=0;
-    	for (long i=1; i <= MAX_QUESTION_COUNT ; i++)
-		{
-			String candidateEntry =request.getParameter(parameterType + i);
-			String questionText =request.getParameter("questionContent" + i);
-			if ((questionText != null) && (questionText.length() > 0))
-			{
-				logger.debug("questionText: " + questionText);
-				if (candidateEntry != null)
-				{
-					mapCounter++;
-					mapTempQuestionsContent.put(new Long(mapCounter).toString(), candidateEntry);
-				}	
-			}
-			
-		}
-    	logger.debug("return repopulated Map: " + mapTempQuestionsContent);
-    	return mapTempQuestionsContent;
-    }
-
-    
-    
     protected McContent createContent(HttpServletRequest request, McAuthoringForm mcAuthoringForm)
     {
     	IMcService mcService =McUtils.getToolService(request);
@@ -1397,7 +1363,89 @@
     }
 
     
+    /**
+     * shrinks the size of the Map to only used entries
+     * 
+     * @param mapQuestionContent
+     * @param request
+     * @return
+     */
+    protected Map repopulateMap(HttpServletRequest request, String parameterType)
+    {
+    	Map mapTempQuestionsContent= new TreeMap(new McComparator());
+    	logger.debug("parameterType: " + parameterType);
+    	
+    	long mapCounter=0;
+    	for (long i=1; i <= MAX_QUESTION_COUNT ; i++)
+		{
+			String candidateQuestionEntry =request.getParameter(parameterType + i);
+			if (
+				(candidateQuestionEntry != null) && 
+				(candidateQuestionEntry.length() > 0)   
+				)
+			{
+				mapCounter++;
+				mapTempQuestionsContent.put(new Long(mapCounter).toString(), candidateQuestionEntry);
+			}
+		}
+    	logger.debug("return repopulated Map: " + mapTempQuestionsContent);
+    	return mapTempQuestionsContent;
+    }
+    
+	
+    protected Map repopulateCurrentWeightsMap(HttpServletRequest request, String parameterType)
+    {
+    	Map mapTempQuestionsContent= new TreeMap(new McComparator());
+    	logger.debug("parameterType: " + parameterType);
+    	
+    	long mapCounter=0;
+    	for (long i=1; i <= MAX_QUESTION_COUNT ; i++)
+		{
+			String candidateEntry =request.getParameter(parameterType + i);
+			String questionText =request.getParameter("questionContent" + i);
+			if ((questionText != null) && (questionText.length() > 0))
+			{
+				logger.debug("questionText: " + questionText);
+				if (candidateEntry != null)
+				{
+					mapCounter++;
+					mapTempQuestionsContent.put(new Long(mapCounter).toString(), candidateEntry);
+				}	
+			}
+			
+		}
+    	logger.debug("return repopulated Map: " + mapTempQuestionsContent);
+    	return mapTempQuestionsContent;
+    }
 
+    
+    protected Map repopulateCurrentCheckBoxStatesMap(HttpServletRequest request)
+    {
+    	Map mapTempContent= new TreeMap(new McComparator());
+    	
+    	long mapCounter=0;
+    	for (long i=1; i <= MAX_OPTION_COUNT ; i++)
+		{
+			String candidateEntry =request.getParameter("checkBoxSelected" + i);
+			String optionText =request.getParameter("optionContent" + i);
+			
+			if (
+					(candidateEntry != null) && 
+					(candidateEntry.length() > 0)   
+					)
+				{
+					if (candidateEntry.equals("Correct"))
+					{
+						mapCounter++;
+						mapTempContent.put(new Long(mapCounter).toString(), optionText);
+					}
+				}
+		}
+    	logger.debug("return repopulated currentCheckBoxStatesMap: " + mapTempContent);
+    	return mapTempContent;
+    }
+
+
     /**
      * persists error messages to request scope
      * @param request
@@ -1410,4 +1458,5 @@
 		logger.debug("add " + message +"  to ActionMessages:");
 		saveErrors(request,errors);	    	    
 	} 
+
 }
Index: lams_tool_lamc/web/BasicContent.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_lamc/web/Attic/BasicContent.jsp,v
diff -u -r1.2 -r1.3
--- lams_tool_lamc/web/BasicContent.jsp	28 Oct 2005 19:15:46 -0000	1.2
+++ lams_tool_lamc/web/BasicContent.jsp	29 Oct 2005 12:02:49 -0000	1.3
@@ -37,13 +37,13 @@
 						<td class="formlabel">
 						</td>
 						<td class="input" align="center">
-							<h5> <b> <bean:message key="label.Questions"/> </b> </h5>
+							 <bean:message key="label.Questions"/> 
 						</td>
 						<td class="input" align="center">
-							<h5> <b> <bean:message key="label.weight"/> </b> </h5>
+							 <bean:message key="label.weight"/> 
 						</td>
 						<td class="input" colspan=2 align="center">
-							<h5> <b> <bean:message key="label.actions"/> </b> </h5>
+							 <bean:message key="label.actions"/> 
 						</td>
 					</tr>
 			  	 		<c:set var="queIndex" scope="session" value="1"/>
Index: lams_tool_lamc/web/authoringMaincontent.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_lamc/web/Attic/authoringMaincontent.jsp,v
diff -u -r1.26 -r1.27
--- lams_tool_lamc/web/authoringMaincontent.jsp	29 Oct 2005 09:42:59 -0000	1.26
+++ lams_tool_lamc/web/authoringMaincontent.jsp	29 Oct 2005 12:02:49 -0000	1.27
@@ -88,7 +88,6 @@
 <html:form  action="/authoring?method=loadQ&validate=false" enctype="multipart/form-data" method="POST" target="_self">
  <!-- tab content one (basic)-->
 <div id='content_b' class="tabbody content_b" >
-<h2><bean:message key="label.authoring.mc.basic"/></h2>
 
 <table align=center> 	  
 <tr>   
@@ -98,7 +97,8 @@
 </tr> 
 </table>
 
-<h2>Please define questions </h2>
+<h2><bean:message key="label.authoring.mc.basic"/></h2>
+
 <div id="formtablecontainer">
 		<jsp:include page="BasicContent.jsp" />
 </div>
Index: lams_tool_lamc/web/editOptionsContent.jsp
===================================================================
RCS file: /usr/local/cvsroot/lams_tool_lamc/web/Attic/editOptionsContent.jsp,v
diff -u -r1.11 -r1.12
--- lams_tool_lamc/web/editOptionsContent.jsp	29 Oct 2005 09:42:59 -0000	1.11
+++ lams_tool_lamc/web/editOptionsContent.jsp	29 Oct 2005 12:02:49 -0000	1.12
@@ -119,17 +119,17 @@
 					<tr>
 					  	<td class="formlabel">  </td>
 						<td class=input colspan=3>
-							<h5> <b> <bean:message key="label.candidateAnswers"/> </h5> </b>
+							&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp
+							 <bean:message key="label.candidateAnswers"/> 
 
-							 &nbsp&nbsp
+							 &nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp&nbsp
 
-
-							<h5> <b> <bean:message key="label.isCorrect"/>  </h5> </b>
+							 <bean:message key="label.isCorrect"/>  
 							
-							&nbsp&nbsp
+							&nbsp&nbsp&nbsp&nbsp
 
 
-							<h5> <b> <bean:message key="label.actions"/> </h5> </b>
+							 <bean:message key="label.actions"/> 
 						</td>
 					</tr>
 
@@ -144,7 +144,7 @@
 								  			<input type="text" name="optionContent<c:out value="${optionIndex}"/>" value="<c:out value="${optionEntry.value}"/>"   
 									  		size="50" maxlength="255"> 
 									  		
-									  		&nbsp&nbsp
+									  		&nbsp
 
 									  	
 			  				  	 		<c:set var="selectedOptionFound" scope="request" value="0"/>
@@ -169,7 +169,7 @@
 											
 					 					</c:if> 			
 									  											
-									  		&nbsp&nbsp
+									  		&nbsp
 
 	  										<html:submit property="addOption" styleClass="linkbutton" 
 											onmouseover="pviiClassNew(this,'linkbutton')" onmouseout="pviiClassNew(this,'linkbutton')">
@@ -186,7 +186,7 @@
 								  			<input type="text" name="optionContent<c:out value="${optionIndex}"/>" value="<c:out value="${optionEntry.value}"/>"   
 									  		size="50" maxlength="255"> 
 
-									  		&nbsp&nbsp
+									  		&nbsp
 
 			  				  	 		<c:set var="selectedOptionFound" scope="request" value="0"/>
 			  							<c:forEach var="selectedOptionEntry" items="${sessionScope.mapSelectedOptions}">
@@ -210,7 +210,7 @@
 
 					 					</c:if> 			
 									  	
-									  		&nbsp&nbsp
+									  		&nbsp
 
 			 								<html:submit property="removeOption" styleClass="linkbutton"  onclick="javascript:document.forms[0].deletableOptionIndex.value=${optionIndex};"
 			 								onmouseover="pviiClassNew(this,'linkbutton')" onmouseout="pviiClassNew(this,'linkbutton')">