diff --git a/app/controllers/repositories_controller.rb b/app/controllers/repositories_controller.rb index d5e1352..938ace9 100644 --- a/app/controllers/repositories_controller.rb +++ b/app/controllers/repositories_controller.rb @@ -72,9 +72,7 @@ class RepositoriesController < ApplicationController @entries ? render(:partial => 'dir_list_content') : render(:nothing => true) else show_error_not_found and return unless @entries - if @path.blank? - @changesets = @repository.latest_changesets(@path, @rev) - end + @changesets = @repository.latest_changesets(@path, @rev) @properties = @repository.properties(@path, @rev) render :action => 'show' end diff --git a/app/views/repositories/_link_to_functions.rhtml b/app/views/repositories/_link_to_functions.rhtml index a699938..412d5f8 100644 --- a/app/views/repositories/_link_to_functions.rhtml +++ b/app/views/repositories/_link_to_functions.rhtml @@ -1,3 +1,5 @@ +<% if @entry && @entry.kind == 'file' %> +
<%= link_to_if action_name != 'changes', l(:label_history), {:action => 'changes', :id => @project, :path => to_path_param(@path), :rev => @rev } %> | <% if @repository.supports_cat? %> @@ -9,3 +11,5 @@ <%= link_to(l(:button_download), {:action => 'entry', :id => @project, :path => to_path_param(@path), :rev => @rev, :format => 'raw' }) if @repository.supports_cat? %> <%= "(#{number_to_human_size(@entry.size)})" if @entry.size %>
+ +<% end %> diff --git a/app/views/repositories/_revisions.rhtml b/app/views/repositories/_revisions.rhtml index fb0131c..8fb1552 100644 --- a/app/views/repositories/_revisions.rhtml +++ b/app/views/repositories/_revisions.rhtml @@ -9,7 +9,7 @@<%= link_to l(:label_view_all_revisions), :action => 'revisions', :id => @project %>
+<%= render :partial => 'revisions', :locals => {:project => @project, :path => @path, :revisions => @changesets, :entry => nil }%> + +<% if @path.blank? %> +<%= link_to l(:label_view_all_revisions), :action => 'revisions', :id => @project %>
+<% else %> +<%= link_to l(:label_view_revisions), :action => 'changes', :path => to_path_param(@path), :id => @project %>
+<% end %> + <% content_for :header_tags do %> <%= auto_discovery_link_tag(:atom, params.merge({:format => 'atom', :action => 'revisions', :id => @project, :page => nil, :key => User.current.rss_key})) %> <% end %> diff --git a/config/locales/fr.yml b/config/locales/fr.yml index 51483f7..f37580b 100644 --- a/config/locales/fr.yml +++ b/config/locales/fr.yml @@ -833,7 +833,7 @@ fr: enumeration_activities: Activités (suivi du temps) label_greater_or_equal: ">=" label_less_or_equal: "<=" - label_view_all_revisions: View all revisions + label_view_all_revisions: Voir toutes les révisions label_tag: Tag label_branch: Branch error_no_tracker_in_project: No tracker is associated to this project. Please check the Project settings. diff --git a/test/functional/repositories_subversion_controller_test.rb b/test/functional/repositories_subversion_controller_test.rb index 8e4affa..fe1b3db 100644 --- a/test/functional/repositories_subversion_controller_test.rb +++ b/test/functional/repositories_subversion_controller_test.rb @@ -75,7 +75,7 @@ class RepositoriesSubversionControllerTest < Test::Unit::TestCase assert_equal ['folder', '.project', 'helloworld.c', 'helloworld.rb', 'textfile.txt'], assigns(:entries).collect(&:name) end - def test_changes + def test_file_changes get :changes, :id => 1, :path => ['subversion_test', 'folder', 'helloworld.rb' ] assert_response :success assert_template 'changes' @@ -94,6 +94,16 @@ class RepositoriesSubversionControllerTest < Test::Unit::TestCase :child => { :tag => 'span', :content => 'native' } } end end + + def test_directory_changes + get :changes, :id => 1, :path => ['subversion_test', 'folder' ] + assert_response :success + assert_template 'changes' + + changesets = assigns(:changesets) + assert_not_nil changesets + assert_equal %w(7 6 5 2), changesets.collect(&:revision) + end def test_entry get :entry, :id => 1, :path => ['subversion_test', 'helloworld.c'] @@ -181,11 +191,22 @@ class RepositoriesSubversionControllerTest < Test::Unit::TestCase } end - def test_diff + def test_revision_diff get :diff, :id => 1, :rev => 3 assert_response :success assert_template 'diff' end + + def test_directory_diff + get :diff, :id => 1, :rev => 6, :rev_to => 2, :path => ['subversion_test', 'folder'] + assert_response :success + assert_template 'diff' + + diff = assigns(:diff) + assert_not_nil diff + # 2 files modified + assert_equal 2, Redmine::UnifiedDiff.new(diff).size + end def test_annotate get :annotate, :id => 1, :path => ['subversion_test', 'helloworld.c']