Revert "Merge branch '22421-fix-issuable-counter-when-more-than-one-label-is-selected' into 'master'
This reverts commit8ed5be5935
, reversing changes made tofb0d1378f4
.
This commit is contained in:
parent
2450625bbc
commit
dfc2fe774d
2 changed files with 9 additions and 26 deletions
|
@ -286,19 +286,8 @@ module ApplicationHelper
|
||||||
}
|
}
|
||||||
|
|
||||||
state_title = titles[state] || state.to_s.humanize
|
state_title = titles[state] || state.to_s.humanize
|
||||||
records_with_state = records.public_send(state)
|
count = records.public_send(state).size
|
||||||
|
html = content_tag :span, state_title
|
||||||
# When filtering by multiple labels, the result of query.count is a Hash
|
|
||||||
# of the form { issuable_id1 => N, issuable_id2 => N }, where N is the
|
|
||||||
# number of labels selected. The ugly "trick" is to load the issuables
|
|
||||||
# as an array and get the size of the array...
|
|
||||||
# We should probably try to solve this properly in the future.
|
|
||||||
# See https://gitlab.com/gitlab-org/gitlab-ce/issues/22414
|
|
||||||
label_names = Array(params.fetch(:label_name, []))
|
|
||||||
records_with_state = records_with_state.to_a if label_names.many?
|
|
||||||
|
|
||||||
count = records_with_state.size
|
|
||||||
html = content_tag :span, state_title
|
|
||||||
|
|
||||||
if count.present?
|
if count.present?
|
||||||
html += " "
|
html += " "
|
||||||
|
|
|
@ -6,19 +6,20 @@ feature 'Issue filtering by Labels', feature: true do
|
||||||
let(:project) { create(:project, :public) }
|
let(:project) { create(:project, :public) }
|
||||||
let!(:user) { create(:user)}
|
let!(:user) { create(:user)}
|
||||||
let!(:label) { create(:label, project: project) }
|
let!(:label) { create(:label, project: project) }
|
||||||
let(:bug) { create(:label, project: project, title: 'bug') }
|
|
||||||
let(:feature) { create(:label, project: project, title: 'feature') }
|
|
||||||
let(:enhancement) { create(:label, project: project, title: 'enhancement') }
|
|
||||||
let(:issue1) { create(:issue, title: "Bugfix1", project: project) }
|
|
||||||
let(:issue2) { create(:issue, title: "Bugfix2", project: project) }
|
|
||||||
let(:issue3) { create(:issue, title: "Feature1", project: project) }
|
|
||||||
|
|
||||||
before do
|
before do
|
||||||
|
bug = create(:label, project: project, title: 'bug')
|
||||||
|
feature = create(:label, project: project, title: 'feature')
|
||||||
|
enhancement = create(:label, project: project, title: 'enhancement')
|
||||||
|
|
||||||
|
issue1 = create(:issue, title: "Bugfix1", project: project)
|
||||||
issue1.labels << bug
|
issue1.labels << bug
|
||||||
|
|
||||||
|
issue2 = create(:issue, title: "Bugfix2", project: project)
|
||||||
issue2.labels << bug
|
issue2.labels << bug
|
||||||
issue2.labels << enhancement
|
issue2.labels << enhancement
|
||||||
|
|
||||||
|
issue3 = create(:issue, title: "Feature1", project: project)
|
||||||
issue3.labels << feature
|
issue3.labels << feature
|
||||||
|
|
||||||
project.team << [user, :master]
|
project.team << [user, :master]
|
||||||
|
@ -158,13 +159,6 @@ feature 'Issue filtering by Labels', feature: true do
|
||||||
wait_for_ajax
|
wait_for_ajax
|
||||||
end
|
end
|
||||||
|
|
||||||
it 'shows a correct "Open" counter' do
|
|
||||||
page.within '.issues-state-filters' do
|
|
||||||
expect(page).not_to have_content "{#{issue2.id} => 1}"
|
|
||||||
expect(page).to have_content "Open 1"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
it 'shows issue "Bugfix2" in issues list' do
|
it 'shows issue "Bugfix2" in issues list' do
|
||||||
expect(page).to have_content "Bugfix2"
|
expect(page).to have_content "Bugfix2"
|
||||||
end
|
end
|
||||||
|
|
Loading…
Reference in a new issue