Merge pull request #249 from openHPI/fix_role_via_LTI

Fix usage of incorrect downcase method
2018-12-20 14:53:44 +01:00
parent ac7e41d8b3 9087d42088
commit 3a4b359e91
1 changed files with 1 additions and 1 deletions
--- a/app/controllers/concerns/lti.rb
+++ b/app/controllers/concerns/lti.rb
@ -56,7 +56,7 @@ module Lti
  def external_user_role(provider)
    result = 'learner'
    provider.roles.each do |role|
-      case role.downcase!
+      case role.downcase
      when 'administrator'
        # We don't want anyone to get admin privileges through LTI
        result = 'teacher' if result == 'learner'