Move Board now takes account of Board status, fixing #88.

dantarian · Oct 12, 2018 · 2e7b4a5 · 2e7b4a5
1 parent 631156d
commit 2e7b4a5
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/app/controllers/boards_controller.rb b/app/controllers/boards_controller.rb
@@ -114,7 +114,7 @@ def destroy
 
     def move_up
         selected_board = Board.find( params[:id] )
-        target_board = Board.where('"order" < ?', selected_board.order).last
+        target_board = Board.where('"order" < ? and closed = ?', selected_board.order, false).last
 
         if target_board
             selected_board.order, target_board.order = target_board.order, selected_board.order
@@ -128,7 +128,7 @@ def move_up
 
     def move_down
         selected_board = Board.find( params[:id] )
-        target_board = Board.where('"order" > ?', selected_board.order).first
+        target_board = Board.where('"order" > ? and closed = ?', selected_board.order, false).first
 
         if target_board
             selected_board.order, target_board.order = target_board.order, selected_board.order