Index: lams_central/src/java/org/lamsfoundation/lams/webservice/xml/LessonManagerServlet.java =================================================================== diff -u -rda98623387cff04c7e30d157e8d37da7ac128de4 -r91188776dc087a53091f168f5c7105aee236c0bf --- lams_central/src/java/org/lamsfoundation/lams/webservice/xml/LessonManagerServlet.java (.../LessonManagerServlet.java) (revision da98623387cff04c7e30d157e8d37da7ac128de4) +++ lams_central/src/java/org/lamsfoundation/lams/webservice/xml/LessonManagerServlet.java (.../LessonManagerServlet.java) (revision 91188776dc087a53091f168f5c7105aee236c0bf) @@ -26,6 +26,7 @@ import javax.xml.transform.dom.DOMSource; import javax.xml.transform.stream.StreamResult; +import org.apache.commons.lang.StringUtils; import org.apache.log4j.Logger; import org.lamsfoundation.lams.integration.ExtCourseClassMap; import org.lamsfoundation.lams.integration.ExtServerOrgMap; @@ -194,11 +195,7 @@ Thread t = new Thread(new AddUsersToLessonThread(serverId, datetime, username, hashValue, lsIdStr, courseId, country, lang, learnerIds, monitorIds, request)); t.start(); - /* - Boolean added = addUsersToLesson(serverId, datetime, username, hashValue, - lsIdStr, courseId, country, lang, learnerIds, monitorIds, request); - log.debug("addUsersToLesson returned boolean value of: " + added); - */ + element = document.createElement(CentralConstants.ELEM_LESSON); element.setAttribute(CentralConstants.ATTR_LESSON_ID, lsIdStr); @@ -691,15 +688,19 @@ if (learnerIds != null) { String[] learnerIdArray = learnerIds.split(","); for (String learnerId : learnerIdArray) { - addUserToLesson(request, serverId, datetime, requestorUsername, hashValue, + if (StringUtils.isNotBlank(learnerId)) { + addUserToLesson(request, serverId, datetime, requestorUsername, hashValue, LoginRequestDispatcher.METHOD_LEARNER, lsIdStr, learnerId, courseId, countryIsoCode, langIsoCode); + } } } if (monitorIds != null) { String[] monitorIdArray = monitorIds.split(","); for (String monitorId : monitorIdArray) { - addUserToLesson(request, serverId, datetime, requestorUsername, hashValue, + if (StringUtils.isNotBlank(monitorId)) { + addUserToLesson(request, serverId, datetime, requestorUsername, hashValue, LoginRequestDispatcher.METHOD_MONITOR, lsIdStr, monitorId, courseId, countryIsoCode, langIsoCode); + } } } return true;