Skip to content

Commit

Permalink
Show deleted issues in the webui
Browse files Browse the repository at this point in the history
Revert commit ff4e4de.

The issue described in #141 was caused by using a wrong old (also known
as old) project in the API endpoint. This issue was fixed in the meantime.
  • Loading branch information
eduardoj committed Aug 28, 2023
1 parent ffa46c2 commit 149f106
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion src/api/app/mixins/parse_package_diff.rb
Expand Up @@ -3,7 +3,6 @@ def issues_hash(sourcediff)
ret = {}
sourcediff.get('issues').elements('issue') do |issue|
next unless issue['name']
next if issue['state'] == 'deleted'

i = Issue.find_by_name_and_tracker(issue['name'], issue['tracker'], nonfatal: 1)
ret[issue['label']] = i.webui_infos if i
Expand Down
2 changes: 1 addition & 1 deletion src/api/spec/mixins/parse_package_diff_spec.rb
Expand Up @@ -119,7 +119,7 @@
it { expect(subject[issue.label][:tracker]).to eq(issue_tracker.name) }

it { expect(subject[:empty]).to be_nil }
it { expect(subject[deleted_issue.label]).to be_nil }
it { expect(subject[deleted_issue.label][:name]).to eq(deleted_issue.name) }
end
end
end

0 comments on commit 149f106

Please sign in to comment.