Merge branch 'dev_aliyun' of http://bdgit.educoder.net/Hjqreturn/educoder into dev_aliyun

app/controllers/exercise_questions_controller.rb

@@ -223,7 +223,6 @@ class ExerciseQuestionsController < ApplicationController
           end
         end
         #试卷未发布时，当标准答案存在时，可修改标准答案内容，可增删标准答案，否则只能修改标准答案，不能增删标准答案
-        st_count = 0
         @exercise_answers_array = @exercise_question.exercise_standard_answers  #问卷的全部标准答案
         if standard_answer.present?
           if @exercise_question.question_type <= 2 #选择题/判断题，标准答案为一个或多个
@@ -232,7 +231,6 @@ class ExerciseQuestionsController < ApplicationController
             old_left_standard_choices = exercise_standard_choices - common_standard_choices # 以前的差集共同的，剩余的表示需要删掉
             new_left_standard_choices = standard_answer - common_standard_choices # 传入的标准答案差集共同的，剩余的表示需要新建
             if old_left_standard_choices.count > 0
-              st_count += 1
               @exercise_answers_array.standard_by_ids(old_left_standard_choices).destroy_all
             end
             if new_left_standard_choices.count > 0  #新建标准答案
@@ -258,7 +256,6 @@ class ExerciseQuestionsController < ApplicationController
 
             #删除多余的选项
             if old_ex_answer_choice_ids.count > new_ex_answer_choice_ids.count  #有减少的填空
-              st_count += 1
               delete_ex_answer_choice_ids = old_ex_answer_choice_ids - new_ex_answer_choice_ids
               old_ex_answer.standard_by_ids(delete_ex_answer_choice_ids).destroy_all
             end
@@ -284,7 +281,6 @@ class ExerciseQuestionsController < ApplicationController
                     ex_answer_pre[n-1].update(standard_option)
                   end
                   if new_add_choice.count > 0  #表示有新增的
-                    st_count += 1
                     new_add_choice.each do |i|
                       standard_option = {
                           :exercise_question_id => @exercise_question.id,
@@ -306,7 +302,6 @@ class ExerciseQuestionsController < ApplicationController
                     ex_answer_pre[index].update(standard_option)
                   end
                   if new_delete_choice.count > 0  #表示填空题的答案有删减的
-                    st_count += 1
                     new_delete_choice.each do |d|
                       ex_answer_pre[d-1].destroy
                     end
@@ -350,8 +345,9 @@ class ExerciseQuestionsController < ApplicationController
           @exercise_question.shixun_name = shixun_name
         end
 
-        #当标准答案修改时，如有已提交的学生，需重新计算分数
+        #当试卷已发布时（试卷的总状态),当标准答案修改时，如有已提交的学生，需重新计算分数.
-        if st_count > 0
+
+        if @exercise.exercise_status == 2
           ex_users_committed = @exercise.exercise_users.exercise_user_committed
           if ex_users_committed.size > 0
             ex_users_committed.each do |ex_user|
@@ -587,15 +583,15 @@ class ExerciseQuestionsController < ApplicationController
     if @exercise_question.present?
       @exercise = @exercise_question.exercise
       if @exercise.blank?
-        tip_exception(404)
+        tip_exception(404,"试卷不存在")
       else
         @course = @exercise.course
         if @course.blank?
-          tip_exception(404)
+          tip_exception(404,"课堂不存在")
         end
       end
     else
-      tip_exception(404)
+      tip_exception(404,"试卷问题不存在")
     end
   end
 

app/controllers/graduation_tasks_controller.rb

@@ -129,7 +129,7 @@ class GraduationTasksController < ApplicationController
         if @user_course_identity >= Course::STUDENT
           tip_exception(403, "无权限操作")
         elsif complete_works == 0
-          normal_status(-1,"暂无用户提交！")
+          normal_status(-1,"暂无用户提交")
         else
           respond_to do |format|
             format.xlsx{
@@ -164,7 +164,7 @@ class GraduationTasksController < ApplicationController
       @work_count = @work_list.count
       @all_work_count = @work_list.count
       if params[:format] == "xlsx" || params[:format] == "zip"
-        normal_status(-1,"作业未发布")
+        normal_status(-1,"毕设任务未发布")
       end
     end
   end

app/controllers/homework_commons_controller.rb

@@ -106,7 +106,7 @@ class HomeworkCommonsController < ApplicationController
 
     student_works = @homework.all_works
     @all_member_count = student_works.size
-    if @homework.publish_time.nil? || @homework.publish_time > Time.now
+    if @homework.publish_time.blank? || (@homework.publish_time > Time.now)
       @student_works = []
       if params[:format] == "xlsx" || params[:format] == "zip"
         normal_status(-1,"作业未发布")

app/controllers/myshixuns_controller.rb

@@ -18,7 +18,7 @@ class MyshixunsController < ApplicationController
   # REDO等删除是否可以做成异步
   # 前段需要按照操作过程提示
   def reset_my_game
-    unless (current_user.admin? || current_user.id == @myshixun.user_id)
+    unless current_user.admin?
       tip_exception("403", "")
     end
 

app/controllers/subjects_controller.rb

@@ -81,6 +81,9 @@ class SubjectsController < ApplicationController
     @tags = ChallengeTag.where(challenge_id: challenge_ids).pluck(:name).uniq
     # 用户获取的实训标签
     @user_tags = @subject.shixuns.map(&:user_tags_name).flatten.uniq
+
+    # 访问数变更
+    @subject.increment!(:visits)
   end
 
   def create

app/services/users/subject_service.rb

@@ -14,7 +14,7 @@ class Users::SubjectService
     subjects = category_scope_subjects
     subjects = user_policy_filter(subjects)
 
-    custom_sort(subjects, :updated_at, params[:sort_direction])
+    custom_sort(subjects.distinct, :updated_at, params[:sort_direction])
   end
 
   private

db/migrate/20190703072611_add_old_exercise_tiankong_choice_id.rb

@@ -0,0 +1,23 @@
+class AddOldExerciseTiankongChoiceId < ActiveRecord::Migration[5.2]
+
+  def up
+    #类型为3 的问题答案及标准答案更新exercise_choice_id 为1，即表示第一空
+    exercise_question_ids = ExerciseQuestion.where("question_type = 3").pluck(:id)
+    ExerciseAnswer.where(exercise_question_id: exercise_question_ids,exercise_choice_id: nil).update_all(exercise_choice_id:1)
+    ExerciseStandardAnswer.where(exercise_question_id: exercise_question_ids,exercise_choice_id: nil).update_all(exercise_choice_id:1)
+
+    exercise_bank_question_ids = ExerciseBankQuestion.where("question_type =3").pluck(:id)
+    ExerciseBankStandardAnswer.where(exercise_bank_question_id: exercise_bank_question_ids).update_all(exercise_bank_choice_id:1)
+  end
+
+  def down
+    #类型为3 的问题答案及标准答案更新exercise_choice_id 为1，即表示第一空
+    exercise_question_ids = ExerciseQuestion.where("question_type = 3").pluck(:id)
+    ExerciseAnswer.where(exercise_question_id: exercise_question_ids,exercise_choice_id: 1).update_all(exercise_choice_id:nil)
+    ExerciseStandardAnswer.where(exercise_question_id: exercise_question_ids,exercise_choice_id: 1).update_all(exercise_choice_id:nil)
+
+    exercise_bank_question_ids = ExerciseBankQuestion.where("question_type =3").pluck(:id)
+    ExerciseBankStandardAnswer.where(exercise_bank_question_id: exercise_bank_question_ids).update_all(exercise_bank_choice_id:nil)
+  end
+
+end