From da55b00da7f1ccaa5662eb01015bb36bd3f0f1bd Mon Sep 17 00:00:00 2001
From: huang <huang.jingquan@163.com>
Date: Fri, 20 Nov 2015 19:05:39 +0800
Subject: [PATCH] =?UTF-8?q?=E5=88=A0=E9=99=A4=E6=8F=92=E5=85=A5=E9=97=AE?=
 =?UTF-8?q?=E9=A2=98?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 app/controllers/exercise_controller.rb | 24 +++++++++++++++++++-----
 1 file changed, 19 insertions(+), 5 deletions(-)

diff --git a/app/controllers/exercise_controller.rb b/app/controllers/exercise_controller.rb
index 448c9a90a..9c4ce463a 100644
--- a/app/controllers/exercise_controller.rb
+++ b/app/controllers/exercise_controller.rb
@@ -269,11 +269,23 @@ class ExerciseController < ApplicationController
   def delete_exercise_question
     @exercise_question = ExerciseQuestion.find params[:exercise_question]
     @exercise = @exercise_question.exercise
-    exercise_questions = @exercise.exercise_questions.where("question_number > #{@exercise_question.question_number}")
-    exercise_questions.each do |question|
-      question.question_number -= 1
-      question.save
+
+    if @exercise_question.question_type == 1
+      ExerciseQuestion.where("question_number>? and question_type=?",params[:quest_num].to_i, 1).update_all(" question_number = question_number - 1")
+      #@exercise.exercise_questions.where("question_number > #{params[:quest_num].to_i} and question_type == 1").update_all(" question_number = question_number + 1")
+    elsif @exercise_question.question_type == 2
+      ExerciseQuestion.where("question_number>? and question_type=?",params[:quest_num].to_i, 2).update_all(" question_number = question_number - 1")
+    else
+      ExerciseQuestion.where("question_number>? and question_type=?",params[:quest_num].to_i, 3).update_all(" question_number = question_number - 1")
     end
+    # @exercise_question_num = params[:quest_num].to_i
+    # @exercise_questions.question_number = params[:quest_num].to_i - 1
+    #
+    # exercise_questions = @exercise.exercise_questions.where("question_number > #{@exercise_question.question_number}")
+    # exercise_questions.each do |question|
+    #   question.question_number -= 1
+    #   question.save
+    # end
     if @exercise_question && @exercise_question.destroy
       respond_to do |format|
         format.js
@@ -568,7 +580,9 @@ class ExerciseController < ApplicationController
     exercises = course.exercises.where("exercise_name=?","")
     unless exercises.empty?
       exercises.each do |exercise|
-        exercise.destroy
+        if exercise.exercise_questions.empty?
+          exercise.destroy
+        end
       end
     end
   end