Skip to content

Commit

Permalink
Parse diffs also in Files Changed between revisions
Browse files Browse the repository at this point in the history
Co-authored-by: Saray Cabrera Padrón <scabrerapadron@suse.de>
  • Loading branch information
eduardoj and saraycp committed Jan 17, 2023
1 parent f621589 commit 7da754c
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 5 deletions.
2 changes: 1 addition & 1 deletion src/api/app/components/sourcediff_component.rb
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ def initialize(bs_request:, action:, index:, refresh:)
@index = index
@refresh = refresh

ActionSourcediffParser.new(action_sourcediff: action[:sourcediff]).call
SourcediffsParser.new(sourcediffs: action[:sourcediff]).call
end

def file_view_path(filename, sourcediff)
Expand Down
2 changes: 2 additions & 0 deletions src/api/app/controllers/webui/package_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -299,6 +299,8 @@ def rdiff
@not_full_diff = @files.any? { |file| file[1]['diff'].try(:[], 'shown') }
@filenames = filenames['filenames']

SourcediffsParser.new(sourcediffs: [filenames]).call if Flipper.enabled?(:request_show_redesign, User.session)

# FIXME: moved from the old view, needs refactoring
@submit_url_opts = {}
if @oproject && @opackage && !@oproject.find_attribute('OBS', 'RejectRequests') && !@opackage.find_attribute('OBS', 'RejectRequests')
Expand Down
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
class ActionSourcediffParser
def initialize(action_sourcediff:)
@action_sourcediff = action_sourcediff
class SourcediffsParser
def initialize(sourcediffs:)
@sourcediffs = sourcediffs
end

def call
@action_sourcediff&.each do |sourcediff|
@sourcediffs&.each do |sourcediff|
sourcediff['filenames']&.each do |filename|
content = sourcediff['files'][filename].dig('diff', '_content')

Expand Down

0 comments on commit 7da754c

Please sign in to comment.