Merge pull request #109 from openHPI/use_names_from_lti

restrict name to be set to lis_person_name_full (should be display na…
This commit is contained in:
rteusner
2017-08-28 16:25:26 +02:00
committed by GitHub
2 changed files with 6 additions and 8 deletions

View File

@@ -5,11 +5,9 @@ class ExternalUser < ActiveRecord::Base
validates :external_id, presence: true
def displayname
result = "User " + id.to_s
if(!consumer.nil? && consumer.name == 'openHPI')
result = Rails.cache.fetch("#{cache_key}/displayname", expires_in: 12.hours) do
Xikolo::UserClient.get(external_id.to_s)[:display_name]
end
result = name
if(result == nil || result == "")
result = "User " + id.to_s
end
result
end