实训作业列表的调整

app/controllers/homework_commons_controller.rb

@@ -167,12 +167,7 @@ class HomeworkCommonsController < ApplicationController
         if params[:work_status].present?
           params_work_status = params[:work_status]
           work_status = params_work_status.map{|status| status.to_i}
-          all_student_works = @student_works.left_joins(:myshixun)
+          @student_works = @student_works.where(compelete_status: work_status)
-          @student_works = all_student_works.where(work_status: work_status) if work_status.include?(0)
-          @student_works = @student_works.or(all_student_works.where(work_status: 0)).or(all_student_works.where(myshixuns: {status: 0})) if work_status.include?(1)
-
-          @student_works = @student_works.or(all_student_works.where(work_status: 0)).or(all_student_works.where(myshixuns: {status: 0})) if work_status.include?(3)
-          @student_works = @student_works.or(all_student_works.where(myshixuns: {status: 1})) if work_status.include?(4)
         end
 
         # 分班情况

app/controllers/student_works_controller.rb

@@ -739,7 +739,8 @@ class StudentWorksController < ApplicationController
                                                                comment: comment)
           challenge_score.create_tiding current_user.id
           if @work.work_status != 0 && @work.myshixun
-            HomeworksService.new.update_myshixun_work_score @work, @work.myshixun, @work.myshixun&.games, @homework, @homework.homework_challenge_settings
+            games = @work.myshixun.games.where(challenge_id: @homework.homework_challenge_settings.pluck(:challenge_id))
+            HomeworksService.new.update_myshixun_work_score @work, @work.myshixun, games, @homework, @homework.homework_challenge_settings
           else
             update_none_commit_work @work, @homework
           end

app/helpers/application_helper.rb

@@ -210,7 +210,7 @@ module ApplicationHelper
   # 普通/分组 作业作品状态数组
   def student_work_status homework, user_id, course, work
     status = []
-    homework_setting = homework.homework_group_setting user_id
+    homework_setting = homework.homework_group_setting user_id, true
     work = work || StudentWork.create(homework_common_id: homework.id, user_id: user_id)
     late_time = homework.late_time || course.end_date
 

app/helpers/homework_commons_helper.rb

@@ -150,7 +150,7 @@ module HomeworkCommonsHelper
 
   # 阶段剩余时间
   def left_time homework, user_id
-    setting = homework.homework_group_setting(user_id)
+    setting = homework.homework_group_setting(user_id, true)
     if setting.publish_time && setting.publish_time < Time.now
       if setting.end_time > Time.now
         status = "剩余提交时间"
@@ -230,10 +230,10 @@ module HomeworkCommonsHelper
 
   # 作品状态
   def practice_homework_status homework, member
-    [{id: 3, name: "未通关", count: homework.un_complete_count(member)},
+    [{id: 0, name: "未提交", count: homework.compelete_status_count(member, 0)},
-     {id: 4, name: "已通关", count: homework.complete_count(member)},
+     {id: 1, name: "未通关", count: homework.compelete_status_count(member, 1)},
-     {id: 1, name: "按时完成", count: homework.finished_count(member)},
+     {id: 2, name: "按时通关", count: homework.compelete_status_count(member, 2)},
-     {id: 2, name: "延时完成", count: homework.delay_finished_count(member)}]
+     {id: 3, name: "迟交通关", count: homework.compelete_status_count(member, 3)}]
   end
 
   # 作品状态

app/models/homework_common.rb

@@ -108,7 +108,7 @@ class HomeworkCommon < ApplicationRecord
 
   # 是否在补交阶段内
   def late_duration
-    homework_setting = self.homework_group_setting(User.current.id)
+    homework_setting = self.homework_group_setting(User.current.id, true)
     !course.is_end && self.publish_time && self.publish_time < Time.now && homework_setting.end_time &&
         homework_setting.end_time < Time.now && self.allow_late && (self.late_time.nil? || self.late_time > Time.now)
   end
@@ -119,7 +119,7 @@ class HomeworkCommon < ApplicationRecord
     if self.course.is_end || (self.allow_late && self.late_time && self.late_time < Time.now)
       status = true
     elsif !self.allow_late
-      homework_setting = self.homework_group_setting(User.current.id)
+      homework_setting = self.homework_group_setting(User.current.id, true)
       status = homework_setting.end_time && homework_setting.end_time < Time.now
     end
     status
@@ -241,14 +241,8 @@ class HomeworkCommon < ApplicationRecord
     self.teacher_works(member).delay_finished.count
   end
 
-  # 未通关数
+  def compelete_status_count member, status
-  def un_complete_count member
+    teacher_works(member).where(compelete_status: status).count
-    teacher_works(member).count - complete_count(member)
-  end
-
-  # 通关数
-  def complete_count member
-    Myshixun.where(id: self.teacher_works(member).pluck(:myshixun_id), status: 1).count
   end
 
   # 分组作业的最大分组id
@@ -257,12 +251,13 @@ class HomeworkCommon < ApplicationRecord
   end
 
   # 作业的分班设置时间
-  def homework_group_setting user_id
+  def homework_group_setting user_id, current_user=false
     if unified_setting
       homework_setting = self
     else
-      member = course.course_member(user_id)
+      # 当前用户是从course_member中取，否则是从学生中取（双重身份的原因）
-      group_setting = self.homework_group_settings.find_by_course_group_id(member.try(:course_group_id))
+      member = current_user ? course.course_member(user_id) : course.students.find_by(user_id: user_id)
+      group_setting = self.homework_group_settings.select{ |setting| setting.course_group_id == member.try(:course_group_id)}.first
       homework_setting = group_setting.present? ? group_setting : self
     end
     homework_setting

app/services/homeworks_service.rb

@@ -322,11 +322,16 @@ class HomeworksService
         work.efficiency = format("%.2f", efficiency)
 
         if myshixun_endtime <= homework_end_or_late_time
-          work.compelete_status = myshixun_endtime < setting_time.publish_time ? 2 : 1
+          # 2是按时通关， 3是迟交通关
+          work.compelete_status = myshixun_endtime < setting_time.end_time ? 2 : 3
 
           # 如果作业的最大效率值有变更则更新所有作品的效率分
           homework.update_column("max_efficiency", work.efficiency) if homework.work_efficiency && homework.max_efficiency < work.efficiency
+        else
+          work.compelete_status = 1 # 未通关
         end
+      else
+        work.compelete_status = 1 # 未通关
       end
 
       work.late_penalty = work.work_status == 2 ? homework.late_penalty : 0

app/views/homework_commons/works_list.json.jbuilder

@@ -44,7 +44,7 @@ elsif @user_course_identity == Course::STUDENT
     json.current_complete_count @work.myshixun.try(:passed_count) if @homework.end_or_late
     json.complete_count @work.myshixun&.time_passed_count(@homework.homework_group_setting(@work.user_id)&.end_time)
     json.view_answer_count @work.myshixun.try(:view_answer_count)
-    json.work_status @work.real_work_status
+    json.work_status @work.compelete_status
   else
     json.(@work, :id, :work_status, :update_time, :ultimate_score)
 
@@ -105,7 +105,7 @@ if @homework.homework_type == "practice"
     json.user_name work.user.try(:real_name)
     json.student_id work.user.try(:student_id)
     json.group_name @students.select{|student| student.user_id == work.user_id}.first.try(:course_group_name)
-    json.work_status work.real_work_status
+    json.work_status work.compelete_status
   end
 elsif @homework.homework_type == "group" || @homework.homework_type == "normal"
   json.anonymous_comment @homework.anonymous_comment

db/migrate/20191213101853_migrate_student_work_compelete_status.rb

@@ -1,5 +1,25 @@
 class MigrateStudentWorkCompeleteStatus < ActiveRecord::Migration[5.2]
   def change
-
+    student_works = StudentWork.where("myshixun_id is not null and myshixun_id != 0")
+    student_works.includes(homework_common: :homework_challenge_settings, myshixun: :games).find_each do |work|
+      if work.myshixun
+        myshixun = work.myshixun
+        homework= work.homework_common
+        setting_time = homework.homework_group_setting myshixun.user_id
+        homework_end_or_late_time = homework.allow_late ? homework.late_time : setting_time.end_time
+        challenge_ids = homework.homework_challenge_settings.pluck(:challenge_id)
+        games = myshixun.games.select{ |game| challenge_ids.include?(game.challenge_id) }
+        myshixun_endtime = games.select{|game| game.status == 2}.size == games.size ? games.map(&:end_time).max : nil
+        if work.work_status != 0 && homework_end_or_late_time.present?
+          if myshixun_endtime.present? && myshixun_endtime <= homework_end_or_late_time
+            # 2是按时通关， 3是迟交通关
+            compelete_status = setting_time.end_time.present? && myshixun_endtime < setting_time.end_time ? 2 : 3
+          else
+            compelete_status = 1 # 未通关
+          end
+          work.update_column("compelete_status", compelete_status)
+        end
+      end
+    end
   end
 end