htwkmooc/codeocean
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

only destroy LTI_parameter when exercise is submitted

Browse Source
This commit is contained in:
Tom Staubitz
2017-01-12 16:03:30 +01:00
parent 43f1735fa0
commit e4daa7ece3
1 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
app/controllers/concerns/lti.rb
View File

@ -19,12 +19,12 @@ module Lti
# exercise_id.exists? ==> the user has submitted the results of an exercise to the consumer. # exercise_id.exists? ==> the user has submitted the results of an exercise to the consumer.
# Only the lti_parameters are deleted. # Only the lti_parameters are deleted.
def clear_lti_session_data(exercise_id = nil) def clear_lti_session_data(exercise_id = nil)
@current_user = ExternalUser.find(@submission.user_id)
if (exercise_id.nil?) if (exercise_id.nil?)
LtiParameter.destroy_all(consumers_id: session[:consumer_id], external_user_id: @current_user.external_id)
session.delete(:consumer_id) session.delete(:consumer_id)
session.delete(:external_user_id) session.delete(:external_user_id)
else else
LtiParameter.destroy_all(consumers_id: session[:consumer_id], LtiParameter.destroy_all(consumers_id: @consumer.id,
external_user_id: @current_user.external_id, external_user_id: @current_user.external_id,
exercises_id: exercise_id) exercises_id: exercise_id)
end end
@ -123,6 +123,7 @@ module Lti
def set_current_user def set_current_user
@current_user = ExternalUser.find_or_create_by(consumer_id: @consumer.id, external_id: @provider.user_id) @current_user = ExternalUser.find_or_create_by(consumer_id: @consumer.id, external_id: @provider.user_id)
#TODO is this really necessary, and does it work at all?
@current_user.update(email: external_user_email(@provider), name: external_user_name(@provider)) @current_user.update(email: external_user_email(@provider), name: external_user_name(@provider))
end end
private :set_current_user private :set_current_user