From 11cb377d2f377212833cc0022a748bc1dcd04402 Mon Sep 17 00:00:00 2001 From: MAEDA Go Date: Sun, 23 Apr 2023 15:59:52 +0900 Subject: [PATCH 2/2] Git: View annotate prior to the change --- app/controllers/repositories_controller.rb | 3 +++ app/views/repositories/annotate.html.erb | 8 ++++++++ config/locales/en.yml | 1 + lib/redmine/scm/adapters/abstract_adapter.rb | 6 ++++-- lib/redmine/scm/adapters/git_adapter.rb | 8 ++++++-- public/stylesheets/scm.css | 1 + test/functional/repositories_git_controller_test.rb | 5 +++++ test/unit/lib/redmine/scm/adapters/git_adapter_test.rb | 4 ++++ 8 files changed, 32 insertions(+), 4 deletions(-) diff --git a/app/controllers/repositories_controller.rb b/app/controllers/repositories_controller.rb index c143ed233..10d67493b 100644 --- a/app/controllers/repositories_controller.rb +++ b/app/controllers/repositories_controller.rb @@ -208,6 +208,9 @@ class RepositoriesController < ApplicationController elsif @annotate.lines.sum(&:size) > Setting.file_max_size_displayed.to_i.kilobyte @annotate = nil @error_message = l(:error_scm_annotate_big_text_file) + else + # true if the SCM adapter supports "View prior to this change" links + @has_pirors = @annotate.priors.any? end @changeset = @repository.find_changeset_by_name(@rev) end diff --git a/app/views/repositories/annotate.html.erb b/app/views/repositories/annotate.html.erb index de30e7c00..2af0852e7 100644 --- a/app/views/repositories/annotate.html.erb +++ b/app/views/repositories/annotate.html.erb @@ -16,6 +16,7 @@ <% line_num = 1; previous_revision = nil %> <% syntax_highlight_lines(@path, Redmine::CodesetUtil.to_utf8_by_setting(@annotate.content)).each do |line| %> <% revision = @annotate.revisions[line_num - 1] %> + <% prior = @annotate.priors[line_num - 1] %> @@ -31,6 +32,13 @@ <%= author.split('<').first %> <% end %> + <% if @has_pirors %> + + <% if prior && revision && revision != previous_revision %> + <%= link_to '', {:action => 'annotate', :id => @project, :repository_id => @repository.identifier_param, :path => to_path_param(prior.split[1] || @path), :rev => prior.split[0] }, :title => l(:label_view_prior_revision), :class => 'icon icon-history' %> + <% end %> + + <% end %> <% if line == "\n" or line == "\r\n" %>
<% else %> diff --git a/config/locales/en.yml b/config/locales/en.yml index 04cbf7792..88625cc6b 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -838,6 +838,7 @@ en: label_latest_revision_plural: Latest revisions label_view_revisions: View revisions label_view_all_revisions: View all revisions + label_view_prior_revision: View prior to this change label_x_revisions: "%{count} revisions" label_max_size: Maximum size label_roadmap: Roadmap diff --git a/lib/redmine/scm/adapters/abstract_adapter.rb b/lib/redmine/scm/adapters/abstract_adapter.rb index 88a62fd6d..5d2d75b06 100644 --- a/lib/redmine/scm/adapters/abstract_adapter.rb +++ b/lib/redmine/scm/adapters/abstract_adapter.rb @@ -423,16 +423,18 @@ module Redmine end class Annotate - attr_reader :lines, :revisions + attr_reader :lines, :revisions, :priors def initialize @lines = [] @revisions = [] + @priors = [] end - def add_line(line, revision) + def add_line(line, revision, prior=nil) @lines << line @revisions << revision + @priors << prior end def content diff --git a/lib/redmine/scm/adapters/git_adapter.rb b/lib/redmine/scm/adapters/git_adapter.rb index 0c916c283..442c2971c 100644 --- a/lib/redmine/scm/adapters/git_adapter.rb +++ b/lib/redmine/scm/adapters/git_adapter.rb @@ -376,11 +376,14 @@ module Redmine identifier = '' # git shows commit author on the first occurrence only authors_by_commit = {} + prior_by_commit = {} content.split("\n").each do |line| if line =~ /^([0-9a-f]{39,40})\s.*/ identifier = $1 elsif line =~ /^author (.+)/ authors_by_commit[identifier] = $1.strip + elsif line =~ /^previous (.+)/ + prior_by_commit[identifier] = $1.strip elsif line =~ /^\t(.*)/ blame.add_line( $1, @@ -388,8 +391,9 @@ module Redmine :identifier => identifier, :revision => identifier, :scmid => identifier, - :author => authors_by_commit[identifier] - ) + :author => authors_by_commit[identifier], + ), + prior_by_commit[identifier] ) identifier = '' author = '' diff --git a/public/stylesheets/scm.css b/public/stylesheets/scm.css index 3bbf6010b..b5db3f186 100644 --- a/public/stylesheets/scm.css +++ b/public/stylesheets/scm.css @@ -104,6 +104,7 @@ table.annotate td.revision { width: 2%; padding-left: 1em; background: inherit; + text-align: left; } table.annotate td.author { diff --git a/test/functional/repositories_git_controller_test.rb b/test/functional/repositories_git_controller_test.rb index 01f745de6..8ffe489a3 100644 --- a/test/functional/repositories_git_controller_test.rb +++ b/test/functional/repositories_git_controller_test.rb @@ -589,9 +589,14 @@ class RepositoriesGitControllerTest < Redmine::RepositoryControllerTest # Line 23, changeset 2f9c0091 assert_select 'tr' do + prior_rev, prior_path = '4a79347ea4b7184938d9bbea0fd421a6079f71bb' ,'sources/watchers_controller.rb' assert_select 'th.line-num a[data-txt=?]', '23' assert_select 'td.revision', :text => /2f9c0091/ assert_select 'td.author', :text => 'jsmith' + assert_select 'td.prior' do + assert_select 'a.icon-history[href=?]', + "/projects/subproject1/repository/#{@repository.id}/revisions/#{prior_rev}/annotate/#{prior_path}" + end assert_select 'td', :text => /remove_watcher/ end end diff --git a/test/unit/lib/redmine/scm/adapters/git_adapter_test.rb b/test/unit/lib/redmine/scm/adapters/git_adapter_test.rb index b81261c41..f3ad2da1c 100644 --- a/test/unit/lib/redmine/scm/adapters/git_adapter_test.rb +++ b/test/unit/lib/redmine/scm/adapters/git_adapter_test.rb @@ -437,6 +437,10 @@ class GitAdapterTest < ActiveSupport::TestCase assert_equal "7234cb2750b63f47bff735edc50a1c0a433c2518", annotate.revisions[4].identifier assert_equal "jsmith", annotate.revisions[4].author + assert_equal "4a79347ea4b7184938d9bbea0fd421a6079f71bb", + annotate.priors[22].split[0] + assert_equal "sources/watchers_controller.rb", + annotate.priors[22].split[1] end def test_annotate_latin_1_identifier -- 2.39.1