From 67af04f82d10e4284f9fd4fe86518ba2728952e4 Mon Sep 17 00:00:00 2001 From: Sebastian Serth Date: Mon, 3 Feb 2020 11:36:42 +0100 Subject: [PATCH] Revert "Prevent updating the user role for LTI launches" This reverts commit d9770396 --- app/controllers/concerns/lti.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/controllers/concerns/lti.rb b/app/controllers/concerns/lti.rb index a2fedcd5..f7bc62ac 100644 --- a/app/controllers/concerns/lti.rb +++ b/app/controllers/concerns/lti.rb @@ -155,7 +155,7 @@ module Lti internal_role = @current_user.role internal_role != 'admin' ? desired_role = external_role : desired_role = internal_role # Update user with new information but change the role only if he is no admin user - @current_user.update(email: external_user_email(@provider), name: external_user_name(@provider))# TODO , role: desired_role) + @current_user.update(email: external_user_email(@provider), name: external_user_name(@provider), role: desired_role) end private :set_current_user