diff options
author | Heinrich Lee Yu <heinrich@gitlab.com> | 2019-07-31 20:57:04 +0300 |
---|---|---|
committer | Heinrich Lee Yu <heinrich@gitlab.com> | 2019-07-31 21:04:10 +0300 |
commit | 8992013689e358328d9ae74623b9b29d80f7b17b (patch) | |
tree | 2242db40a6b13180192ea29544aac3fc85bb6036 /app/controllers/boards | |
parent | 2e7f4bbb66b4bae61c9dd09234e8435c91e7e986 (diff) |
Fix bug when moving batches of items to the end
Starts from START_POSITION when there are no existing
positions.
Also improves the test to actually test the behavior
Diffstat (limited to 'app/controllers/boards')
-rw-r--r-- | app/controllers/boards/issues_controller.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/app/controllers/boards/issues_controller.rb b/app/controllers/boards/issues_controller.rb index 90528f75ffd..1d1a72d21f1 100644 --- a/app/controllers/boards/issues_controller.rb +++ b/app/controllers/boards/issues_controller.rb @@ -26,7 +26,7 @@ module Boards list_service = Boards::Issues::ListService.new(board_parent, current_user, filter_params) issues = list_service.execute issues = issues.page(params[:page]).per(params[:per] || 20).without_count - Issue.move_to_end(issues) if Gitlab::Database.read_write? + Issue.move_nulls_to_end(issues) if Gitlab::Database.read_write? issues = issues.preload(:milestone, :assignees, project: [ |