Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayToolActivityAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayToolActivityAction.java,v
diff -u -r1.5 -r1.6
--- lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayToolActivityAction.java	10 May 2006 05:30:46 -0000	1.5
+++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayToolActivityAction.java	8 Jun 2006 01:03:02 -0000	1.6
@@ -71,8 +71,7 @@
 		
 		ToolActivity toolActivity = (ToolActivity)activity;
 
-		String url = actionMappings.getLearnerToolURL(learnerProgress.getLesson().getLessonId(),
-				toolActivity, learnerProgress.getUser());
+		String url = actionMappings.getLearnerToolURL(learnerProgress.getLesson(), toolActivity, learnerProgress.getUser());
 		try 
 		{
 		    response.sendRedirect(url);
Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/LearnerAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/LearnerAction.java,v
diff -u -r1.23 -r1.24
--- lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/LearnerAction.java	6 Jun 2006 02:35:26 -0000	1.23
+++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/LearnerAction.java	8 Jun 2006 01:03:02 -0000	1.24
@@ -376,7 +376,7 @@
 	        //preparing tranfer object for flash
 	        LearnerProgress learnerProgress = LearningWebUtil.getLearnerProgressByUser(request,getServlet().getServletContext());
 	        ProgressActivityDTO activityDTO = new ProgressActivityDTO(new Long(activityId),
-	                 activityMapping.calculateActivityURLForProgressView(learnerProgress.getLesson().getLessonId(),learner,requestedActivity));
+	                 activityMapping.calculateActivityURLForProgressView(learnerProgress.getLesson(),learner,requestedActivity));
 	        //send data back to flash.
 	        message = new FlashMessage("getLearnerActivityURL",activityDTO);
 
Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/LoadToolActivityAction.java
===================================================================
RCS file: /usr/local/cvsroot/lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/LoadToolActivityAction.java,v
diff -u -r1.6 -r1.7
--- lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/LoadToolActivityAction.java	10 May 2006 05:30:46 -0000	1.6
+++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/LoadToolActivityAction.java	8 Jun 2006 01:03:02 -0000	1.7
@@ -71,9 +71,8 @@
 				
 		form.setActivityId(activity.getActivityId());
 		
-		if (activity.isToolActivity() || activity.isSystemToolActivity() ) { 
-			String url = actionMappings.getLearnerToolURL(learnerProgress.getLesson().getLessonId(),
-					activity, learnerProgress.getUser());
+		if (activity.isToolActivity() || activity.isSystemToolActivity() ) {
+			String url = actionMappings.getLearnerToolURL(learnerProgress.getLesson(),activity, learnerProgress.getUser());
 			form.addActivityURL(new ActivityURL(activity.getActivityId(),url));
 		} else {
 		    log.error(className+": activity not ToolActivity");
Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/util/ActivityMapping.java
===================================================================
RCS file: /usr/local/cvsroot/lams_learning/src/java/org/lamsfoundation/lams/learning/web/util/ActivityMapping.java,v
diff -u -r1.18 -r1.19
--- lams_learning/src/java/org/lamsfoundation/lams/learning/web/util/ActivityMapping.java	6 Jun 2006 02:35:26 -0000	1.18
+++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/util/ActivityMapping.java	8 Jun 2006 01:03:02 -0000	1.19
@@ -40,6 +40,7 @@
 import org.lamsfoundation.lams.learningdesign.Activity;
 import org.lamsfoundation.lams.learningdesign.ToolActivity;
 import org.lamsfoundation.lams.lesson.LearnerProgress;
+import org.lamsfoundation.lams.lesson.Lesson;
 import org.lamsfoundation.lams.tool.exception.LamsToolServiceException;
 import org.lamsfoundation.lams.tool.service.ILamsCoreToolService;
 import org.lamsfoundation.lams.usermanagement.User;
@@ -293,15 +294,17 @@
      * @param activity, the Activity to be displayed
      * @param progress, the current LearnerProgress, used to get activity status
      */
-    public String getLearnerToolURL(Long lessonID, Activity activity, User learner)
+    public String getLearnerToolURL(Lesson lesson, Activity activity, User learner)
     {
         try
         {
-            return toolService.getToolLearnerURL(lessonID,activity,learner);
+        	if ( lesson.isPreviewLesson() )
+        		return toolService.getToolLearnerPreviewURL(lesson.getLessonId(),activity,learner);
+        	else 
+        		return toolService.getToolLearnerURL(lesson.getLessonId(),activity,learner);
         }
         catch (LamsToolServiceException e)
         {
-            //TODO define an exception at web layer
             throw new LearnerServiceException(e.getMessage());
         }
     }
@@ -369,13 +372,13 @@
      * @param activity the activity the learner want to view
      * @return the url for that tool.
      */
-    public String calculateActivityURLForProgressView(Long lessonID, User learner,
+    public String calculateActivityURLForProgressView(Lesson lesson, User learner,
                                                       Activity activity)
     {
 
         if (activity.isToolActivity())
         {
-            return getLearnerToolURL(lessonID, ((ToolActivity) activity), learner);
+            return getLearnerToolURL(lesson, ((ToolActivity) activity), learner);
         }
         else if (activity.isGroupingActivity())
             //TODO need to be changed when group action servlet is done