Index: lams_bb_integration/src/org/lamsfoundation/ld/integration/blackboard/LamsSecurityUtil.java =================================================================== diff -u -r0afe65522880b0c7c8c4c19162c1ff35b371e4a9 -r95c0d30771017e8a809cf954c5235e486092192a --- lams_bb_integration/src/org/lamsfoundation/ld/integration/blackboard/LamsSecurityUtil.java (.../LamsSecurityUtil.java) (revision 0afe65522880b0c7c8c4c19162c1ff35b371e4a9) +++ lams_bb_integration/src/org/lamsfoundation/ld/integration/blackboard/LamsSecurityUtil.java (.../LamsSecurityUtil.java) (revision 95c0d30771017e8a809cf954c5235e486092192a) @@ -296,16 +296,17 @@ String lang = getLanguage(locale); String method = (isPreview) ? "preview" : "start"; - // courseId not needed for preview but pass it through if we have it. + // courseId aways needed to check roles + // if it is preview, then can use the DUMMY_COURSE String courseId = null; if ( isPreview ) - courseId = ctx.getCourse() != null ? ctx.getCourse().getCourseId() : null; + courseId = ctx.getCourse()!=null ? ctx.getCourse().getCourseId() : DUMMY_COURSE; else courseId = ctx.getCourse().getCourseId(); - if (serverId == null || serverAddr == null || serverKey == null) { - logger.info("Unable to start lesson, one or more lams configuration properties is null"); - throw new RuntimeException("Unable to start lesson, one or more lams configuration properties is null"); + if (courseId == null || serverId == null || serverAddr == null || serverKey == null) { + logger.info("Unable to start lesson, one or more lams configuration properties or the course id is null"); + throw new RuntimeException("Unable to start lesson, one or more lams configuration properties or the course id is null"); } try {