Index: lams_central/src/java/org/lamsfoundation/lams/web/ShibLearnerServlet.java
===================================================================
RCS file: /usr/local/cvsroot/lams_central/src/java/org/lamsfoundation/lams/web/Attic/ShibLearnerServlet.java,v
diff -u -r1.1.2.2 -r1.1.2.3
--- lams_central/src/java/org/lamsfoundation/lams/web/ShibLearnerServlet.java	17 Jul 2007 05:36:22 -0000	1.1.2.2
+++ lams_central/src/java/org/lamsfoundation/lams/web/ShibLearnerServlet.java	18 Jul 2007 02:49:19 -0000	1.1.2.3
@@ -81,12 +81,15 @@
 			Lesson lesson = lessonId != null ? getLessonService().getLesson(lessonId) : null;
 			if (lesson != null) {
 				LessonClass lessonClass = lesson.getLessonClass();
-				if (user != null && lessonClass != null && !lesson.getLessonClass().getLearners().contains(user)) {
-					lessonClass.addLearner(user);
-					log.info("Added shib user "+user.getLogin()+" to lesson with id "+lessonIdStr);
+				if (user != null && lessonClass != null && !lessonClass.getLearners().contains(user)) {
+					boolean result = lessonService.addLearner(lessonId, user.getUserId());
+					if (result) {
+						log.info("Added shib user "+user.getLogin()+" to lesson with id "+lessonIdStr);
+					}
 				}
 			}
-			log.warn("Shib user "+user.getLogin()+" tried to access learner without "+Role.LEARNER+" role.");
+		} else {
+			log.warn("Shib user "+username+" tried to access learner without "+Role.LEARNER+" role.");
 		}
 		
 		response.sendRedirect("/lams/home.do?method=learner&lessonID="+lessonIdStr);