Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/ChooseActivity.java
===================================================================
RCS file: /usr/local/cvsroot/lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/Attic/ChooseActivity.java,v
diff -u -r1.3 -r1.4
--- lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/ChooseActivity.java	21 Feb 2005 05:31:39 -0000	1.3
+++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/ChooseActivity.java	24 Feb 2005 05:38:11 -0000	1.4
@@ -61,29 +61,29 @@
 		ActivityForm form = (ActivityForm) actionForm;
 		ActionMappings actionMappings = getActionMappings();
 		
-		SessionBean sessionBean = ActivityAction.getSessionBean(request);
+		SessionBean sessionBean = getSessionBean(request);
 		if (sessionBean == null) {
 			// forward to the no session error page
-			return mapping.findForward(actionMappings.NO_SESSION_ERROR);
+			return mapping.findForward(ActionMappings.NO_SESSION_ERROR);
 		}
 		
 		// check token
 		if (!this.isTokenValid(request, true)) {
 			// didn't come here from options page
 		    log.info(className+": No valid token in request");
-			return mapping.findForward(actionMappings.DOUBLE_SUBMIT_ERROR);
+			return mapping.findForward(ActionMappings.DOUBLE_SUBMIT_ERROR);
 		}
 		
 		// Get learner
-		User learner = sessionBean.getLeaner();
+		User learner = sessionBean.getLearner();
 		Lesson lesson = sessionBean.getLesson();
 		
 		LearnerProgress progress = getLearnerProgress(request, form);
 		Activity activity = getActivity(request, form, progress);
 		
 		if (activity == null) {
 		    log.error(className+": No activity in request or session");
-			return mapping.findForward(actionMappings.ERROR);
+			return mapping.findForward(ActionMappings.ERROR);
 		}
 
 		ILearnerService learnerService = getLearnerService(request);
Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/CompleteActivity.java
===================================================================
RCS file: /usr/local/cvsroot/lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/Attic/CompleteActivity.java,v
diff -u -r1.4 -r1.5
--- lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/CompleteActivity.java	21 Feb 2005 05:31:39 -0000	1.4
+++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/CompleteActivity.java	24 Feb 2005 05:38:11 -0000	1.5
@@ -64,29 +64,29 @@
 		ActivityForm form = (ActivityForm)actionForm;
 		ActionMappings actionMappings = getActionMappings();
 		
-		SessionBean sessionBean = ActivityAction.getSessionBean(request);
+		SessionBean sessionBean = getSessionBean(request);
 		if (sessionBean == null) {
 			// forward to the no session error page
-			return mapping.findForward(actionMappings.NO_SESSION_ERROR);
+			return mapping.findForward(ActionMappings.NO_SESSION_ERROR);
 		}
 		
 		// check token
 		if (!this.isTokenValid(request, true)) {
 			// didn't come here from options page
 		    log.info(className+": No valid token in request");
-			return mapping.findForward(actionMappings.DOUBLE_SUBMIT_ERROR);
+			return mapping.findForward(ActionMappings.DOUBLE_SUBMIT_ERROR);
 		}
 		
 		// Get learner
-		User learner = sessionBean.getLeaner();
+		User learner = sessionBean.getLearner();
 		Lesson lesson = sessionBean.getLesson();
 		
 		LearnerProgress progress = getLearnerProgress(request, form);
 		Activity activity = getActivity(request, form, progress);
 		
 		if (activity == null) {
 		    log.error(className+": No activity in request or session");
-			return mapping.findForward(actionMappings.ERROR);
+			return mapping.findForward(ActionMappings.ERROR);
 		}
 
 		ILearnerService learnerService = getLearnerService(request);
Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayActivity.java
===================================================================
RCS file: /usr/local/cvsroot/lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/Attic/DisplayActivity.java,v
diff -u -r1.5 -r1.6
--- lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayActivity.java	21 Feb 2005 05:31:39 -0000	1.5
+++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayActivity.java	24 Feb 2005 05:38:11 -0000	1.6
@@ -23,8 +23,6 @@
 
 import javax.servlet.http.*;
 
-import java.util.*;
-
 import org.apache.struts.action.ActionForm;
 import org.apache.struts.action.ActionForward;
 import org.apache.struts.action.ActionMapping;
@@ -62,14 +60,14 @@
 		ActivityForm form = (ActivityForm) actionForm;
 		ActionMappings actionMappings = getActionMappings();
 		
-		SessionBean sessionBean = ActivityAction.getSessionBean(request);
+		SessionBean sessionBean = getSessionBean(request);
 		if (sessionBean == null) {
 			// forward to the no session error page
-			return mapping.findForward(actionMappings.NO_SESSION_ERROR);
+			return mapping.findForward(ActionMappings.NO_SESSION_ERROR);
 		}
 		
 		// Get learner
-		User learner = sessionBean.getLeaner();
+		User learner = sessionBean.getLearner();
 		Lesson lesson = sessionBean.getLesson();
 		
 		LearnerProgress learnerProgress = getLearnerProgress(request, form);
Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayLoadToolActivity.java
===================================================================
RCS file: /usr/local/cvsroot/lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/Attic/DisplayLoadToolActivity.java,v
diff -u -r1.3 -r1.4
--- lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayLoadToolActivity.java	21 Feb 2005 05:31:39 -0000	1.3
+++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayLoadToolActivity.java	24 Feb 2005 05:38:11 -0000	1.4
@@ -43,7 +43,7 @@
  * @struts:action path="/DisplayLoadToolActivity" name="activityForm"
  *                validate="false" scope="request"
  * 
- * @struts:action-forward name="displayTool" path=".toolActivity"
+ * @struts:action-forward name="displayTool" path=".loadToolActivity"
  * 
  */
 public class DisplayLoadToolActivity extends ActivityAction {
@@ -64,7 +64,7 @@
 		Activity activity = getActivity(request, form, learnerProgress);
 		if (!(activity instanceof ToolActivity)) {
 		    log.error(className+": activity not ToolActivity");
-			return mapping.findForward(actionMappings.ERROR);
+			return mapping.findForward(ActionMappings.ERROR);
 		}
 		
 		ToolActivity toolActivity = (ToolActivity)activity;
Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayOptionsActivity.java
===================================================================
RCS file: /usr/local/cvsroot/lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/Attic/DisplayOptionsActivity.java,v
diff -u -r1.6 -r1.7
--- lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayOptionsActivity.java	21 Feb 2005 05:31:39 -0000	1.6
+++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayOptionsActivity.java	24 Feb 2005 05:38:11 -0000	1.7
@@ -72,7 +72,7 @@
 		Activity activity = getActivity(request, form, learnerProgress);
 		if (!(activity instanceof OptionsActivity)) {
 		    log.error(className+": activity not OptionsActivity "+activity.getActivityId());
-			return mapping.findForward(actionMappings.ERROR);
+			return mapping.findForward(ActionMappings.ERROR);
 		}
 
 		OptionsActivity optionsActivity = (OptionsActivity)activity;
Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayParallelActivity.java
===================================================================
RCS file: /usr/local/cvsroot/lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/Attic/DisplayParallelActivity.java,v
diff -u -r1.6 -r1.7
--- lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayParallelActivity.java	21 Feb 2005 05:31:39 -0000	1.6
+++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayParallelActivity.java	24 Feb 2005 05:38:11 -0000	1.7
@@ -65,7 +65,7 @@
 		Activity activity = getActivity(request, form, learnerProgress);
 		if (!(activity instanceof ParallelActivity)) {
 		    log.error(className+": activity not ParallelActivity "+activity.getActivityId());
-			return mapping.findForward(actionMappings.ERROR);
+			return mapping.findForward(ActionMappings.ERROR);
 		}
 
 		ParallelActivity parallelActivity = (ParallelActivity)activity;
@@ -84,7 +84,7 @@
 		}
 		if (activityURLs.size() == 0) {
 		    log.error(className+": No sub-activity URLs for activity "+activity.getActivityId());
-			return mapping.findForward(actionMappings.ERROR);
+			return mapping.findForward(ActionMappings.ERROR);
 		}
 		form.setActivityURLs(activityURLs);
 		
Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayToolActivity.java
===================================================================
RCS file: /usr/local/cvsroot/lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/Attic/DisplayToolActivity.java,v
diff -u -r1.6 -r1.7
--- lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayToolActivity.java	21 Feb 2005 05:31:39 -0000	1.6
+++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayToolActivity.java	24 Feb 2005 05:38:11 -0000	1.7
@@ -23,12 +23,9 @@
 
 import javax.servlet.http.*;
 
-import java.util.*;
-
 import org.apache.struts.action.ActionForm;
 import org.apache.struts.action.ActionForward;
 import org.apache.struts.action.ActionMapping;
-import org.lamsfoundation.lams.learning.web.bean.ActivityURL;
 import org.lamsfoundation.lams.learning.web.form.ActivityForm;
 
 import org.lamsfoundation.lams.learningdesign.*;
@@ -38,6 +35,8 @@
 /** 
  * Action class to forward the user to a Tool.
  * 
+ * @author daveg
+ * 
  * XDoclet definition:
  * 
  * @struts:action path="/DisplayToolActivity" name="activityForm"
@@ -62,7 +61,7 @@
 		Activity activity = getActivity(request, form, learnerProgress);
 		if (!(activity instanceof ToolActivity)) {
 		    log.error(className+": activity not ToolActivity");
-			return mapping.findForward(actionMappings.ERROR);
+			return mapping.findForward(ActionMappings.ERROR);
 		}
 		
 		ToolActivity toolActivity = (ToolActivity)activity;
@@ -72,7 +71,7 @@
 		    response.sendRedirect(url);
 		}
 		catch (java.io.IOException e) {
-		    return mapping.findForward(actionMappings.ERROR);
+		    return mapping.findForward(ActionMappings.ERROR);
 		}
 		return null;
 	}