Index: lams_bb_integration/RELEASE_NOTES.TXT =================================================================== diff -u -r15727fba2560b07c19aa85591b35791515be1a8e -rb36ef67f811fceb8dfd0d6371d5c715268afd986 --- lams_bb_integration/RELEASE_NOTES.TXT (.../RELEASE_NOTES.TXT) (revision 15727fba2560b07c19aa85591b35791515be1a8e) +++ lams_bb_integration/RELEASE_NOTES.TXT (.../RELEASE_NOTES.TXT) (revision b36ef67f811fceb8dfd0d6371d5c715268afd986) @@ -133,3 +133,4 @@ * LDEV-3927: Fixed a FF bug: "Open Lesson" not open in firefox browser for the second time after closing the opened window * LDEV-3510: Remove previously added line that made LAMS Lessons always appear at top of content list * LDEV-3958: Add a button to admin page that can fix grade center columns +* LDEV-4080: Blackboard sends _course_id instead of courseId string to LAMS on creating new lessons Index: lams_bb_integration/src/org/lamsfoundation/ld/integration/util/BlackboardUtil.java =================================================================== diff -u -r58494a5f77b6880147f0e079241abc4c535cdb6e -rb36ef67f811fceb8dfd0d6371d5c715268afd986 --- lams_bb_integration/src/org/lamsfoundation/ld/integration/util/BlackboardUtil.java (.../BlackboardUtil.java) (revision 58494a5f77b6880147f0e079241abc4c535cdb6e) +++ lams_bb_integration/src/org/lamsfoundation/ld/integration/util/BlackboardUtil.java (.../BlackboardUtil.java) (revision b36ef67f811fceb8dfd0d6371d5c715268afd986) @@ -249,11 +249,11 @@ // create a new thread to pre-add students and monitors to a lesson (in order to do this task in parallel not to // slow down later work) final User userFinal = user; - final String courseIdStrFinal = _course_id; + final Course courseFinal = course; Thread preaddLearnersMonitorsThread = new Thread(new Runnable() { @Override public void run() { - LamsSecurityUtil.preaddLearnersMonitorsToLesson(userFinal, courseIdStrFinal, LamsLessonIdLong); + LamsSecurityUtil.preaddLearnersMonitorsToLesson(userFinal, courseFinal, LamsLessonIdLong); } }, "LAMS_preaddLearnersMonitors_thread"); preaddLearnersMonitorsThread.start(); Index: lams_bb_integration/src/org/lamsfoundation/ld/integration/util/LamsSecurityUtil.java =================================================================== diff -u -r4cbb8f0a143a5ceed24a3de664315b501a8cbbdf -rb36ef67f811fceb8dfd0d6371d5c715268afd986 --- lams_bb_integration/src/org/lamsfoundation/ld/integration/util/LamsSecurityUtil.java (.../LamsSecurityUtil.java) (revision 4cbb8f0a143a5ceed24a3de664315b501a8cbbdf) +++ lams_bb_integration/src/org/lamsfoundation/ld/integration/util/LamsSecurityUtil.java (.../LamsSecurityUtil.java) (revision b36ef67f811fceb8dfd0d6371d5c715268afd986) @@ -664,7 +664,7 @@ * @param lessonId * the lesoon id that was just started */ - public static void preaddLearnersMonitorsToLesson(User user, String courseIdStr, long lessonId) { + public static void preaddLearnersMonitorsToLesson(User user, Course course, long lessonId) { String serverId = getServerID(); String serverAddr = getServerAddress(); String serverKey = getServerKey(); @@ -693,7 +693,7 @@ BbPersistenceManager bbPm = PersistenceServiceFactory.getInstance().getDbPersistenceManager(); CourseMembershipDbLoader courseMemLoader = CourseMembershipDbLoader.Default.getInstance(); - Id courseId = bbPm.generateId(Course.DATA_TYPE, courseIdStr); + Id courseId = course.getId(); BbList studentCourseMemberships = courseMemLoader.loadByCourseIdAndRole(courseId, CourseMembership.Role.STUDENT, null, true); for (CourseMembership courseMembership : studentCourseMemberships) { @@ -754,7 +754,7 @@ String serviceURL = serverAddr + "/services/xml/LessonManager?" + "&serverId=" + URLEncoder.encode(serverId, "utf8") + "&datetime=" + timestamp + "&username=" + URLEncoder.encode(username, "utf8") + "&hashValue=" + hash + "&courseId=" - + URLEncoder.encode(courseIdStr, "utf8") + "&lsId=" + lessonId + "&country=" + country + "&lang=" + + URLEncoder.encode(course.getCourseId(), "utf8") + "&lsId=" + lessonId + "&country=" + country + "&lang=" + lang + "&method=join" + "&firstNames=" + firstNames + "&lastNames=" + lastNames + "&emails=" + emails; if (!monitorIds.isEmpty()) {