E2023 2 Track the time students look at other submissions

app/controllers/response_controller.rb

@@ -105,7 +105,11 @@ def edit
   # Update the response and answers when student "edit" existing response
   def update
     render nothing: true unless action_allowed?
-
+
+    # the response to be updated
+    @response = Response.find(params[:id])
+    @current_round = @response.round
+
     msg = ""
     begin
       # the response to be updated
@@ -330,6 +334,7 @@ def assign_instance_vars
       @header = 'Edit'
       @next_action = 'update'
       @response = Response.find(params[:id])
+      @current_round = @response.round
       @map = @response.map
       @contributor = @map.contributor
     when 'new'

app/controllers/submission_viewing_events_controller.rb

@@ -0,0 +1,75 @@
+class SubmissionViewingEventsController < ApplicationController
+  def action_allowed?
+    true
+  end
+
+  # record time when link or file is opened in new window
+  def record_start_time
+    param_args = params[:submission_viewing_event]
+    # check if this link is already opened and timed
+    submission_viewing_event_records = SubmissionViewingEvent.where(map_id: param_args[:map_id], round: param_args[:round], link: param_args[:link])
+    # if opened, end these records with current time
+    if submission_viewing_event_records
+      submission_viewing_event_records.each do |time_record|
+        if time_record.end_at.nil?
+          # time_record.update_attribute('end_at', start_at)
+          time_record.destroy
+        end
+      end
+    end
+    # create new response time record for current link
+    submission_viewing_event = SubmissionViewingEvent.new(submission_viewing_event_params)
+    submission_viewing_event.save
+
+    #if creating start time for expertiza update end times for all other links.
+    if param_args[:link]=='Expertiza Review'
+      params[:submission_viewing_event][:link] = nil
+      params[:submission_viewing_event][:end_at] = params[:submission_viewing_event][:start_at]
+      record_end_time()
+    end
+    render nothing: true
+  end
+
+  # record time when link or file window is closed
+  def record_end_time
+    data = params.require(:submission_viewing_event)
+    if data[:link].nil?
+      submission_viewing_event_records = SubmissionViewingEvent.where(map_id: data[:map_id], round: data[:round], end_at: nil).where.not(link: "Expertiza Review")
+    else
+      submission_viewing_event_records = SubmissionViewingEvent.where(map_id: data[:map_id], round: data[:round], link: data[:link])
+    end
+    submission_viewing_event_records.each do |time_record|
+      if time_record.end_at.nil?
+        time_record.update_attribute('end_at', data[:end_at])
+        # break
+      end
+    end
+    respond_to do |format|
+      format.json { head :no_content }
+    end
+  end
+
+  # mark end_at review time for all uncommited links/files
+  def mark_end_time
+    data = params.require(:submission_viewing_event)
+    @link_array = []
+    submission_viewing_event_records = SubmissionViewingEvent.where(map_id: data[:map_id], round: data[:round])
+    submission_viewing_event_records.each do |submissionviewingevent_entry|
+      if submissionviewingevent_entry.end_at.nil?
+        @link_array.push(submissionviewingevent_entry.link)
+        submissionviewingevent_entry.update_attribute('end_at', data[:end_at])
+      end
+    end
+    respond_to do |format|
+      format.json { render json: @link_array }
+    end
+  end
+
+  private
+
+  # Only allow a trusted parameter "white list" through.
+  def submission_viewing_event_params
+    params.require(:submission_viewing_event).permit(:map_id, :round, :link, :start_at, :end_at)
+  end
+
+end

app/controllers/tree_display_controller.rb

@@ -213,10 +213,14 @@ def children_node_ng
     flash[:error] = "Invalid JSON in the TreeList" unless json_valid? params[:reactParams][:child_nodes]
     child_nodes = child_nodes_from_params(params[:reactParams][:child_nodes])
     tmp_res = {}
-    unless child_nodes.blank?
+
+    begin
       child_nodes.each do |node|
         initialize_fnode_update_children(params, node, tmp_res)
       end
+      flash[:error] = "Invalid child nodes in the TreeList"
+    rescue
+
     end
     res = res_node_for_child(tmp_res)
     res['Assignments'] = res['Assignments'].sort_by {|x| [x['instructor'], -1 * x['creation_date'].to_i] } if res.key?('Assignments')

app/helpers/submission_viewing_events_helper.rb

@@ -0,0 +1,2 @@
+module SubmissionViewingEventsHelper
+end

app/helpers/submitted_content_helper.rb

@@ -24,11 +24,13 @@ def display_directory_tree(participant, files, display_to_reviewer_flag)
           ret += link_to File.basename(file), :controller => 'submitted_content', :action => 'edit', :id => participant.id, "current_folder[name]" => file
         else
           ret += "\n      "
-          ret += link_to File.basename(file), :controller => 'submitted_content',
-                                              :action => 'download',
-                                              :id => participant.id,
-                                              :download => File.basename(file),
-                                              "current_folder[name]" => File.dirname(file)
+          ret += link_to File.basename(file), {:controller => 'submitted_content',
+                                               :action => 'download',
+                                               :id => participant.id,
+                                               :download => File.basename(file),
+                                               "current_folder[name]" => File.dirname(file)},
+                                               :class => "fileLink",
+                                               :download => File.basename(file).to_s
         end
         ret += "\n   </td>\n   <td valign = top>\n"
         ret += File.size(file).to_s

app/models/submission_viewing_event.rb

@@ -0,0 +1,2 @@
+class SubmissionViewingEvent < ActiveRecord::Base
+end

app/views/popup/view_review_scores_popup.html.erb

@@ -52,8 +52,9 @@
                       <% answer = Answer.where(response_id: response_id, question_id: question.id).first %>
                       <% if !answer.nil? %>
                           <tr>
-                            <% team = Team.find(ResponseMap.find(Response.find(response_id).map_id).reviewee_id) %>
-                            <% #TODO does this one need reviewer fixed? %>>
+                            <% index = ResponseMap.find(Response.find(response_id).map_id).reviewee_id %>
+                            <% team = Team.find(index)%>
+
                             <% response_map = ReviewResponseMap.where(reviewed_object_id: @assignment_id,
                                                                       reviewer_id: @reviewer_id,
                                                                       reviewee_id: team.id).last %>

app/views/reports/_review_report.html.erb

@@ -1,3 +1,4 @@
+<% @l = -1 %>
 <!-- the table creation is changed top accommodate functionality of tablesorter, which requires thead and tbody seperately. Previously, headers were in tbody only -->
 <div class='reviewreport'>
   <!-- Added headers for tablesorter -->
@@ -48,7 +49,7 @@
     </tr>
     </thead>
     <tbody>
-
+    <% timeHash = Hash.new %>
     <% sort_reviewer_by_review_volume_desc.each do |r| %>
       <% @response_maps, @rspan = get_data_for_review_report(@id, r.id, @type) %>
       <tr>
@@ -98,6 +99,9 @@
                   </div>
                 </div>
 
+                  <!--Display total time spent on reviewing as a hyperlink to popup which shows the breakdown of time spent between expertiza and other links as piechart-->
+                  <%= render partial: 'review_submissions_time_spent',  locals: {bgcolor: @bgcolor, map_id: ri.id,id: r.id, reviewee_id: ri.reviewee_id, timeHash: timeHash} %>
+
                 <!--Author feedback-->
                 <!--Dr.Kidd required to add a "author feedback" column that shows the average score the reviewers received from the authors. In this case, she can see all the data on a single screen.-->
                 <!--Dr.Kidd's course-->