Commit 4596ab5b authored by Sean McGivern's avatar Sean McGivern

Merge branch 'issue-boards-closed-list-all' into 'master'

Fixed issue boards closed list not listing all issues

Closes #33850

See merge request !12271
parents f7a9e713 9aa83228
...@@ -3,7 +3,7 @@ module Boards ...@@ -3,7 +3,7 @@ module Boards
class ListService < BaseService class ListService < BaseService
def execute def execute
issues = IssuesFinder.new(current_user, filter_params).execute issues = IssuesFinder.new(current_user, filter_params).execute
issues = without_board_labels(issues) unless movable_list? issues = without_board_labels(issues) unless movable_list? || closed_list?
issues = with_list_label(issues) if movable_list? issues = with_list_label(issues) if movable_list?
issues.order_by_position_and_priority issues.order_by_position_and_priority
end end
...@@ -21,7 +21,15 @@ module Boards ...@@ -21,7 +21,15 @@ module Boards
end end
def movable_list? def movable_list?
@movable_list ||= list.present? && list.movable? return @movable_list if defined?(@movable_list)
@movable_list = list.present? && list.movable?
end
def closed_list?
return @closed_list if defined?(@closed_list)
@closed_list = list.present? && list.closed?
end end
def filter_params def filter_params
......
---
title: Fixed issue boards closed list not showing all closed issues
merge_request:
author:
...@@ -67,7 +67,7 @@ describe Boards::Issues::ListService, services: true do ...@@ -67,7 +67,7 @@ describe Boards::Issues::ListService, services: true do
issues = described_class.new(project, user, params).execute issues = described_class.new(project, user, params).execute
expect(issues).to eq [closed_issue4, closed_issue2, closed_issue3, closed_issue1] expect(issues).to eq [closed_issue4, closed_issue2, closed_issue5, closed_issue3, closed_issue1]
end end
it 'returns opened issues that have label list applied when listing issues from a label list' do it 'returns opened issues that have label list applied when listing issues from a label list' do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment