Skip to content

Commit

Permalink
Fix bug: pass the bs_request superseded object, not the number
Browse files Browse the repository at this point in the history
  • Loading branch information
ncounter committed Mar 22, 2024
1 parent 9218201 commit 49027f3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/api/app/controllers/webui/request_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -542,7 +542,7 @@ def prepare_request_data
@staging_status = staging_status(@bs_request, target_project) if Staging::Workflow.find_by(project: target_project)

# Collecting all issues in a hash. Each key is the issue name and the value is a hash containing all the issue details.
@issues = @action.webui_sourcediff({ diff_to_superseded: @diff_to_superseded_id, cacheonly: 1 }).reduce({}) { |accumulator, sourcediff| accumulator.merge(sourcediff.fetch('issues', {})) }
@issues = @action.webui_sourcediff({ diff_to_superseded: @diff_to_superseded, cacheonly: 1 }).reduce({}) { |accumulator, sourcediff| accumulator.merge(sourcediff.fetch('issues', {})) }

Check warning on line 545 in src/api/app/controllers/webui/request_controller.rb

View check run for this annotation

Codecov / codecov/patch

src/api/app/controllers/webui/request_controller.rb#L545

Added line #L545 was not covered by tests

# retrieve a list of all package maintainers that are assigned to at least one target package
@package_maintainers = target_package_maintainers
Expand Down

0 comments on commit 49027f3

Please sign in to comment.