Set bonus points for survey via LTI parameter

openHPI · Oct 13, 2023 · 5f764f0 · 5f764f0
1 parent a258e2a
commit 5f764f0
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/app/controllers/sessions_controller.rb b/app/controllers/sessions_controller.rb
@@ -72,14 +72,17 @@ def destroy
   private
 
   def redirect_to_survey
-    # The following code is taken from store_lti_session_data(params) & send_score_for(submission, user)
-    # It gives a bonus point to users who opened the survey
-    begin
-      lti_parameters = params.slice(*Lti::SESSION_PARAMETERS).permit!.to_h
-      provider = build_tool_provider(consumer: current_user.consumer, parameters: lti_parameters)
-      provider.post_replace_result!(1.0)
-    rescue IMS::LTI::XMLParseError, Net::OpenTimeout, Net::ReadTimeout, Errno::ECONNRESET, SocketError, EOFError
-      # We don't do anything here because it is only a bonus point and we want the users to do the survey
+
+    if params[:custom_bonus_points]
+      # The following code is taken from store_lti_session_data(params) & send_score_for(submission, user)
+      # It gives a bonus point to users who opened the survey
+      begin
+        lti_parameters = params.slice(*Lti::SESSION_PARAMETERS).permit!.to_h
+        provider = build_tool_provider(consumer: current_user.consumer, parameters: lti_parameters)
+        provider.post_replace_result!(params[:custom_bonus_points])
+      rescue IMS::LTI::XMLParseError, Net::OpenTimeout, Net::ReadTimeout, Errno::ECONNRESET, SocketError, EOFError
+        # We don't do anything here because it is only a bonus point and we want the users to do the survey
+      end
     end
 
     # This method is taken from Xikolo and slightly adapted.