Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/ActivityAction.java =================================================================== diff -u -re5049e0948ac38f849ed086dfa9b8b7dfd8d76ae -r7da79aec2f320705164e3faae07352a54d1de68a --- lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/ActivityAction.java (.../ActivityAction.java) (revision e5049e0948ac38f849ed086dfa9b8b7dfd8d76ae) +++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/ActivityAction.java (.../ActivityAction.java) (revision 7da79aec2f320705164e3faae07352a54d1de68a) @@ -59,11 +59,12 @@ return learnerService; } + /** Setup the progress string and the lesson id in the actionForm */ public ActionForward setupProgressString(ActionForm actionForm, HttpServletRequest request) { LearnerProgress learnerProgress = LearningWebUtil.getLearnerProgress(request,getLearnerService()); - ActivityForm activityForm = (ActivityForm) actionForm; + ActivityForm activityForm = (ActivityForm) actionForm; // Calculate the progress summary. On join this method gets called twice, and we // only want to calculate once @@ -73,6 +74,11 @@ activityForm.setProgressSummary(progressSummary); } + Lesson currentLesson = learnerProgress.getLesson(); + if(currentLesson != null){ + activityForm.setLessonID(currentLesson.getLessonId()); + } + if(log.isDebugEnabled()) log.debug("Entering activity: progress summary is "+activityForm.getProgressSummary()); @@ -127,11 +133,6 @@ progressSummary.append(currentActivity.getActivityId()); } - progressSummary.append("&lessonID="); - Lesson currentLesson = learnerProgress.getLesson(); - if(currentLesson != null){ - progressSummary.append(currentLesson.getLessonId()); - } } return progressSummary.toString(); } Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayOptionsActivityAction.java =================================================================== diff -u -re5049e0948ac38f849ed086dfa9b8b7dfd8d76ae -r7da79aec2f320705164e3faae07352a54d1de68a --- lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayOptionsActivityAction.java (.../DisplayOptionsActivityAction.java) (revision e5049e0948ac38f849ed086dfa9b8b7dfd8d76ae) +++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayOptionsActivityAction.java (.../DisplayOptionsActivityAction.java) (revision 7da79aec2f320705164e3faae07352a54d1de68a) @@ -83,7 +83,7 @@ OptionsActivity optionsActivity = (OptionsActivity)activity; - form.setActivityId(activity.getActivityId()); + form.setActivityID(activity.getActivityId()); List activityURLs = new ArrayList(); Set subActivities = optionsActivity.getActivities(); Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayParallelActivityAction.java =================================================================== diff -u -r5aff7aa069793bc34e60b304e69e9e9fceb996f5 -r7da79aec2f320705164e3faae07352a54d1de68a --- lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayParallelActivityAction.java (.../DisplayParallelActivityAction.java) (revision 5aff7aa069793bc34e60b304e69e9e9fceb996f5) +++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/DisplayParallelActivityAction.java (.../DisplayParallelActivityAction.java) (revision 7da79aec2f320705164e3faae07352a54d1de68a) @@ -80,7 +80,7 @@ ParallelActivity parallelActivity = (ParallelActivity)activity; - form.setActivityId(activity.getActivityId()); + form.setActivityID(activity.getActivityId()); List activityURLs = new ArrayList(); Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/LoadToolActivityAction.java =================================================================== diff -u -re5049e0948ac38f849ed086dfa9b8b7dfd8d76ae -r7da79aec2f320705164e3faae07352a54d1de68a --- lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/LoadToolActivityAction.java (.../LoadToolActivityAction.java) (revision e5049e0948ac38f849ed086dfa9b8b7dfd8d76ae) +++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/action/LoadToolActivityAction.java (.../LoadToolActivityAction.java) (revision 7da79aec2f320705164e3faae07352a54d1de68a) @@ -78,7 +78,7 @@ LearnerProgress learnerProgress = LearningWebUtil.getLearnerProgress(request,learnerService); Activity activity = LearningWebUtil.getActivityFromRequest(request, learnerService); - form.setActivityId(activity.getActivityId()); + form.setActivityID(activity.getActivityId()); if (activity.isToolActivity() || activity.isSystemToolActivity() ) { Index: lams_learning/src/java/org/lamsfoundation/lams/learning/web/form/ActivityForm.java =================================================================== diff -u -r7d5e18b6ddc40f4de3d34eefa114afc297724289 -r7da79aec2f320705164e3faae07352a54d1de68a --- lams_learning/src/java/org/lamsfoundation/lams/learning/web/form/ActivityForm.java (.../ActivityForm.java) (revision 7d5e18b6ddc40f4de3d34eefa114afc297724289) +++ lams_learning/src/java/org/lamsfoundation/lams/learning/web/form/ActivityForm.java (.../ActivityForm.java) (revision 7da79aec2f320705164e3faae07352a54d1de68a) @@ -45,14 +45,17 @@ * LearnerProgress (or Learner) and Activity. Note that the activity may already * be complete. */ - private Long activityId; + private Long activityID; /** List of ActivityURL, will only contain one if a simple activity */ private List activityURLs; /** Progress summary suitable for Flash. In the form attempted=13_14&completed=10_11¤t=12.*/ private String progressSummary; + /** Lesson to which this activity belongs. Id comes from the learner progress */ + private Long lessonID; + /** * Method reset * @param mapping @@ -68,11 +71,11 @@ public void setActivityURLs(List activityURLs) { this.activityURLs = activityURLs; } - public Long getActivityId() { - return activityId; + public Long getActivityID() { + return activityID; } - public void setActivityId(Long activityId) { - this.activityId = activityId; + public void setActivityID(Long activityID) { + this.activityID = activityID; } public void addActivityURL(ActivityURL activityUrl) @@ -89,4 +92,12 @@ public void setProgressSummary(String progressSummary) { this.progressSummary = progressSummary; } + + public Long getLessonID() { + return lessonID; + } + + public void setLessonID(Long lessonID) { + this.lessonID = lessonID; + } } \ No newline at end of file Index: lams_learning/web/optionsActivity.jsp =================================================================== diff -u -ra0722fe03d763f9da5efc82987034d929e6a23a1 -r7da79aec2f320705164e3faae07352a54d1de68a --- lams_learning/web/optionsActivity.jsp (.../optionsActivity.jsp) (revision a0722fe03d763f9da5efc82987034d929e6a23a1) +++ lams_learning/web/optionsActivity.jsp (.../optionsActivity.jsp) (revision 7da79aec2f320705164e3faae07352a54d1de68a) @@ -160,7 +160,7 @@ - " /> + " />