Distinguish between intermediate and final feedback

* Also add more information to user_exercise_feedback
* Migrate existing feedback to enrich with submissions

app/controllers/exercises_controller.rb

@@ -617,7 +617,7 @@ class ExercisesController < ApplicationController
       end
     else
       # redirect to feedback page if score is less than 100 percent
-      if @exercise.needs_more_feedback? && !@embed_options[:disable_redirect_to_feedback]
+      if @exercise.needs_more_feedback?(@submission) && !@embed_options[:disable_redirect_to_feedback]
         clear_lti_session_data(@submission.exercise_id, @submission.user_id, session[:consumer_id])
         redirect_to_user_feedback
       else

app/controllers/user_exercise_feedbacks_controller.rb

@@ -97,7 +97,26 @@ class UserExerciseFeedbacksController < ApplicationController
   end
 
   def uef_params
-    params[:user_exercise_feedback].permit(:feedback_text, :difficulty, :exercise_id, :user_estimated_worktime).merge(user_id: current_user.id, user_type: current_user.class.name) if params[:user_exercise_feedback].present?
+    return unless params[:user_exercise_feedback].present?
+
+    exercise_id = if params[:user_exercise_feedback].nil?
+                    params[:exercise_id]
+                  else
+                    params[:user_exercise_feedback][:exercise_id]
+                  end
+
+    user_id = current_user.id
+    user_type = current_user.class.name
+    latest_submission = Submission
+                        .where(user_id: user_id, user_type: user_type, exercise_id: exercise_id)
+                        .order(created_at: :desc).first
+
+    params[:user_exercise_feedback]
+      .permit(:feedback_text, :difficulty, :exercise_id, :user_estimated_worktime)
+      .merge(user_id: user_id,
+             user_type: user_type,
+             submission: latest_submission,
+             normalized_score: latest_submission.normalized_score)
   end
 
   def validate_inputs(uef_params)

app/models/exercise.rb

@@ -46,7 +46,7 @@ class Exercise < ApplicationRecord
   @working_time_statistics = nil
   attr_reader :working_time_statistics
 
-  MAX_EXERCISE_FEEDBACKS = 20
+  MAX_GROUP_EXERCISE_FEEDBACKS = 20
 
   def average_percentage
     if average_score && (maximum_score != 0.0) && submissions.exists?(cause: 'submit')
@@ -550,8 +550,12 @@ class Exercise < ApplicationRecord
   end
   private :valid_submission_deadlines?
 
-  def needs_more_feedback?
-    user_exercise_feedbacks.size <= MAX_EXERCISE_FEEDBACKS
+  def needs_more_feedback?(submission)
+    if submission.normalized_score == 1.00
+      user_exercise_feedbacks.final.size <= MAX_GROUP_EXERCISE_FEEDBACKS
+    else
+      user_exercise_feedbacks.intermediate.size <= MAX_GROUP_EXERCISE_FEEDBACKS
+    end
   end
 
   def last_submission_per_user

app/models/submission.rb

@@ -101,9 +101,9 @@ class Submission < ApplicationRecord
   end
 
   def redirect_to_feedback?
-    # Redirect 10% of users to the exercise feedback page. Ensure, that always the
-    # same users get redirected per exercise and different users for different exercises.
-    # If desired, the number of feedbacks can be limited with exercise.needs_more_feedback?
+    # Redirect 10% of users to the exercise feedback page. Ensure, that always the same
+    # users get redirected per exercise and different users for different exercises. If
+    # desired, the number of feedbacks can be limited with exercise.needs_more_feedback?(submission)
     (user_id + exercise.created_at.to_i) % 10 == 1
   end
 

app/models/user_exercise_feedback.rb

@@ -2,10 +2,14 @@ class UserExerciseFeedback < ApplicationRecord
   include Creation
 
   belongs_to :exercise
+  belongs_to :submission, optional: true
   has_one :execution_environment, through: :exercise
 
   validates :user_id, uniqueness: { scope: [:exercise_id, :user_type] }
 
+  scope :intermediate, -> { where.not(normalized_score: 1.00) }
+  scope :final, -> { where(normalized_score: 1.00) }
+
   def to_s
     "User Exercise Feedback"
   end