diff --git a/app/controllers/exercise_controller.rb b/app/controllers/exercise_controller.rb
index c8e5bf27c..cd1f32fa9 100644
--- a/app/controllers/exercise_controller.rb
+++ b/app/controllers/exercise_controller.rb
@@ -275,12 +275,12 @@ class ExerciseController < ApplicationController
     @exercise = @exercise_question.exercise
 
     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")
+      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")
+      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")
+      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