Fixed: Wiki export link doesn't work for users without :view_wiki_edits permission...

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@7971 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/app/views/wiki/show.html.erb b/app/views/wiki/show.html.erb
index 564fc8980f390cf318654f8a46cf7a8e1f958ddb..22f3bab72d35816168fced787369da419c5e28fb 100644 (file)
--- a/app/views/wiki/show.html.erb
+++ b/app/views/wiki/show.html.erb
@@ -53,9 +53,9 @@
 <% end %>
 
 <% other_formats_links do |f| %>
-  <%= f.link_to 'PDF', :url => {:id => @page.title, :version => @content.version} %>
-  <%= f.link_to 'HTML', :url => {:id => @page.title, :version => @content.version} %>
-  <%= f.link_to 'TXT', :url => {:id => @page.title, :version => @content.version} %>
+  <%= f.link_to 'PDF', :url => {:id => @page.title, :version => params[:version]} %>
+  <%= f.link_to 'HTML', :url => {:id => @page.title, :version => params[:version]} %>
+  <%= f.link_to 'TXT', :url => {:id => @page.title, :version => params[:version]} %>
 <% end if User.current.allowed_to?(:export_wiki_pages, @project) %>
 
 <% content_for :header_tags do %>

diff --git a/test/functional/wiki_controller_test.rb b/test/functional/wiki_controller_test.rb
index e5b4bed71abab4146fb06a298958e8c9fbaba218..8b887ecdb5ad34031e96051bc2c3a3aa43ac091b 100644 (file)
--- a/test/functional/wiki_controller_test.rb
+++ b/test/functional/wiki_controller_test.rb
@@ -45,6 +45,13 @@ class WikiControllerTest < ActionController::TestCase
                            :child => { :tag => 'a', :attributes => { :href => '/projects/ecookbook/wiki/Page_with_an_inline_image' },
                                                     :content => 'Page with an inline image' } }
   end
+  
+  def test_export_link
+    Role.anonymous.add_permission! :export_wiki_pages
+    get :show, :project_id => 'ecookbook'
+    assert_response :success
+    assert_tag 'a', :attributes => {:href => '/projects/ecookbook/wiki/CookBook_documentation.txt'}
+  end
 
   def test_show_page_with_name
     get :show, :project_id => 1, :id => 'Another_page'