Index: lams_central/src/java/org/lamsfoundation/lams/web/LoginRequestLtiServlet.java =================================================================== diff -u -r0bb35103dafb599b99c0284f48d4058b5d106ea0 -r4d916717581aaac21676d79af6d9c9c53d92787d --- lams_central/src/java/org/lamsfoundation/lams/web/LoginRequestLtiServlet.java (.../LoginRequestLtiServlet.java) (revision 0bb35103dafb599b99c0284f48d4058b5d106ea0) +++ lams_central/src/java/org/lamsfoundation/lams/web/LoginRequestLtiServlet.java (.../LoginRequestLtiServlet.java) (revision 4d916717581aaac21676d79af6d9c9c53d92787d) @@ -129,18 +129,14 @@ } ExtServerLessonMap lesson = integrationService.getLtiConsumerLesson(consumerKey, resourceLinkId); - //Determine method based on role parameter. Monitor roles can be either LTI standard ones or tool consumer's custom ones set + //Determine method based on the "role" parameter. Author roles can be either LTI standard ones or tool consumer's custom ones set //In case of ContentItemSelectionRequest user must still be a stuff member in order to create a lesson. - String method; boolean isCustomMonitorRole = LtiUtils.isToolConsumerCustomRole(roles, extServer.getLtiToolConsumerMonitorRoles()); - if (LtiUtils.isStaff(roles) || LtiUtils.isAdmin(roles) || isCustomMonitorRole) { - method = (lesson == null) ? LoginRequestDispatcher.METHOD_AUTHOR : LoginRequestDispatcher.METHOD_MONITOR; + String method = LtiUtils.isStaff(roles) || LtiUtils.isAdmin(roles) || isCustomMonitorRole + ? LoginRequestDispatcher.METHOD_AUTHOR + : LoginRequestDispatcher.METHOD_LEARNER_STRICT_AUTHENTICATION; - } else { - method = LoginRequestDispatcher.METHOD_LEARNER_STRICT_AUTHENTICATION; - } - //provide empty lessonId in case of learner accesses LTI link before teacher authored it String lessonId = lesson == null ? "" : lesson.getLessonId().toString();